[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140310140238.GA15558@lee--X1>
Date: Mon, 10 Mar 2014 14:02:38 +0000
From: Lee Jones <lee.jones@...aro.org>
To: "Opensource [Steve Twiss]" <stwiss.opensource@...semi.com>
Cc: Mark Brown <broonie@...aro.org>,
Philipp Zabel <p.zabel@...gutronix.de>,
Samuel Ortiz <sameo@...ux.intel.com>,
David Dajun Chen <david.chen@...semi.com>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH V1 1/2] mfd: da9063: Upgrade of register definitions to
support production silicon
> From: Opensource [Steve Twiss] <stwiss.opensource@...semi.com>
>
> This patch updates the register definitions for DA9063 to support the
> production silicon variant code ID (0x5). These changes are not backwards
> compatible with the previous register definitions and can only be used
> with the production variant of DA9063.
>
> Signed-off-by: Opensource [Steve Twiss] <stwiss.opensource@...semi.com>
> ---
> Checks performed with next-20140306/scripts/checkpatch.pl
> registers.h total: 0 errors, 0 warnings, 1032 lines checked
>
> A brief summary of the changes include:
>
> - Introduction of a new DA9063_REG_ALARM_S register (now appearing at
> addr=0x46) allowing alarm second resolutions and which causes a shift
> in the register map for the RTC registers;
> - Two new configuration registers DA9063_REG_CONFIG_M/N at 0x112 & 0x113;
> - Modified register ranges for MON_REG_[1-6] which now appears at the
> addr=0x114 onwards;
> - New register addresses for the general purpose registers GP_ID_[0-19]
> (now appearing at 0x121 onwards);
> - Renaming of some definitions to match our hardware design conventions;
> - There are also some bit-pattern additions that define some functionality
> alterations of the registers. One notable addition is:
> DA9063_BUCK_SLOWSTART can also be found in the CONTROL_B register and
> enables a BUCK slow start (reduced inrush current; increased startup
> time);
>
> include/linux/mfd/da9063/registers.h | 120 ++++++++++++++++++----------------
> 1 file changed, 62 insertions(+), 58 deletions(-)
Well, as this doesn't seem to cause any issues when building and I'm
going to guess that the other patch in the set won't be a problem (as
you didn't send it to me, so I'm gussing it's insignificant), I'll
apply the patch.
Applied, thanks.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
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/
Powered by blists - more mailing lists