Re: [PATCH v6 08/52] dt-bindings: memory: tegra20: emc: Document mfd-simple compatible and statistics sub-device

From: Krzysztof Kozlowski
Date: Tue Oct 27 2020 - 15:45:45 EST


On Tue, Oct 27, 2020 at 10:22:19PM +0300, Dmitry Osipenko wrote:
> 27.10.2020 12:02, Krzysztof Kozlowski пишет:
> >> @@ -31,17 +32,34 @@ Example:
> >> ...
> >> };
> >>
> >> + emc_bw_dfs_opp_table: emc_opp_table1 {
> > Hyphens for node name.
>
> We already use underscores for the Tegra CPU OPP table.
>
> https://elixir.bootlin.com/linux/v5.10-rc1/source/arch/arm/boot/dts/tegra20-cpu-opp.dtsi#L4
>
> What makes you think that hyphens will be a better choice? Is it a
> documented naming convention?

Unfortunately that's the source of confusion also for me because
Devicetree spec mentions both of them (and does not specify preferences).

The choice of dashes/hyphens comes now explicitly from all dtschema
files. Previously, the documentation were emails from Rob. :)

Best regards,
Krzysztof