[U-Boot] [PATCH v7 00/17] sf: Add support for quad and dual_flash
Jagan Teki
jagannadh.teki at gmail.com
Mon Jan 13 12:00:20 CET 2014
Hi,
On Mon, Jan 13, 2014 at 3:15 PM, Gerhard Sittig <gsi at denx.de> wrote:
> On Sun, Jan 12, 2014 at 22:29 +0530, Jagannadha Sutradharudu Teki wrote:
>>
>> This series is a combo of quad cmds and dual_flash support
>> Refer [1] and [2] for more functional description about this changes.
>>
>> [ ... ]
>>
>> Please let me know for any issues/concerns/questions.
>>
>> --
>> Thanks,
>> Jagan.
>>
>> Changes for v7:
>> - cmdsz on read_ops
>> - sf param flags are placed spi_flash.h
>> - code cleanup
>> - updated dual_flash with BAR support
>> [ ... ]
>
> Mail setup related nits:
>
> Please don't do this. The dash line correctly separates your
> signature, which is good (and not a given either). But putting
> more context below that line will make it look like it's part of
> the signature, and thus probably will get lost (either by
> automatic trimming, or folding, or colorization and/or mental
> filtering).
Yes - will take this input.
>
> Then I noticed that the series and your cover letter aren't
> "connected". The series should be a reply to the cover letter,
> while you seem to create and send them out in separate steps.
> Which breaks their relation, and has them appear in random order
> and with other messages in between them.
Probably the random order appearance is some issue on my host server side.
Will fix to encounter all the mails properly.
>
> And the recipient list suggests that there is an issue with
> inappropriately quoted entries, there's another nonexistent
> @theia address as has been in the past. Please fix this (I did
> upon reply, check your original message).
Try many times to fix this, any idea would appreciate.
--
Thanks,
Jagan.
More information about the U-Boot
mailing list