[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