[PATCH 0/5] ppc/pnv: Introduce GPIO lines to drive the PSIHB device

Cédric Le Goater posted 5 patches 2 years, 1 month ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220323072846.1780212-1-clg@kaod.org
Maintainers: "Cédric Le Goater" <clg@kaod.org>
include/hw/ppc/pnv_lpc.h |  8 ++------
include/hw/ppc/pnv_occ.h |  7 ++-----
include/hw/ppc/pnv_psi.h |  7 +------
hw/ppc/pnv.c             | 30 ++++++++++++++++++------------
hw/ppc/pnv_lpc.c         | 19 ++++---------------
hw/ppc/pnv_occ.c         | 16 ++++------------
hw/ppc/pnv_psi.c         | 36 +++++++++++-------------------------
7 files changed, 42 insertions(+), 81 deletions(-)
[PATCH 0/5] ppc/pnv: Introduce GPIO lines to drive the PSIHB device
Posted by Cédric Le Goater 2 years, 1 month ago
Hello, 

The PSIHB OCC and LPC interrupts are driven by a complex framework
using Object links and class handlers. Simplify the whole with GPIO
lines.

Thanks,

C.

Cédric Le Goater (5):
  ppc/pnv: Fix PSI IRQ definition
  ppc/pnv: Remove PnvLpcController::psi link
  ppc/pnv: Remove PnvOCC::psi link
  ppc/pnv: Remove PnvPsiClas::irq_set
  ppc/pnv: Remove usless checks in set_irq handlers

 include/hw/ppc/pnv_lpc.h |  8 ++------
 include/hw/ppc/pnv_occ.h |  7 ++-----
 include/hw/ppc/pnv_psi.h |  7 +------
 hw/ppc/pnv.c             | 30 ++++++++++++++++++------------
 hw/ppc/pnv_lpc.c         | 19 ++++---------------
 hw/ppc/pnv_occ.c         | 16 ++++------------
 hw/ppc/pnv_psi.c         | 36 +++++++++++-------------------------
 7 files changed, 42 insertions(+), 81 deletions(-)

-- 
2.34.1
Re: [PATCH 0/5] ppc/pnv: Introduce GPIO lines to drive the PSIHB device
Posted by Daniel Henrique Barboza 2 years, 1 month ago

On 3/23/22 04:28, Cédric Le Goater wrote:
> Hello,
> 
> The PSIHB OCC and LPC interrupts are driven by a complex framework
> using Object links and class handlers. Simplify the whole with GPIO
> lines.


Series LGTM. All patches:


Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>




Daniel


> 
> Thanks,
> 
> C.
> 
> Cédric Le Goater (5):
>    ppc/pnv: Fix PSI IRQ definition
>    ppc/pnv: Remove PnvLpcController::psi link
>    ppc/pnv: Remove PnvOCC::psi link
>    ppc/pnv: Remove PnvPsiClas::irq_set
>    ppc/pnv: Remove usless checks in set_irq handlers
> 
>   include/hw/ppc/pnv_lpc.h |  8 ++------
>   include/hw/ppc/pnv_occ.h |  7 ++-----
>   include/hw/ppc/pnv_psi.h |  7 +------
>   hw/ppc/pnv.c             | 30 ++++++++++++++++++------------
>   hw/ppc/pnv_lpc.c         | 19 ++++---------------
>   hw/ppc/pnv_occ.c         | 16 ++++------------
>   hw/ppc/pnv_psi.c         | 36 +++++++++++-------------------------
>   7 files changed, 42 insertions(+), 81 deletions(-)
> 

Re: [PATCH 0/5] ppc/pnv: Introduce GPIO lines to drive the PSIHB device
Posted by Daniel Henrique Barboza 2 years ago

On 3/23/22 04:28, Cédric Le Goater wrote:
> Hello,
> 
> The PSIHB OCC and LPC interrupts are driven by a complex framework
> using Object links and class handlers. Simplify the whole with GPIO
> lines.
> 
> Thanks,

Queued in gitlab.com/danielhb/qemu/tree/ppc-next. Thanks,


Daniel

> 
> C.
> 
> Cédric Le Goater (5):
>    ppc/pnv: Fix PSI IRQ definition
>    ppc/pnv: Remove PnvLpcController::psi link
>    ppc/pnv: Remove PnvOCC::psi link
>    ppc/pnv: Remove PnvPsiClas::irq_set
>    ppc/pnv: Remove usless checks in set_irq handlers
> 
>   include/hw/ppc/pnv_lpc.h |  8 ++------
>   include/hw/ppc/pnv_occ.h |  7 ++-----
>   include/hw/ppc/pnv_psi.h |  7 +------
>   hw/ppc/pnv.c             | 30 ++++++++++++++++++------------
>   hw/ppc/pnv_lpc.c         | 19 ++++---------------
>   hw/ppc/pnv_occ.c         | 16 ++++------------
>   hw/ppc/pnv_psi.c         | 36 +++++++++++-------------------------
>   7 files changed, 42 insertions(+), 81 deletions(-)
>