From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Date: Sun, 28 Sep 2014 20:10:51 +0200 Subject: [U-Boot] [linux-sunxi] Re: [PATCH v2 10/10] ARM: sun6i: Add Colombus board defconfig In-Reply-To: <1411921238.17796.7.camel@hellion.org.uk> References: <1411545673-5591-1-git-send-email-wens@csie.org> <1411545673-5591-11-git-send-email-wens@csie.org> <1411672141.3824.28.camel@hellion.org.uk> <1411918437.17796.2.camel@hellion.org.uk> <54282C03.70306@redhat.com> <1411921238.17796.7.camel@hellion.org.uk> Message-ID: <54284F2B.9060106@redhat.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 >>>> >>>>> --- >>>>> 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 >>> >>> 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