qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Blue Swirl" <blauwirbel@gmail.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] qemu Makefile.target vl.h hw/cuda.c hw/grackle_...
Date: Thu, 1 Nov 2007 21:53:22 +0200	[thread overview]
Message-ID: <f43fc5580711011253i7f14b0fahf68d936bd83d66e2@mail.gmail.com> (raw)
In-Reply-To: <f43fc5580711011241s3de6c1afqbdba99ed618da5d6@mail.gmail.com>

On 11/1/07, Blue Swirl <blauwirbel@gmail.com> wrote:
> On 10/29/07, Jocelyn Mayer <l_indien@magic.fr> wrote:
> > CVSROOT:        /sources/qemu
> > Module name:    qemu
> > Changes by:     Jocelyn Mayer <j_mayer> 07/10/28 23:42:18
> >
> > Modified files:
> >         .              : Makefile.target vl.h
> >         hw             : cuda.c grackle_pci.c heathrow_pic.c ppc.c
> >                          ppc_chrp.c ppc_prep.c
> > Added files:
> >         hw             : mac_dbdma.c mac_nvram.c macio.c ppc_mac.h
> >                          ppc_oldworld.c
> >
> > Log message:
> >         * sort the PowerPC target object files
> >         * make PowerPC NVRAM accessors generic to be able to use a MacIO NVRAM
> >           instead of the M48T59 one
> >         * split PowerMac targets code:
> >          - move all PowerMac related definitions and prototypes into hw/ppc_mac.h
> >          - add hw/mac_dbdma.c, hw/mac_nvram.c and macio.c
> >            which implements shared PowerMac devices
> >          - define the g3bw machine in a new hw/ppc_oldworld.c file
> >         * Fix the g3bw target:
> >          - fix the Grackle host PCI device
> >          - connect the Heathrow PIC to the PowerPC 6xx bus pins
> >
> > CVSWeb URLs:
> > http://cvs.savannah.gnu.org/viewcvs/qemu/Makefile.target?cvsroot=qemu&r1=1.212&r2=1.213
> > http://cvs.savannah.gnu.org/viewcvs/qemu/vl.h?cvsroot=qemu&r1=1.280&r2=1.281
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/cuda.c?cvsroot=qemu&r1=1.16&r2=1.17
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/grackle_pci.c?cvsroot=qemu&r1=1.6&r2=1.7
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/heathrow_pic.c?cvsroot=qemu&r1=1.5&r2=1.6
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/ppc.c?cvsroot=qemu&r1=1.34&r2=1.35
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/ppc_chrp.c?cvsroot=qemu&r1=1.44&r2=1.45
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/ppc_prep.c?cvsroot=qemu&r1=1.47&r2=1.48
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/mac_dbdma.c?cvsroot=qemu&rev=1.1
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/mac_nvram.c?cvsroot=qemu&rev=1.1
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/macio.c?cvsroot=qemu&rev=1.1
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/ppc_mac.h?cvsroot=qemu&rev=1.1
> > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/ppc_oldworld.c?cvsroot=qemu&rev=1.1
>
> You broke sparc64-softmmu build with this patch.

I think the best solution to fix this is to put the nvram helpers to
m48t59.h as inline functions  instead of duplicating the code in
several places.

  reply	other threads:[~2007-11-01 19:53 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-28 23:42 [Qemu-devel] qemu Makefile.target vl.h hw/cuda.c hw/grackle_ Jocelyn Mayer
2007-10-29  0:59 ` Stuart Brady
2007-10-29  1:21   ` J. Mayer
2007-11-01 19:41 ` Blue Swirl
2007-11-01 19:53   ` Blue Swirl [this message]
2007-11-01 22:13     ` J. Mayer
2007-11-01 22:43       ` J. Mayer
2007-11-02 15:18         ` Blue Swirl
2007-11-02 18:53           ` Jocelyn Mayer
2007-11-02 20:33             ` Blue Swirl
2007-11-02 22:25               ` J. Mayer

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=f43fc5580711011253i7f14b0fahf68d936bd83d66e2@mail.gmail.com \
    --to=blauwirbel@gmail.com \
    --cc=qemu-devel@nongnu.org \
    /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;
as well as URLs for NNTP newsgroup(s).