linux-kernel - regulator: tps65020 support

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-next>] [day] [month] [year] [list]

Message-ID: <CAM8FrKK1z7euvURCH+ou8uNYumF82bn13oYfBOAiMNiB9zdBJA@mail.gmail.com>
Date:	Tue, 26 Jul 2011 00:03:46 +0200
From:	Marcus Folkesson <marcus.folkesson@...il.com>
To:	lrg@...mlogic.co.uk, broonie@...nsource.wolfsonmicro.com
Cc:	linux-kernel@...r.kernel.org
Subject: regulator: tps65020 support

Hi,

This patch gives support for the tps65020 voltage regulator from Texas
Instruments.
The support is added to the existing tps65023 regulator.

It also fixes two bugs:
* The original driver does not allow core voltage adjustments by the
I2C bus. (clearing CORE ADJ bit in CTRL2 register).
* The original driver does not either telling the chip that the
DEFCORE register i changed by setting the GO bit in CTRL2.

I have read SubmittingPatches and think all should be okay (this is my
first submit :-) ), but I still have two questions:
* The patch is renaming tps65023-regulator.c to tps6502x-regulator.c
(and edits the corresponding Kconfig/Makefile) since it now handles
both tps65020 and tps65023. Is this allowed or is the Kconfig and
Makefile files holy or something?
* checkpatch.pl is showing two warnings about lines over 80
characters, but this is not really truth. The lines is splitted up so
it should not complain. Please take a look.

Med vänliga hälsningar / Best regards
Marcus Folkesson

View attachment "0001-regulator-TPS65020-support.patch" of type "text/x-patch" (37646 bytes)