Re: [PATCH V1 1/2] mfd: da9063: Upgrade of register definitions to support production silicon

From: Lee Jones
Date: Mon Mar 10 2014 - 12:28:03 EST


> >> From: Opensource [Steve Twiss] <stwiss.opensource@xxxxxxxxxxx>
> >>
> >> 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@xxxxxxxxxxx>
> >> ---
> >> 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.
>
> Hi Lee,
>
> Thanks for this.
>
> It looks like made a mess of the To: and Cc: e-mail list in my submission patches
> for this one. The second patch (the RTC driver) should have been CC'ed to you
> also.
> Sorry. I will try to get my e-mail lists in order for my next submission.
>
> The RTC driver component relies on the register change patch which you just
> applied. This RTC component was reviewed and Ack'ed by Alessandro Zummo
> here:
> http://www.kernelhub.org/?p=2&msg=427642
>
> There was no way for you to know any of this because I had not CC'ed you
> properly on this part.
>
> It is possible to add the RTC component as well or shall I resend that part
> again?

It needs to be in my inbox for me to apply it.

Can you send it again with Alessandro and my Acks applied please?

--
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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/