Re: [PATCH v2 00/23] Rid W=1 warnings in MTD

From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Lee Jones <lee.jones@linaro.org>
Cc: "Boris BREZILLON" <b.brezillon.dev@gmail.com>,
	"Vignesh Raghavendra" <vigneshr@ti.com>,
	"Benjamin Herrenschmidt" <benh@kernel.crashing.org>,
	"Sergey Lapin" <slapin@ossfans.org>,
	dri-devel@lists.freedesktop.org, linux-mtd@lists.infradead.org,
	"Frieder Schrempf" <frieder.schrempf@kontron.de>,
	"Choudary Kalluri" <punnaia@xilinx.com>,
	"Robert Jarzmik" <robert.jarzmik@free.fr>,
	"Sumit Semwal" <sumit.semwal@linaro.org>,
	"Paul Mackerras" <paulus@samba.org>,
	"Dan Brown" <dan_brown@ieee.org>,
	linux-samsung-soc@vger.kernel.org,
	"Adrian Hunter" <ext-adrian.hunter@nokia.com>,
	"Kamal Dasu" <kdasu.kdev@gmail.com>,
	"Michael Ellerman" <mpe@ellerman.id.au>,
	"Krzysztof Kozlowski" <krzk@kernel.org>,
	"Thomas Gleixner" <gleixner@linutronix.de>,
	"Chen-Yu Tsai" <wens@csie.org>, "Kukjin Kim" <kgene@kernel.org>,
	bcm-kernel-feedback-list@broadcom.com,
	linux-media@vger.kernel.org,
	"Tudor Ambarus" <tudor.ambarus@microchip.com>,
	"Maxime Ripard" <mripard@kernel.org>,
	linaro-mm-sig@lists.linaro.org, "Dmitriy B" <rzk333@gmail.com>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"Jochen Schäuble" <psionic@psionic.de>,
	"Naga Sureshkumar Relli" <nagasure@xilinx.com>,
	"Yoshio Furuyama" <ytc-mb-yfuruyama7@kioxia.com>,
	"Nicolas Pitre" <nico@fluxnic.net>,
	linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org,
	"Ben Dooks" <ben@simtec.co.uk>,
	"Kyungmin Park" <kyungmin.park@samsung.com>,
	"Boris Brezillon" <boris.brezillon@collabora.com>,
	"Qiang Yu" <yuq825@gmail.com>,
	"Philipp Zabel" <p.zabel@pengutronix.de>,
	"Richard Weinberger" <richard@nod.at>,
	"Jian Zhang" <jzhang@ti.com>,
	"Brian Norris" <computersforpeace@gmail.com>,
	"David Woodhouse" <dwmw2@infradead.org>,
	"Christian König" <christian.koenig@amd.com>
Subject: Re: [PATCH v2 00/23] Rid W=1 warnings in MTD
Date: Mon, 9 Nov 2020 09:28:55 +0100	[thread overview]
Message-ID: <20201109092855.495174dc@xps13> (raw)
In-Reply-To: <20201106213655.1838861-1-lee.jones@linaro.org>

Hi Lee,

Lee Jones <lee.jones@linaro.org> wrote on Fri,  6 Nov 2020 21:36:32
+0000:

> This set is part of a larger effort attempting to clean-up W=1
> kernel builds, which are currently overwhelmingly riddled with
> niggly little warnings.
> 
> v1 => v2:
>   - Added tags
>   - Satisfied Miquel's review comments
> 

You probably missed my request to update the titles. That's why I
wanted the entire series to be resent. Anyway, as I forgot a few,
please find below the prefixes that should be used:

> Lee Jones (23):
>   mtd: mtdpart: Fix misdocumented function parameter 'mtd'
>   mtd: devices: phram: File headers are not good candidates for
>     kernel-doc
>   mtd: nand: onenand: onenand_base: Fix expected kernel-doc formatting

mtd: onenand: Fix...

>   mtd: devices: docg3: Fix kernel-doc 'bad line' and 'excessive doc'
>     issues
>   mtd: mtdcore: Fix misspelled function parameter 'section'

mtd: Fix...

>   mtd: nand: onenand: onenand_bbt: Fix expected kernel-doc formatting

mtd: onenand: Fix...

>   mtd: spi-nor: controllers: hisi-sfc: Demote non-conformant kernel-doc

mtd: spi-nor: hisi-sfc: Demote...

>   mtd: ubi: build: Document 'ubi_num' in struct mtd_dev_param
>   mtd: nand: spi: toshiba: Demote non-conformant kernel-doc header

mtd: spinand: toshiba: Demote...

>   mtd: ubi: kapi: Correct documentation for 'ubi_leb_read_sg's 'sgl'
>     parameter
>   mtd: ubi: eba: Fix a couple of misdocumentation issues
>   mtd: ubi: wl: Fix a couple of kernel-doc issues
>   mtd: nand: raw: brcmnand: brcmnand: Demote non-conformant kernel-doc
>     headers

mtd: rawnand: brcmnand: Demote...

>   mtd: ubi: gluebi: Fix misnamed function parameter documentation
>   mtd: nand: raw: diskonchip: Marking unused variables as
>     __always_unused

mtd: rawnand: diskonchip: Marking...

>   mtd: nand: raw: cafe_nand: Remove superfluous param doc and add
>     another

mtd: rawnand: cafe: Remove

>   mtd: nand: raw: s3c2410: Add documentation for 2 missing struct
>     members

mtd: rawnand: s3c2410: Add...

>   mtd: nand: raw: omap_elm: Finish half populated function header,
>     demote empty ones

mtd: rawnand: omap_elm: Finish

>   mtd: nand: raw: omap2: Fix a bunch of kernel-doc misdemeanours

mtd:r rawnand: omap2: Fix

>   mtd: nand: raw: sunxi_nand: Document 'sunxi_nfc's 'caps' member

mtd: rawnand: sunxi: Document

>   mtd: nand: raw: arasan-nand-controller: Document 'anfc_op's 'buf'
>     member

mtd: rawnand: arasan: Document

>   mtd: nand: onenand: onenand_base: Fix some kernel-doc misdemeanours

mtd: onenand: Fix

>   mtd: devices: powernv_flash: Add function names to headers and fix
>     'dev'

Otherwise the content of the series looks good to me.

Thanks,
Miquèl

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/