From: David Brownell <david-b@pacbell.net>
To: Mingkai Hu <Mingkai.hu@freescale.com>,
David Woodhouse <dwmw2@infradead.org>,
Grant Likely <grant.likely@secretlab.ca>
Cc: linuxppc-dev@ozlabs.org, kumar.gala@freescale.com,
spi-devel-general@lists.sourceforge.net
Subject: Re: [spi-devel-general] [PATCH v2 4/6] mtd: m25p80: add a read function to read page by page
Date: Tue, 10 Aug 2010 06:44:03 -0700 (PDT) [thread overview]
Message-ID: <220473.37389.qm@web180310.mail.gq1.yahoo.com> (raw)
In-Reply-To: <AANLkTikM5je7Ckt=7bP6fj7OKCaroSn_AqgcS4T+Bv5d@mail.gmail.com>
=0A=0A--- On Tue, 8/10/10, Grant Likely <grant.likely@secretlab.ca> wrote:=
=0A=0A> This one bothers me, but I can't put my=0A> finger on it.=A0The fla=
g feels=0A> like a controller specific hack.=0A=0AThat's because it *IS* ..=
.=0A=0ANot clear what a good fix would look like.=0ABut in general, SPI mas=
ter controllers are=0Aresponsible for returning all bytes requested,=0Ainst=
ead of (as with this one) a subset.=0A=0ASuggesting the real issue is a bug=
gy SPI=0Amaster controller and/or driver. Can't it=0Aissue multiple reads =
to collect all the data=0Arequested?? =0A=0A- Dave=0A=0A=0A
next prev parent reply other threads:[~2010-08-10 13:44 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-02 7:51 [PATCH v2 0/6] refactor spi_mpc8xxx.c and add eSPI controller support Mingkai Hu
2010-08-02 7:51 ` [PATCH v2 1/6] spi/mpc8xxx: refactor the common code for SPI/eSPI controller Mingkai Hu
2010-08-02 7:52 ` [PATCH v2 2/6] eSPI: add eSPI controller support Mingkai Hu
2010-08-02 7:52 ` [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions Mingkai Hu
2010-08-02 7:52 ` [PATCH v2 4/6] mtd: m25p80: add a read function to read page by page Mingkai Hu
2010-08-02 7:52 ` [PATCH v2 5/6] powerpc/of: add eSPI controller dts bindings Mingkai Hu
2010-08-02 7:52 ` [PATCH v2 6/6] DTS: add SPI flash(s25fl128p01) support on p4080ds and mpc8536ds board Mingkai Hu
2010-08-10 7:10 ` Grant Likely
2010-09-02 8:30 ` Hu Mingkai-B21284
2010-08-10 7:00 ` [PATCH v2 4/6] mtd: m25p80: add a read function to read page by page Grant Likely
2010-08-10 13:44 ` David Brownell [this message]
2010-09-02 9:04 ` Hu Mingkai-B21284
2010-08-10 6:56 ` [PATCH v2 3/6] mtd: m25p80: add support to parse the SPI flash's partitions Grant Likely
2010-08-10 7:14 ` [spi-devel-general] " David Brownell
2010-08-10 7:16 ` Grant Likely
2010-08-10 6:44 ` [PATCH v2 2/6] eSPI: add eSPI controller support Grant Likely
2010-09-02 8:28 ` Hu Mingkai-B21284
2010-08-10 6:40 ` [PATCH v2 1/6] spi/mpc8xxx: refactor the common code for SPI/eSPI controller Grant Likely
2010-09-02 8:27 ` Hu Mingkai-B21284
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=220473.37389.qm@web180310.mail.gq1.yahoo.com \
--to=david-b@pacbell.net \
--cc=Mingkai.hu@freescale.com \
--cc=dwmw2@infradead.org \
--cc=grant.likely@secretlab.ca \
--cc=kumar.gala@freescale.com \
--cc=linuxppc-dev@ozlabs.org \
--cc=spi-devel-general@lists.sourceforge.net \
/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