All of lore.kernel.org
 help / color / mirror / Atom feed
From: Scott Wood <scottwood@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/4] mpc8xx: Correct cast of im_cpm.cp_dparam
Date: Wed, 15 May 2013 18:16:01 -0500	[thread overview]
Message-ID: <1368659761.8202.36@snotra> (raw)
In-Reply-To: <20130514102619.A4E2E3804AF@gemini.denx.de> (from wd@denx.de on Tue May 14 05:26:19 2013)

On 05/14/2013 05:26:19 AM, Wolfgang Denk wrote:
> Dear Tom,
> 
> In message <20130513223447.5D86C3804AF@gemini.denx.de> I wrote:
> >
> > In message <1368477117-32669-2-git-send-email-trini@ti.com> you  
> wrote:
> > > We must cast this to unsigned char not unsigned short to avoid  
> warnings.
> >
> > But this is wrong.  We're actually reading a 16 bit value here.
> >
> > I think there was previous discussion, and Scott Wodd has a patch  
> out
> > there, which just needed some minor fixing...
> 
> Scott's patch is here:
> 
> http://patchwork.ozlabs.org/patch/210599/
> 
> Scott, do you have any plans to submit an updated version?

I'll try to get to it soon.

-Scott

  reply	other threads:[~2013-05-15 23:16 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-13 20:31 [U-Boot] [PATCH 1/4] powerpc: Correct im_dprambase cast Tom Rini
2013-05-13 20:31 ` [U-Boot] [PATCH 2/4] mpc8xx: Correct cast of im_cpm.cp_dparam Tom Rini
2013-05-13 22:34   ` Wolfgang Denk
2013-05-14 10:26     ` Wolfgang Denk
2013-05-15 23:16       ` Scott Wood [this message]
2013-05-14 13:14     ` Tom Rini
2013-05-13 20:31 ` [U-Boot] [PATCH 3/4] ELPPC: Don't write magic numbers via short Tom Rini
2013-05-13 22:40   ` Wolfgang Denk
2013-05-13 20:31 ` [U-Boot] [PATCH 4/4] mpc8323erdb: Update crc32 check of MAC Tom Rini
2013-05-13 22:38   ` Wolfgang Denk
2013-05-13 22:37 ` [U-Boot] [PATCH 1/4] powerpc: Correct im_dprambase cast Wolfgang Denk

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=1368659761.8202.36@snotra \
    --to=scottwood@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.