lkml.org 
[lkml]   [2021]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] dt-bindings: convert power domain node for rockchip DW MIPI DSI
On Sat, Dec 11, 2021 at 11:55:58AM +0100, Heiko Stübner wrote:
> Hi David,
>
> Am Montag, 6. Dezember 2021, 22:26:50 CET schrieb David Heidelberg:
> > Convert into YAML format into format, which can be validated.
> >
> > Changes:
> > - drop panel from example
>
> the patch subject is strange, talking about a "power domain node".
> That needs a fix.
>
> Some more things below.
>
>
> > +properties:
> > + compatible:
> > + oneOf:
> > + - items:
> > + - enum:
> > + - rockchip,px30-mipi-dsi
> > + - rockchip,rk3288-mipi-dsi
> > + - rockchip,rk3399-mipi-dsi
> > + - const: snps,dw-mipi-dsi
>
> > + - items:
> > + - const: rockchip,px30-mipi-dsi
> > + - items:
> > + - const: rockchip,rk3288-mipi-dsi
> > + - items:
> > + - const: rockchip,rk3399-mipi-dsi
>
> what are these for?
>
> I see that px30 uses the dsi without the snps part, but you
> can also just add a patch adding that second compatible to px30.dtsi

Or drop snps,dw-mipi-dsi as it doesn't seem to be all that meaningful.
But I guess that's more invasive so maybe stick with changing px30.

>
> I don't think we need to support both ways.

Agreed.

Rob

\
 
 \ /
  Last update: 2021-12-13 23:51    [W:0.216 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site