Messages in this thread | | | Date | Sat, 28 Jun 2014 19:12:17 -0700 | From | Randy Dunlap <> | Subject | Re: [PATCH v2] Documentation: Rename hpfall.c => freefall.c, fixes for Dell |
| |
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@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
-- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |