[Skiboot] [PATCH] platforms/astbmc: Move prd_init calls to astbmc_early_init()
Joel Stanley
joel at jms.id.au
Mon Sep 14 13:10:50 AEST 2015
On Mon, Sep 14, 2015 at 12:02 PM, Jeremy Kerr <jk at ozlabs.org> wrote:
>
> Currently, most astbmc platforms do their own call to prd_init(), but
> garrison is out-of-sync.
>
> This change moves the prd_init call to astbmc_early_init, so we don't
> need to enable it on every platform.
Good idea.
> Signed-off-by: Jeremy Kerr <jk at ozlabs.org>
Reviewed-by: Joel Stanley <joel at jms.id.au>
> ---
> platforms/astbmc/common.c | 2 ++
> platforms/astbmc/firestone.c | 1 -
> platforms/astbmc/habanero.c | 2 --
> platforms/astbmc/palmetto.c | 2 --
> 4 files changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/platforms/astbmc/common.c b/platforms/astbmc/common.c
> index 8d37785..40a9fc8 100644
> --- a/platforms/astbmc/common.c
> +++ b/platforms/astbmc/common.c
> @@ -348,4 +348,6 @@ void astbmc_early_init(void)
>
> /* Setup UART and use it as console with interrupts */
> uart_init(true);
> +
> + prd_init();
Should it always be last? Do we need a comment?
> }
> diff --git a/platforms/astbmc/firestone.c b/platforms/astbmc/firestone.c
> index 4eed429..aaed13e 100644
> --- a/platforms/astbmc/firestone.c
> +++ b/platforms/astbmc/firestone.c
> @@ -138,7 +138,6 @@ static bool firestone_probe(void)
>
> /* Lot of common early inits here */
> astbmc_early_init();
> - prd_init();
> slot_table_init(firestone_phb_table);
>
> return true;
> diff --git a/platforms/astbmc/habanero.c b/platforms/astbmc/habanero.c
> index 4ab2cdc..738aa63 100644
> --- a/platforms/astbmc/habanero.c
> +++ b/platforms/astbmc/habanero.c
> @@ -134,8 +134,6 @@ static bool habanero_probe(void)
> /* Lot of common early inits here */
> astbmc_early_init();
>
> - prd_init();
> -
> slot_table_init(habanero_phb_table);
>
> return true;
> diff --git a/platforms/astbmc/palmetto.c b/platforms/astbmc/palmetto.c
> index ca91908..033f103 100644
> --- a/platforms/astbmc/palmetto.c
> +++ b/platforms/astbmc/palmetto.c
> @@ -39,8 +39,6 @@ static bool palmetto_probe(void)
> /* Lot of common early inits here */
> astbmc_early_init();
>
> - prd_init();
> -
> return true;
> }
>
> --
> 2.1.4
>
> _______________________________________________
> Skiboot mailing list
> Skiboot at lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/skiboot
More information about the Skiboot
mailing list