[U-Boot] [linux-sunxi] Re: [PATCH v2 10/10] ARM: sun6i: Add Colombus board defconfig

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 09/28/2014 06:20 PM, Ian Campbell wrote:
> On Sun, 2014-09-28 at 17:40 +0200, Hans de Goede wrote:
>> Hi Ian,
>>
>> On 09/28/2014 05:33 PM, Ian Campbell wrote:
>>> On Thu, 2014-09-25 at 20:09 +0100, Ian Campbell wrote:
>>>> On Wed, 2014-09-24 at 16:01 +0800, Chen-Yu Tsai wrote:
>>>>> The Colombus board is an A31 evaluation board from WITS Technology.
>>>>>
>>>>> Signed-off-by: Chen-Yu Tsai <wens@csie.org>
>>>>
>>>>> ---
>>>>>  configs/Colombus_defconfig | 4 ++++
>>>>>  1 file changed, 4 insertions(+)
>>>>>  create mode 100644 configs/Colombus_defconfig
>>>>>
>>>>> diff --git a/configs/Colombus_defconfig b/configs/Colombus_defconfig
>>>>> new file mode 100644
>>>>> index 0000000..16800de
>>>>> --- /dev/null
>>>>> +++ b/configs/Colombus_defconfig
>>>>> @@ -0,0 +1,4 @@
>>>>> +CONFIG_SYS_EXTRA_OPTIONS="COLOMBUS"
>>>>
>>>> Does this do anything other than define an unused #define?
>>>>
>>>> Ah, I suppose eventually it will cause the inclusion of a suitable dram
>>>> file. Really ought to start moving things out of SYS_EXTRA though, but I
>>>> don't think you need to shave that yakk just to get this patch in, so:
>>>>
>>>> Acked-by: Ian Campbell <ijc@hellion.org.uk>
>>>
>>> Although I've just noticed that lacks a board/sunxi/MAINTAINERS entry.
>>>
>>> I think there is no need to resend the whole series, just this one
>>> patch. With this minor tweak I think it's time add this to
>>> u-boot-sunxi#next.
>>
>> Before you do that, note that I've just added 2 patches there, which I would
>> like to get into v2014.10. Specifically I'm hoping that I can get some
>> positive testing feedback on the bananapi gmac patch I've send (off-list),
>> and I believe we really should try to get the bananapi fix into v2014.10,
>> and if we're going todo a pull-req for v2014.10, we might as well include
>> the 2 patches I've just added to next. Do you agree ?
> 
> You mean these two?
>         sun7i: Add support for Olimex A20-OLinuXino-LIME2
>         mmc: sunxi: add SDHC support for sun6i/sun7i/sun8i

Yes.

> The latter seems like a feature to me, or at least the changelog doesn't
> give any rationale why it should go in now rather than waiting for the
> next merge window (i.e. why it's a bugfix, what the upside is to justify
> its inclusion now). How much testing has it had and what are the
> potential downsides?

AFAIK the downside is that High Capacity cards will not work without it.

Looking at the code if this bit is set, then for some commands
drivers/mmc/mmc.c or-s in OCR_HCS into the mmc cmdarg, so I guess you're
right that this may cause some undesirable side effects, so lets delay
this one.

> WRT the new board (and new boards generally), I'm in two minds. On the
> one hand they are pretty low risk (can't regress anything else, at least
> not in this case), on the other we are 6 weeks past the close of the
> merge window and 2 from the release date, so we are pretty far along.
> Where do we draw the line?

Normally I would not include new boards at this moment in the cycle, but
since we need to do a pull-req for the gmac anyways I thought it would
be nice to have it included, esp. since many distros only spin things
like sdcard boot images once, so if we do not include it now, many distros
will not get it for a significant amount of time.

Either way let me know how you want to proceed, if you think we should not
include this, then I'll send a pull-req with only the gmac fix.

> The gmac fix is a clear bug fix and once it is properly posted publicly
> I will ack and then I agree it should go in.

I was hoping for Stephen to get around to testing it today, and then I wanted
to send it out with his Tested-by. I'll just go and send it as is for now.

>> Still feel free to merge the sun6i series into next, I can just cherry pick
>> the 3 patches in question directly into master when I'm ready to send the
>> pull-req.
> 
> Ack, that's what I expected to happen.

Regards,

Hans