Skip to content

Add new 'chassis-type' root node property#45

Merged
robherring merged 1 commit intodevicetree-org:masterfrom
a-wai:master
Sep 13, 2021
Merged

Add new 'chassis-type' root node property#45
robherring merged 1 commit intodevicetree-org:masterfrom
a-wai:master

Conversation

@a-wai
Copy link
Contributor

@a-wai a-wai commented Sep 8, 2021

ACPI-based systems generally provide DMI data allowing userspace software to identify the form-factor of the running system.

Such information can be useful (for example, in order to implement different behavior on a desktop and on a smartphone) but is not readily available on DT-based systems.

This patch proposes adding a new optional property to the root node, allowing a way to specify the system form factor.

The new property is named chassis-type, using the same values as systemd's hostnamed.

ACPI-based systems generally provide DMI data allowing userspace
software to identify the form-factor of the running system.

Such information can be useful (for example, in order to implement
different behavior on a desktop and on a smartphone) but is not readily
available on DT-based systems.

This patch proposes adding a new optional property to the root node,
allowing a way to specify the system form factor.

The new property is named `chassis-type`, using the same values as
systemd's hostnamed.

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>
@robherring
Copy link
Member

'embedded' seems rather pointless, but okay.

Please also add this to the corresponding schema in https://github.com/devicetree-org/dt-schema/blob/master/schemas/root-node.yaml

@a-wai
Copy link
Contributor Author

a-wai commented Sep 8, 2021

'embedded' seems rather pointless, but okay.

I can see it being used for random dev boards, but I included it mostly to stay in line with systemd.

Please also add this to the corresponding schema in https://github.com/devicetree-org/dt-schema/blob/master/schemas/root-node.yaml

Sure thing, here it is: devicetree-org/dt-schema#58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants