On 16.07.2012, at 21:32, Scott Wood wrote: > On 07/16/2012 02:29 PM, Yoder Stuart-B08248 wrote: >> >> >>> -----Original Message----- >>> From: kvm-ppc-owner@xxxxxxxxxxxxxxx [mailto:kvm-ppc-owner@xxxxxxxxxxxxxxx] On Behalf Of Scott Wood >>> Sent: Monday, July 16, 2012 12:12 PM >>> To: Bhushan Bharat-R65777 >>> Cc: Wood Scott-B07421; Alexander Graf; qemu-ppc@xxxxxxxxxx; kvm-ppc@xxxxxxxxxxxxxxx >>> Subject: Re: [PATCH] openpic: Added BRR1 register >>> >>> On 07/16/2012 12:09 PM, Bhushan Bharat-R65777 wrote: >>>> >>>> >>>>> -----Original Message----- >>>>> From: Wood Scott-B07421 >>>>> Sent: Monday, July 16, 2012 10:34 PM >>>>> To: Bhushan Bharat-R65777 >>>>> Cc: Alexander Graf; qemu-ppc@xxxxxxxxxx; kvm-ppc@xxxxxxxxxxxxxxx >>>>> Subject: Re: [PATCH] openpic: Added BRR1 register >>>>> >>>>> On 07/16/2012 11:21 AM, Bhushan Bharat-R65777 wrote: >>>>>>>> + case 0x00: /* BRR1 */ >>>>>>>> + retval = 0x00400200; >>>>>>> >>>>>>> Please unmagicify this one :) >>>>>> >>>>>> /* BRR1 ( Block revision register ) */ >>>>>> >>>>>> #define IPID 0x00400000 /* IP-block ID */ #define IPMJ 0x00000200 /* >>>>>> IP major number */ #define IPMN 0x00000200 /* IP minor number */ >>>>> >>>>> IPMN looks wrong. >>>> >>>> Opps : copy paste error here :) >>>> >>>> Btw, I am not aware of all MPIC IP versions, differences in them and what is the latest/best version >>> this emulated code supports. I drive this value from mpc8544 Reference Manual. >>>> >>>> I requested Alex to suggest the most updated version. >>> >>> QEMU does not have a complete implementation of any version of the FSL MPIC. >> >> So what revision numbers should we use? > > How about 0.0 for now? Would guests care (read: break)? Alex -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html