Messages in this thread | | | Subject | Re: [PATCH] backlight: ili922x: fix kernel-doc warnings & notation | From | Randy Dunlap <> | Date | Mon, 15 Nov 2021 18:25:08 -0800 |
| |
On 11/15/21 3:38 AM, Daniel Thompson wrote:
> > Thanks for the fixes. Just a could of quibbles about full > stops/periods. > > >> --- >> drivers/video/backlight/ili922x.c | 29 ++++++++++++++++++---------- >> 1 file changed, 19 insertions(+), 10 deletions(-) >> >> --- linux-next-20211102.orig/drivers/video/backlight/ili922x.c >> +++ linux-next-20211102/drivers/video/backlight/ili922x.c
>> /** >> - * CHECK_FREQ_REG(spi_device s, spi_transfer x) - Check the frequency >> - * for the SPI transfer. According to the datasheet, the controller >> - * accept higher frequency for the GRAM transfer, but it requires >> + * CHECK_FREQ_REG() - Check the frequency for the SPI transfer >> + * According to the datasheet, the controller >> + * accepts higher frequency for the GRAM transfer, but it requires > > Also missing the full stop/period in the first sentence of the summary. > > Note that here the missing full stop does not benefit from a new line to > conceal it and we will generate bad text as a result. > > Check the frequency for the SPI transfer According to the data > sheet, the controller accepts...
Got it. I'll send a v2.
Thanks. -- ~Randy
| |