[PATCH] spi: spi.h: add missing kernel-doc for @last_cs_index_mask

Randy Dunlap posted 1 patch 1 year, 10 months ago
There is a newer version of this series
include/linux/spi/spi.h |    1 +
1 file changed, 1 insertion(+)
[PATCH] spi: spi.h: add missing kernel-doc for @last_cs_index_mask
Posted by Randy Dunlap 1 year, 10 months ago
kernel-doc complains about last_cs_index_mask not described, so add its
decscription.

spi.h:778: warning: Function parameter or struct member 'last_cs_index_mask' not described in 'spi_controller'

Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
---
Cc: Mark Brown <broonie@kernel.org>
Cc: Andy Shevchenko <andy@kernel.org>
Cc: linux-spi@vger.kernel.org

 include/linux/spi/spi.h |    1 +
 1 file changed, 1 insertion(+)

diff -- a/include/linux/spi/spi.h b/include/linux/spi/spi.h
--- a/include/linux/spi/spi.h
+++ b/include/linux/spi/spi.h
@@ -453,6 +453,7 @@ extern struct spi_device *spi_new_ancill
  * @last_cs_mode_high: was (mode & SPI_CS_HIGH) true on the last call to set_cs.
  * @last_cs: the last chip_select that is recorded by set_cs, -1 on non chip
  *           selected
+ * @last_cs_index_mask: bit mask the last chip selects that were used
  * @xfer_completion: used by core transfer_one_message()
  * @busy: message pump is busy
  * @running: message pump is running
Re: [PATCH] spi: spi.h: add missing kernel-doc for @last_cs_index_mask
Posted by Andy Shevchenko 1 year, 10 months ago
On Thu, Mar 28, 2024 at 6:38 AM Randy Dunlap <rdunlap@infradead.org> wrote:
>
> kernel-doc complains about last_cs_index_mask not described, so add its
> decscription.

description

> spi.h:778: warning: Function parameter or struct member 'last_cs_index_mask' not described in 'spi_controller'

Do we need a Fixes tag?

-- 
With Best Regards,
Andy Shevchenko
Re: [PATCH] spi: spi.h: add missing kernel-doc for @last_cs_index_mask
Posted by Randy Dunlap 1 year, 10 months ago

On 3/28/24 01:51, Andy Shevchenko wrote:
> On Thu, Mar 28, 2024 at 6:38 AM Randy Dunlap <rdunlap@infradead.org> wrote:
>>
>> kernel-doc complains about last_cs_index_mask not described, so add its
>> decscription.
> 
> description

Fixed in v2.

>> spi.h:778: warning: Function parameter or struct member 'last_cs_index_mask' not described in 'spi_controller'
> 
> Do we need a Fixes tag?

I'll add one.

Thanks.
-- 
#Randy