Re: [PATCH v3 0/7] LMU Common code intro

[Lee Jones]

On Wed, 01 May 2019, Jacek Anaszewski wrote:

> Ekhm, I forgot to add the main recipient.
> 
> Adding Lee.
> 
> On 5/1/19 12:05 AM, Jacek Anaszewski wrote:
> > Hi Lee,
> > 
> > This patch set has dependency on the previous one for lm3532, which
> > also touches ti-lmu.txt bindings, and for which I already created
> > immutable branch. Now if I created another immutable branch for
> > this patch set we would have to resolve conflicts between the two,
> > as they would both be based on 5.1-rc1. Adds much overhead and
> > is error prone. Therefore I propose to apply this patch set on
> > top of my merge of LED tree with the immutable branch for lm3532.
> > 
> > Please let me know if you see it differently. I'll wait for your
> > response until Friday, and then proceed with the above as I think
> > it should be harmless for MFD.

Looks like this set wasn't sent until -rc6 which is when I normally
stop applying significant patches.  I think it's best that we wait
until the next rotation.  That way all this dependency related
hoop-jumping just goes away.

> > On 4/30/19 9:17 PM, Dan Murphy wrote:
> > > Hello
> > > 
> > > I have added the Reviewed-by for dt bindings as well as made the
> > > Kconfig change
> > > pointed out for the common code flag
> > > 
> > > Dan
> > > 
> > > Dan Murphy (7):
> > > ÂÂ dt-bindings: mfd: LMU: Fix lm3632 dt binding example
> > > ÂÂ dt-bindings: mfd: LMU: Add the ramp up/down property
> > > ÂÂ dt-bindings: mfd: LMU: Add ti,brightness-resolution
> > > ÂÂ leds: TI LMU: Add common code for TI LMU devices
> > > ÂÂ dt-bindings: ti-lmu: Modify dt bindings for the LM3697
> > > ÂÂ mfd: ti-lmu: Remove support for LM3697
> > > ÂÂ leds: lm3697: Introduce the lm3697 driver
> > > 
> > >  .../devicetree/bindings/leds/leds-lm3697.txt | 73 ++++
> > > Â .../devicetree/bindings/mfd/ti-lmu.txtÂÂÂÂÂÂÂ |Â 56 ++-
> > > Â drivers/leds/KconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |Â 15 +
> > > Â drivers/leds/MakefileÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 2 +
> > > Â drivers/leds/leds-lm3697.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ | 395 ++++++++++++++++++
> > > Â drivers/leds/ti-lmu-led-common.cÂÂÂÂÂÂÂÂÂÂÂÂÂ | 155 +++++++
> > > Â drivers/mfd/KconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 2 +-
> > > Â drivers/mfd/ti-lmu.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |Â 17 -
> > > Â include/linux/mfd/ti-lmu-register.hÂÂÂÂÂÂÂÂÂÂ |Â 44 --
> > > Â include/linux/mfd/ti-lmu.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 1 -
> > > Â include/linux/ti-lmu-led-common.hÂÂÂÂÂÂÂÂÂÂÂÂ |Â 47 +++
> > > Â 11 files changed, 712 insertions(+), 95 deletions(-)
> > > Â create mode 100644
> > > Documentation/devicetree/bindings/leds/leds-lm3697.txt
> > > Â create mode 100644 drivers/leds/leds-lm3697.c
> > > Â create mode 100644 drivers/leds/ti-lmu-led-common.c
> > > Â create mode 100644 include/linux/ti-lmu-led-common.h
> > > 
> > 
> > 
> 

-- 
Lee Jones [æçæ]
Linaro Services Technical Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog