linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: khilman@ti.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/5] OMAP3EVM: Board bug fixes/enhancements
Date: Thu, 27 Jan 2011 12:05:13 -0800	[thread overview]
Message-ID: <87lj26kt8m.fsf@ti.com> (raw)
In-Reply-To: <19F8576C6E063C45BE387C64729E739404BD7BE82E@dbde02.ent.ti.com> (Vaibhav Hiremath's message of "Fri, 28 Jan 2011 00:16:07 +0530")

Hi Vaibhav,

"Hiremath, Vaibhav" <hvaibhav@ti.com> writes:

[...]

>> 
>> >   OMAP3EVM: Made backlight GPIO default state to off
>> >   OMAP3EVM: Set TSC wakeup option in pad config
>> 
>> And this one doesn't apply as it seems to apply on top of code that
>> doesn't exist upstream.
>> 
>
> You should first apply Ethernet-reset patch first, and then all
> patches will get applied cleanly. I have tested it here at my end.

OK, sorry to have missed that one.

In the future, dependencies need to be clearly stated in the patch 0
cover letter.  When reviewers/maintainers are looking at lots of
patches, it is not easy to keep context for all the patches and series
arriving on the list.  Additional information such as dependencies and
change history are extremely valuable to reviewers & maintainers.

> Since you already acked Ethernet patch, I thought it will make its way
> to mainline before all these patches. And also while posting, I posted
> Ethernet patch again first and then this series.

It will likely make it's way upstream sooner since it's a regression
fix, however, it is not yet queued anywhere so dependencies on it should
be clearly stated here.

Thanks,

Kevin

      reply	other threads:[~2011-01-27 20:05 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-25 17:37 [PATCH 0/5] OMAP3EVM: Board bug fixes/enhancements Vaibhav Hiremath
2011-01-25 17:37 ` [PATCH 1/5] OMAP:board-omap3evm: Change TWL related gpio API's to gpio*_cansleep Vaibhav Hiremath
2011-01-28  7:56   ` Varadarajan, Charulatha
2011-01-25 17:37 ` [PATCH 2/5] OMAP3EVM: Add vio regulator supply required for ads7846 TSC driver Vaibhav Hiremath
2011-01-25 17:37 ` [PATCH 3/5] AM/DM37x: DSS mux configuration for >Rev-B processor cards Vaibhav Hiremath
2011-01-25 17:37 ` [PATCH 4/5] OMAP3EVM: Made backlight GPIO default state to off Vaibhav Hiremath
2011-01-25 17:37 ` [PATCH 5/5] OMAP3EVM: Set TSC wakeup option in pad config Vaibhav Hiremath
2011-01-27 18:42   ` Kevin Hilman
2011-02-17 22:19     ` Tony Lindgren
2011-02-22  6:01       ` Hiremath, Vaibhav
2011-01-27 18:41 ` [PATCH 0/5] OMAP3EVM: Board bug fixes/enhancements Kevin Hilman
2011-01-27 18:46   ` Hiremath, Vaibhav
2011-01-27 20:05     ` Kevin Hilman [this message]

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=87lj26kt8m.fsf@ti.com \
    --to=khilman@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).