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

Dear Alex,
in message <1125386352.431408702216a@webmail.televes.com:443> you wrote:
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:
Signed-off-by: Thomas Schaefer thomas.schaefer@kontron.com Signed-off-by: Alexandre Bastos alebas@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] ...
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?
Best regards,
Wolfgang Denk

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@kontron.com Signed-off-by: Alexandre Bastos alebas@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
participants (2)
-
Alex BASTOS
-
Wolfgang Denk