On 30-07-15, 11:24, Rob Herring wrote: > On Thu, Jul 30, 2015 at 8:49 AM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote: > > On 30-07-15, 08:18, Rob Herring wrote: > >> On Wed, Jul 29, 2015 at 5:52 AM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote: > >> > 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 <rob.herring@xxxxxxxxxx> > >> > >> 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? > > Cc: Rob Herring <rob.herring@xxxxxxxxxx> > > Still wrong... Let me send the patch as reply to this one, with personally cc'ing all people you want me to. -- viresh -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html