[patch/resend] nand: add containing structure for NAND platform data

[patch/resend] nand: add containing structure for NAND platform data

[Vitaly]

In order to make it more convenient to pass platform-specific NAND stuff 
to a generic driver which is hopefully to come out next, I've introduced
a container structure for chip-specific and controller-specific data.
I'd expect that use cases in platform code will be like

struct platform_nand_data nand_data = {
	...
};

struct platform_device nand_device = {
	...
	.dev = {
		.platform_data = &nand_data,
		...
	},
};

So, please find the patch below.

 include/linux/mtd/nand.h |   10 ++++++++++
 1 files changed, 10 insertions(+)

Signed-off-by: Vitaly Wool <vitalywool@gmail.com> <mailto:vitalywool@gmail.com>

Re: [patch/resend] nand: add containing structure for NAND platform data

[Sergei Shtylyov]

Hello.

Vitaly wrote:

> In order to make it more convenient to pass platform-specific NAND stuff 
> to a generic driver which is hopefully to come out next, I've introduced
> a container structure for chip-specific and controller-specific data.
> I'd expect that use cases in platform code will be like
> 
> struct platform_nand_data nand_data = {
> 	...
> };
> 
> struct platform_device nand_device = {
> 	...
> 	.dev = {
> 		.platform_data = &nand_data,
> 		...
> 	},
> };
> 
> So, please find the patch below.
> 
>  include/linux/mtd/nand.h |   10 ++++++++++
>  1 files changed, 10 insertions(+)
> 
> Signed-off-by: Vitaly Wool <vitalywool@gmail.com> <mailto:vitalywool@gmail.com>

   Erm, I can't. ;-)

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

MBR, Sergei

Re: [patch/resend] nand: add containing structure for NAND platform data

[Vitaly Wool]

> > So, please find the patch below.
> >
> >  include/linux/mtd/nand.h |   10 ++++++++++
> >  1 files changed, 10 insertions(+)
> >
> > Signed-off-by: Vitaly Wool <vitalywool@gmail.com> <mailto:vitalywool@gmail.com>
>
>    Erm, I can't. ;-)

I did mess it up. Sorry guys. Will resend shortly.

Vitaly

[patch/resend] nand: add containing structure for NAND platform data

[Vitaly Wool]

In order to make it more convenient to pass platform-specific NAND stuff
to a generic driver which is hopefully to come out next, I've introduced
a container structure for chip-specific and controller-specific data.
I'd expect that use cases in platform code will be like

struct platform_nand_data nand_data = {
	...
};

struct platform_device nand_device = {
	...
	.dev = {
		.platform_data = &nand_data,
		...
	},
};

So, please find the patch below.

 include/linux/mtd/nand.h |   10 ++++++++++
 1 files changed, 10 insertions(+)

Index: linux-2.6/include/linux/mtd/nand.h
===================================================================
--- linux-2.6.orig/include/linux/mtd/nand.h
+++ linux-2.6/include/linux/mtd/nand.h
@@ -592,6 +592,16 @@ struct platform_nand_ctrl {
 	void		*priv;
 };
 
+/**
+ * struct platform_nand_data - container structure for platform-specific data
+ * @chip:		chip level chip structure
+ * @ctrl:		controller level device structure
+ */
+struct platform_nand_data {
+	struct platform_nand_chip	chip;
+	struct platform_nand_ctrl	ctrl;
+};
+
 /* Some helpers to access the data structures */
 static inline
 struct platform_nand_chip *get_platform_nandchip(struct mtd_info *mtd)