Re: [PATCH v3 7/7] tty: add pruss SUART driver

[Subhasish Ghosh]

Hello,

Thank you for your comments.
Would really appreciate more comments and ways to fix this patch and bring 
it
up to Linux Standards.
If I have missed something/misunderstood then please let me know.


> > +config SERIAL_PRUSS_SUART
> > + depends on ARCH_DAVINCI && ARCH_DAVINCI_DA850
> > + select SERIAL_CORE
> > + tristate "PRUSS based SoftUART emulation on DA8XX"
> > + ---help---
> > + This driver emulates upto eight different UARTs on the PRUSS.
>
>                      up to
SG -- Ok Will do.


> > + You may modify the NR_SUARTS macro in the driver to emulate
> > + less number of UARTS as per your requirement.
> > + If not sure, mark N
>
>              mark N.
SG -- Ok Will do.


> > +    Enter the McASP number to use with SUART (0, 1 or 2).
> > + You will need to recompile the kernel if this is changed.
>
> The 2 help text lines should be indented the same.
SG - Ok will do.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/