On Thu, Aug 4, 2011 at 12:31 AM, Geoff Levand <ge...@infradead.org> wrote:
> On 08/01/2011 01:02 PM, Andre Heider wrote:
>> ---
>> arch/powerpc/include/asm/ps3.h | 7 +++++++
>> arch/powerpc/platforms/ps3/platform.h | 4 ++++
>> arch/powerpc/platforms/ps3/repository.c | 19 +++++++++++++++++++
>> arch/powerpc/platforms/ps3/setup.c | 22 ++++++++++++++++++++++
>> 4 files changed, 52 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/powerpc/include/asm/ps3.h b/arch/powerpc/include/asm/ps3.h
>> 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 {
>> void ps3_get_firmware_version(union ps3_firmware_version *v);
>> int ps3_compare_firmware_version(u16 major, u16 minor, u16 rev);
>>
>> +enum ps3_ss_laid {
>> + PS3_SS_LAID_GAMEOS = 0x1070000002000001UL,
>> + PS3_SS_LAID_OTHEROS = 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.
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
