| Message ID | 20150608043902.7342.52407.stgit@drishya |
|---|---|
| State | Accepted |
| Headers | show |
Hey Vaidy, On Mon, Jun 8, 2015 at 2:13 PM, Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com> wrote: > --- a/platforms/astbmc/firestone.c > +++ b/platforms/astbmc/firestone.c > @@ -30,6 +30,7 @@ static bool firestone_probe(void) > > /* Lot of common early inits here */ > astbmc_early_init(); > + prd_init(); Is there any reason not to put it in astbmc_early_init? Cheers, Joel
Hi Joel, >> /* Lot of common early inits here */ >> astbmc_early_init(); >> + prd_init(); > > Is there any reason not to put it in astbmc_early_init? I was thinking about doing this, but it may be possible that we have an astbmc machine without PRD support; the two aren't necessarily linked. Cheers, Jeremy
Hi Vaidy, > Call prd_init() on astbmc/firestone platform > in order to enable PRD in host kernel. > > Signed-off-by: Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com> Acked-by: Jeremy Kerr <jk@ozlabs.org> Thanks! Jeremy
diff --git a/platforms/astbmc/firestone.c b/platforms/astbmc/firestone.c index 4d3be19..362f1b6 100644 --- a/platforms/astbmc/firestone.c +++ b/platforms/astbmc/firestone.c @@ -30,6 +30,7 @@ static bool firestone_probe(void) /* Lot of common early inits here */ astbmc_early_init(); + prd_init(); return true; }
Call prd_init() on astbmc/firestone platform in order to enable PRD in host kernel. Signed-off-by: Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com> --- Tested on Firestone system with custom HBRT. PRD memory region parsing in OPAL and host kernel have been verified. Host kernel PRD driver and application are functional. platforms/astbmc/firestone.c | 1 + 1 file changed, 1 insertion(+)