[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20200323212711.GA6856@bogus>
Date: Mon, 23 Mar 2020 15:27:11 -0600
From: Rob Herring <robh@...nel.org>
To: Pascal Roeleven <dev@...calroeleven.nl>
Cc: Sam Ravnborg <sam@...nborg.org>,
Thierry Reding <thierry.reding@...il.com>,
David Airlie <airlied@...ux.ie>,
Daniel Vetter <daniel@...ll.ch>,
Mark Rutland <mark.rutland@....com>,
Maxime Ripard <mripard@...nel.org>,
Chen-Yu Tsai <wens@...e.org>, dri-devel@...ts.freedesktop.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-sunxi@...glegroups.com
Subject: Re: [PATCH 1/2] drm/panel: Add Starry KR070PE2T
On Wed, Mar 11, 2020 at 11:23:27AM +0100, Pascal Roeleven wrote:
> On 2020-03-10 19:54, Sam Ravnborg wrote:
> > A few things to improve.
> >
> > The binding should be a separate patch.
> > subject - shall start with dt-bindings:
> > Shall be sent to deveicetree mailing list.
>
> Hi Sam,
>
> Thank you very much for your review.
> I did consider this. The reason I combined the patches, is that the binding
> depends on the display so I thought they were related in some way. Didn't
> know the correct procedure to handle this. I will split them apart in v2.
FYI, checkpatch.pl will tell you both bindings should be a separate
patch and that they should be in DT schema format.
Rob
Powered by blists - more mailing lists