linux-kernel - Re: [PATCH 00/14] Assorted small patches for regulators

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 24 Feb 2010 10:03:15 +0000
From:	Mark Brown <broonie@...nsource.wolfsonmicro.com>
To:	Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc:	Liam Girdwood <lrg@...mlogic.co.uk>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 00/14] Assorted small patches for regulators

On Tue, Feb 23, 2010 at 11:37:39PM -0800, Dmitry Torokhov wrote:

> I happend to peek into drivers/regulator and saw a bunch of small issues, so
> here goes. The patches are against linux-next, compile-tested only since I
> don't have the hardware,

I'm working through these now, but a few general issues:

 - It'd be very much easier to review patches that do one thing at once,
   especially when the patches do things more invasive than just adding
   annotations or changing types.  The random cleanup stuff makes the
   invasive changes much harder to see and review.
 - Frequently your patches include additional changes above those that
   you list in the changelog which again increases the effort require to
   reviwe - things like random whitespace changes and the addition of
   module annotations seem particularly prone to this.
 - Please always use a subject line for your patches which fits the
   style of the subsystem.  You're using "Regulators:" as a prefix when
   pretty much everything else uses "regulator:".
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/