public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

      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