Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for powerpc and POWER9 indirect type 1 scom accesses #21

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mikey
Copy link

@mikey mikey commented May 2, 2017

A fix for powerpc and an update for POWER9.

mikey added 3 commits May 2, 2017 14:48
Signed-off-by: Michael Neuling <[email protected]>
Without this we get this on gcc 6.3:
misc.c:261:29: error: ‘cpuid_params’ defined but not used [-Werror=unused-const-variable=]
 MAKE_PREREQ_PARAMS_VAR_ARGS(cpuid_params, 3, 4, "<cpu> <function> [index]", 0);
                             ^
Signed-off-by: Michael Neuling <[email protected]>
Skiboot recently added support for type 1 indirect scom addresses.
skiboot commit is here:

  commit 74311a97190d33e3753dc09a5fe6064120c8363a
  Author: Michael Neuling <[email protected]>
  xscom: Add indirect form 1 scoms

This now uses the whole top nibble of the scom address to determine
the indirection type.

As a consequence, the bit mangling for the kernel interface needs to
be updated (although it's backwards compatible) due to kernel commit.

  commit 517c27570cf38f182e7a688d50a9b978333f8ea8
  Author: Michael Neuling <[email protected]>
  powerpc/powernv: Fix XSCOM address mangling for form 1 indirect

This borrows the code from skiboot scom utils to add address
mangling. Code is from skiboot commit:

  commit 034d545b1503bf3980c290c361ed1a99ce9d308d
  Author: Michael Neuling <[email protected]>
  external: Update xscom utils for type 1 indirect accesses

Signed-off-by: Michael Neuling <[email protected]>
@andy-shev
Copy link

Looks like this repo is abandoned. So I merged the request to my fork.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants