Add config file for customizing DRM Lease names

Fixed

Description

Currently each DRM lease manages a signle connector only.  The DRM lease names are derived from the connector name and the DRM device number.  Eg. card0-HDMI-A-1, or card1-LVDS-1.

A configuration file would allow the system designer to group multiple connectors into a single lease, and to give the lease an arbitrary name (maybe related to how the outputs will be used, such as "ivi" or "cluster")

It also would allow for DRM planes to be assigned to specific leases, which is useful for platforms where a given plane could be used on one of several CRTCs.

Environment

None

Activity

Show:

Walt Miner 
July 21, 2022 at 7:55 PM

Close for NN Milestone 1

Damian Hobson-Garcia 
April 5, 2022 at 8:35 AM

 san,

I have pushed an update to the drm-lease-manager to load configuration data from a file.

For now it is in my sandbox branch: named-lease-and-config for testing.

Currently all connectors are "optional".  If a connector listed in the config file is not available on the system, a warning will be logged and
the next connector in the lease will be processed.  If there are no connectors in a lease after all processing is done that lease will not be created.

Given this spec do you still want to specify some connectors as "mandatory"? Or is it sufficient to have all connectors as "optional"?

 

 

Naoto YAMAGUCHI 
March 23, 2022 at 7:12 AM

+1

Damian Hobson-Garcia 
March 18, 2022 at 4:19 AM
(edited)

One possible option for a configuration file format:

A toml file with the following structure

 

A suggestion was made to have some connectors be optional, while others are mandatory ie. drm-lease-manager will fail on launch if a mandatory connector is not available on launch, but just issues a warning if an optional connector is not present.

To add per-connector options could be added to the configuration file for these kinds of settings.  eg.

san, san, please let me know if you have any other configuration requirements that need to be added to the specification.

Marius Vlad 
February 22, 2021 at 9:39 AM

Oh, I see. I haven't been using the same lease for multiple connectors, just a lease per connector but good to know it works. 

Details

Assignee

Reporter

Fix versions

Labels

Components

Priority

Created February 22, 2021 at 7:51 AM
Updated July 21, 2022 at 7:55 PM
Resolved May 11, 2022 at 3:03 AM