From: xiaolei li <xiaolei.li@mediatek.com>
To: "Rafał Miłecki" <zajec5@gmail.com>
Cc: "David Woodhouse" <dwmw2@infradead.org>,
"Brian Norris" <computersforpeace@gmail.com>,
"Boris Brezillon" <boris.brezillon@free-electrons.com>,
"Marek Vasut" <marek.vasut@gmail.com>,
"Richard Weinberger" <richard@nod.at>,
"Kyungmin Park" <kyungmin.park@samsung.com>,
"Miquel Raynal" <miquel.raynal@bootlin.com>,
linux-mtd@lists.infradead.org, "Rafał Miłecki" <rafal@milecki.pl>
Subject: Re: [PATCH 1/4] mtd: rawnand: use mtd_device_register() where applicable
Date: Mon, 16 Jul 2018 11:55:09 +0800 [thread overview]
Message-ID: <1531713309.10234.7.camel@mhfsdcap03> (raw)
In-Reply-To: <20180713092734.16408-2-zajec5@gmail.com>
On Fri, 2018-07-13 at 11:27 +0200, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> If driver doesn't specify parsers it can use that little helper.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
> drivers/mtd/nand/raw/cmx270_nand.c | 4 ++--
> drivers/mtd/nand/raw/cs553x_nand.c | 3 +--
> drivers/mtd/nand/raw/davinci_nand.c | 3 +--
> drivers/mtd/nand/raw/jz4740_nand.c | 5 ++---
> drivers/mtd/nand/raw/mtk_nand.c | 2 +-
> drivers/mtd/nand/raw/s3c2410.c | 4 ++--
> drivers/mtd/nand/raw/txx9ndfmc.c | 2 +-
> 7 files changed, 10 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/mtd/nand/raw/cmx270_nand.c b/drivers/mtd/nand/raw/cmx270_nand.c
> index 02d6751e9efe..b66e254b6802 100644
> --- a/drivers/mtd/nand/raw/cmx270_nand.c
> +++ b/drivers/mtd/nand/raw/cmx270_nand.c
> @@ -200,8 +200,8 @@ static int __init cmx270_init(void)
> }
>
> /* Register the partitions */
> - ret = mtd_device_parse_register(cmx270_nand_mtd, NULL, NULL,
> - partition_info, NUM_PARTITIONS);
> + ret = mtd_device_register(cmx270_nand_mtd, partition_info,
> + NUM_PARTITIONS);
> if (ret)
> goto err_scan;
>
> diff --git a/drivers/mtd/nand/raw/cs553x_nand.c b/drivers/mtd/nand/raw/cs553x_nand.c
> index 82269fde9e66..beafad62e7d5 100644
> --- a/drivers/mtd/nand/raw/cs553x_nand.c
> +++ b/drivers/mtd/nand/raw/cs553x_nand.c
> @@ -310,8 +310,7 @@ static int __init cs553x_init(void)
> for (i = 0; i < NR_CS553X_CONTROLLERS; i++) {
> if (cs553x_mtd[i]) {
> /* If any devices registered, return success. Else the last error. */
> - mtd_device_parse_register(cs553x_mtd[i], NULL, NULL,
> - NULL, 0);
> + mtd_device_register(cs553x_mtd[i], NULL, 0);
> err = 0;
> }
> }
> diff --git a/drivers/mtd/nand/raw/davinci_nand.c b/drivers/mtd/nand/raw/davinci_nand.c
> index cd12e5abafde..451abe3ad749 100644
> --- a/drivers/mtd/nand/raw/davinci_nand.c
> +++ b/drivers/mtd/nand/raw/davinci_nand.c
> @@ -803,8 +803,7 @@ static int nand_davinci_probe(struct platform_device *pdev)
> goto err;
>
> if (pdata->parts)
> - ret = mtd_device_parse_register(mtd, NULL, NULL,
> - pdata->parts, pdata->nr_parts);
> + ret = mtd_device_register(mtd, pdata->parts, pdata->nr_parts);
> else
> ret = mtd_device_register(mtd, NULL, 0);
> if (ret < 0)
> diff --git a/drivers/mtd/nand/raw/jz4740_nand.c b/drivers/mtd/nand/raw/jz4740_nand.c
> index 613b00a9604b..e04597a764a8 100644
> --- a/drivers/mtd/nand/raw/jz4740_nand.c
> +++ b/drivers/mtd/nand/raw/jz4740_nand.c
> @@ -466,9 +466,8 @@ static int jz_nand_probe(struct platform_device *pdev)
> goto err_unclaim_banks;
> }
>
> - ret = mtd_device_parse_register(mtd, NULL, NULL,
> - pdata ? pdata->partitions : NULL,
> - pdata ? pdata->num_partitions : 0);
> + ret = mtd_device_register(mtd, pdata ? pdata->partitions : NULL,
> + pdata ? pdata->num_partitions : 0);
>
> if (ret) {
> dev_err(&pdev->dev, "Failed to add mtd device\n");
> diff --git a/drivers/mtd/nand/raw/mtk_nand.c b/drivers/mtd/nand/raw/mtk_nand.c
> index 75c845adb050..e6b14b79c8a8 100644
> --- a/drivers/mtd/nand/raw/mtk_nand.c
> +++ b/drivers/mtd/nand/raw/mtk_nand.c
> @@ -1357,7 +1357,7 @@ static int mtk_nfc_nand_chip_init(struct device *dev, struct mtk_nfc *nfc,
> if (ret)
> return ret;
>
> - ret = mtd_device_parse_register(mtd, NULL, NULL, NULL, 0);
> + ret = mtd_device_register(mtd, NULL, 0);
> if (ret) {
> dev_err(dev, "mtd parse partition error\n");
> nand_release(mtd);
Acked-by: Xiaolei Li <xiaolei.li@mediatek.com>
Thanks very much!
Xiaolei
> diff --git a/drivers/mtd/nand/raw/s3c2410.c b/drivers/mtd/nand/raw/s3c2410.c
> index 19661c5d3220..10d81f367d26 100644
> --- a/drivers/mtd/nand/raw/s3c2410.c
> +++ b/drivers/mtd/nand/raw/s3c2410.c
> @@ -802,8 +802,8 @@ static int s3c2410_nand_add_partition(struct s3c2410_nand_info *info,
>
> mtdinfo->name = set->name;
>
> - return mtd_device_parse_register(mtdinfo, NULL, NULL,
> - set->partitions, set->nr_partitions);
> + return mtd_device_register(mtdinfo, set->partitions,
> + set->nr_partitions);
> }
>
> return -ENODEV;
> diff --git a/drivers/mtd/nand/raw/txx9ndfmc.c b/drivers/mtd/nand/raw/txx9ndfmc.c
> index b567d212fe7d..fe954e2222f5 100644
> --- a/drivers/mtd/nand/raw/txx9ndfmc.c
> +++ b/drivers/mtd/nand/raw/txx9ndfmc.c
> @@ -366,7 +366,7 @@ static int __init txx9ndfmc_probe(struct platform_device *dev)
> }
> mtd->name = txx9_priv->mtdname;
>
> - mtd_device_parse_register(mtd, NULL, NULL, NULL, 0);
> + mtd_device_register(mtd, NULL, 0);
> drvdata->mtds[i] = mtd;
> }
>
next prev parent reply other threads:[~2018-07-16 3:55 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-13 9:27 [PATCH 0/4] mtd: use mtd_device_register() where applicable Rafał Miłecki
2018-07-13 9:27 ` [PATCH 1/4] mtd: rawnand: " Rafał Miłecki
2018-07-16 3:55 ` xiaolei li [this message]
2018-07-13 9:27 ` [PATCH 2/4] mtd: onenand: " Rafał Miłecki
2018-07-13 9:27 ` [PATCH 3/4] mtd: maps: " Rafał Miłecki
2018-07-13 9:27 ` [PATCH 4/4] mtd: sst25l: use mtd_device_register() Rafał Miłecki
2018-07-18 7:48 ` [PATCH 0/4] mtd: use mtd_device_register() where applicable Miquel Raynal
2018-07-18 14:37 ` Boris Brezillon
2018-07-18 18:28 ` Boris Brezillon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1531713309.10234.7.camel@mhfsdcap03 \
--to=xiaolei.li@mediatek.com \
--cc=boris.brezillon@free-electrons.com \
--cc=computersforpeace@gmail.com \
--cc=dwmw2@infradead.org \
--cc=kyungmin.park@samsung.com \
--cc=linux-mtd@lists.infradead.org \
--cc=marek.vasut@gmail.com \
--cc=miquel.raynal@bootlin.com \
--cc=rafal@milecki.pl \
--cc=richard@nod.at \
--cc=zajec5@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox