[Skiboot] [PATCH 0/2] Enable HIOMAP on P8

[Andrew Jeffery]

On Tue, 26 Feb 2019, at 16:29, Stewart Smith wrote:
> Andrew Jeffery <andrew at aj.id.au> writes:
> > I'm back again with enablement patches for hiomap on p8 systems. This time with
> > more testing in the loop.
> >
> > The bug with bd9839684d48 ("astbmc: Try IPMI HIOMAP for P8") turned out to be
> > an issue with ordering of operations in ast_sio_is_enabled(), which broke
> > assumptions of ast-sf-ctrl (i.e. that the SuperIO controller was unlocked with
> > the correct device selected). Patch 1/2 reworks the test sequence to leave
> > SuperIO unlocked with the prior device selected if a device was selected on
> > entry to the test.
> >
> > Patch 2/2 has not changed from bd9839684d48, aside from being tested on
> > Garrison, Palmetto (without and _with_ hiomap) and Witherspoon systems.
> >
> > Please review, CI, test and all the other things that we can do to prove I
> > haven't broken booting again.
> 
> Well, seemed to work on CI I threw at things. So, merged to master as of
> c826e1ca9e5b8a21989325536f2f9eb5f2c87607.
> 
> My guess is that these should go to stable too (at least once we reckon
> everything is okay and not-terrible)?

Yeah. As a note there's one more patch I need to send to add the IPMI netfn/cmd
for AMI on the 2400.

> 
> 
> -- 
> Stewart Smith
> OPAL Architect, IBM.
> 
>