[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CACRpkdavjBPaNq1sRfi-FYiT7T3W19fU-mjha0pAtooUyf-8XQ@mail.gmail.com>
Date: Fri, 10 Mar 2023 23:50:57 +0100
From: Linus Walleij <linus.walleij@...aro.org>
To: Jianhua Lu <lujianhua000@...il.com>
Cc: Konrad Dybcio <konrad.dybcio@...aro.org>,
Thierry Reding <thierry.reding@...il.com>,
Sam Ravnborg <sam@...nborg.org>,
David Airlie <airlied@...il.com>,
Daniel Vetter <daniel@...ll.ch>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Neil Armstrong <neil.armstrong@...aro.org>,
dri-devel@...ts.freedesktop.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org,
~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org
Subject: Re: [PATCH v4 2/2] drm/panel: Add driver for Novatek NT36523
On Fri, Mar 10, 2023 at 7:35 PM Jianhua Lu <lujianhua000@...il.com> wrote:
> On Fri, Mar 10, 2023 at 07:10:21PM +0100, Konrad Dybcio wrote:
> >
> > > +#define mipi_dsi_dual_dcs_write_seq(dsi0, dsi1, cmd, seq...) \
> > > + do { \
> > > + mipi_dsi_dcs_write_seq(dsi0, cmd, seq); \
> > > + mipi_dsi_dcs_write_seq(dsi1, cmd, seq); \
> > > + } while (0)
> > This should be in the same file as mipi_dsi_dcs_write_seq, imo
> I have sent a patch to do it, upstream don't think this wrapper is a proper
> approach to deal with dual dsi and wrap all of mipi_dsi_* function is
> useless.
> https://lore.kernel.org/lkml/20230310110542.6649-1-lujianhua000@gmail.com/
We can keep it locally if the fight isn't worthwhile, but I will try to enter
the discussion.
Yours,
Linus Walleij
Powered by blists - more mailing lists