2022-06-08 12:56:24

by David Jander

[permalink] [raw]
Subject: [PATCH] include: linux: spi: spi.h: Add missing documentation for struct members

Fixes "make htmldocs" warnings.

Reported-by: Stephen Rothwell <[email protected]>
Signed-off-by: David Jander <[email protected]>
---
include/linux/spi/spi.h | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/include/linux/spi/spi.h b/include/linux/spi/spi.h
index eac8d3caf954..2e63b4935deb 100644
--- a/include/linux/spi/spi.h
+++ b/include/linux/spi/spi.h
@@ -35,7 +35,8 @@ extern struct bus_type spi_bus_type;

/**
* struct spi_statistics - statistics for spi transfers
- * @lock: lock protecting this structure
+ * @syncp: seqcount to protect members in this struct for per-cpu udate
+ * on 32-bit systems
*
* @messages: number of spi-messages handled
* @transfers: number of spi_transfers handled
@@ -155,7 +156,7 @@ extern int spi_delay_exec(struct spi_delay *_delay, struct spi_transfer *xfer);
* @cs_inactive: delay to be introduced by the controller after CS is
* deasserted. If @cs_change_delay is used from @spi_transfer, then the
* two delays will be added up.
- * @statistics: statistics for the spi_device
+ * @pcpu_statistics: statistics for the spi_device
*
* A @spi_device is used to interchange data between an SPI slave
* (usually a discrete chip) and CPU memory.
@@ -439,7 +440,7 @@ extern struct spi_device *spi_new_ancillary_device(struct spi_device *spi, u8 ch
* @max_native_cs: When cs_gpiods is used, and this field is filled in,
* spi_register_controller() will validate all native CS (including the
* unused native CS) against this value.
- * @statistics: statistics for the spi_controller
+ * @pcpu_statistics: statistics for the spi_controller
* @dma_tx: DMA transmit channel
* @dma_rx: DMA receive channel
* @dummy_rx: dummy receive buffer for full-duplex devices
--
2.32.0


2022-06-08 12:58:22

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] include: linux: spi: spi.h: Add missing documentation for struct members

On Wed, Jun 08, 2022 at 02:29:17PM +0200, David Jander wrote:
> Fixes "make htmldocs" warnings.
>
> Reported-by: Stephen Rothwell <[email protected]>
> Signed-off-by: David Jander <[email protected]>

Please submit patches using subject lines reflecting the style for the
subsystem, this makes it easier for people to identify relevant patches.
Look at what existing commits in the area you're changing are doing and
make sure your subject lines visually resemble what they're doing.
There's no need to resubmit to fix this alone.


Attachments:
(No filename) (546.00 B)
signature.asc (499.00 B)
Download all attachments

2022-06-08 14:14:12

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH] include: linux: spi: spi.h: Add missing documentation for struct members

On Wed, Jun 8, 2022 at 2:56 PM David Jander <[email protected]> wrote:
>
> Fixes "make htmldocs" warnings.

Can you provide the warnings?

> Reported-by: Stephen Rothwell <[email protected]>

Perhaps a Fixes tag?

> Signed-off-by: David Jander <[email protected]>

--
With Best Regards,
Andy Shevchenko