Messages in this thread | | | Date | Mon, 23 Aug 2021 17:43:45 -0500 | From | Rob Herring <> | Subject | Re: [PATCH v4 6/6] phy: dt-bindings: cdns,dphy: add Rx DPHY compatible |
| |
On Mon, Aug 23, 2021 at 04:44:54AM +0300, Laurent Pinchart wrote: > Hi Pratyush, > > Thank you for the patch. > > On Sat, Aug 21, 2021 at 12:33:46AM +0530, Pratyush Yadav wrote: > > The DPHY is treated to be in Tx mode by default. Add a new compatible > > for Rx mode DPHYs. > > > > Signed-off-by: Pratyush Yadav <p.yadav@ti.com> > > > > --- > > > > Changes in v4: > > - New in v4. > > > > Documentation/devicetree/bindings/phy/cdns,dphy.yaml | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/phy/cdns,dphy.yaml b/Documentation/devicetree/bindings/phy/cdns,dphy.yaml > > index d5a5e1f0b671..48b6a841152e 100644 > > --- a/Documentation/devicetree/bindings/phy/cdns,dphy.yaml > > +++ b/Documentation/devicetree/bindings/phy/cdns,dphy.yaml > > @@ -11,8 +11,9 @@ maintainers: > > > > properties: > > compatible: > > - items: > > + oneOf: > > - const: cdns,dphy > > + - const: cdns,dphy-rx > > It's customary to use an enum instead: > > compatibles: > enum: > - cdns,dphy > - cdns,dphy-tx > > I don't know if Rob has a preference for consistency (or for any other > reason). In either case,
Yes, use enum.
Rob
| |