[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CA+cxXhmugPkSNoXCrVTJdY9rq-vMP3AXrCKFxdMT9yhPVQhPaQ@mail.gmail.com>
Date: Tue, 24 Aug 2021 18:14:20 -0700
From: Philip Chen <philipchen@...omium.org>
To: Doug Anderson <dianders@...omium.org>
Cc: LKML <linux-kernel@...r.kernel.org>,
Stephen Boyd <swboyd@...omium.org>,
Philip Chen <philipchen@...gle.com>,
Andrzej Hajda <a.hajda@...sung.com>,
Daniel Vetter <daniel@...ll.ch>,
David Airlie <airlied@...ux.ie>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Jonas Karlman <jonas@...boo.se>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
Neil Armstrong <narmstrong@...libre.com>,
Robert Foss <robert.foss@...aro.org>,
dri-devel <dri-devel@...ts.freedesktop.org>
Subject: Re: [PATCH] drm/bridge: parade-ps8640: Reorg the macros
Hi,
On Tue, Aug 24, 2021 at 3:37 PM Doug Anderson <dianders@...omium.org> wrote:
>
> Hi,
>
> On Fri, Aug 20, 2021 at 4:22 PM Philip Chen <philipchen@...omium.org> wrote:
> >
> > From: Philip Chen <philipchen@...omium.org>
> >
> > Reorg the macros as follows:
> > (1) Group the registers on the same page together.
> > (2) Group the register and its bit operation together while indenting
> > the macros of the bit operation with one space.
> >
> > Also fix a misnomer for the number of mipi data lanes.
> >
> > Signed-off-by: Philip Chen <philipchen@...omium.org>
> > Signed-off-by: Philip Chen <philipchen@...gle.com>
> > ---
> >
> > drivers/gpu/drm/bridge/parade-ps8640.c | 18 ++++++++++--------
> > 1 file changed, 10 insertions(+), 8 deletions(-)
>
> This seems fine to me other than the slightly mixed up Signed-off-by
> lines. I think that the git config from wherever you ran "git
> send-email" had your @google.com address even if you authored the
> patch with your @chomium.org address. Once that's fixed then I'm happy
> to add my Reviewed-by and (unless someone objects) I'll plan to apply
> this patch to drm-misc-next.
Thank you.
I just posted v2 with the double sign-off lines fixed.
PTAL.
>
> -Doug
Powered by blists - more mailing lists