Re: [PATCH v2] serial: Convert serial_rs485 to kernel doc

From: Ilpo Järvinen
Date: Tue Sep 27 2022 - 07:10:33 EST


On Tue, 27 Sep 2022, Jiri Slaby wrote:

> On 27. 09. 22, 12:47, Ilpo Järvinen wrote:
> > For reasons unknown to me, the formatting in the flags doesn't produce the
> > effect promised by kerneldoc's documentation:
> >
> > https://www.kernel.org/doc/html/latest/doc-guide/kernel-doc.html#return-values
>
> Returns are special.

Might be but I understood that formatting thing to be more general.

> > * Serial interface for controlling RS485 settings on chips with suitable
> > * support. Set with TIOCSRS485 and get with TIOCGRS485 if supported by
> > your
> > * platform. The set function returns the new state, with any unsupported
> > bits
> > * reverted appropriately.
> > + *
> > + * serial_rs485::flags bits are:
>
> Put one more \n here.
>
> > + * * %SER_RS485_ENABLED - RS485 enabled.
> > + * * %SER_RS485_RTS_ON_SEND - Logical level for RTS pin when sending.
> > + * * %SER_RS485_RTS_AFTER_SEND - Logical level for RTS pin after
> > sent.
> > + * * %SER_RS485_RX_DURING_TX - Full-duplex RS485 line.
> > + * * %SER_RS485_TERMINATE_BUS - Enable bus termination (if
> > supported).
> > + * * %SER_RS485_ADDRB - Enable RS485 addressing mode.
> > + * * %SER_RS485_ADDR_RECV - Receive address filter (enables @addr_recv).
> > + * * Requires %SER_RS485_ADDRB.
>
> And perhaps remove the second * here?

It gave me error earlier but now when I added also that newline above you
suggested, it seems to no longer triggers the error. Thus, I ended up
making both of the changes you suggested.


--
i.