Thread (14 messages) read the whole thread 14 messages, 5 authors, 2013-12-12

[PATCH v5 1/3] mfd: tps6586x: add version detection

From: Lee Jones <hidden>
Date: 2013-12-06 15:36:14
Also in: linux-tegra, lkml

On Fri, 06 Dec 2013, Stefan Agner wrote:
Am 2013-12-06 15:26, schrieb Thierry Reding:
quoted
On Fri, Dec 06, 2013 at 01:51:45PM +0100, Stefan Agner wrote:
quoted
Use the VERSIONCRC to determine the exact device version. According to
the datasheet this register can be used as device identifier. The
identification is needed since some tps6586x regulators use a different
voltage table.

Signed-off-by: Stefan Agner <stefan@agner.ch>
---
Changes since v4:
  - Use variable named version instead of ret

Changes since v2:
  - Avoid moving devm_kzalloc
---
 drivers/mfd/tps6586x.c       | 50 +++++++++++++++++++++++++++++++++++++-------
 include/linux/mfd/tps6586x.h |  7 +++++++
 2 files changed, 49 insertions(+), 8 deletions(-)
I think I also already gave my Reviewed-by on both this and 2/3, neither
of which changed significantly enough to warrant that to be dropped. In
general it's fine to keep Reviewed-by and Acked-by as long as you're
only changing minor things.

Thierry
Ok, sorry, didn't realize that is my job to pick them up when doing new
revisions. Lee, can you add this when you apply this patch?
Will it's not legislation, but it does help us a great deal.
Reviewed-by: Thierry Reding <redacted>
Sure.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help