From: Brian Norris <computersforpeace@gmail.com>
To: linux-mtd@lists.infradead.org
Cc: linux-kernel@vger.kernel.org,
"Daniel Kurtz" <djkurtz@chromium.org>,
"Marek Vasut" <marex@denx.de>,
"Bayi Cheng" <bayi.cheng@mediatek.com>,
"Rafał Miłecki" <zajec5@gmail.com>,
"Felix Fietkau" <nbd@openwrt.org>,
"Eddie Huang (黃智傑)" <eddie.huang@mediatek.com>,
"Milton Chiang (江明晏)" <Milton.Chiang@mediatek.com>
Subject: Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)
Date: Tue, 5 Jan 2016 16:02:10 -0800 [thread overview]
Message-ID: <20160106000210.GJ109450@google.com> (raw)
In-Reply-To: <20160105022928.GF109450@google.com>
On Mon, Jan 04, 2016 at 06:29:28PM -0800, Brian Norris wrote:
> On Tue, Dec 15, 2015 at 10:48:20AM -0800, Brian Norris wrote:
> > Spansion and Winbond have occasionally used the same manufacturer ID,
> > and they don't support the same features. Particularly, writing SR=0
> > seems to break read access for Spansion's s25fl064k. Unfortunately, we
> > don't currently have a way to differentiate these Spansion and Winbond
> > parts, so rather than regressing support for these Spansion flash, let's
> > drop the new Winbond lock/unlock support for now. We can try to address
> > Winbond support during the next release cycle.
> >
> > Original discussion:
> >
> > http://patchwork.ozlabs.org/patch/549173/
> > http://patchwork.ozlabs.org/patch/553683/
> >
> > Fixes: 357ca38d4751 ("mtd: spi-nor: support lock/unlock/is_locked for Winbond")
> > Fixes: c6fc2171b249 ("mtd: spi-nor: disable protection for Winbond flash at startup")
> > Signed-off-by: Brian Norris <computersforpeace@gmail.com>
> > Reported-by: Felix Fietkau <nbd@openwrt.org>
> > Cc: Felix Fietkau <nbd@openwrt.org>
>
> Felix,
>
> Can I get a Tested-by? I'm going to send this for 4.4 still, if
> possible.
Despite the lack of response, pushed both to linux-mtd.git, as they are
obvious responses to the reported regressions/bugs.
Brian
next prev parent reply other threads:[~2016-01-06 0:02 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-15 18:48 [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond) Brian Norris
2015-12-15 18:48 ` [PATCH for-4.4 2/2] mtd: spi-nor: fix stm_is_locked_sr() parameters Brian Norris
2016-01-05 2:30 ` Brian Norris
2016-01-05 2:29 ` [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond) Brian Norris
2016-01-06 0:02 ` Brian Norris [this message]
2016-01-06 0:03 ` Felix Fietkau
2016-01-06 2:07 ` bayi cheng
2016-03-26 18:57 ` Matthias Schiffer
2016-03-27 22:52 ` Matthias Schiffer
2016-03-28 20:56 ` Brian Norris
2016-03-29 19:14 ` Matthias Schiffer
2016-03-30 12:47 ` Cyrille Pitchen
2016-04-01 3:05 ` James Cameron
2016-04-01 20:27 ` Brian Norris
2016-04-04 15:33 ` Cyrille Pitchen
2016-04-26 5:54 ` Brian Norris
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=20160106000210.GJ109450@google.com \
--to=computersforpeace@gmail.com \
--cc=Milton.Chiang@mediatek.com \
--cc=bayi.cheng@mediatek.com \
--cc=djkurtz@chromium.org \
--cc=eddie.huang@mediatek.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=marex@denx.de \
--cc=nbd@openwrt.org \
--cc=zajec5@gmail.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.