Re: [PATCH V3 02/16] PM / OPP: Create a directory for opp bindings

On 30-07-15, 11:24, Rob Herring wrote:

On Thu, Jul 30, 2015 at 8:49 AM, Viresh Kumar [off-list ref] wrote:

quoted

On 30-07-15, 08:18, Rob Herring wrote:

quoted

On Wed, Jul 29, 2015 at 5:52 AM, Viresh Kumar [off-list ref] wrote:

quoted

More platform specific extended opp bindings will follow and it would be
easy to manage them with a directory for opp. Lets create that and move
the existing opp bindings into it.

Cc: Rob Herring <redacted>

Use get_maintainers.pl please so you send to right people and right email.

Already done, you were just cc'd as an extra guy :)

No, because no one that is a maintainer of
Documentation/devicetree/bindings is cc'ed here. You did on the other
patch, but not here as neither Ian, Mark, Kumar nor Pawel are cc'ed.

No. I sent all of them together and get_maintainers is invoked for all
of them. I checked again and its a bug with get_maintainers.

@Joe: Can you please take a look at that?

Problem description: Because my patch only had this:

-----------8<-------------

---
 Documentation/devicetree/bindings/{power => opp}/opp.txt | 0
 1 file changed, 0 insertions(+), 0 deletions(-)
 rename Documentation/devicetree/bindings/{power => opp}/opp.txt (100%)

diff --git a/Documentation/devicetree/bindings/power/opp.txt b/Documentation/devicetree/bindings/opp/opp.txt
similarity index 100%
rename from Documentation/devicetree/bindings/power/opp.txt
rename to Documentation/devicetree/bindings/opp/opp.txt

------------>8------------

get_maintainers reports this for the patch:

scripts/get_maintainer.pl: file
'0001-PM-OPP-Create-a-directory-for-opp-bindings.patch' doesn't appear
to be a patch.  Add -f to options?

 

quoted

Cc: Rob Herring <redacted>

Still wrong...

Let me send the patch as reply to this one, with personally cc'ing all
people you want me to.

-- 
viresh