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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20201109092855.495174dc@xps13>
Date:   Mon, 9 Nov 2020 09:28:55 +0100
From:   Miquel Raynal <miquel.raynal@...tlin.com>
To:     Lee Jones <lee.jones@...aro.org>
Cc:     linux-kernel@...r.kernel.org,
        Adrian Hunter <ext-adrian.hunter@...ia.com>,
        bcm-kernel-feedback-list@...adcom.com,
        Ben Dooks <ben@...tec.co.uk>,
        Benjamin Herrenschmidt <benh@...nel.crashing.org>,
        Boris BREZILLON <b.brezillon.dev@...il.com>,
        Boris Brezillon <boris.brezillon@...labora.com>,
        Brian Norris <computersforpeace@...il.com>,
        Chen-Yu Tsai <wens@...e.org>,
        Choudary Kalluri <punnaia@...inx.com>,
        Christian König <christian.koenig@....com>,
        Dan Brown <dan_brown@...e.org>,
        David Woodhouse <dwmw2@...radead.org>,
        Dmitriy B <rzk333@...il.com>, dri-devel@...ts.freedesktop.org,
        Frieder Schrempf <frieder.schrempf@...tron.de>,
        Jian Zhang <jzhang@...com>,
        Jochen Schäuble <psionic@...onic.de>,
        Kamal Dasu <kdasu.kdev@...il.com>,
        Krzysztof Kozlowski <krzk@...nel.org>,
        Kukjin Kim <kgene@...nel.org>,
        Kyungmin Park <kyungmin.park@...sung.com>,
        linaro-mm-sig@...ts.linaro.org, linux-media@...r.kernel.org,
        linux-mtd@...ts.infradead.org, linuxppc-dev@...ts.ozlabs.org,
        linux-samsung-soc@...r.kernel.org,
        Maxime Ripard <mripard@...nel.org>,
        Michael Ellerman <mpe@...erman.id.au>,
        Naga Sureshkumar Relli <nagasure@...inx.com>,
        Nicolas Pitre <nico@...xnic.net>,
        Paul Mackerras <paulus@...ba.org>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        Qiang Yu <yuq825@...il.com>,
        Richard Weinberger <richard@....at>,
        Robert Jarzmik <robert.jarzmik@...e.fr>,
        Sergey Lapin <slapin@...fans.org>,
        Sumit Semwal <sumit.semwal@...aro.org>,
        Thomas Gleixner <gleixner@...utronix.de>,
        Thomas Gleixner <tglx@...utronix.de>,
        Tudor Ambarus <tudor.ambarus@...rochip.com>,
        Vignesh Raghavendra <vigneshr@...com>,
        Yoshio Furuyama <ytc-mb-yfuruyama7@...xia.com>
Subject: Re: [PATCH v2 00/23] Rid W=1 warnings in MTD

Hi Lee,

Lee Jones <lee.jones@...aro.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