22/09/2017 09:37, Shreyansh Jain: > On Friday 22 September 2017 12:23 PM, Thomas Monjalon wrote: > > 22/09/2017 08:47, Shreyansh Jain: > >> On Friday 22 September 2017 03:26 AM, Thomas Monjalon wrote: > >>> 23/08/2017 16:11, Shreyansh Jain: > >>>> --- a/MAINTAINERS > >>>> +++ b/MAINTAINERS > >>>> @@ -409,6 +409,7 @@ NXP dpaa > >>>> M: Hemant Agrawal <hemant.agra...@nxp.com> > >>>> M: Shreyansh Jain <shreyansh.j...@nxp.com> > >>>> F: drivers/bus/dpaa/ > >>>> +F: drivers/mempool/dpaa/ > >>>> F: doc/guides/nics/dpaa.rst > >>>> F: doc/guides/nics/features/dpaa.ini > >>> > >>> This kind of patch can be squashed in the first patch introducing > >>> this new directory. > >> > >> Then the patch script (devtools/check-git-log.sh) reports error - I > >> think. That is the primary reason I split them across multiple patches. > >> You sure that doesn't matter? > > > > Which error? > > > > To be clear I suggest to squash with patch 19 where > > drivers/mempool/dpaa/Makefile is introduced. > > Yes, I understand that. > It would report error that the headline is wrong because I am hitting > different directories - "MAINTAINERS" and "drivers/mempool/*" with the > same patch having headline "mempool/*".
The test you are talking about has this comment: # check headline prefix when touching only drivers, e.g. net/<driver name> If you hit a warning, there is a bug.