Hi Greg , Thanks for the review. On Mon, Apr 1, 2019 at 11:09 PM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > On Mon, Apr 01, 2019 at 10:07:19PM +0530, shubhrajyoti.datta@xxxxxxxxx wrote: > > From: Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxxxxx> > > > > Vivado has a configurationo for selecting the modem control. > > Add a dt binding to check for the same. > > > > Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxxxxx> > > --- > > Documentation/devicetree/bindings/serial/cdns,uart.txt | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/serial/cdns,uart.txt b/Documentation/devicetree/bindings/serial/cdns,uart.txt > > index 227bb77..3e96619 100644 > > --- a/Documentation/devicetree/bindings/serial/cdns,uart.txt > > +++ b/Documentation/devicetree/bindings/serial/cdns,uart.txt > > @@ -12,6 +12,10 @@ Required properties: > > See ../clocks/clock-bindings.txt for details. > > > > > > +Optional properties: > > +- xlnx,nomodem: The presence of this property indicates that the > > + UART does not support modem lines for RTS/CTS hardware flow control. > > Shouldn't that be something like "noflowcontrol" or "nortscts"? "modem" > has nothing to do with rts/cts flow control, right? Updated in the v2. Thanks > > thanks, > > greg k-h
