lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Tue, 05 Dec 2017 14:27:47 -0800
From:   Joe Perches <joe@...ches.com>
To:     Hans-Frieder Vogt <hfvogt@....net>, linux-kernel@...r.kernel.org
Cc:     zbr@...emap.net
Subject: Re: [PATCH v2] w1: driver for serial linkUSB, linkOEM, LINK devices

On Sun, 2017-11-12 at 22:04 +0100, Hans-Frieder Vogt wrote:
> Hi,

hi back.

> attached is the revised version of a driver that supports the serial OneWire masters from iButtonLink(TM) in the w1 kernel driver. In order to be usable it needs an updated userland tool (patch included in documentation file). The kernel patch is against linux-next. Please try and comment.
> 
> v2 of the patch set changes the following:
> - based on review input from Evgeniy (thanks!), introduced a helper function to re-use more code and generally improved the style of the code (couldn't get rid of all problems checkpatch identified, but many of them)
> - introduced a version check to make sure the hardware belongs to the supported devices
> - removed a dependency on little endian system
> - further testing revealed re-entry problems. The protection of critical code areas has therefore been changed from local spinlocks to a more toplevel mutex-based system to enforce serialisation. I have not run into any problems with unexpected feedback from the link device since.

Please wrap your commit messages.
A maximum line length at 72 or so is preferred.

> Signed-off by: Hans-Frieder Vogt <hfvogt@....net>

Note the space between off and by.

checkpatch does correctly warn about this as
"missing Signed-off-by:"

Not much can be done to quiet checkpatch when
the patches contain an inter-diff.

Powered by blists - more mailing lists