[v2,05/23] mtd: mtdcore: Fix misspelled function parameter 'section'

Message ID 20201106213655.1838861-6-lee.jones@linaro.org
State Superseded
Headers show
Series
  • [v2,01/23] mtd: mtdpart: Fix misdocumented function parameter 'mtd'
Related show

Commit Message

Lee Jones Nov. 6, 2020, 9:36 p.m.
Fixes the following W=1 kernel build warning(s):

 drivers/mtd/mtdcore.c:1592: warning: Function parameter or member 'section' not described in 'mtd_ooblayout_find_eccregion'
 drivers/mtd/mtdcore.c:1592: warning: Excess function parameter 'sectionp' description in 'mtd_ooblayout_find_eccregion'

Cc: Miquel Raynal <miquel.raynal@bootlin.com>
Cc: Richard Weinberger <richard@nod.at>
Cc: Vignesh Raghavendra <vigneshr@ti.com>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: linux-mtd@lists.infradead.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>

---
 drivers/mtd/mtdcore.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.25.1


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

Comments

Alexander Dahl Nov. 9, 2020, 2:19 p.m. | #1
Hello,

Am Freitag, 6. November 2020, 22:36:37 CET schrieb Lee Jones:
> Fixes the following W=1 kernel build warning(s):

> 

>  drivers/mtd/mtdcore.c:1592: warning: Function parameter or member 'section'

> not described in 'mtd_ooblayout_find_eccregion' drivers/mtd/mtdcore.c:1592:

> warning: Excess function parameter 'sectionp' description in

> 'mtd_ooblayout_find_eccregion'

> 

> Cc: Miquel Raynal <miquel.raynal@bootlin.com>

> Cc: Richard Weinberger <richard@nod.at>

> Cc: Vignesh Raghavendra <vigneshr@ti.com>

> Cc: David Woodhouse <dwmw2@infradead.org>

> Cc: linux-mtd@lists.infradead.org

> Signed-off-by: Lee Jones <lee.jones@linaro.org>

> ---

>  drivers/mtd/mtdcore.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c

> index e9e163ae9d863..31d1be4b22617 100644

> --- a/drivers/mtd/mtdcore.c

> +++ b/drivers/mtd/mtdcore.c

> @@ -1578,7 +1578,7 @@ static int mtd_ooblayout_find_region(struct mtd_info

> *mtd, int byte, *				  ECC byte

>   * @mtd: mtd info structure

>   * @eccbyte: the byte we are searching for

> - * @sectionp: pointer where the section id will be stored

> + * @section: pointer where the section id will be stored

>   * @oobregion: OOB region information

>   *

>   * Works like mtd_ooblayout_find_region() except it searches for a specific

> ECC


Reviewed-by: Alexander Dahl <ada@thorsis.com>


Greets
Alex




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

Patch

diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c
index e9e163ae9d863..31d1be4b22617 100644
--- a/drivers/mtd/mtdcore.c
+++ b/drivers/mtd/mtdcore.c
@@ -1578,7 +1578,7 @@  static int mtd_ooblayout_find_region(struct mtd_info *mtd, int byte,
  *				  ECC byte
  * @mtd: mtd info structure
  * @eccbyte: the byte we are searching for
- * @sectionp: pointer where the section id will be stored
+ * @section: pointer where the section id will be stored
  * @oobregion: OOB region information
  *
  * Works like mtd_ooblayout_find_region() except it searches for a specific ECC