[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/8] hw/ppc/ppc4xx_devs: Make code style fixes to UIC code
From: |
David Gibson |
Subject: |
Re: [PATCH 1/8] hw/ppc/ppc4xx_devs: Make code style fixes to UIC code |
Date: |
Mon, 14 Dec 2020 16:43:08 +1100 |
On Sat, Dec 12, 2020 at 12:15:30AM +0000, Peter Maydell wrote:
> In a following commit we will move the PPC UIC implementation to
> its own file in hw/intc. To prevent checkpatch complaining about that
> code-motion, fix up the minor style issues first.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Applied to ppc-for-6.0, thanks.
> ---
> hw/ppc/ppc4xx_devs.c | 25 +++++++++++++++----------
> 1 file changed, 15 insertions(+), 10 deletions(-)
>
> diff --git a/hw/ppc/ppc4xx_devs.c b/hw/ppc/ppc4xx_devs.c
> index f1651e04d9a..f2f9ca4ffec 100644
> --- a/hw/ppc/ppc4xx_devs.c
> +++ b/hw/ppc/ppc4xx_devs.c
> @@ -105,7 +105,7 @@ struct ppcuic_t {
> qemu_irq *irqs;
> };
>
> -static void ppcuic_trigger_irq (ppcuic_t *uic)
> +static void ppcuic_trigger_irq(ppcuic_t *uic)
> {
> uint32_t ir, cr;
> int start, end, inc, i;
> @@ -156,26 +156,28 @@ static void ppcuic_trigger_irq (ppcuic_t *uic)
> }
> }
>
> -static void ppcuic_set_irq (void *opaque, int irq_num, int level)
> +static void ppcuic_set_irq(void *opaque, int irq_num, int level)
> {
> ppcuic_t *uic;
> uint32_t mask, sr;
>
> uic = opaque;
> - mask = 1U << (31-irq_num);
> + mask = 1U << (31 - irq_num);
> LOG_UIC("%s: irq %d level %d uicsr %08" PRIx32
> " mask %08" PRIx32 " => %08" PRIx32 " %08" PRIx32 "\n",
> __func__, irq_num, level,
> uic->uicsr, mask, uic->uicsr & mask, level << irq_num);
> - if (irq_num < 0 || irq_num > 31)
> + if (irq_num < 0 || irq_num > 31) {
> return;
> + }
> sr = uic->uicsr;
>
> /* Update status register */
> if (uic->uictr & mask) {
> /* Edge sensitive interrupt */
> - if (level == 1)
> + if (level == 1) {
> uic->uicsr |= mask;
> + }
> } else {
> /* Level sensitive interrupt */
> if (level == 1) {
> @@ -188,11 +190,12 @@ static void ppcuic_set_irq (void *opaque, int irq_num,
> int level)
> }
> LOG_UIC("%s: irq %d level %d sr %" PRIx32 " => "
> "%08" PRIx32 "\n", __func__, irq_num, level, uic->uicsr, sr);
> - if (sr != uic->uicsr)
> + if (sr != uic->uicsr) {
> ppcuic_trigger_irq(uic);
> + }
> }
>
> -static uint32_t dcr_read_uic (void *opaque, int dcrn)
> +static uint32_t dcr_read_uic(void *opaque, int dcrn)
> {
> ppcuic_t *uic;
> uint32_t ret;
> @@ -220,13 +223,15 @@ static uint32_t dcr_read_uic (void *opaque, int dcrn)
> ret = uic->uicsr & uic->uicer;
> break;
> case DCR_UICVR:
> - if (!uic->use_vectors)
> + if (!uic->use_vectors) {
> goto no_read;
> + }
> ret = uic->uicvr;
> break;
> case DCR_UICVCR:
> - if (!uic->use_vectors)
> + if (!uic->use_vectors) {
> goto no_read;
> + }
> ret = uic->uicvcr;
> break;
> default:
> @@ -238,7 +243,7 @@ static uint32_t dcr_read_uic (void *opaque, int dcrn)
> return ret;
> }
>
> -static void dcr_write_uic (void *opaque, int dcrn, uint32_t val)
> +static void dcr_write_uic(void *opaque, int dcrn, uint32_t val)
> {
> ppcuic_t *uic;
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- [PATCH 0/8] hw/ppc: Convert UIC device to QOM, Peter Maydell, 2020/12/11
- [PATCH 1/8] hw/ppc/ppc4xx_devs: Make code style fixes to UIC code, Peter Maydell, 2020/12/11
- [PATCH 2/8] ppc: Convert PPC UIC to a QOM device, Peter Maydell, 2020/12/11
- Re: [PATCH 2/8] ppc: Convert PPC UIC to a QOM device, Edgar E. Iglesias, 2020/12/13
- Re: [PATCH 2/8] ppc: Convert PPC UIC to a QOM device, David Gibson, 2020/12/14
- Re: [PATCH 2/8] ppc: Convert PPC UIC to a QOM device, BALATON Zoltan, 2020/12/14
- [PATCH 3/8] hw/ppc/virtex_ml507: Drop use of ppcuic_init(), Peter Maydell, 2020/12/11