linux-kernel - Re: [PATCH] drm/bridge: ti-tfp410: switch to using fwnode_gpiod_get

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: <CACRpkdYY_W8_L4---iMORt6vriUa9wKEi0d_kiMRbB_NQatRog@mail.gmail.com>
Date:   Wed, 13 Nov 2019 14:52:23 +0100
From:   Linus Walleij <linus.walleij@...aro.org>
To:     Daniel Vetter <daniel@...ll.ch>
Cc:     Dmitry Torokhov <dmitry.torokhov@...il.com>,
        Andrzej Hajda <a.hajda@...sung.com>,
        Neil Armstrong <narmstrong@...libre.com>,
        Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
        Jonas Karlman <jonas@...boo.se>,
        Jernej Skrabec <jernej.skrabec@...l.net>,
        David Airlie <airlied@...ux.ie>,
        Heikki Krogerus <heikki.krogerus@...ux.intel.com>,
        "open list:DRM PANEL DRIVERS" <dri-devel@...ts.freedesktop.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] drm/bridge: ti-tfp410: switch to using fwnode_gpiod_get_index()

On Tue, Nov 5, 2019 at 4:41 PM Daniel Vetter <daniel@...ll.ch> wrote:
> On Tue, Nov 5, 2019 at 4:29 PM Linus Walleij <linus.walleij@...aro.org> wrote:
> > On Tue, Nov 5, 2019 at 1:40 AM Dmitry Torokhov
> > <dmitry.torokhov@...il.com> wrote:

> > I'm happy to merge it into the GPIO tree if some DRM maintainer can
> > provide an ACK.
>
> Ack.

Thanks!

> > Getting ACK from DRM people is problematic and a bit of friction in the
> > community, DVetter usually advice to seek mutual reviews etc, but IMO
> > it would be better if some people felt more compelled to review stuff
> > eventually. (And that has the problem that it doesn't scale.)
>
> This has a review already plus if you merge your implied review.

Yeah I missed Laurent's review tag. I needed some kund of consent
to take it into the GPIO tree I suppose.

> That's more than good enough imo, so not seeing the issue here?

No issue.

What freaked me out was the option of having to pull in an
immutable branch from my GPIO tree into drm-misc. That would
have been scary. Keeping it all in my tree works fine.

Yours,
Linus Walleij