| Message ID | 20230920152015.1376838-4-hugo@hugovil.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/4] serial: sc16is7xx: use device_property APIs when configuring irda mode | expand |
On Thu, 21 Sep 2023 12:29:13 +0200 m.brock@vanmierlo.com wrote: > Hugo Villeneuve wrote on 2023-09-20 17:20: > > --- a/Documentation/devicetree/bindings/serial/nxp,sc16is7xx.txt > > +++ b/Documentation/devicetree/bindings/serial/nxp,sc16is7xx.txt > > @@ -21,8 +21,8 @@ Optional properties: > > the second cell is used to specify the GPIO polarity: > > 0 = active high, > > 1 = active low. > > -- irda-mode-ports: An array that lists the indices of the port that > > - should operate in IrDA mode. > > +- nxp,irda-mode-ports: An array that lists the indices of the port > > that > > + should operate in IrDA mode. > > - nxp,modem-control-line-ports: An array that lists the indices of > > the port that > > Are you sure you want this line removed? Hi Maarten, the line "- nxp,modem-control-line-ports..." is not removed, but because it starts with a dash in the source file, maybe it is confusing when looking at the patch only? Hugo. > > > should have shared GPIO lines configured as > > modem control lines. > > @@ -80,8 +80,8 @@ Optional properties: > > the second cell is used to specify the GPIO polarity: > > 0 = active high, > > 1 = active low. > > -- irda-mode-ports: An array that lists the indices of the port that > > - should operate in IrDA mode. > > +- nxp,irda-mode-ports: An array that lists the indices of the port > > that > > + should operate in IrDA mode. > > - nxp,modem-control-line-ports: An array that lists the indices of > > the port that > > Same here. > > > should have shared GPIO lines configured as > > modem control lines. > > Greets, > Maarten > >
On Thu, 21 Sep 2023 11:24:24 -0500 Rob Herring <robh@kernel.org> wrote: > On Wed, Sep 20, 2023 at 11:20:14AM -0400, Hugo Villeneuve wrote: > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > > The NXP-specific "irda-mode-ports" property lacks a proper vendor > > prefix. Add "nxp," prefix to comply with DT best practises. > > The opposite direction would be worth carrying in the driver, but I > don't think adding a vendor prefix is. I would just leave this as-is. > > Rob Hi Rob, I am not following you, can you clarify? Hugo.
On Thu, Sep 21, 2023 at 11:32 AM Hugo Villeneuve <hugo@hugovil.com> wrote: > > On Thu, 21 Sep 2023 11:24:24 -0500 > Rob Herring <robh@kernel.org> wrote: > > > On Wed, Sep 20, 2023 at 11:20:14AM -0400, Hugo Villeneuve wrote: > > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > > > > The NXP-specific "irda-mode-ports" property lacks a proper vendor > > > prefix. Add "nxp," prefix to comply with DT best practises. > > > > The opposite direction would be worth carrying in the driver, but I > > don't think adding a vendor prefix is. I would just leave this as-is. > > > > Rob > > Hi Rob, > I am not following you, can you clarify? Just keep the property as-is and don't add a 'nxp,' prefixed version. It's not worth carrying the fallback in the driver. If you were dropping a vendor prefix from a property, then that's worth it because there might be other users added. Rob
On Fri, 22 Sep 2023 14:52:06 -0500 Rob Herring <robh@kernel.org> wrote: > On Thu, Sep 21, 2023 at 11:32 AM Hugo Villeneuve <hugo@hugovil.com> wrote: > > > > On Thu, 21 Sep 2023 11:24:24 -0500 > > Rob Herring <robh@kernel.org> wrote: > > > > > On Wed, Sep 20, 2023 at 11:20:14AM -0400, Hugo Villeneuve wrote: > > > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > > > > > > The NXP-specific "irda-mode-ports" property lacks a proper vendor > > > > prefix. Add "nxp," prefix to comply with DT best practises. > > > > > > The opposite direction would be worth carrying in the driver, but I > > > don't think adding a vendor prefix is. I would just leave this as-is. > > > > > > Rob > > > > Hi Rob, > > I am not following you, can you clarify? > > Just keep the property as-is and don't add a 'nxp,' prefixed version. > It's not worth carrying the fallback in the driver. > > If you were dropping a vendor prefix from a property, then that's > worth it because there might be other users added. > > Rob Hi Rob, ok, then I will drop this patch as well as the other titled 'serial: sc16is7xx: recognize "nxp,irda-mode-ports" DT property' for V2. Hugo.
diff --git a/Documentation/devicetree/bindings/serial/nxp,sc16is7xx.txt b/Documentation/devicetree/bindings/serial/nxp,sc16is7xx.txt index 1a7e4bff0456..d89815c5c562 100644 --- a/Documentation/devicetree/bindings/serial/nxp,sc16is7xx.txt +++ b/Documentation/devicetree/bindings/serial/nxp,sc16is7xx.txt @@ -21,8 +21,8 @@ Optional properties: the second cell is used to specify the GPIO polarity: 0 = active high, 1 = active low. -- irda-mode-ports: An array that lists the indices of the port that - should operate in IrDA mode. +- nxp,irda-mode-ports: An array that lists the indices of the port that + should operate in IrDA mode. - nxp,modem-control-line-ports: An array that lists the indices of the port that should have shared GPIO lines configured as modem control lines. @@ -80,8 +80,8 @@ Optional properties: the second cell is used to specify the GPIO polarity: 0 = active high, 1 = active low. -- irda-mode-ports: An array that lists the indices of the port that - should operate in IrDA mode. +- nxp,irda-mode-ports: An array that lists the indices of the port that + should operate in IrDA mode. - nxp,modem-control-line-ports: An array that lists the indices of the port that should have shared GPIO lines configured as modem control lines.