linux-kernel - Re: [PATCH V1] DA9210 driver files

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:	Mon, 15 Jul 2013 17:57:35 +0100
From:	Mark Brown <broonie@...nel.org>
To:	Steve Twiss <stwiss.opensource@...semi.com>
Cc:	Liam Girdwood <lgirdwood@...il.com>,
	David Dajun Chen <david.chen@...semi.com>,
	Guennadi Liakhovetski <g.liakhovetski@....de>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH V1] DA9210 driver files

On Mon, Jul 15, 2013 at 03:06:59PM +0100, Steve Twiss wrote:
> From: Steve Twiss <stwiss.opensource@...semi.com>
> 
> This patch describes the regulator driver for the DA9210 Multi-phase Buck
> from Dialog Semiconductor Ltd. It has been made aginst kernel version
> linux-next next-20130715

To repeat *again* you should submit patches in the format covered in
SubmittingPatches.  This includes things like the commit message, if
your commit message doesn't look like other commit messages for the code
you're changing that's an issue.

> +struct da9210 {
> +	struct device *dev;

This is never used as far as I can tell.

> +MODULE_ALIAS("platform: da9210");

This is for a platform device but this is an I2C device so would never
match.  Also for a platform device the string should have no space in
it.

Otherwise this looks OK, though it'd be better to fill in the register
map information more (listing the valid registers for example).

Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)