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]
Date:   Tue, 25 Aug 2020 15:22:47 -0700
From:   Rob Clark <robdclark@...il.com>
To:     Stephen Boyd <swboyd@...omium.org>
Cc:     linux-arm-msm <linux-arm-msm@...r.kernel.org>,
        Rob Clark <robdclark@...omium.org>,
        Douglas Anderson <dianders@...omium.org>,
        Matthias Kaehlcke <mka@...omium.org>,
        Atul Dhudase <adhudase@...eaurora.org>,
        Venkata Lakshmi Narayana Gubba <gubbaven@...eaurora.org>,
        Evan Green <evgreen@...omium.org>,
        Cheng-Yi Chiang <cychiang@...omium.org>,
        Ajit Pandey <ajitp@...eaurora.org>,
        Alexandru Stan <amstan@...omium.org>,
        Sujit Kautkar <sujitka@...omium.org>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] arm64: dts: qcom: sc7180-trogdor: add initial trogdor and
 lazor dt

On Tue, Aug 25, 2020 at 2:21 AM Stephen Boyd <swboyd@...omium.org> wrote:
>
> Quoting Rob Clark (2020-08-24 17:33:39)
> > diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> > new file mode 100644
> > index 000000000000..b04987ab6c22
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> > @@ -0,0 +1,1364 @@
> > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> > +/*
> [...]
> > +
> > +       gpio_keys: gpio-keys {
> > +               compatible = "gpio-keys";
> > +               status = "disabled";
> > +               pinctrl-names = "default";
> > +               pinctrl-0 = <&pen_pdct_l>;
> > +
> > +               pen-insert {
> > +                       label = "Pen Insert";
> > +
> > +                       /* Insert = low, eject = high */
> > +                       gpios = <&tlmm 52 GPIO_ACTIVE_LOW>;
> > +                       linux,code = <SW_PEN_INSERTED>;
> > +                       linux,input-type = <EV_SW>;
> > +                       wakeup-source;
> > +               };
> > +       };
> > +
> > +       max98357a: max98357a {
>
> Maybe node name should be 'audio-codec' or 'dac' or 'amp' or 'speaker'
> or 'codec'?

looking at other boards, it looks like 2 votes for "audio-codec-0",
one vote for "codec0", and 2 votes for "max98357a"..  so seems to be a
codec rather than amp/speaker.. and "audio-codec" seems to be the
popular choice..

BR,
-R

Powered by blists - more mailing lists