[PATCH v3 03/14] mtd: rawnand: mxc: rename the local ->set/get_features() implementation

Boris Brezillon boris.brezillon at bootlin.com
Fri Mar 2 14:38:03 PST 2018


On Fri,  2 Mar 2018 15:24:11 +0100
Miquel Raynal <miquel.raynal at bootlin.com> wrote:

> MXC NAND driver embeds its own implementation of the
> ->set/get_features() hooks. These hooks were once flagged "onfi" but a  
> lot of not-ONFI compliant NAND chips use it so they have been renamed
> to remove the "onfi" relationship. Rename these functions the same way
> for consistency.

Should be squashed in patch 2 IMO.

> 
> Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
> ---
>  drivers/mtd/nand/raw/mxc_nand.c | 14 ++++++--------
>  1 file changed, 6 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/mtd/nand/raw/mxc_nand.c b/drivers/mtd/nand/raw/mxc_nand.c
> index d331305f45c1..61501654e708 100644
> --- a/drivers/mtd/nand/raw/mxc_nand.c
> +++ b/drivers/mtd/nand/raw/mxc_nand.c
> @@ -1421,9 +1421,8 @@ static void mxc_nand_command(struct mtd_info *mtd, unsigned command,
>  	}
>  }
>  
> -static int mxc_nand_onfi_set_features(struct mtd_info *mtd,
> -				      struct nand_chip *chip, int addr,
> -				      u8 *subfeature_param)
> +static int mxc_nand_set_features(struct mtd_info *mtd, struct nand_chip *chip,
> +				 int addr, u8 *subfeature_param)
>  {
>  	struct nand_chip *nand_chip = mtd_to_nand(mtd);
>  	struct mxc_nand_host *host = nand_get_controller_data(nand_chip);
> @@ -1447,9 +1446,8 @@ static int mxc_nand_onfi_set_features(struct mtd_info *mtd,
>  	return 0;
>  }
>  
> -static int mxc_nand_onfi_get_features(struct mtd_info *mtd,
> -				      struct nand_chip *chip, int addr,
> -				      u8 *subfeature_param)
> +static int mxc_nand_get_features(struct mtd_info *mtd, struct nand_chip *chip,
> +				 int addr, u8 *subfeature_param)
>  {
>  	struct nand_chip *nand_chip = mtd_to_nand(mtd);
>  	struct mxc_nand_host *host = nand_get_controller_data(nand_chip);
> @@ -1752,8 +1750,8 @@ static int mxcnd_probe(struct platform_device *pdev)
>  	this->read_word = mxc_nand_read_word;
>  	this->write_buf = mxc_nand_write_buf;
>  	this->read_buf = mxc_nand_read_buf;
> -	this->set_features = mxc_nand_onfi_set_features;
> -	this->get_features = mxc_nand_onfi_get_features;
> +	this->set_features = mxc_nand_set_features;
> +	this->get_features = mxc_nand_get_features;
>  
>  	host->clk = devm_clk_get(&pdev->dev, NULL);
>  	if (IS_ERR(host->clk))



-- 
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com



More information about the linux-mtd mailing list