Re: [PATCH 05/15] ps3: Detect the current lpar environment

On Thu, Aug 4, 2011 at 12:31 AM, Geoff Levand [off-list ref] wrote:

On 08/01/2011 01:02 PM, Andre Heider wrote:

quoted

---
=A0arch/powerpc/include/asm/ps3.h =A0 =A0 =A0 =A0 =A0| =A0 =A07 +++++++
=A0arch/powerpc/platforms/ps3/platform.h =A0 | =A0 =A04 ++++
=A0arch/powerpc/platforms/ps3/repository.c | =A0 19 +++++++++++++++++++
=A0arch/powerpc/platforms/ps3/setup.c =A0 =A0 =A0| =A0 22 ++++++++++++++=

++++++++

quoted

=A04 files changed, 52 insertions(+), 0 deletions(-)

diff --git a/arch/powerpc/include/asm/ps3.h b/arch/powerpc/include/asm/p=

s3.h

quoted

index 7f065e1..136354a 100644

--- a/arch/powerpc/include/asm/ps3.h
+++ b/arch/powerpc/include/asm/ps3.h

@@ -39,6 +39,13 @@ union ps3_firmware_version {

=A0void ps3_get_firmware_version(union ps3_firmware_version *v);
=A0int ps3_compare_firmware_version(u16 major, u16 minor, u16 rev);

+enum ps3_ss_laid {
+ =A0 =A0 PS3_SS_LAID_GAMEOS =3D 0x1070000002000001UL,
+ =A0 =A0 PS3_SS_LAID_OTHEROS =3D 0x1080000004000001UL,

Only PS3_SS_LAID_OTHEROS is used for anything outside ps3_setup_arch(),
so I think it makes sense to split this into two patches with one adding
just PS3_SS_LAID_OTHEROS and ps3_get_ss_laid() with a comment that
it adds the ps3_get_ss_laid routine.

Sounds reasonable, I will split that into two patches then.