Re: [PATCH v3 21/27] MAINTAINERS: get rid of non-existing Documentation/fpga
From: Alan Tull
Date: Thu Jun 14 2018 - 14:10:26 EST
On Thu, Jun 14, 2018 at 11:52 AM, Mauro Carvalho Chehab
<mchehab+samsung@xxxxxxxxxx> wrote:
> Em Thu, 14 Jun 2018 11:39:20 -0500
> Alan Tull <atull@xxxxxxxxxx> escreveu:
>
>> On Thu, Jun 14, 2018 at 11:09 AM, Mauro Carvalho Chehab
>> <mchehab+samsung@xxxxxxxxxx> wrote:
>> > This directory seems to have moved to a new place
>> > (Documentation/driver-api/fpga/), with is already at MAINTAINERS.
>> >
>> > So, get rid of the old location.
>>
>> Wow, you caught that fast!
>
> Just a coincidence :-) I had to rebase a pending patch series close to
> the end of the merge window.
>
> When I ran the check tool:
> ./scripts/documentation-file-ref-check
>
> It detected this.
Nice tool. Thanks for the explanation, it's always good to learn
about another one.
>
>> There's other documentation coming to this
>> directory soon. IMO we might as well leave it. We don't need to keep
>> adding and deleting this line from MAINTAINERS.
>
> Yeah, sure. I'll drop this hunk for v4.
>
Thanks!
>>
>> Also, why didn't you cc linux-fpga mailing list on this? I guess you
>> ran get_maintainer.pl after adding this patch? :)
>
> Yeah, that was the case. I also had to sanitize the c/c for this
> specific patch as get_maintainers.pl isn't good enough to check for
> MAINTAINERS access... it usually just get some random noise...
>
>>
>> Alan
>>
>> >
>> > Cc: Alan Tull <atull@xxxxxxxxxx>
>> > Cc: Moritz Fischer <mdf@xxxxxxxxxx>
>> > Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@xxxxxxxxxx>
>> > ---
>> > MAINTAINERS | 1 -
>> > 1 file changed, 1 deletion(-)
>> >
>> > diff --git a/MAINTAINERS b/MAINTAINERS
>> > index 783ce44b3e7a..67641f5bb373 100644
>> > --- a/MAINTAINERS
>> > +++ b/MAINTAINERS
>> > @@ -5632,7 +5632,6 @@ L: linux-fpga@xxxxxxxxxxxxxxx
>> > S: Maintained
>> > T: git git://git.kernel.org/pub/scm/linux/kernel/git/atull/linux-fpga.git
>> > Q: http://patchwork.kernel.org/project/linux-fpga/list/
>> > -F: Documentation/fpga/
>> > F: Documentation/driver-api/fpga/
>> > F: Documentation/devicetree/bindings/fpga/
>> > F: drivers/fpga/
>> > --
>> > 2.17.1
>> >
>
>
>
> Thanks,
> Mauro