From: Holger Brunck <holger.brunck@keymile.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 03/14] arm/km: convert mgcoge3un target to km_kirkwood
Date: Tue, 03 Jul 2012 14:31:49 +0200 [thread overview]
Message-ID: <4FF2E635.7050303@keymile.com> (raw)
In-Reply-To: <F766E4F80769BD478052FB6533FA745D1A2FE3025B@SC-VEXCH4.marvell.com>
On 07/03/2012 01:19 PM, Prafulla Wadaskar wrote:
>
>
>> -----Original Message-----
>> From: Holger Brunck [mailto:holger.brunck at keymile.com]
>> Sent: 03 July 2012 16:08
>> To: Prafulla Wadaskar
>> Cc: u-boot at lists.denx.de; Valentin Longchamp
>> Subject: Re: [PATCH v2 03/14] arm/km: convert mgcoge3un target to
>> km_kirkwood
>>
>> Hi Prafulla,
>>
>> On 07/03/2012 10:05 AM, Prafulla Wadaskar wrote:
>>>
>>>
>>>> -----Original Message-----
>>>> From: Holger Brunck [mailto:holger.brunck at keymile.com]
>>>> Sent: 13 June 2012 19:03
>>>> To: u-boot at lists.denx.de
>>>> Cc: Holger Brunck; Valentin Longchamp; Prafulla Wadaskar
>>>> Subject: [PATCH v2 03/14] arm/km: convert mgcoge3un target to
>>>> km_kirkwood
>>>>
>>>> Use the generic header km_kirkwood.h and get rid of the
>>>> board specific header.
>>>>
>>>> Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
>>>> Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
>>>> cc: Prafulla Wadaskar <prafulla@marvell.com>
>>>> ---
>>>> changes for v2:
>>>> - rebased because of changes in other patches
>>>>
>>>> board/keymile/km_arm/km_arm.c | 9 ++--
>>>> boards.cfg | 2 +-
>>>> include/configs/km_kirkwood.h | 46 +++++++++++++++++++++
>>>> include/configs/mgcoge3un.h | 87 -----------------------------
>> ---
>>>> ---------
>>>> 4 files changed, 51 insertions(+), 93 deletions(-)
>>>> delete mode 100644 include/configs/mgcoge3un.h
>>>
>>> This patch makes sense to me since it since it shrinks overall code.
>>> Doe not have dependency in patch series, can be accepted if outside
>> the series.
>>>
>>
>> again, but there are a lot of dependencies between 01-02 and 03-04
>> because all
>> doing a lot in km_kirkwood.h. So do you have any particular
>> objections against
>> the first two patches, beside your input for the kwbimage.cfg which I
>> answered
>> in a different mail?
>
> As I suggested earlier
>
> You may have patch series for the bugfixes/improvements to the currently supported code.
>
> You may have new board support as separate patch, if those have dependency, get it addressed first.
>
> As such I do not have any objection about longer patch series but
> Generally having a long patch series requires longer time to be get pulled in.
>
so and when do you think to pull this in? AFAIK we are shortly before rc1. And I
would like to see this in, everything was published before the merge window was
closed and we have reacted on every input you gave. But we can't react on inputs
which are not clearly stated.
So how do we proceed here to get this in for v2012.07? If I summarize the
situation you don't have any particular objections against 01-08 of this series.
So should I resend these eight patches as a standalone serie to get at least
these patches in?
Regards
Holger
next prev parent reply other threads:[~2012-07-03 12:31 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-13 13:33 [U-Boot] [PATCH v2 00/14] updates for Keymile Marvell boards Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 01/14] arm/km: add kmnusa board support Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 02/14] arm/km: add kmcoge5un " Holger Brunck
2012-07-03 8:04 ` Prafulla Wadaskar
2012-07-03 9:37 ` Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 03/14] arm/km: convert mgcoge3un target to km_kirkwood Holger Brunck
2012-07-03 8:05 ` Prafulla Wadaskar
2012-07-03 10:37 ` Holger Brunck
2012-07-03 11:19 ` Prafulla Wadaskar
2012-07-03 12:31 ` Holger Brunck [this message]
2012-07-03 12:38 ` Prafulla Wadaskar
2012-07-03 13:00 ` Holger Brunck
2012-07-03 13:07 ` Prafulla Wadaskar
2012-07-03 13:43 ` Holger Brunck
2012-07-03 14:39 ` Prafulla Wadaskar
2012-07-03 18:00 ` Wolfgang Denk
2012-07-04 9:21 ` Prafulla Wadaskar
2012-07-05 5:54 ` Holger Brunck
2012-07-05 6:04 ` Prafulla Wadaskar
2012-07-05 7:15 ` Holger Brunck
2012-07-05 12:09 ` Prafulla Wadaskar
2012-07-05 13:43 ` Holger Brunck
2012-07-05 13:48 ` Prafulla Wadaskar
2012-07-05 15:44 ` Holger Brunck
2012-07-04 8:24 ` Holger Brunck
2012-07-04 9:23 ` Prafulla Wadaskar
2012-07-03 14:00 ` Detlev Zundel
2012-06-13 13:33 ` [U-Boot] [PATCH v2 04/14] arm/km: remove portl2.h and use km_kirkwood instead Holger Brunck
2012-07-03 8:12 ` Prafulla Wadaskar
2012-06-13 13:33 ` [U-Boot] [PATCH v2 05/14] arm/km: correct init of 88e6352 switch in the reset_phy function Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 06/14] arm/km: enable BOCO2 FPGA download support Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 07/14] arm/km: cleanup km_kirkwood boards Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 08/14] arm/km: redefine piggy 4 reg names to avoid conflicts Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 09/14] arm/km: add support for external switch configuration Holger Brunck
2012-06-26 15:31 ` [U-Boot] [PATCH v3 " Valentin Longchamp
2012-06-13 13:33 ` [U-Boot] [PATCH v2 10/14] arm/km: enable external switch configuration for kmnusa Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 11/14] arm/km: skip FPGA config when already configured Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 12/14] arm/km: support the 2 PCIe fpga resets Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 13/14] arm/km: add implementation for read_dip_switch Holger Brunck
2012-06-13 13:33 ` [U-Boot] [PATCH v2 14/14] arm/km: remove calls to kw_gpio_* in board_early_init_f Holger Brunck
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=4FF2E635.7050303@keymile.com \
--to=holger.brunck@keymile.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox