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

[Pali RohÃr]

Hello Randy, sorry but now I do not have any free time...

On Sunday 29 June 2014 04:12:17 Randy Dunlap wrote:
> Hi,
> 
> This should really be done as 2 separate patches:
> One to move/rename the file and one to make the
> changes/additions to it for DELL.
> 
> Can you resend it as 2 patches like that, please?
> 
> Thanks... and sorry for the delay.
> 
> On 06/11/14 01:02, 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@xxxxxxxxx>
> > Cc: Sonal Santan <sonal.santan@xxxxxxxxx>
> > ---
> > 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

-- 
Pali RohÃr
pali.rohar@xxxxxxxxx

Attachment: signature.asc
Description: This is a digitally signed message part.