[PATCH] PXA: Fix wakealarm capability for pxa-rtc

Robert Jarzmik robert.jarzmik at free.fr
Sat Jul 31 17:07:06 EDT 2010


Marek Vasut <marek.vasut at gmail.com> writes:

> Dne So 31. července 2010 17:38:06 Robert Jarzmik napsal(a):
>> Marek Vasut <marek.vasut at gmail.com> writes:
>> > The wakealarm capability was incorrectly set after the RTC was
>> > registered, which resulted into the sysfs node not being created.
>> 
>> On my platform, without this patch:
>> > cat /sys/devices/platform/pxa-rtc/power/wakeup
>> > enabled
>> 
>> The sysfs node is created. What's the issue you're having ?
>
> It's not created for me without this patch (and if you read the code, it is 
> impossible to be created without this patch).

I read the code, and I see:
rtc_device_register()
  -> device_register()
    -> device_initialize()
      -> device_init_wakeup(dev, 0)

So with or without your patch, device_init_wakeup() is called. The difference is
that with your patch, we end up with a disabled wakeup capability.
I showed the code sequence, your turn to show your "if you read the code".

Moreover, my platform without your patch prooves that the sysfs entry is created.

So unless you prove me wrong, I will reject the patch.

Cheers.

-- 
Robert



More information about the linux-arm-kernel mailing list