On Tue, 20 May 2014 10:56:47 +0200, Pali Rohár said:
> Hm? Which errors? Here is output from checkpacth:
>
> total: 0 errors, 0 warnings, 276 lines checked
>
> 0001-platform-x86-dell-smo8800-Dell-Latitude-freefall-dri.patch
> has no obvious style problems and is ready for submission.
>
> total: 0 errors, 0 warnings, 181 lines checked
>
> 0002-Documentation-Rename-hpfall.c-freefall.c-fixes-for-D.patch
> has no obvious style problems and is ready for submission.
Hmm.. what tree are you building against? I wonder if your checkpatch is
a different version than mine (next-20140519). What I see:
[/usr/src/linux-next] scripts/checkpatch.pl patches/freefall.1
WARNING: Missing a blank line after declarations
#135: FILE: drivers/platform/x86/dell-smo8800.c:41:
+ struct smo8800_device *smo8800 = data;
+ atomic_inc(&smo8800->count);
WARNING: Missing a blank line after declarations
#143: FILE: drivers/platform/x86/dell-smo8800.c:49:
+ struct smo8800_device *smo8800 = data;
+ dev_info(smo8800->dev, "detected free fall\n");
WARNING: Missing a blank line after declarations
#318: FILE: drivers/platform/x86/dell-smo8800.c:224:
+ struct smo8800_device *smo8800 = device->driver_data;
+ free_irq(smo8800->irq, smo8800);
total: 0 errors, 3 warnings, 276 lines checked
patches/freefall.1 has style problems, please review.
[/usr/src/linux-next] scripts/checkpatch.pl patches/freefall.2
WARNING: Missing a blank line after declarations
#126: FILE: Documentation/laptops/freefall.c:47:
+ int fd = open(unload_heads_path, O_RDONLY);
+ if (fd < 0) {
WARNING: Missing a blank line after declarations
#139: FILE: Documentation/laptops/freefall.c:60:
+ int fd = open(path, O_RDWR);
+ if (fd < 0) {
WARNING: Missing a blank line after declarations
#161: FILE: Documentation/laptops/freefall.c:82:
+ const char *str = (seconds == 0) ? "Unparked" : "Parked";
+ write_int(unload_heads_path, seconds*1000);
total: 0 errors, 3 warnings, 181 lines checked
Attachment:
pgpB4AW6gbXZ3.pgp
Description: PGP signature
