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

From: Pali RohÃr
Date: Mon Jun 16 2014 - 07:55:17 EST


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@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
>
> Acked-by: Pavel Machek <pavel@xxxxxx>
>
> Pavel

Randy, Matthew: PING

first patch which implementing dell-smo8800.ko was merged, but
this patch which updating documentation and fall daemon *not*.

--
Pali RohÃr
pali.rohar@xxxxxxxxx

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