RE: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-15 Thread Caizhiyong
> -Original Message-
> From: Huang Shijie [mailto:b32...@freescale.com]
> Sent: Monday, December 16, 2013 11:07 AM
> To: Caizhiyong
> Cc: Huang Shijie; Brian Norris; David Woodhouse; Quyaxin;
> linux-kernel@vger.kernel.org; linux-...@lists.infradead.org; Artem Bityutskiy;
> Wanglin (Albert); David Woodhouse
> Subject: Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize 
> NAND
> 
> On Mon, Dec 16, 2013 at 02:03:54AM +, Caizhiyong wrote:
> > > >
> > > > There are a lot of 16K page size NAND, they are used in android system.
> > > >
> > > > Many NAND need read retry and data randomization, kernel does not 
> > > > support
> this
> > > feature.
> > > Brian is adding the read-retry feature.
> > >
> > > I think the data randomization should be done by the hardware.
> > > If we do it by software, it costs lot of the cpu cycles.
> > >
> > >
> > > > Kernel also does not support synchronous NAND.
> > >
> > > The synchronous NAND should be supported by the nand controller driver,
> > > not the kernel.
> >
> > Yes. Those feature should be support by NAND controller hardware and driver.
> > But the driver does not know what NAND should use those feature.
> > Perhaps, the NAND id table, should add something.
> We can get the synchronous info from the ONFI nand.
> 
> In actually, if you do not add this patch, i will change it too.
> I have several 16K nand chips on my hands now. :)

I have not found a good way to solve these problems. I expect your patch.

> 
> thanks
> Huang Shijie
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-15 Thread Huang Shijie
On Mon, Dec 16, 2013 at 02:03:54AM +, Caizhiyong wrote:
> > >
> > > There are a lot of 16K page size NAND, they are used in android system.
> > >
> > > Many NAND need read retry and data randomization, kernel does not support 
> > > this
> > feature.
> > Brian is adding the read-retry feature.
> > 
> > I think the data randomization should be done by the hardware.
> > If we do it by software, it costs lot of the cpu cycles.
> >
> > 
> > > Kernel also does not support synchronous NAND.
> > 
> > The synchronous NAND should be supported by the nand controller driver,
> > not the kernel.
> 
> Yes. Those feature should be support by NAND controller hardware and driver.
> But the driver does not know what NAND should use those feature.
> Perhaps, the NAND id table, should add something.
We can get the synchronous info from the ONFI nand.

In actually, if you do not add this patch, i will change it too.
I have several 16K nand chips on my hands now. :)

thanks
Huang Shijie


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-15 Thread Caizhiyong
> >
> > There are a lot of 16K page size NAND, they are used in android system.
> >
> > Many NAND need read retry and data randomization, kernel does not support 
> > this
> feature.
> Brian is adding the read-retry feature.
> 
> I think the data randomization should be done by the hardware.
> If we do it by software, it costs lot of the cpu cycles.
>
> 
> > Kernel also does not support synchronous NAND.
> 
> The synchronous NAND should be supported by the nand controller driver,
> not the kernel.

Yes. Those feature should be support by NAND controller hardware and driver.
But the driver does not know what NAND should use those feature.
Perhaps, the NAND id table, should add something.

Looking forward to your feedback.
Best regards.
Cai Zhiyong.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-15 Thread Caizhiyong
 
  There are a lot of 16K page size NAND, they are used in android system.
 
  Many NAND need read retry and data randomization, kernel does not support 
  this
 feature.
 Brian is adding the read-retry feature.
 
 I think the data randomization should be done by the hardware.
 If we do it by software, it costs lot of the cpu cycles.

 
  Kernel also does not support synchronous NAND.
 
 The synchronous NAND should be supported by the nand controller driver,
 not the kernel.

Yes. Those feature should be support by NAND controller hardware and driver.
But the driver does not know what NAND should use those feature.
Perhaps, the NAND id table, should add something.

Looking forward to your feedback.
Best regards.
Cai Zhiyong.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-15 Thread Huang Shijie
On Mon, Dec 16, 2013 at 02:03:54AM +, Caizhiyong wrote:
  
   There are a lot of 16K page size NAND, they are used in android system.
  
   Many NAND need read retry and data randomization, kernel does not support 
   this
  feature.
  Brian is adding the read-retry feature.
  
  I think the data randomization should be done by the hardware.
  If we do it by software, it costs lot of the cpu cycles.
 
  
   Kernel also does not support synchronous NAND.
  
  The synchronous NAND should be supported by the nand controller driver,
  not the kernel.
 
 Yes. Those feature should be support by NAND controller hardware and driver.
 But the driver does not know what NAND should use those feature.
 Perhaps, the NAND id table, should add something.
We can get the synchronous info from the ONFI nand.

In actually, if you do not add this patch, i will change it too.
I have several 16K nand chips on my hands now. :)

thanks
Huang Shijie


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-15 Thread Caizhiyong
 -Original Message-
 From: Huang Shijie [mailto:b32...@freescale.com]
 Sent: Monday, December 16, 2013 11:07 AM
 To: Caizhiyong
 Cc: Huang Shijie; Brian Norris; David Woodhouse; Quyaxin;
 linux-kernel@vger.kernel.org; linux-...@lists.infradead.org; Artem Bityutskiy;
 Wanglin (Albert); David Woodhouse
 Subject: Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize 
 NAND
 
 On Mon, Dec 16, 2013 at 02:03:54AM +, Caizhiyong wrote:
   
There are a lot of 16K page size NAND, they are used in android system.
   
Many NAND need read retry and data randomization, kernel does not 
support
 this
   feature.
   Brian is adding the read-retry feature.
  
   I think the data randomization should be done by the hardware.
   If we do it by software, it costs lot of the cpu cycles.
  
  
Kernel also does not support synchronous NAND.
  
   The synchronous NAND should be supported by the nand controller driver,
   not the kernel.
 
  Yes. Those feature should be support by NAND controller hardware and driver.
  But the driver does not know what NAND should use those feature.
  Perhaps, the NAND id table, should add something.
 We can get the synchronous info from the ONFI nand.
 
 In actually, if you do not add this patch, i will change it too.
 I have several 16K nand chips on my hands now. :)

I have not found a good way to solve these problems. I expect your patch.

 
 thanks
 Huang Shijie
 

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-13 Thread Huang Shijie
On Fri, Dec 13, 2013 at 11:10:10AM +, Caizhiyong wrote:
> 
> There are a lot of 16K page size NAND, they are used in android system.
> 
> Many NAND need read retry and data randomization, kernel does not support 
> this feature.
Brian is adding the read-retry feature.

I think the data randomization should be done by the hardware.
If we do it by software, it costs lot of the cpu cycles.


> Kernel also does not support synchronous NAND.

The synchronous NAND should be supported by the nand controller driver,
not the kernel.


thanks
Huang Shijie
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-13 Thread Caizhiyong
> On Fri, Dec 13, 2013 at 05:03:49AM +, Caizhiyong wrote:
> > From: Cai Zhiyong 
> > Date: Fri, 13 Dec 2013 12:52:46 +0800
> > Subject: [PATCH] mtd: increase max page/OOB size to support 16K pagesize 
> > NAND.
> >
> > The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes.
> > So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE.
> 
> It would help if we had nand_base/nand_ids support for TC58TEG5DCJTA
> before we try to increase NAND_MAX_PAGESIZE and NAND_MAX_OOBSIZE yet
> again. AFAICT, nand_base will not detect new, large Toshiba flash
> properly anyway.

There are a lot of 16K page size NAND, they are used in android system.

Many NAND need read retry and data randomization, kernel does not support this 
feature.
Kernel also does not support synchronous NAND.
As you know, is someone doing some work to support those feature now?

If Kernel support TC58TEG5DCJTA, the read retry and data randomization also
need be supported, it will be a big project.

> 
> > Signed-off-by: Cai Zhiyong 
> > ---
> >  include/linux/mtd/nand.h | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
> > index f3ea8da..2f0a7f2 100644
> > --- a/include/linux/mtd/nand.h
> > +++ b/include/linux/mtd/nand.h
> > @@ -56,8 +56,8 @@ extern int nand_unlock(struct mtd_info *mtd, loff_t ofs,
> uint64_t len);
> >   * is supported now. If you add a chip with bigger oobsize/page
> >   * adjust this accordingly.
> >   */
> > -#define NAND_MAX_OOBSIZE   744
> > -#define NAND_MAX_PAGESIZE  8192
> > +#define NAND_MAX_OOBSIZE   1280
> > +#define NAND_MAX_PAGESIZE  16384
> 
> It might be time to draw a line in the sand: next person who needs to
> increase MAX_{PAGE,OOB}SIZE gets the job of killing the macro? We are
> long overdue for dynamic buffer allocation.
> 
> There are two drivers that use the macros, plus nand_chip.buffers.*. The
> drivers can be weaned off by allocating a small, fixed-size temporary
> buffer for the few tasks that need it. With nand_base.c, I think we can
> get by with waiting to allocate nand_chip.buffers until after we detect
> the writesize/oobsize.
> 
> So Cai, are you up for this?

Ok, I will try update it.

> 
> >
> >  /*
> >   * Constants for hardware specific CLE/ALE/NCE function
> >
> 
> Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-13 Thread Brian Norris
On Fri, Dec 13, 2013 at 03:27:12PM +0800, Huang Shijie wrote:
> On Fri, Dec 13, 2013 at 05:03:49AM +, Caizhiyong wrote:
> > --- a/include/linux/mtd/nand.h
> > +++ b/include/linux/mtd/nand.h
> > @@ -56,8 +56,8 @@ extern int nand_unlock(struct mtd_info *mtd, loff_t ofs, 
> > uint64_t len);
> >   * is supported now. If you add a chip with bigger oobsize/page
> >   * adjust this accordingly.
> >   */
> > -#define NAND_MAX_OOBSIZE   744
> > -#define NAND_MAX_PAGESIZE  8192
> > +#define NAND_MAX_OOBSIZE   1280
> > +#define NAND_MAX_PAGESIZE  16384
> maybe it is time to remove these two macros.

It's like you read my mind :)

Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-13 Thread Brian Norris
On Fri, Dec 13, 2013 at 05:03:49AM +, Caizhiyong wrote:
> From: Cai Zhiyong 
> Date: Fri, 13 Dec 2013 12:52:46 +0800
> Subject: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND.
> 
> The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes.
> So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE.

It would help if we had nand_base/nand_ids support for TC58TEG5DCJTA
before we try to increase NAND_MAX_PAGESIZE and NAND_MAX_OOBSIZE yet
again. AFAICT, nand_base will not detect new, large Toshiba flash
properly anyway.

> Signed-off-by: Cai Zhiyong 
> ---
>  include/linux/mtd/nand.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
> index f3ea8da..2f0a7f2 100644
> --- a/include/linux/mtd/nand.h
> +++ b/include/linux/mtd/nand.h
> @@ -56,8 +56,8 @@ extern int nand_unlock(struct mtd_info *mtd, loff_t ofs, 
> uint64_t len);
>   * is supported now. If you add a chip with bigger oobsize/page
>   * adjust this accordingly.
>   */
> -#define NAND_MAX_OOBSIZE 744
> -#define NAND_MAX_PAGESIZE8192
> +#define NAND_MAX_OOBSIZE 1280
> +#define NAND_MAX_PAGESIZE16384

It might be time to draw a line in the sand: next person who needs to
increase MAX_{PAGE,OOB}SIZE gets the job of killing the macro? We are
long overdue for dynamic buffer allocation.

There are two drivers that use the macros, plus nand_chip.buffers.*. The
drivers can be weaned off by allocating a small, fixed-size temporary
buffer for the few tasks that need it. With nand_base.c, I think we can
get by with waiting to allocate nand_chip.buffers until after we detect
the writesize/oobsize.

So Cai, are you up for this?

>  
>  /*
>   * Constants for hardware specific CLE/ALE/NCE function
> 

Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-13 Thread Brian Norris
On Fri, Dec 13, 2013 at 05:03:49AM +, Caizhiyong wrote:
 From: Cai Zhiyong caizhiy...@huawei.com
 Date: Fri, 13 Dec 2013 12:52:46 +0800
 Subject: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND.
 
 The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes.
 So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE.

It would help if we had nand_base/nand_ids support for TC58TEG5DCJTA
before we try to increase NAND_MAX_PAGESIZE and NAND_MAX_OOBSIZE yet
again. AFAICT, nand_base will not detect new, large Toshiba flash
properly anyway.

 Signed-off-by: Cai Zhiyong caizhiy...@huawei.com
 ---
  include/linux/mtd/nand.h | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
 
 diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
 index f3ea8da..2f0a7f2 100644
 --- a/include/linux/mtd/nand.h
 +++ b/include/linux/mtd/nand.h
 @@ -56,8 +56,8 @@ extern int nand_unlock(struct mtd_info *mtd, loff_t ofs, 
 uint64_t len);
   * is supported now. If you add a chip with bigger oobsize/page
   * adjust this accordingly.
   */
 -#define NAND_MAX_OOBSIZE 744
 -#define NAND_MAX_PAGESIZE8192
 +#define NAND_MAX_OOBSIZE 1280
 +#define NAND_MAX_PAGESIZE16384

It might be time to draw a line in the sand: next person who needs to
increase MAX_{PAGE,OOB}SIZE gets the job of killing the macro? We are
long overdue for dynamic buffer allocation.

There are two drivers that use the macros, plus nand_chip.buffers.*. The
drivers can be weaned off by allocating a small, fixed-size temporary
buffer for the few tasks that need it. With nand_base.c, I think we can
get by with waiting to allocate nand_chip.buffers until after we detect
the writesize/oobsize.

So Cai, are you up for this?

  
  /*
   * Constants for hardware specific CLE/ALE/NCE function
 

Brian
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-13 Thread Brian Norris
On Fri, Dec 13, 2013 at 03:27:12PM +0800, Huang Shijie wrote:
 On Fri, Dec 13, 2013 at 05:03:49AM +, Caizhiyong wrote:
  --- a/include/linux/mtd/nand.h
  +++ b/include/linux/mtd/nand.h
  @@ -56,8 +56,8 @@ extern int nand_unlock(struct mtd_info *mtd, loff_t ofs, 
  uint64_t len);
* is supported now. If you add a chip with bigger oobsize/page
* adjust this accordingly.
*/
  -#define NAND_MAX_OOBSIZE   744
  -#define NAND_MAX_PAGESIZE  8192
  +#define NAND_MAX_OOBSIZE   1280
  +#define NAND_MAX_PAGESIZE  16384
 maybe it is time to remove these two macros.

It's like you read my mind :)

Brian
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-13 Thread Caizhiyong
 On Fri, Dec 13, 2013 at 05:03:49AM +, Caizhiyong wrote:
  From: Cai Zhiyong caizhiy...@huawei.com
  Date: Fri, 13 Dec 2013 12:52:46 +0800
  Subject: [PATCH] mtd: increase max page/OOB size to support 16K pagesize 
  NAND.
 
  The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes.
  So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE.
 
 It would help if we had nand_base/nand_ids support for TC58TEG5DCJTA
 before we try to increase NAND_MAX_PAGESIZE and NAND_MAX_OOBSIZE yet
 again. AFAICT, nand_base will not detect new, large Toshiba flash
 properly anyway.

There are a lot of 16K page size NAND, they are used in android system.

Many NAND need read retry and data randomization, kernel does not support this 
feature.
Kernel also does not support synchronous NAND.
As you know, is someone doing some work to support those feature now?

If Kernel support TC58TEG5DCJTA, the read retry and data randomization also
need be supported, it will be a big project.

 
  Signed-off-by: Cai Zhiyong caizhiy...@huawei.com
  ---
   include/linux/mtd/nand.h | 4 ++--
   1 file changed, 2 insertions(+), 2 deletions(-)
 
  diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
  index f3ea8da..2f0a7f2 100644
  --- a/include/linux/mtd/nand.h
  +++ b/include/linux/mtd/nand.h
  @@ -56,8 +56,8 @@ extern int nand_unlock(struct mtd_info *mtd, loff_t ofs,
 uint64_t len);
* is supported now. If you add a chip with bigger oobsize/page
* adjust this accordingly.
*/
  -#define NAND_MAX_OOBSIZE   744
  -#define NAND_MAX_PAGESIZE  8192
  +#define NAND_MAX_OOBSIZE   1280
  +#define NAND_MAX_PAGESIZE  16384
 
 It might be time to draw a line in the sand: next person who needs to
 increase MAX_{PAGE,OOB}SIZE gets the job of killing the macro? We are
 long overdue for dynamic buffer allocation.
 
 There are two drivers that use the macros, plus nand_chip.buffers.*. The
 drivers can be weaned off by allocating a small, fixed-size temporary
 buffer for the few tasks that need it. With nand_base.c, I think we can
 get by with waiting to allocate nand_chip.buffers until after we detect
 the writesize/oobsize.
 
 So Cai, are you up for this?

Ok, I will try update it.

 
 
   /*
* Constants for hardware specific CLE/ALE/NCE function
 
 
 Brian
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-13 Thread Huang Shijie
On Fri, Dec 13, 2013 at 11:10:10AM +, Caizhiyong wrote:
 
 There are a lot of 16K page size NAND, they are used in android system.
 
 Many NAND need read retry and data randomization, kernel does not support 
 this feature.
Brian is adding the read-retry feature.

I think the data randomization should be done by the hardware.
If we do it by software, it costs lot of the cpu cycles.


 Kernel also does not support synchronous NAND.

The synchronous NAND should be supported by the nand controller driver,
not the kernel.


thanks
Huang Shijie
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-12 Thread Huang Shijie
On Fri, Dec 13, 2013 at 05:03:49AM +, Caizhiyong wrote:
> From: Cai Zhiyong 
> Date: Fri, 13 Dec 2013 12:52:46 +0800
> Subject: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND.
> 
> The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes.
> So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE.
> 
> Signed-off-by: Cai Zhiyong 
> ---
>  include/linux/mtd/nand.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
> index f3ea8da..2f0a7f2 100644
> --- a/include/linux/mtd/nand.h
> +++ b/include/linux/mtd/nand.h
> @@ -56,8 +56,8 @@ extern int nand_unlock(struct mtd_info *mtd, loff_t ofs, 
> uint64_t len);
>   * is supported now. If you add a chip with bigger oobsize/page
>   * adjust this accordingly.
>   */
> -#define NAND_MAX_OOBSIZE 744
> -#define NAND_MAX_PAGESIZE8192
> +#define NAND_MAX_OOBSIZE 1280
> +#define NAND_MAX_PAGESIZE16384
maybe it is time to remove these two macros.

thanks
Huang Shijie

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-12 Thread Brian Norris
On Fri, Dec 13, 2013 at 05:01:56AM +, Caizhiyong wrote:
> The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes.
> So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE.
> 
> Signed-off-by: Huang Shijie 
> Signed-off-by: Artem Bityutskiy 
> Signed-off-by: David Woodhouse 

Huh? I hope you're not intentionally forging Signed-off-by lines...
Presumably the follow-up (re)send is because this one was an accident?

> Signed-off-by: Cai Zhiyong 

Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-12 Thread Brian Norris
On Fri, Dec 13, 2013 at 05:01:56AM +, Caizhiyong wrote:
 The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes.
 So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE.
 
 Signed-off-by: Huang Shijie b32...@freescale.com
 Signed-off-by: Artem Bityutskiy artem.bityuts...@linux.intel.com
 Signed-off-by: David Woodhouse david.woodho...@intel.com

Huh? I hope you're not intentionally forging Signed-off-by lines...
Presumably the follow-up (re)send is because this one was an accident?

 Signed-off-by: Cai Zhiyong caizhiy...@huawei.com

Brian
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-12 Thread Huang Shijie
On Fri, Dec 13, 2013 at 05:03:49AM +, Caizhiyong wrote:
 From: Cai Zhiyong caizhiy...@huawei.com
 Date: Fri, 13 Dec 2013 12:52:46 +0800
 Subject: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND.
 
 The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes.
 So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE.
 
 Signed-off-by: Cai Zhiyong caizhiy...@huawei.com
 ---
  include/linux/mtd/nand.h | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
 
 diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
 index f3ea8da..2f0a7f2 100644
 --- a/include/linux/mtd/nand.h
 +++ b/include/linux/mtd/nand.h
 @@ -56,8 +56,8 @@ extern int nand_unlock(struct mtd_info *mtd, loff_t ofs, 
 uint64_t len);
   * is supported now. If you add a chip with bigger oobsize/page
   * adjust this accordingly.
   */
 -#define NAND_MAX_OOBSIZE 744
 -#define NAND_MAX_PAGESIZE8192
 +#define NAND_MAX_OOBSIZE 1280
 +#define NAND_MAX_PAGESIZE16384
maybe it is time to remove these two macros.

thanks
Huang Shijie

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/