[U-Boot-Users] [PATCH] AMD/Spansion write buffer support, splitted

Alex BASTOS alebas at televes.com
Thu Oct 6 15:14:21 CEST 2005


Wolfgang,

> >
> > This patch:
> >  - Adds flash_write_cfibuffer support for AMD/Spansion Flashes.
> >
> > This was splitted from previous patch.
>
> I am confused. I have two messages  from  you  (sent  from  different
> addresses, but both on Tue, 30 Aug 2005, with two different patches:
>

Yeah, this is quite confusing. First one was submitted which my
proposed changes. But after sending it I read some comments from
Yuli Barcohen, so I send the second one with the changes for those
comments (mainly, not using different command defines, but shifting
the actual command).

> > Signed-off-by: Thomas Schaefer <thomas.schaefer at kontron.com>
> > Signed-off-by: Alexandre Bastos <alebas at televes.com>
> >
> > CHANGELOG:
> > * Add support for AMD/Spansion Flashes in flash_write_cfibuffer
> >   Patch by Alex Bastos and Thomas Schaefer, 2005-08-29
> ...
> > Content-Type: text/x-patch; name="20050830_cfi_flash_A.diff"
>
>
> and
>
> [no CHANGELOG entry]

The CHANGELOG should be the same than in thefirst patch

> > It may be enough shifting the address each time it is needed
> > (in fact, only two times).
> >
> >Suggestions?
> ...
> > Content-Type: text/x-patch; name="20050830_cfi_flash_B.diff"
>
>
> Are these two patches in alternative?  Or  additional?  Or  just  for
> disussion? Or what?
>


So, Should I say alternative?. Maybe the second is most a RFC.
Sorry  about the confusing stuff.
Working with mixed environments (OS, email clients, hours, etc)


Best regards,

Alex BASTOS






More information about the U-Boot mailing list