Re: [PATCH 2/2] MAINTAINERS: ARM Versatile Express platform

[Pawel Moll <pawel.moll@arm.com>]

On Wed, 2014-11-26 at 01:59 +0000, Joe Perches wrote:
> On Tue, 2014-11-25 at 17:54 -0800, Mike Turquette wrote:
> > Quoting Pawel Moll (2014-11-25 10:17:35)
> > > This patch adds a separate section for the ARM
> > > Versatile Express platform maintainers, listing
> > > all different bits and bobs used by it.
> []
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> []
> > > +ARM/VERSATILE EXPRESS PLATFORM
> []
> > > +F:     arch/arm/boot/dts/vexpress-*
> > > +F:     arch/arm/mach-vexpress/*
> > > +F:     drivers/bus/vexpress-config.c
> > > +F:     drivers/clk/versatile/clk-vexpress-osc.c
> > > +F:     drivers/clocksource/versatile.c
> > > +F:     drivers/cpufreq/vexpress-spc-cpufreq.c
> > > +F:     drivers/hwmon/vexpress.c
> > > +F:     drivers/mfd/vexpress-sysreg.c
> > > +F:     drivers/misc/vexpress-vexpress-syscfg.c
> > > +F:     drivers/power/reset/vexpress-poweroff.c
> > > +F:     drivers/regulator/vexpress.c
> > > +F:     include/linux/vexpress.h
> > >
> 
> maybe
> F:	arch/arm/boot/dts/vexpress*

Ok.

> F:	arch/arm/mach-vexpress/

Right, of course. Whole directory.

> F:	*/*/vexpress*

Cool, didn't think about multiple wildcards. 

> F:	*/*/*/*vexpress*

I'd keep the clk-vexpress-osc.c explicit - it's an odd one, and don't
feel comfortable with the wildcard before vexpress.

> F:	drivers/clocksource/versatile.c

Thanks for the suggestion, sill spin the patch in a second.

Pawel