Re: [PATCH v2] Documentation: Rename hpfall.c => freefall.c, fixes for Dell

[Randy Dunlap <rdunlap@infradead.org>]

On 06/27/14 12:08, Pali Rohár wrote:
> On Monday 16 June 2014 13:55:03 Pali Rohár wrote:
>> On Saturday 14 June 2014 10:53:29 Pavel Machek wrote:
>>> On Wed 2014-06-11 10:02:07, Pali Roh?r wrote:
>>>> Dell kernel driver dell-smo8800 provides same freefall
>>>> interface as hp_accel so program hpfall.c works also on
>>>> Dell laptops. So rename it to freefall.c.
>>>>
>>>> Dell driver does not provide hp::hddprotect led so make
>>>> sure that freefall.c works also if hp::hddprotect does
>>>> not exist in sysfs.
>>>>
>>>> Additionally write info to syslog.
>>>>
>>>> Signed-off-by: Pali Rohár <pali.rohar@gmail.com>
>>>> Cc: Sonal Santan <sonal.santan@gmail.com>
>>>> ---
>>>> Changes since v1:
>>>> * Fixed warnings: Missing a blank line after declarations
>>>> ---
>>>>
>>>>  Documentation/laptops/00-INDEX   |    4 +-
>>>>  Documentation/laptops/freefall.c |  177
>>>>  ++++++++++++++++++++++++++++++++++++++
>>>>  Documentation/laptops/hpfall.c   |  146
>>>>  ------------------------------- 3 files changed, 179
>>>>  insertions(+), 148 deletions(-) create mode 100644
>>>>  Documentation/laptops/freefall.c delete mode 100644
>>>>  Documentation/laptops/hpfall.c
>>>
>>> Acked-by: Pavel Machek <pavel@ucw.cz>
>>>
>>> 						Pavel
>>
>> Randy, Matthew: PING
>>
>> first patch which implementing dell-smo8800.ko was merged, but
>> this patch which updating documentation and fall daemon *not*.
> 
> PING.
> 
> What is with this patch? Why was still not set to Linus in doc or 
> platform pull request?
> 

Sorry, I was on vacation this week, but that does not explain the previous
delays.

I'll merge it sometime this weekend.

Thanks.

-- 
~Randy