[OpenWrt-Devel] [PATCH] linux:generic: mtd backport for Spansion S25FL164K

L. D. Pinney ldpinney at gmail.com
Mon Jun 29 19:38:28 EDT 2015


The space was removed because "git apply" complained....I'll ignore the
complaint.
Please mark this one rejected...
I'll do better next time.
Thanks for your patience and guidance.

Larry

On Mon, Jun 29, 2015 at 2:16 AM, Rafał Miłecki <zajec5 at gmail.com> wrote:

> On 29 June 2015 at 07:54, L. D. Pinney <ldpinney at gmail.com> wrote:
> > Backport for the Spansion S25FL164K
> > It's an 8 MiB flash with 4 KiB erase sectors.
> >
> > Signed-off-by: L. D. Pinney <ldpinney at gmail.com>
>
> It looks much better but this patch doesn't seem to be refreshed, see:
>
> Applying patch
> generic/091-mtd-spi-nor-add-support-for-the-Spansion_S25FL164K
> patching file drivers/mtd/spi-nor/spi-nor.c
> Hunk #1 succeeded at 566 (offset -7 lines).
>
> Applying patch
> generic/472-mtd-m25p80-add-support-for-Winbond-W25X05-flash.patch
> patching file drivers/mtd/spi-nor/spi-nor.c
> Hunk #1 succeeded at 617 (offset 1 line).
>
> And there is some general problem with your patch, see below
>
>
> > diff --git
> a/target/linux/generic/patches-3.18/091-mtd-spi-nor-add-support-for-the-Spansion_S25FL164K
> b/target/linux/generic/patches-3.18/091-mtd-spi-nor-add-support-for-the-Spansion_S25FL164K
> > new file mode 100644
> > index 0000000..866567a
> > --- /dev/null
> > +++
> b/target/linux/generic/patches-3.18/091-mtd-spi-nor-add-support-for-the-Spansion_S25FL164K
> > @@ -0,0 +1,10 @@
> > +--- a/drivers/mtd/spi-nor/spi-nor.c
> > ++++ b/drivers/mtd/spi-nor/spi-nor.c
> > +@@ -573,6 +573,7 @@ static const struct spi_device_id spi_nor_ids[] = {
> > +       { "s25fl008k",  INFO(0xef4014,      0,  64 * 1024,  16, SECT_4K)
> },
> > +       { "s25fl016k",  INFO(0xef4015,      0,  64 * 1024,  32, SECT_4K)
> },
> > +       { "s25fl064k",  INFO(0xef4017,      0,  64 * 1024, 128, SECT_4K)
> },
> > ++      { "s25fl164k",  INFO(0x014017,      0,  64 * 1024, 128, SECT_4K)
> },
> > +
> > +       /* SST -- large erase sizes are "overlays", "sectors" are 4K */
> > +       { "sst25vf040b", INFO(0xbf258d, 0, 64 * 1024,  8, SECT_4K |
> SST_WRITE) },
>
> A diff (patch) file should have space at the beginning for every
> unmodified (reference) file. The file you added doesn't have it. How
> did you prepare this file?
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.infradead.org/pipermail/openwrt-devel/attachments/20150629/2bf585fb/attachment.htm>
-------------- next part --------------
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


More information about the openwrt-devel mailing list