public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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 15:43:17 +0200	[thread overview]
Message-ID: <4FF2F6F5.2080309@keymile.com> (raw)
In-Reply-To: <F766E4F80769BD478052FB6533FA745D1A2FE3027C@SC-VEXCH4.marvell.com>

On 07/03/2012 03:07 PM, Prafulla Wadaskar wrote:
>>>>>>>
>>>>>>> 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?
>>>
>>> Yes, Please send the patch series for bug fix and improvements ONLY,
>> I will pull them ASAP.
>>>
>>> The others we can converge soon.
>>>
>>
>> But 01-08 are not only bugfixes there are also two new boards in these
>> patches.
>> So will you pull these eight patches in if I post them again without
>> 09-14?
> 
> Pls post bug fixes and improvement patches first those will be pulled faster.
> 
This was already done from my side:
http://lists.denx.de/pipermail/u-boot/2012-May/125219.html

Sorry Prafulla but I have admit that this whole discussion costs a lot of time
which I do not have. We are moving in circles your last comment was already
discussed several times e.g.:
http://lists.denx.de/pipermail/u-boot/2012-June/126081.html

So again you have no specific inputs of improvements for 01-08 right? All you
rephrase are some general comments which are not very helpfull to improve our
situation and to get our own board code into mainline.

We try to stick to every rule which is valid for u-boot development. But we
can't stick to rules which are not clearly stated.

Regards
Holger

  reply	other threads:[~2012-07-03 13:43 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
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 [this message]
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=4FF2F6F5.2080309@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