From: York Sun <yorksun@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC] MAKEALL: Fix case substitution for old bash
Date: Mon, 1 Apr 2013 11:19:58 -0700 [thread overview]
Message-ID: <5159CFCE.3020207@freescale.com> (raw)
In-Reply-To: <CAPnjgZ2ObSo_0sMemhs2+Z407qhT5oSgwoYm46Fyb5beV+CWOA@mail.gmail.com>
On 03/30/2013 03:30 PM, Simon Glass wrote:
> +U-Boot
>
> Hi Allen, York,
>
> On Fri, Mar 29, 2013 at 11:05 AM, Allen Martin <amartin@nvidia.com
> <mailto:amartin@nvidia.com>> wrote:
>
> On Fri, Mar 29, 2013 at 10:56:33AM -0700, York Sun wrote:
> > On 03/29/2013 10:52 AM, Allen Martin wrote:
> > > On Fri, Mar 29, 2013 at 10:49:06AM -0700, York Sun wrote:
> > >> On 03/29/2013 10:29 AM, Allen Martin wrote:
> > >>> On Fri, Mar 29, 2013 at 09:56:41AM -0700, York Sun wrote:
> > >>>> On 03/27/2013 12:55 PM, Tom Rini wrote:
> > >>>>> On Thu, Mar 21, 2013 at 01:38:37PM -0700, York Sun wrote:
> > >>>>>> On 03/21/2013 01:28 PM, Allen Martin wrote:
> > >>>>> [snip]
> > >>>>>>> PS: What happened with your bool fixup patch? I'm a big
> fan of that
> > >>>>>>> patch :^)
> > >>>>>>>
> > >>>>>>
> > >>>>>> Tom asked on March 4th. I think he will accept it.
> > >>>>>
> > >>>>> I never got the expected acks on v7 that I thought would be
> forthcoming.
> > >>>>>
> > >>>>
> > >>>> Allen,
> > >>>>
> > >>>> Can you ack the v7 of the bool fixup patch? Tom seems waiting
> for you.
> > >>>
> > >>> Oh sorry, I didn't realize there was a newer version of the
> patch I
> > >>> hadn't ack'ed yet. I'll take a look today. Do you know if it
> still
> > >>> applies cleanly to TOT u-boot/master, or will it need to be
> rebased?
> > >>>
> > >>
> > >> It almost applies cleanly.
> > >>
> > >> Checking patch board/xilinx/common/xbasic_types.h...
> > >> error: board/xilinx/common/xbasic_types.h: does not exist in index
> > >>
> > >> due to commit
> > >>
> > >> commit 151eeeb2754da4b2d26bb878206d45ce58276fe0
> > >> Author: Michal Simek <michal.simek@xilinx.com
> <mailto:michal.simek@xilinx.com>>
> > >> Date: Wed Jan 23 14:11:50 2013 +0100
> > >>
> > >> board: xilinx: Remove common folder
> > >>
> > >> All these files was used for ancient xilinx drivers
> > >> which are finally gone.
> > >>
> > >> Signed-off-by: Michal Simek <michal.simek@xilinx.com
> <mailto:michal.simek@xilinx.com>>
> > >> Tested-by: Rommel Custodio <sessyargc@gmail.com
> <mailto:sessyargc@gmail.com>>
> > >>
> > >> board/xilinx/common/xbasic_types.c | 165 ----
> > >> board/xilinx/common/xbasic_types.h | 283 ------
> > >> board/xilinx/common/xbuf_descriptor.h | 252 ------
> > >> board/xilinx/common/xdma_channel.c | 738 ----------------
> > >> board/xilinx/common/xdma_channel.h | 291 ------
> > >> board/xilinx/common/xdma_channel_i.h | 110 ---
> > >> board/xilinx/common/xdma_channel_sg.c | 1317
> > >> ----------------------------
> > >> board/xilinx/common/xio.h | 81 --
> > >> board/xilinx/common/xipif_v1_23_b.c | 331 -------
> > >> board/xilinx/common/xipif_v1_23_b.h | 746 ----------------
> > >> board/xilinx/common/xpacket_fifo_v1_00_b.c | 448 ----------
> > >> board/xilinx/common/xpacket_fifo_v1_00_b.h | 306 -------
> > >> board/xilinx/common/xstatus.h | 347 --------
> > >> board/xilinx/common/xversion.c | 350 --------
> > >> board/xilinx/common/xversion.h | 97 --
> > >> board/xilinx/zynq/Makefile | 3 -
> > >> 16 files changed, 5865 deletions(-)
> > >>
> > >>
> > >> Do I need to send a newer version?
> > >>
> > >
> > > Tom will probably make you send a newer one anyway. I'll fix it up
> > > locally and start testing in the mean time.
> > >
> >
> > Allen,
> >
> > I didn't change anything for the version, only removed "RFC" from the
> > subject. I have been there for quite a while. I thought Tom was
> going to
> > pick it up.
>
> Yeah sorry, I would have re-acked it if I saw there was a new
> version. Since it no longer applies cleanly I'm pretty sure he'll ask
> you to send a new version that does even if there are no other
> differences.
>
> Maybe we can talk Simon Glass into testing it out with his super fast
> multi-arch u-boot builder. It's way fancier than the perl script I
> use. Adding Simon.
>
> The patch in question is:
> http://patchwork.ozlabs.org/patch/217088/
>
>
> Yes, it needs a rebase, but the change is minor - I hope we can get this
> in. York can you do a rebase?
>
> Here are the boards that seem to fail: EVB64260 ZUMA EVB64260_750CX P3G4
>
Thanks, Simon. I will rebase the patch and address those failure.
York
prev parent reply other threads:[~2013-04-01 18:19 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-21 19:58 [U-Boot] [RFC] MAKEALL: Fix case substitution for old bash York Sun
2013-03-21 20:28 ` Allen Martin
2013-03-21 20:38 ` York Sun
2013-03-27 19:55 ` Tom Rini
[not found] ` <5155C7C9.2010306@freescale.com>
[not found] ` <20130329172922.GA18937@badger>
[not found] ` <5155D412.1020802@freescale.com>
[not found] ` <20130329175205.GB18937@badger>
[not found] ` <5155D5D1.1000802@freescale.com>
[not found] ` <20130329180520.GA1867@badger>
2013-03-30 22:30 ` Simon Glass
2013-04-01 18:19 ` York Sun [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=5159CFCE.3020207@freescale.com \
--to=yorksun@freescale.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