linux-kernel - Re: [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <785296E3-FA91-4F5E-A8C0-1FAA8C576A17@gmail.com>
Date:   Fri, 2 Oct 2020 19:09:05 +0400
From:   Christian Hewitt <christianshewitt@...il.com>
To:     Jerome Brunet <jbrunet@...libre.com>
Cc:     Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Kevin Hilman <khilman@...libre.com>,
        devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-amlogic@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L


> On 2 Oct 2020, at 6:44 pm, Jerome Brunet <jbrunet@...libre.com> wrote:
> 
> On Fri 02 Oct 2020 at 16:16, Christian Hewitt <christianshewitt@...il.com> wrote:
> 
>> VIM3L now inherits the sound node from the VIM3 common dtsi but is
>> an SM1 device, so label it as such, and stop users blaming future
>> support issues on the distro/app "wrongly detecting" their device.
>> 
>> Signed-off-by: Christian Hewitt <christianshewitt@...il.com>
>> ---
>> arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts | 4 ++++
>> 1 file changed, 4 insertions(+)
>> 
>> diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts b/arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts
>> index 4b517ca72059..f46f0ecc37ec 100644
>> --- a/arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts
>> +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts
>> @@ -32,6 +32,10 @@
>> 		regulator-boot-on;
>> 		regulator-always-on;
>> 	};
>> +
>> +	sound {
>> +		model = "SM1-KHADAS-VIM3L";
>> +	};
> 
> The sound card is the same so I don't see why the sm1 board should have
> a different name. If you are not happy with the name, please update it
> in the common file.

It’s a nice-to-have not a must-have, but the current LE images that are
in circulation use 5.7 with the previous board-correct name so I was
looking for continuity. We do see user forum reports (infrequent but
recurring) of wrongly detected hardware with other SoC platforms where
similar name inheritance surfaces the ‘wrong’ device name in GUIs, and
I like anything that avoids support work.

I’d suggest KHADAS-VIM3-VIM3L as a common name, but then it’s the only
device in the current device-tree set that is not prefixed with the SoC
identifier, which (OCD) feels wrong.

Christian