linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] spi: Fix make htmldocs issues
@ 2016-03-10 18:01 Javier Martinez Canillas
  2016-03-10 18:01 ` [PATCH 1/2] spi: Fix htmldocs build error due struct spi_replaced_transfers Javier Martinez Canillas
  2016-03-10 18:01 ` [PATCH 2/2] spi: Add gfp parameter to kernel-doc to fix build warning Javier Martinez Canillas
  0 siblings, 2 replies; 3+ messages in thread
From: Javier Martinez Canillas @ 2016-03-10 18:01 UTC (permalink / raw)
  To: linux-kernel; +Cc: Mark Brown, linux-spi, Javier Martinez Canillas

Hello,

The make htmldocs target is broken in today's next (next-20160310) and
this is caused by an error in the SPI kernel-doc.

This small series fix that SPI kernel-doc error and also a warning that
I noticed when the docs were finally built.

Best regards,
Javier


Javier Martinez Canillas (2):
  spi: Fix htmldocs build error due struct spi_replaced_transfers
  spi: Add gfp parameter to kernel-doc to fix build warning

 drivers/spi/spi.c       | 1 +
 include/linux/spi/spi.h | 8 ++++----
 2 files changed, 5 insertions(+), 4 deletions(-)

-- 
2.5.0

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 1/2] spi: Fix htmldocs build error due struct spi_replaced_transfers
  2016-03-10 18:01 [PATCH 0/2] spi: Fix make htmldocs issues Javier Martinez Canillas
@ 2016-03-10 18:01 ` Javier Martinez Canillas
  2016-03-10 18:01 ` [PATCH 2/2] spi: Add gfp parameter to kernel-doc to fix build warning Javier Martinez Canillas
  1 sibling, 0 replies; 3+ messages in thread
From: Javier Martinez Canillas @ 2016-03-10 18:01 UTC (permalink / raw)
  To: linux-kernel; +Cc: Mark Brown, linux-spi, Javier Martinez Canillas

The kernel-doc has to be just before the structure definition but the one
for struct spi_replaced_transfers was before a structure declaration and
that confuses kernel-doc which complains with the following build error:

.//include/linux/spi/spi.h:933: error: Cannot parse struct or union!

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 include/linux/spi/spi.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/linux/spi/spi.h b/include/linux/spi/spi.h
index 520a23d46df6..857a9a1d82b5 100644
--- a/include/linux/spi/spi.h
+++ b/include/linux/spi/spi.h
@@ -911,6 +911,10 @@ spi_max_transfer_size(struct spi_device *spi)
 
 /* SPI transfer replacement methods which make use of spi_res */
 
+struct spi_replaced_transfers;
+typedef void (*spi_replaced_release_t)(struct spi_master *master,
+				       struct spi_message *msg,
+				       struct spi_replaced_transfers *res);
 /**
  * struct spi_replaced_transfers - structure describing the spi_transfer
  *                                 replacements that have occurred
@@ -930,10 +934,6 @@ spi_max_transfer_size(struct spi_device *spi)
  * if some extra allocation is requested, so alignment will be the same
  * as for spi_transfers
  */
-struct spi_replaced_transfers;
-typedef void (*spi_replaced_release_t)(struct spi_master *master,
-				       struct spi_message *msg,
-				       struct spi_replaced_transfers *res);
 struct spi_replaced_transfers {
 	spi_replaced_release_t release;
 	void *extradata;
-- 
2.5.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH 2/2] spi: Add gfp parameter to kernel-doc to fix build warning
  2016-03-10 18:01 [PATCH 0/2] spi: Fix make htmldocs issues Javier Martinez Canillas
  2016-03-10 18:01 ` [PATCH 1/2] spi: Fix htmldocs build error due struct spi_replaced_transfers Javier Martinez Canillas
@ 2016-03-10 18:01 ` Javier Martinez Canillas
  1 sibling, 0 replies; 3+ messages in thread
From: Javier Martinez Canillas @ 2016-03-10 18:01 UTC (permalink / raw)
  To: linux-kernel; +Cc: Mark Brown, linux-spi, Javier Martinez Canillas

The spi_split_transfers_maxsize() gfp parameter is missing in the
function kernel-doc so building gives the following warning:

.//drivers/spi/spi.c:2359: warning: No description found for parameter 'gfp'

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

---

 drivers/spi/spi.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c
index 78b7e899ce76..f88161247a2c 100644
--- a/drivers/spi/spi.c
+++ b/drivers/spi/spi.c
@@ -2349,6 +2349,7 @@ static int __spi_split_transfer_maxsize(struct spi_master *master,
  * @master:    the @spi_master for this transfer
  * @msg:   the @spi_message to transform
  * @maxsize:  the maximum when to apply this
+ * @gfp: GFP allocation flags
  *
  * Return: status of transformation
  */
-- 
2.5.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-03-10 18:02 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-10 18:01 [PATCH 0/2] spi: Fix make htmldocs issues Javier Martinez Canillas
2016-03-10 18:01 ` [PATCH 1/2] spi: Fix htmldocs build error due struct spi_replaced_transfers Javier Martinez Canillas
2016-03-10 18:01 ` [PATCH 2/2] spi: Add gfp parameter to kernel-doc to fix build warning Javier Martinez Canillas

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).