[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 10/14] omap-intc: remove PROP_PTR
From: |
Peter Maydell |
Subject: |
Re: [PATCH 10/14] omap-intc: remove PROP_PTR |
Date: |
Fri, 18 Oct 2019 17:55:25 +0100 |
On Fri, 18 Oct 2019 at 16:43, Marc-André Lureau
<address@hidden> wrote:
>
> Since clock are not QOM objects, replace PROP_PTR of clocks with
> setters methods.
>
> (in theory there should probably be different methods for omap1 &
> omap2 intc, but this is left as a future improvement)
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> diff --git a/include/hw/arm/omap.h b/include/hw/arm/omap.h
> index 2fda996648..6b7897ba27 100644
> --- a/include/hw/arm/omap.h
> +++ b/include/hw/arm/omap.h
> @@ -67,6 +67,16 @@ void omap_clk_setrate(omap_clk clk, int divide, int
> multiply);
> int64_t omap_clk_getrate(omap_clk clk);
> void omap_clk_reparent(omap_clk clk, omap_clk parent);
>
> +/* omap_intc.c */
> +#define TYPE_OMAP_INTC "common-omap-intc"
> +#define OMAP_INTC(obj) \
> + OBJECT_CHECK(omap_intr_handler, (obj), TYPE_OMAP_INTC)
> +
> +typedef struct omap_intr_handler_s omap_intr_handler;
> +
If you add
/*
* TODO: Ideally we should have a clock framework that
* let us wire these clocks up with QOM properties or links.
*/
> +void omap_intc_set_iclk(omap_intr_handler *intc, omap_clk clk);
> +void omap_intc_set_fclk(omap_intr_handler *intc, omap_clk clk);
> +
then
Reviewed-by: Peter Maydell <address@hidden>
thanks
-- PMM
- [PATCH 07/14] RFC: mips/cps: fix setting saar property, (continued)
- [PATCH 07/14] RFC: mips/cps: fix setting saar property, Marc-André Lureau, 2019/10/18
- [PATCH 08/14] cris: replace PROP_PTR with PROP_LINK for interrupt vector, Marc-André Lureau, 2019/10/18
- [PATCH 09/14] smbus-eeprom: remove PROP_PTR, Marc-André Lureau, 2019/10/18
- [PATCH 10/14] omap-intc: remove PROP_PTR, Marc-André Lureau, 2019/10/18
- Re: [PATCH 10/14] omap-intc: remove PROP_PTR,
Peter Maydell <=
- [PATCH 11/14] omap-i2c: remove PROP_PTR, Marc-André Lureau, 2019/10/18
- [PATCH 12/14] omap-gpio: remove PROP_PTR, Marc-André Lureau, 2019/10/18
- [PATCH 13/14] qdev: remove PROP_MEMORY_REGION, Marc-André Lureau, 2019/10/18
- [PATCH 14/14] Remove QDEV_PROP_PTR, Marc-André Lureau, 2019/10/18