Messages in this thread | | | Date | Sat, 5 Sep 2020 19:07:14 +0300 | From | Laurent Pinchart <> | Subject | Re: [PATCH 4/5] dt-bindings: display/bridge: nwl-dsi: Document fsl,clock-drop-level property |
| |
Hi Robert,
Thank you for the patch.
On Fri, Aug 28, 2020 at 02:13:31PM +0300, Robert Chiras (OSS) wrote: > From: Robert Chiras <robert.chiras@nxp.com> > > Add documentation for a new property: 'fsl,clock-drop-level'. > > Signed-off-by: Robert Chiras <robert.chiras@nxp.com> > --- > Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml b/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml > index 8b5741b..b415f4e 100644 > --- a/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml > @@ -143,6 +143,10 @@ properties: > > additionalProperties: false > > + clock-drop-level: > + description: > + Specifies the level at wich the crtc_clock should be dropped > +
There's no "crtc_clock" defined in the bindings. As DT bindings shouldn't be tied to a particular driver implementation, could you document this property without referring to concepts specific to the driver ? I think the documentation should also be extended, looking at this patch I have no idea what this does and how to compute the value that should be set.
> patternProperties: > "^panel@[0-9]+$": > type: object
-- Regards,
Laurent Pinchart
| |