Subject: Re: [patch 6/6] PS3: Add os-area database routines

From: Geoff Levand <[email-masked]>
Date: Mon, 08 Oct 2007 18:12:24 -0700

Geert Uytterhoeven wrote:
> On Sat, 6 Oct 2007 [email-masked] wrote:
>> --- a/arch/powerpc/platforms/ps3/os-area.c
>> +++ b/arch/powerpc/platforms/ps3/os-area.c
> 
>> +static int db_get_video_mode(const struct os_area_db *db,
>> +	unsigned int *video_mode)
>         ^^^^^^^^^^^^^^
>> +{
>> +	return db_get_64(db, &os_area_db_id_video_mode, (uint64_t*)video_mode);
>                                                         ^^^^^^^^^^^
>> +}
> 
> Woops, memory corruption, when writing a 64-bit value to a 32-bit variable.

Whoa!  That routines is not even used, so I removed it.

-Geoff