[PATCH V2] MAINTAINERS: Add Dialog PMIC search terms for missing documentation and header files

[PATCH V2] MAINTAINERS: Add Dialog PMIC search terms for missing documentation and header files

[Steve Twiss]

From: Steve Twiss <stwiss.opensource@diasemi.com>

Dialog Semiconductor support would like to follow files by adding to the
existing MAINTAINERS search terms. The update will allow us to follow
files for PMIC documentation bindings and header files.

The full list is:

DT bindings
 - Documentation/devicetree/bindings/mfd/da9052-i2c.txt
 - Documentation/devicetree/bindings/mfd/da9055.txt
 - Documentation/devicetree/bindings/mfd/da9062.txt
 - Documentation/devicetree/bindings/mfd/da9063.txt
 - Documentation/devicetree/bindings/mfd/da9150.txt
 - Documentation/devicetree/bindings/regulator/da9210.txt
 - Documentation/devicetree/bindings/regulator/da9211.txt
Header files
 - include/linux/mfd/da9062/core.h
 - include/linux/mfd/da9062/registers.h
 - include/linux/regulator/da9211.h

Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com>

---
Hi Lee & Mark,

The majority of these updates are for MFD documentation and headers,
although there is a mixture with the regulators as well.

I previously send a patch TO: Lee and CC:'ed Mark, but I think now I
need to resend V2 with Mark in the TO: field.
https://lkml.org/lkml/2016/5/11/419

I have also updated this V2 patch to align with linux-next/v4.6
and removed the change to the onkey driver.

Lee: with Mark's Ack, would it be possible for you to take these
through please?

Regards,
Steve


 MAINTAINERS | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 9c567a4..742a860 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3574,6 +3574,8 @@ M:	Support Opensource <support.opensource@diasemi.com>
 W:	http://www.dialog-semiconductor.com/products
 S:	Supported
 F:	Documentation/hwmon/da90??
+F:	Documentation/devicetree/bindings/mfd/da90*.txt
+F:	Documentation/devicetree/bindings/regulator/da92*.txt
 F:	Documentation/devicetree/bindings/sound/da[79]*.txt
 F:	drivers/gpio/gpio-da90??.c
 F:	drivers/hwmon/da90??-hwmon.c
@@ -3594,8 +3596,10 @@ F:	drivers/watchdog/da90??_wdt.c
 F:	include/linux/mfd/da903x.h
 F:	include/linux/mfd/da9052/
 F:	include/linux/mfd/da9055/
+F:	include/linux/mfd/da9062/
 F:	include/linux/mfd/da9063/
 F:	include/linux/mfd/da9150/
+F:	include/linux/regulator/da9211.h
 F:	include/sound/da[79]*.h
 F:	sound/soc/codecs/da[79]*.[ch]
 
-- 
end-of-patch for PATCH V2

Re: [PATCH V2] MAINTAINERS: Add Dialog PMIC search terms for missing documentation and header files

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 1077 bytes --]

On Tue, May 24, 2016 at 08:53:46AM +0100, Steve Twiss wrote:

> The majority of these updates are for MFD documentation and headers,
> although there is a mixture with the regulators as well.

> I previously send a patch TO: Lee and CC:'ed Mark, but I think now I
> need to resend V2 with Mark in the TO: field.
> https://lkml.org/lkml/2016/5/11/419

No, the issue here is that this is MFD patch with a non-obvious subject
line, it's getting a lot of new versions (which usually indicates that
there's stuff going on and it might well change again quickly) and it's
getting replies (which would normally suggest it's being taken care of
or needs more revision).  This all means that it looks like something
best left for later.

> I have also updated this V2 patch to align with linux-next/v4.6
> and removed the change to the onkey driver.

> Lee: with Mark's Ack, would it be possible for you to take these
> through please?

Acked-by: Mark Brown <broonie@kernel.org>

We *really* shouldn't be blocking on my ack for trivial changes outside
the subsystem like this though :(

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

RE: [PATCH V2] MAINTAINERS: Add Dialog PMIC search terms for missing documentation and header files

[Opensource [Steve Twiss]]

On 08 June 2016 15:19, Mark Brown wrote:

> Subject: Re: [PATCH V2] MAINTAINERS: Add Dialog PMIC search terms for
> missing documentation and header files
> 
> On Tue, May 24, 2016 at 08:53:46AM +0100, Steve Twiss wrote:
> 
> > The majority of these updates are for MFD documentation and headers,
> > although there is a mixture with the regulators as well.
> 
> > I previously send a patch TO: Lee and CC:'ed Mark, but I think now I
> > need to resend V2 with Mark in the TO: field.
> > https://lkml.org/lkml/2016/5/11/419
> 
> No, the issue here is that this is MFD patch with a non-obvious subject
> line,

Thanks Mark,

I originally used "Dialog PMIC" in the Subject line to describe the coverage
of MFD, regulator and input/ONKEY. I will be more explicit in future.

> it's getting a lot of new versions (which usually indicates that
> there's stuff going on and it might well change again quickly) and it's
> getting replies (which would normally suggest it's being taken care of
> or needs more revision).  This all means that it looks like something
> best left for later.
> 
> > I have also updated this V2 patch to align with linux-next/v4.6
> > and removed the change to the onkey driver.
> 
> > Lee: with Mark's Ack, would it be possible for you to take these
> > through please?
> 
> Acked-by: Mark Brown <broonie@kernel.org>
> 
> We *really* shouldn't be blocking on my ack for trivial changes outside
> the subsystem like this though :(

There had been no reply since my first e-mail round about a month ago, so I
figured there was something wrong with the patch submission.

Lee has since applied PATCH V3 which I resent earlier today. That version finally
removed the regulator files and deleted you from the TO: and CC: e-mail fields 
thereby removing you as a dependency.

I will have to send the regulator updates separately at a later date.

In future I will split up the responsibility and define the Subject line better to
remove any ambiguity.

Regards,
Steve

Re: [PATCH V2] MAINTAINERS: Add Dialog PMIC search terms for missing documentation and header files

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 699 bytes --]

On Wed, Jun 08, 2016 at 05:25:30PM +0000, Opensource [Steve Twiss] wrote:
> On 08 June 2016 15:19, Mark Brown wrote:

> > We *really* shouldn't be blocking on my ack for trivial changes outside
> > the subsystem like this though :(

> There had been no reply since my first e-mail round about a month ago, so I
> figured there was something wrong with the patch submission.

It was the activity pattern on the patch more than anything else, and of
course the merge window was in that month as well.

> In future I will split up the responsibility and define the Subject line better to
> remove any ambiguity.

Like I say a trivial maintainers update like this really should just get
applied anyway.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

Re: [PATCH V2] MAINTAINERS: Add Dialog PMIC search terms for missing documentation and header files

[Lee Jones]

On Wed, 08 Jun 2016, Mark Brown wrote:
> On Wed, Jun 08, 2016 at 05:25:30PM +0000, Opensource [Steve Twiss] wrote:
> > On 08 June 2016 15:19, Mark Brown wrote:
> 
> > > We *really* shouldn't be blocking on my ack for trivial changes outside
> > > the subsystem like this though :(
> 
> > There had been no reply since my first e-mail round about a month ago, so I
> > figured there was something wrong with the patch submission.
> 
> It was the activity pattern on the patch more than anything else, and of
> course the merge window was in that month as well.
> 
> > In future I will split up the responsibility and define the Subject line better to
> > remove any ambiguity.
> 
> Like I say a trivial maintainers update like this really should just get
> applied anyway.

Right.  I was about to apply this regardless.

The issue was not specific to this patch. I tend to go dark during
the merge-window to concentrate on my 'real job' (TM).  I applied your
other patch because it's the one I came across first.  This one was
marked as "to review" (as I am doing now).

Don't worry about splitting, I will apply this and remove the other.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog