All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sourav Poddar <sourav.poddar@ti.com>
To: Kamlakant Patel <kamlakant.patel@broadcom.com>
Cc: rnayak@ti.com, linux-kernel@vger.kernel.org, balbi@ti.com,
	broonie@kernel.org, linux-mtd@lists.infradead.org,
	tqnguyen@micron.com, grant.likely@linaro.org,
	spi-devel-general@lists.sourceforge.net,
	linux-omap@vger.kernel.org, dwmw2@infradead.org,
	manonuevo@micron.com
Subject: Re: [PATCH 1/3] drivers: mtd: spinand: Add generic spinand frameowrk and micron driver.
Date: Thu, 27 Jun 2013 10:21:21 +0530	[thread overview]
Message-ID: <51CBC4C9.7080809@ti.com> (raw)
In-Reply-To: <20130626152243.GA9414@kpatel.netlogicmircro.com>

Hi Kamlakant,
On Wednesday 26 June 2013 08:52 PM, Kamlakant Patel wrote:
> On Wed, Jun 26, 2013 at 01:11:10PM +0530, Sourav Poddar wrote:
>> From: Mona Anonuevo<manonuevo@micron.com>
>>
>> This patch adds support for a generic spinand framework(spinand_mtd.c).
>> This frameowrk can be used for other spi based flash devices also. The idea
>> is to have a common model under drivers/mtd, as also present for other no spi
>> devices(there is a generic framework and device part simply attaches itself to it.)
>>
>> The generic frework will be used later by me for a SPI based spansion S25FL256 device.
>> The patch also contains a micron driver attaching itself to generic framework.
>>
>> Signed-off-by: Mona Anonuevo<manonuevo@micron.com>
>> Signed-off-by: Tuan Nguyen<tqnguyen@micron.com>
>> Signed-off-by: Sourav Poddar<sourav.poddar@ti.com>
>> ----
>> [I picked this as a standalone patch, can split it into generic and device part
>> based on community feedback.]
>>
>>   drivers/mtd/Kconfig               |    2 +
>>   drivers/mtd/Makefile              |    2 +
>>   drivers/mtd/spinand/Kconfig       |   24 ++
>>   drivers/mtd/spinand/Makefile      |   10 +
>>   drivers/mtd/spinand/spinand_lld.c |  776 +++++++++++++++++++++++++++++++++++++
>>   drivers/mtd/spinand/spinand_mtd.c |  690 +++++++++++++++++++++++++++++++++
>>   include/linux/mtd/spinand.h       |  155 ++++++++
>>   7 files changed, 1659 insertions(+), 0 deletions(-)
>>   create mode 100644 drivers/mtd/spinand/Kconfig
>>   create mode 100644 drivers/mtd/spinand/Makefile
>>   create mode 100644 drivers/mtd/spinand/spinand_lld.c
>>   create mode 100644 drivers/mtd/spinand/spinand_mtd.c
>>   create mode 100644 include/linux/mtd/spinand.h
>>
> I am working on Micron SPINAND(Micron MT29F1G01ZACH). I tried this patch, but it's not working.
> It is throwing following error message while mounting:
> [  260.232000] jffs2: cannot read OOB for EB at 00000000, requested 8 bytes, read 0 bytes, error -22
> mount: mounting /dev/mtdblock5 on /mnt/ failed: Input/output error
>
> I am working on it to fix into the driver, will send an updated patch with the fix.
>
Thanks for replying on this.
Since, this patch is already posted, I think it will be better if you post
the delta fix on top of this patch.
> Thanks,
> Kamlakant Patel
>

WARNING: multiple messages have this Message-ID (diff)
From: Sourav Poddar <sourav.poddar-l0cyMroinI0@public.gmane.org>
To: Kamlakant Patel
	<kamlakant.patel-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
Cc: rnayak-l0cyMroinI0@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	balbi-l0cyMroinI0@public.gmane.org,
	broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	tqnguyen-AL4WhLSQfzjQT0dZR+AlfA@public.gmane.org,
	grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
	linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org,
	manonuevo-AL4WhLSQfzjQT0dZR+AlfA@public.gmane.org
Subject: Re: [PATCH 1/3] drivers: mtd: spinand: Add generic spinand frameowrk and micron driver.
Date: Thu, 27 Jun 2013 10:21:21 +0530	[thread overview]
Message-ID: <51CBC4C9.7080809@ti.com> (raw)
In-Reply-To: <20130626152243.GA9414-+J/lxNz7zLijYOuY2elzOeoJsYJ2OyzEAL8bYrjMMd8@public.gmane.org>

Hi Kamlakant,
On Wednesday 26 June 2013 08:52 PM, Kamlakant Patel wrote:
> On Wed, Jun 26, 2013 at 01:11:10PM +0530, Sourav Poddar wrote:
>> From: Mona Anonuevo<manonuevo-AL4WhLSQfzjQT0dZR+AlfA@public.gmane.org>
>>
>> This patch adds support for a generic spinand framework(spinand_mtd.c).
>> This frameowrk can be used for other spi based flash devices also. The idea
>> is to have a common model under drivers/mtd, as also present for other no spi
>> devices(there is a generic framework and device part simply attaches itself to it.)
>>
>> The generic frework will be used later by me for a SPI based spansion S25FL256 device.
>> The patch also contains a micron driver attaching itself to generic framework.
>>
>> Signed-off-by: Mona Anonuevo<manonuevo-AL4WhLSQfzjQT0dZR+AlfA@public.gmane.org>
>> Signed-off-by: Tuan Nguyen<tqnguyen-AL4WhLSQfzjQT0dZR+AlfA@public.gmane.org>
>> Signed-off-by: Sourav Poddar<sourav.poddar-l0cyMroinI0@public.gmane.org>
>> ----
>> [I picked this as a standalone patch, can split it into generic and device part
>> based on community feedback.]
>>
>>   drivers/mtd/Kconfig               |    2 +
>>   drivers/mtd/Makefile              |    2 +
>>   drivers/mtd/spinand/Kconfig       |   24 ++
>>   drivers/mtd/spinand/Makefile      |   10 +
>>   drivers/mtd/spinand/spinand_lld.c |  776 +++++++++++++++++++++++++++++++++++++
>>   drivers/mtd/spinand/spinand_mtd.c |  690 +++++++++++++++++++++++++++++++++
>>   include/linux/mtd/spinand.h       |  155 ++++++++
>>   7 files changed, 1659 insertions(+), 0 deletions(-)
>>   create mode 100644 drivers/mtd/spinand/Kconfig
>>   create mode 100644 drivers/mtd/spinand/Makefile
>>   create mode 100644 drivers/mtd/spinand/spinand_lld.c
>>   create mode 100644 drivers/mtd/spinand/spinand_mtd.c
>>   create mode 100644 include/linux/mtd/spinand.h
>>
> I am working on Micron SPINAND(Micron MT29F1G01ZACH). I tried this patch, but it's not working.
> It is throwing following error message while mounting:
> [  260.232000] jffs2: cannot read OOB for EB at 00000000, requested 8 bytes, read 0 bytes, error -22
> mount: mounting /dev/mtdblock5 on /mnt/ failed: Input/output error
>
> I am working on it to fix into the driver, will send an updated patch with the fix.
>
Thanks for replying on this.
Since, this patch is already posted, I think it will be better if you post
the delta fix on top of this patch.
> Thanks,
> Kamlakant Patel
>


------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev

WARNING: multiple messages have this Message-ID (diff)
From: Sourav Poddar <sourav.poddar@ti.com>
To: Kamlakant Patel <kamlakant.patel@broadcom.com>
Cc: <linux-mtd@lists.infradead.org>,
	<spi-devel-general@lists.sourceforge.net>, <broonie@kernel.org>,
	<dwmw2@infradead.org>, <manonuevo@micron.com>,
	<tqnguyen@micron.com>, <grant.likely@linaro.org>,
	<linux-omap@vger.kernel.org>, <rnayak@ti.com>,
	<linux-kernel@vger.kernel.org>, <balbi@ti.com>
Subject: Re: [PATCH 1/3] drivers: mtd: spinand: Add generic spinand frameowrk and micron driver.
Date: Thu, 27 Jun 2013 10:21:21 +0530	[thread overview]
Message-ID: <51CBC4C9.7080809@ti.com> (raw)
In-Reply-To: <20130626152243.GA9414@kpatel.netlogicmircro.com>

Hi Kamlakant,
On Wednesday 26 June 2013 08:52 PM, Kamlakant Patel wrote:
> On Wed, Jun 26, 2013 at 01:11:10PM +0530, Sourav Poddar wrote:
>> From: Mona Anonuevo<manonuevo@micron.com>
>>
>> This patch adds support for a generic spinand framework(spinand_mtd.c).
>> This frameowrk can be used for other spi based flash devices also. The idea
>> is to have a common model under drivers/mtd, as also present for other no spi
>> devices(there is a generic framework and device part simply attaches itself to it.)
>>
>> The generic frework will be used later by me for a SPI based spansion S25FL256 device.
>> The patch also contains a micron driver attaching itself to generic framework.
>>
>> Signed-off-by: Mona Anonuevo<manonuevo@micron.com>
>> Signed-off-by: Tuan Nguyen<tqnguyen@micron.com>
>> Signed-off-by: Sourav Poddar<sourav.poddar@ti.com>
>> ----
>> [I picked this as a standalone patch, can split it into generic and device part
>> based on community feedback.]
>>
>>   drivers/mtd/Kconfig               |    2 +
>>   drivers/mtd/Makefile              |    2 +
>>   drivers/mtd/spinand/Kconfig       |   24 ++
>>   drivers/mtd/spinand/Makefile      |   10 +
>>   drivers/mtd/spinand/spinand_lld.c |  776 +++++++++++++++++++++++++++++++++++++
>>   drivers/mtd/spinand/spinand_mtd.c |  690 +++++++++++++++++++++++++++++++++
>>   include/linux/mtd/spinand.h       |  155 ++++++++
>>   7 files changed, 1659 insertions(+), 0 deletions(-)
>>   create mode 100644 drivers/mtd/spinand/Kconfig
>>   create mode 100644 drivers/mtd/spinand/Makefile
>>   create mode 100644 drivers/mtd/spinand/spinand_lld.c
>>   create mode 100644 drivers/mtd/spinand/spinand_mtd.c
>>   create mode 100644 include/linux/mtd/spinand.h
>>
> I am working on Micron SPINAND(Micron MT29F1G01ZACH). I tried this patch, but it's not working.
> It is throwing following error message while mounting:
> [  260.232000] jffs2: cannot read OOB for EB at 00000000, requested 8 bytes, read 0 bytes, error -22
> mount: mounting /dev/mtdblock5 on /mnt/ failed: Input/output error
>
> I am working on it to fix into the driver, will send an updated patch with the fix.
>
Thanks for replying on this.
Since, this patch is already posted, I think it will be better if you post
the delta fix on top of this patch.
> Thanks,
> Kamlakant Patel
>


  reply	other threads:[~2013-06-27  4:52 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-26  7:41 [PATCH 0/3] spi/mtd generic framework, ti qspi controller and spansion driver Sourav Poddar
2013-06-26  7:41 ` [PATCH 0/3] spi/mtd generic framework,ti " Sourav Poddar
2013-06-26  7:41 ` [PATCH 0/3] spi/mtd generic framework, ti " Sourav Poddar
2013-06-26  7:41 ` Sourav Poddar
2013-06-26  7:41 ` [PATCH 1/3] drivers: mtd: spinand: Add generic spinand frameowrk and micron driver Sourav Poddar
2013-06-26  7:41   ` Sourav Poddar
2013-06-26  7:41   ` Sourav Poddar
2013-06-26  7:41   ` Sourav Poddar
2013-06-26 14:15   ` Florian Fainelli
2013-06-26 14:15     ` Florian Fainelli
2013-06-26 15:22   ` Kamlakant Patel
2013-06-26 15:22     ` Kamlakant Patel
2013-06-26 15:22     ` Kamlakant Patel
2013-06-27  4:51     ` Sourav Poddar [this message]
2013-06-27  4:51       ` Sourav Poddar
2013-06-27  4:51       ` Sourav Poddar
2013-07-01  5:17   ` Sourav Poddar
2013-07-01  5:17     ` Sourav Poddar
2013-07-01  5:17     ` Sourav Poddar
2013-07-01  5:17     ` Sourav Poddar
2013-06-26  7:41 ` [PATCH 2/3] drivers: spi: Add qspi flash controller Sourav Poddar
2013-06-26  7:41   ` Sourav Poddar
2013-06-26  7:41   ` Sourav Poddar
2013-06-26  7:41   ` Sourav Poddar
2013-07-01  5:19   ` Sourav Poddar
2013-07-01  5:19     ` Sourav Poddar
2013-07-01  5:19     ` Sourav Poddar
2013-07-01  5:19     ` Sourav Poddar
2013-07-01 10:56   ` Mark Brown
2013-07-01 10:56     ` Mark Brown
2013-07-01 10:56     ` Mark Brown
2013-07-01 11:15     ` Sourav Poddar
2013-07-01 11:15       ` Sourav Poddar
2013-07-01 11:15       ` Sourav Poddar
2013-06-26  7:41 ` [PATCH 3/3] drivers: mtd: spinand: Add qspi spansion " Sourav Poddar
2013-06-26  7:41   ` Sourav Poddar
2013-06-26  7:41   ` Sourav Poddar
2013-06-26  7:41   ` Sourav Poddar
2013-07-01  5:18   ` Sourav Poddar
2013-07-01  5:18     ` Sourav Poddar
2013-07-01  5:18     ` Sourav Poddar
2013-07-01  5:18     ` Sourav Poddar
2013-07-01  5:17 ` [PATCH 0/3] spi/mtd generic framework,ti qspi controller and spansion driver Sourav Poddar
2013-07-01  5:17   ` Sourav Poddar
2013-07-01  5:17   ` Sourav Poddar
2013-07-01  5:17   ` Sourav Poddar

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=51CBC4C9.7080809@ti.com \
    --to=sourav.poddar@ti.com \
    --cc=balbi@ti.com \
    --cc=broonie@kernel.org \
    --cc=dwmw2@infradead.org \
    --cc=grant.likely@linaro.org \
    --cc=kamlakant.patel@broadcom.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=manonuevo@micron.com \
    --cc=rnayak@ti.com \
    --cc=spi-devel-general@lists.sourceforge.net \
    --cc=tqnguyen@micron.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.