Re: [PATCH 3/8] Drivers:MTD:Nand: remove the redefinition error

[Boris Brezillon <boris.brezillon@free-electrons.com>]

Hi Venkat,

On Fri, 15 Dec 2017 09:27:09 +0530
venkat.prashanth2498@gmail.com wrote:

> From: Venkat Prashanth B U <venkat.prashanth2498@gmail.com>
> 
> This is a patch to the omap_elm.c file which fixes the redefinition
> error in the function‘elm_decode_bch_error_page’

You obviously don't know what you're doing. The symbol is not
redefined, it's just that we have a dummy implementation when
CONFIG_MTD_NAND_OMAP_BCH is disabled, which is perfectly valid,
since the omap_elm.c file is not compiled is this case.

And you also posted a patch without even trying to compile it, otherwise
you would have noticed that you're breaking the build when
CONFIG_MTD_NAND_OMAP_BCH is enabled.

So please, next time, think twice before sending this kind of 'fix'.

Thanks,

Boris

> 
> Signed-off-by: Venkat Prashanth B U <venkat.prashanth2498@gmail.com>
> ---
>  drivers/mtd/nand/omap_elm.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/mtd/nand/omap_elm.c b/drivers/mtd/nand/omap_elm.c
> index 235ec79..06c24f3 100644
> --- a/drivers/mtd/nand/omap_elm.c
> +++ b/drivers/mtd/nand/omap_elm.c
> @@ -335,7 +335,7 @@ static void elm_error_correction(struct elm_info *info,
>  }
> 
>  /**
> - * elm_decode_bch_error_page - Locate error position
> + * elm_decode_bch_err_pg - Locate error position
>   * @dev:	device pointer
>   * @ecc_calc:	calculated ECC bytes from GPMC
>   * @err_vec:	elm error vectors
> @@ -343,7 +343,7 @@ static void elm_error_correction(struct elm_info *info,
>   * Called with one or more error reported vectors & vectors with
>   * error reported is updated in err_vec[].error_reported
>   */
> -void elm_decode_bch_error_page(struct device *dev, u8 *ecc_calc,
> +void elm_decode_bch_err_pg(struct device *dev, u8 *ecc_calc,
>  		struct elm_errorvec *err_vec)
>  {
>  	struct elm_info *info = dev_get_drvdata(dev);
> @@ -368,7 +368,7 @@ void elm_decode_bch_error_page(struct device *dev, u8 *ecc_calc,
>  	elm_write_reg(info, ELM_IRQENABLE, reg_val & ~INTR_EN_PAGE_MASK);
>  	elm_error_correction(info, err_vec);
>  }
> -EXPORT_SYMBOL(elm_decode_bch_error_page);
> +EXPORT_SYMBOL(elm_decode_bch_err_pg);
> 
>  static irqreturn_t elm_isr(int this_irq, void *dev_id)
>  {
> --
> 1.9.1
> 
> 
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/