Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1739337

Re: [PATCH] MAINTAINERS: Hibernation or S2R doesn't cover entire power directory

Show all headers | View raw

On 25-09-17, 00:44, Rafael J. Wysocki wrote:
> On Wed, Sep 20, 2017 at 5:58 PM, Viresh Kumar <viresh.kumar@linaro.org> wrote:
> > The command:
> >
> >  $ scripts/get_maintainer.pl -f drivers/base/power/opp/core.c
> >
> > is incorrectly picking maintainers from Hibernation and Suspend to RAM
> > as well, simply because the entries in MAINTAINERS file contains the
> > entire power directory. Though the only concerning file is main.c
> > really.
> 
> No, it is not the only one.  The others are common.c, trace.c,
> wakeup.c, wakeirq.c, sysfs.c, generic_ops.c and power.h (if I have not
> overlooked any).
> 
> I'm not actually sure how to fix this issue other than by listing all
> of the files in question directly in there.

Should I just use *.c for now? Yeah, that will still cover lot more files, but
at least the directories, like OPP, are not going to show those maintainers.

Or I was actually thinking of moving the entire OPP directory out of
drivers/base/power, and move to drivers/opp, just like cpufreq and cpuidle. Will
that be okay?

-- 
viresh

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] MAINTAINERS: Hibernation or S2R doesn't cover entire power directory Viresh Kumar <viresh.kumar@linaro.org> - 2017-09-20 18:00 +0200
  Re: [PATCH] MAINTAINERS: Hibernation or S2R doesn't cover entire  power directory "Rafael J. Wysocki" <rafael@kernel.org> - 2017-09-25 00:50 +0200
    Re: [PATCH] MAINTAINERS: Hibernation or S2R doesn't cover entire  power directory Viresh Kumar <viresh.kumar@linaro.org> - 2017-09-26 01:20 +0200
      Re: [PATCH] MAINTAINERS: Hibernation or S2R doesn't cover entire  power directory "Rafael J. Wysocki" <rafael@kernel.org> - 2017-09-26 01:20 +0200

csiph-web