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 <zaj...@gmail.com> wrote:

> On 29 June 2015 at 07:54, L. D. Pinney <ldpin...@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 <ldpin...@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?
>
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to