public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Pull request: u-boot-mips/master
Date: Thu, 13 Sep 2012 10:03:52 -0700	[thread overview]
Message-ID: <505211F8.7050501@ti.com> (raw)
In-Reply-To: <CACUy__VJ_2tbo7Bmh3+qGWOwax+5XbsL_8uGGBYNnsHrD4RucA@mail.gmail.com>

On 09/13/2012 06:38 AM, Daniel Schwierzeck wrote:
> Hi Tom,
> 
> 2012/9/13 Tom Rini <trini@ti.com>:
>> On Thu, Sep 13, 2012 at 01:17:03AM +0200, Daniel Schwierzeck wrote:
> [...]
>>
>> With eldk-5.2.1 I see a number of warnings on qemu_mips64, of the form
>> "oh, this is 64bit and some assumption is wrong now about types".  Can
>> you please fix these problems as well?
> 
> yes, but all warnings are not MIPS specific. Most likely there are
> many more locations in common
> U-Boot code which make false assumptions and need to be fixed for 64 bit.

Exactly.  Since you guys are the first to hit it, can you please try and
fix them?

>> And this is really past the
>> merge window but since it's isolated I'm willing to go with your
>> discretion on pulling it now, once the warnings are dealt with.  Thanks!
>>
> 
> the qemu_mips64 patches were submitted at the end of the merge window,
> but the review took me four weeks.
> I can reschedule the qemu_mips64 support to the next merge window if
> you are more comfortable
> with that. But the remaining patches are bugfixes and cleanups and
> should be merged for the upcoming release.
> 
> Shall I send a new pull request without qemu_mips64?

OK, I didn't recall seeing them in time.  I'm OK with pulling it in so
long as someone is going to address all of the warnings this shows.

-- 
Tom

  reply	other threads:[~2012-09-13 17:03 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-12 23:17 [U-Boot] Pull request: u-boot-mips/master Daniel Schwierzeck
2012-09-12 23:43 ` Tom Rini
2012-09-13 13:38   ` Daniel Schwierzeck
2012-09-13 17:03     ` Tom Rini [this message]
2012-09-14  2:03       ` Daniel Schwierzeck
2012-09-14  2:04 ` Daniel Schwierzeck
2012-09-18 14:50   ` Tom Rini
  -- strict thread matches above, loose matches on Subject: below --
2017-05-10 19:46 Daniel Schwierzeck
2017-05-11  0:13 ` Tom Rini
2017-05-12 21:08 Daniel Schwierzeck
2017-05-14  0:12 ` Tom Rini
2017-05-31 19:38 Daniel Schwierzeck
2017-06-01 12:10 ` Tom Rini

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=505211F8.7050501@ti.com \
    --to=trini@ti.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