On 01/05/2008 03:29 AM, Arnd Bergmann wrote:
> On Saturday 05 January 2008, Geoff Levand wrote:
>> + struct layout {
>> + struct ps3_system_bus_device dev;
>> + } *p;
>
> What's the point of this data structure? You don't use the
> struct anywhere, and it only has one mem
On Saturday 05 January 2008, Geoff Levand wrote:
> + struct layout {
> + struct ps3_system_bus_device dev;
> + } *p;
What's the point of this data structure? You don't use the
struct anywhere, and it only has one member, so you could
just declare that directly.
> +
Add PS3 logical performance monitor (lpm) device support to the
PS3 system-bus and platform device registration routines.
Signed-off-by: Geoff Levand <[EMAIL PROTECTED]>
---
arch/powerpc/platforms/ps3/device-init.c | 95 ++-
arch/powerpc/platforms/ps3/system-bus.c |