linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: js@sig21.net (Johannes Stezenbach)
To: linux-arm-kernel@lists.infradead.org
Subject: SPI: DUAL/QUAD support
Date: Fri, 5 Jul 2013 09:35:50 +0200	[thread overview]
Message-ID: <20130705073550.GA24258@sig21.net> (raw)
In-Reply-To: <20980858CB6D3A4BAE95CA194937D5E73E9E4C2E@DBDE04.ent.ti.com>

On Fri, Jul 05, 2013 at 06:45:01AM +0000, Gupta, Pekon wrote:
> 
> > Just as Johannes said. In my patch m25p80's changes is incomplete.
> > Because my inicial aim is to add the transfer width member to tell controller
> > the mode flash in. And I will attach new patch about m25p80 later.
> > Also you are right. There are really series of flashes do not support dual/quad
> > transfer. So I don't know whether there are any standard for serial-flash just
> > like the CFI for parallel-flash. Personally, to make a general standard and
> > a general code for serial-flash is necessary. So that we do not need to add
> > special function in m25p80.c.
> > 
> [Pekon]: Does below generic framework for spianand, suit your driver?
> http://lists.infradead.org/pipermail/linux-mtd/2013-July/047434.html
> 
> you should be able to extend the generic spinand API for all
> serial modes (SPI, Dual-SPI, Quad-SPI).
> As QSPI and Dual-SPI devices should implicitly support SPI (MISO/MOSI)
> So, you can do device probing using default SPI mode. And then based on
> DT inputs and device support upgrade to QSPI / Dual-SPI mode.
> And going forward extend it for direct memory-mapped device for XIP.

Not sure what you have in mind since NAND and NOR flash work very
differently, and SPI vs. memory-mapped is also very different.

But to add to the spinand review comments two things caught
my eye on quick glance over the code:

> +#define mu_spi_nand_driver_version "Beagle-MTD_01.00_Linux2.6.33_20100507"

seems like an unused leftover?  if the version number is
important then maybe better put it in commit message?

> +/bin/bash: 4: command not found

how did this get in there?


Johannes

  reply	other threads:[~2013-07-05  7:35 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-04 11:36 SPI: DUAL/QUAD support yuhang wang
2013-07-04 13:00 ` Johannes Stezenbach
2013-07-04 14:58   ` Thomas.Betker at rohde-schwarz.com
2013-07-04 15:49     ` Mark Brown
2013-07-04 16:04       ` Thomas.Betker at rohde-schwarz.com
2013-07-05  6:25         ` yuhang wang
2013-07-05  6:45           ` Gupta, Pekon
2013-07-05  7:35             ` Johannes Stezenbach [this message]
2013-07-05  7:41               ` Sourav Poddar
2013-07-05  8:04               ` Gupta, Pekon
2013-07-05  7:40           ` Sourav Poddar
2013-07-05  8:48             ` yuhang wang
2013-07-05  8:55               ` Sourav Poddar
2013-07-05  9:07                 ` yuhang wang
2013-07-05  9:08                   ` Sourav Poddar
2013-07-05  9:17                     ` yuhang wang
2013-07-05  9:27                       ` Sourav Poddar
2013-07-05 10:24                         ` yuhang wang
2013-07-05 14:34                           ` Johannes Stezenbach
2013-07-05 15:41                             ` yuhang wang
2013-07-04 14:36 ` Mark Brown
2013-07-04 18:06   ` Johannes Stezenbach
2013-07-04 19:12     ` Mark Brown
2013-07-05  9:41       ` yuhang wang
2013-07-05 10:12         ` Mark Brown
  -- strict thread matches above, loose matches on Subject: below --
2013-07-04  7:07 SPI : " 王宇航
2013-07-04  9:00 ` Mark Brown

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=20130705073550.GA24258@sig21.net \
    --to=js@sig21.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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;
as well as URLs for NNTP newsgroup(s).