Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-12 Thread yajin
>> >> Bonito north bridge is built on FPGA now, VENDOR_ID/DEVICE_ID are >>  temporary value so I didn't put them in pci_ids.h > > In that case, perhaps the code should be committed after the design > has stabilized a bit more? > In fact, the FPGA north bridge is very stable. It has been shipped in

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-11 Thread chen huacai
On Wed, May 12, 2010 at 3:52 AM, Blue Swirl wrote: > On 5/11/10, chen huacai wrote: >> >>  +    s->pci = qemu_mallocz(sizeof(*s->pci)); >>  >>  +    assert(s->pci != NULL); >>  >>  +    bonito_state = s; >>  >>  + >>  >>  +    /* get the north bridge pci bus */ >>  >>  +    s->pci->bus = pci_regi

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-11 Thread Blue Swirl
On 5/11/10, chen huacai wrote: > >> +s->pci = qemu_mallocz(sizeof(*s->pci)); > >> +assert(s->pci != NULL); > >> +bonito_state = s; > >> + > >> +/* get the north bridge pci bus */ > >> +s->pci->bus = pci_register_bus(NULL, "pci", pci_bonito_set_irq, > >> +

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-10 Thread chen huacai
>>  +    s->pci = qemu_mallocz(sizeof(*s->pci)); >>  +    assert(s->pci != NULL); >>  +    bonito_state = s; >>  + >>  +    /* get the north bridge pci bus */ >>  +    s->pci->bus = pci_register_bus(NULL, "pci", pci_bonito_set_irq, >>  +                                   pci_bonito_map_irq, pic, 0x

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-10 Thread Blue Swirl
On 5/9/10, chen huacai wrote: > This patch add initial support of bonito north bridge used by fulong mini pc > > Signed-off-by: Huacai Chen > - > diff --git a/Makefile.target b/Makefile.target > index c092900..fc4c59f 100644 > --- a/Makefile.target > +++ b/Makefile.target > @@ -218,7 +

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-10 Thread Stefan Weil
Am 10.05.2010 13:21, schrieb chen huacai: --- a/hw/mips.h +++ b/hw/mips.h @@ -5,6 +5,9 @@ /* gt64xxx.c */ PCIBus *pci_gt64120_init(qemu_irq *pic); +/* bonito.c */ +PCIBus *bonito_init_2e(qemu_irq pic); + /* ds1225y.c */ void *ds1225y_init(target_phys_addr_t mem_base, const char *filename);

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-10 Thread chen huacai
>> --- a/hw/mips.h >> +++ b/hw/mips.h >> @@ -5,6 +5,9 @@ >>  /* gt64xxx.c */ >>  PCIBus *pci_gt64120_init(qemu_irq *pic); >> >> +/* bonito.c */ >> +PCIBus *bonito_init_2e(qemu_irq pic); >> + >>  /* ds1225y.c */ >>  void *ds1225y_init(target_phys_addr_t mem_base, const char *filename); >>  void ds12

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-09 Thread yajin
> 'long long unsigned int', but argument 2 has type 'target_phys_addr_t' > /home/stefan/src/qemu/hw/bonito.c: In function 'bonito_spciconf_readw': I think TARGET_FMT_plx should be used instead of using %llx to avoid this warning. yajin http://vm-kernel.org 2010/5/9 Stefan Weil : > Am 09.05.201

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-09 Thread Stefan Weil
Am 09.05.2010 13:25, schrieb chen huacai: obj-mips-y += mips_addr.o mips_timer.o mips_int.o obj-mips-y += dma.o vga.o i8259.o obj-mips-y += g364fb.o jazz_led.o -obj-mips-y += gt64xxx.o pckbd.o mc146818rtc.o +obj-mips-y += gt64xxx.o bonito.o pckbd.o mc146818rtc.o Is fulong 64 bit only? Mayb

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-09 Thread chen huacai
>>  obj-mips-y += mips_addr.o mips_timer.o mips_int.o >>  obj-mips-y += dma.o vga.o i8259.o >>  obj-mips-y += g364fb.o jazz_led.o >> -obj-mips-y += gt64xxx.o pckbd.o mc146818rtc.o >> +obj-mips-y += gt64xxx.o bonito.o pckbd.o mc146818rtc.o >> > > Is fulong 64 bit only? > > Maybe it would be better t

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-09 Thread Stefan Weil
Am 09.05.2010 04:00, schrieb chen huacai: This patch add initial support of bonito north bridge used by fulong mini pc Signed-off-by: Huacai Chen ... + +} BonitoState; + +BonitoState * bonito_state; Add static attribute? With DEBUG enabled, I get a lot of compiler warnings: /home/s

Re: [Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-09 Thread Stefan Weil
Am 09.05.2010 04:00, schrieb chen huacai: This patch add initial support of bonito north bridge used by fulong mini pc Signed-off-by: Huacai Chen - diff --git a/Makefile.target b/Makefile.target index c092900..fc4c59f 100644 --- a/Makefile.target +++ b/Makefile.target @@ -218,7 +218,7 @@ obj

[Qemu-devel] [RFC][MIPS][PATCH 1/6] Initial support of bonito north bridge used by fulong mini pc

2010-05-08 Thread chen huacai
This patch add initial support of bonito north bridge used by fulong mini pc Signed-off-by: Huacai Chen - diff --git a/Makefile.target b/Makefile.target index c092900..fc4c59f 100644 --- a/Makefile.target +++ b/Makefile.target @@ -218,7 +218,7 @@ obj-mips-y = mips_r4k.o mips_jazz.o mips_malta