linux-kernel - Re: [PATCH V7 1/4] Documentation/devicetree/bindings: b850v3_lvds

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: <74f0-58704480-5-27259840@173563636>
date:   Sat, 07 Jan 2017 01:29:52 +0000
from:   "Peter Senna Tschudin" <peter.senna@...labora.co.uk>
to:     "Rob Herring" <robh@...nel.org>
cc:     "Mauro Carvalho Chehab" <mchehab@....samsung.com>,
        "Archit Taneja" <architt@...eaurora.org>,
        devicetree@...r.kernel.org 
        <devicetree@...r.kernel.org>,
        heiko@...ech.de <heiko@...ech.de>,
        "Greg Kroah-Hartman" <gregkh@...uxfoundation.org>,
        "David Miller" <davem@...emloft.net>,
        linux-arm-kernel@...ts.infradead.org 
        <linux-arm-kernel@...ts.infradead.org>,
        "Martyn Welch" <martyn.welch@...labora.co.uk>,
        "Guenter Roeck" <linux@...ck-us.net>,
        kernel@...gutronix.de <kernel@...gutronix.de>,
        "Kumar Gala" <galak@...eaurora.org>,
        linux-kernel@...r.kernel.org 
        <linux-kernel@...r.kernel.org>, "David Airlie" <airlied@...ux.ie>,
        "Peter Senna Tschudin" <peter.senna@...labora.com>,
        "Fabio Estevam" <fabio.estevam@....com>, martin.donnelly@...com,
        "Jiri Slaby" <jslaby@...e.cz>,
        "Thierry Reding" <thierry.reding@...il.com>,
        "Mark Rutland" <mark.rutland@....com>,
        "Enric Balletbo i Serra" <eballetbo@...il.com>,
        "Takashi Iwai" <tiwai@...e.com>,
        "Philipp Zabel" <p.zabel@...gutronix.de>,
        enric.balletbo@...labora.com,
        "Peter Senna Tschudin" <peter.senna@...il.com>,
        "Javier Martinez Canillas" <javier@...hile0.org>,
        "Yakir Yang" <ykk@...k-chips.com>,
        "Thierry Reding" <treding@...dia.com>,
        "Ian Campbell" <ijc+devicetree@...lion.org.uk>,
        "Shawn Guo" <shawnguo@...nel.org>,
        "dri-devel" <dri-devel@...ts.freedesktop.org>,
        "Daniel Vetter" <daniel.vetter@...ll.ch>,
        "Russell King" <rmk+kernel@...linux.org.uk>,
        "Pawel Moll" <pawel.moll@....com>,
        "Russell King" <linux@...linux.org.uk>,
        "Andrew Morton" <akpm@...ux-foundation.org>
subject: Re: [PATCH V7 1/4] Documentation/devicetree/bindings: b850v3_lvds_dp

 
On 04 January, 2017 21:39 CET, Rob Herring <robh@...nel.org> wrote: 
 
> On Tue, Jan 3, 2017 at 5:34 PM, Peter Senna Tschudin
> <peter.senna@...labora.co.uk> wrote:
> >  Hi Rob,
> >
> > Thank you for the review.
> >
> > On 03 January, 2017 23:51 CET, Rob Herring <robh@...nel.org> wrote:
> >
> >> On Sun, Jan 01, 2017 at 09:24:29PM +0100, Peter Senna Tschudin wrote:
> >> > Devicetree bindings documentation for the GE B850v3 LVDS/DP++
> >> > display bridge.
> >> >
> >> > Cc: Martyn Welch <martyn.welch@...labora.co.uk>
> >> > Cc: Martin Donnelly <martin.donnelly@...com>
> >> > Cc: Javier Martinez Canillas <javier@...hile0.org>
> >> > Cc: Enric Balletbo i Serra <enric.balletbo@...labora.com>
> >> > Cc: Philipp Zabel <p.zabel@...gutronix.de>
> >> > Cc: Rob Herring <robh@...nel.org>
> >> > Cc: Fabio Estevam <fabio.estevam@....com>
> >> > Signed-off-by: Peter Senna Tschudin <peter.senna@...labora.com>
> >> > ---
> >> > There was an Acked-by from Rob Herring <robh@...nel.org> for V6, but I changed
> >> > the bindings to use i2c_new_secondary_device() so I removed it from the commit
> >> > message.
> >> >
> >> >  .../devicetree/bindings/ge/b850v3-lvds-dp.txt      | 39 ++++++++++++++++++++++
> >>
> >> Generally, bindings are not organized by vendor. Put in
> >> bindings/display/bridge/... instead.
> >
> > Will change that.
> >
> >>
> >> >  1 file changed, 39 insertions(+)
> >> >  create mode 100644 Documentation/devicetree/bindings/ge/b850v3-lvds-dp.txt
> >> >
> >> > diff --git a/Documentation/devicetree/bindings/ge/b850v3-lvds-dp.txt b/Documentation/devicetree/bindings/ge/b850v3-lvds-dp.txt
> >> > new file mode 100644
> >> > index 0000000..1bc6ebf
> >> > --- /dev/null
> >> > +++ b/Documentation/devicetree/bindings/ge/b850v3-lvds-dp.txt
> >> > @@ -0,0 +1,39 @@
> >> > +Driver for GE B850v3 LVDS/DP++ display bridge
> >> > +
> >> > +Required properties:
> >> > +  - compatible : should be "ge,b850v3-lvds-dp".
> >>
> >> Isn't '-lvds-dp' redundant? The part# should be enough.
> >
> > b850v3 is the name of the product, this is why the proposed name. What about, b850v3-dp2 dp2 indicating the second DP output?
> 
> Humm, b850v3 is the board name? This node should be the name of the bridge chip.

>From the cover letter:

-- // --
There are two physical bridges on the video signal pipeline: a STDP4028(LVDS to
DP) and a STDP2690(DP to DP++).  The hardware and firmware made it complicated
for this binding to comprise two device tree nodes, as the design goal is to
configure both bridges based on the LVDS signal, which leave the driver
powerless to control the video processing pipeline. The two bridges behaves as
a single bridge, and the driver is only needed for telling the host about EDID /
HPD, and for giving the host powers to ack interrupts. The video signal
pipeline is as follows:

  Host -> LVDS|--(STDP4028)--|DP -> DP|--(STDP2690)--|DP++ -> Video output
-- // --