[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/19] openpic: Remove unused code
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH 01/19] openpic: Remove unused code |
Date: |
Sat, 8 Dec 2012 16:14:36 +0100 |
On 08.12.2012, at 16:12, Andreas Färber wrote:
> Am 08.12.2012 14:44, schrieb Alexander Graf:
>> The openpic code had a few WIP bits left that nobody reanimated within
>> the last few years. Remove that code.
>>
>> Signed-off-by: Alexander Graf <address@hidden>
>
> Reviewed-by: Andreas Färber <address@hidden>
>
> I'm not aware of any GW80314 users, CC'ing Hervé just to be sure.
> The BeBox has an i82378 (sitting on its MPC105), using i8259 PICs.
>
> But even if there were something using them, this is most definitely the
> wrong way to model variations nowadays. :)
>
> Alex, did you find out what was broken about it?
I don't even know what model was supposed to ever use this. IIUC most of that
code was for the Intel OpenPIC flavor (whichever one that is). So it's really
more about untestable code rather than it being broken.
I'd say if anyone wants to introduce a board that requires these bits, I'm more
than happy to see them added back in in a properly modeled way :)
Alex
- [Qemu-devel] [PATCH 15/19] openpic: make brr1 model specific, (continued)
- [Qemu-devel] [PATCH 15/19] openpic: make brr1 model specific, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 07/19] openpic: merge mpic and openpic timer handling, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 03/19] openpic: update to proper memory api, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 13/19] openpic: remove irq_out, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 05/19] openpic: Convert subregions to memory api, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 11/19] openpic: convert simple reg operations to builtin bitops, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 18/19] PPC: e500: Declare pci bridge as bridge, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 17/19] PPC: e500: Add MSI support, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 01/19] openpic: Remove unused code, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 16/19] openpic: add Shared MSI support, Alexander Graf, 2012/12/08
- Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/19] openpic: add Shared MSI support, Scott Wood, 2012/12/10
- Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/19] openpic: add Shared MSI support, Alexander Graf, 2012/12/11
- Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/19] openpic: add Shared MSI support, Scott Wood, 2012/12/11
- Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/19] openpic: add Shared MSI support, Alexander Graf, 2012/12/11
- Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/19] openpic: add Shared MSI support, Scott Wood, 2012/12/11
- Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/19] openpic: add Shared MSI support, Alexander Graf, 2012/12/12
[Qemu-devel] [PATCH 12/19] openpic: rename openpic_t to OpenPICState, Alexander Graf, 2012/12/08
[Qemu-devel] [PATCH 08/19] openpic: combine openpic and mpic reset functions, Alexander Graf, 2012/12/08