* [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L @ 2020-10-02 14:16 Christian Hewitt 2020-10-02 14:44 ` Jerome Brunet 2020-10-02 18:51 ` Kevin Hilman 0 siblings, 2 replies; 7+ messages in thread From: Christian Hewitt @ 2020-10-02 14:16 UTC (permalink / raw) To: Rob Herring, Mark Rutland, Kevin Hilman, devicetree, linux-arm-kernel, linux-amlogic, linux-kernel Cc: Christian Hewitt 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@gmail.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"; + }; }; &cpu0 { -- 2.17.1 _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L 2020-10-02 14:16 [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L Christian Hewitt @ 2020-10-02 14:44 ` Jerome Brunet 2020-10-02 15:09 ` Christian Hewitt 2020-10-02 18:51 ` Kevin Hilman 1 sibling, 1 reply; 7+ messages in thread From: Jerome Brunet @ 2020-10-02 14:44 UTC (permalink / raw) To: Christian Hewitt, Rob Herring, Mark Rutland, Kevin Hilman, devicetree, linux-arm-kernel, linux-amlogic, linux-kernel On Fri 02 Oct 2020 at 16:16, Christian Hewitt <christianshewitt@gmail.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@gmail.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. > }; > > &cpu0 { _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L 2020-10-02 14:44 ` Jerome Brunet @ 2020-10-02 15:09 ` Christian Hewitt 2020-10-02 18:45 ` Kevin Hilman 0 siblings, 1 reply; 7+ messages in thread From: Christian Hewitt @ 2020-10-02 15:09 UTC (permalink / raw) To: Jerome Brunet Cc: Mark Rutland, devicetree, Kevin Hilman, linux-kernel, Rob Herring, linux-amlogic, linux-arm-kernel > On 2 Oct 2020, at 6:44 pm, Jerome Brunet <jbrunet@baylibre.com> wrote: > > On Fri 02 Oct 2020 at 16:16, Christian Hewitt <christianshewitt@gmail.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@gmail.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 _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L 2020-10-02 15:09 ` Christian Hewitt @ 2020-10-02 18:45 ` Kevin Hilman 2020-10-02 18:53 ` Jerome Brunet 0 siblings, 1 reply; 7+ messages in thread From: Kevin Hilman @ 2020-10-02 18:45 UTC (permalink / raw) To: Christian Hewitt, Jerome Brunet Cc: Mark Rutland, devicetree, linux-kernel, Rob Herring, linux-amlogic, linux-arm-kernel Christian Hewitt <christianshewitt@gmail.com> writes: >> On 2 Oct 2020, at 6:44 pm, Jerome Brunet <jbrunet@baylibre.com> wrote: >> >> On Fri 02 Oct 2020 at 16:16, Christian Hewitt <christianshewitt@gmail.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@gmail.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. True, but turns out there's nothing SoC specific about this sound block since it's identical across SoCs, so specifying the SoC is being too specific. OTOH, while I agree it looks "wrong", it's pretty common in Linux DT to have the SoC prefix to mean only that it's "compatible" with that SoC, not that it *is* that SoC. However, I agree that that can lead to confusion with end users, so since this change has not functional change, and only a UX issue in userspace, I'm fine to apply it. Kevin _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L 2020-10-02 18:45 ` Kevin Hilman @ 2020-10-02 18:53 ` Jerome Brunet 2020-10-02 19:15 ` Kevin Hilman 0 siblings, 1 reply; 7+ messages in thread From: Jerome Brunet @ 2020-10-02 18:53 UTC (permalink / raw) To: Kevin Hilman, Christian Hewitt Cc: Mark Rutland, devicetree, linux-kernel, Rob Herring, linux-amlogic, linux-arm-kernel On Fri 02 Oct 2020 at 20:45, Kevin Hilman <khilman@baylibre.com> wrote: > Christian Hewitt <christianshewitt@gmail.com> writes: > >>> On 2 Oct 2020, at 6:44 pm, Jerome Brunet <jbrunet@baylibre.com> wrote: >>> >>> On Fri 02 Oct 2020 at 16:16, Christian Hewitt <christianshewitt@gmail.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@gmail.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. > > True, but turns out there's nothing SoC specific about this sound block > since it's identical across SoCs, so specifying the SoC is being too > specific. > > OTOH, while I agree it looks "wrong", it's pretty common in Linux DT to > have the SoC prefix to mean only that it's "compatible" with that SoC, > not that it *is* that SoC. > > However, I agree that that can lead to confusion with end users, so > since this change has not functional change, and only a UX issue in > userspace, I'm fine to apply it. It is not UX only. This string is used by alsa-utils to match the card. For example, the string will be matched to restore the controls settings with alsactl on boot. VIM3 and VIM3L are the same sound card AFAICT, so it should be the same string. > > Kevin _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L 2020-10-02 18:53 ` Jerome Brunet @ 2020-10-02 19:15 ` Kevin Hilman 0 siblings, 0 replies; 7+ messages in thread From: Kevin Hilman @ 2020-10-02 19:15 UTC (permalink / raw) To: Jerome Brunet, Christian Hewitt Cc: Mark Rutland, devicetree, linux-kernel, Rob Herring, linux-amlogic, linux-arm-kernel Jerome Brunet <jbrunet@baylibre.com> writes: > On Fri 02 Oct 2020 at 20:45, Kevin Hilman <khilman@baylibre.com> wrote: > >> Christian Hewitt <christianshewitt@gmail.com> writes: >> >>>> On 2 Oct 2020, at 6:44 pm, Jerome Brunet <jbrunet@baylibre.com> wrote: >>>> >>>> On Fri 02 Oct 2020 at 16:16, Christian Hewitt <christianshewitt@gmail.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@gmail.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. >> >> True, but turns out there's nothing SoC specific about this sound block >> since it's identical across SoCs, so specifying the SoC is being too >> specific. >> >> OTOH, while I agree it looks "wrong", it's pretty common in Linux DT to >> have the SoC prefix to mean only that it's "compatible" with that SoC, >> not that it *is* that SoC. >> >> However, I agree that that can lead to confusion with end users, so >> since this change has not functional change, and only a UX issue in >> userspace, I'm fine to apply it. > > It is not UX only. This string is used by alsa-utils to match the > card. For example, the string will be matched to restore the controls > settings with alsactl on boot. VIM3 and VIM3L are the same sound card > AFAICT, so it should be the same string. Ah, OK, thanks for clarifying. Then I would say if it gets changed, it gets changed in the common file. Kevin _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L 2020-10-02 14:16 [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L Christian Hewitt 2020-10-02 14:44 ` Jerome Brunet @ 2020-10-02 18:51 ` Kevin Hilman 1 sibling, 0 replies; 7+ messages in thread From: Kevin Hilman @ 2020-10-02 18:51 UTC (permalink / raw) To: linux-kernel, Mark Rutland, Rob Herring, devicetree, Christian Hewitt, linux-amlogic, linux-arm-kernel On Fri, 2 Oct 2020 14:16:19 +0000, Christian Hewitt 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. Applied, thanks! [1/1] arm64: dts: meson: add SM1 soundcard name to VIM3L commit: 0fe4c285ef8bd0475da7e288c844cb55fc850970 Best regards, -- Kevin Hilman <khilman@baylibre.com> _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-10-02 19:15 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-10-02 14:16 [PATCH] arm64: dts: meson: add SM1 soundcard name to VIM3L Christian Hewitt 2020-10-02 14:44 ` Jerome Brunet 2020-10-02 15:09 ` Christian Hewitt 2020-10-02 18:45 ` Kevin Hilman 2020-10-02 18:53 ` Jerome Brunet 2020-10-02 19:15 ` Kevin Hilman 2020-10-02 18:51 ` Kevin Hilman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox