From: Gerhard Sittig <gsi@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v7 00/17] sf: Add support for quad and dual_flash
Date: Mon, 13 Jan 2014 10:45:55 +0100 [thread overview]
Message-ID: <20140113094555.GF20094@book.gsilab.sittig.org> (raw)
In-Reply-To: <1b0355e1-3aac-4ae8-9ac9-b57d28a91cf0@CO9EHSMHS030.ehs.local>
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).
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.
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).
virtually yours
Gerhard Sittig
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr. 5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
next prev parent reply other threads:[~2014-01-13 9:45 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-12 16:59 [U-Boot] [PATCH v7 00/17] sf: Add support for quad and dual_flash Jagannadha Sutradharudu Teki
2014-01-13 9:45 ` Gerhard Sittig [this message]
2014-01-13 11:00 ` Jagan Teki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140113094555.GF20094@book.gsilab.sittig.org \
--to=gsi@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.