Re: [PATCH V2 1/3] Thermal: rename thermal_sys.c to thermal_core.c

[Eduardo Valentin]

On 11-04-2013 21:12, Zhang Rui wrote:
> On Thu, 2013-04-04 at 16:24 -0400, Eduardo Valentin wrote:
> > On 29-03-2013 10:26, Zhang Rui wrote:
> > > this is the preparation work to build all the thermal core framework
> > > source file, like governors, cpu cooling, etc, into one module.
> > >
> > > No functional change in this patch.
> > >
> > > Signed-off-by: Zhang Rui <rui.zhang@xxxxxxxxx>
> >
> > Durga and me have already reviewed and acked this patch. Can you please
> > add our Acked-by on this patch?
>
> sure.
> BTW, I've talked with Rafael, and we agree on adding prefix "acpi_" for
> ACPI drivers, and then we can rename thermal_sys.ko to thermal.ko.

Good!

> But I'll try to push this patch set for 3.10, to fix the problem on
> hand, also for easily back porting to 3.8 and 3.9 stable kernel.
> Then, we'll rename a couple of ACPI drivers and then thermal driver in
> 3.11.
>
> what do you think?

I am OK with this approach.

Thanks for keeping this up.

> thanks,
> rui
>
> > > ---
> > >    drivers/thermal/Makefile                          |    1 +
> > >    drivers/thermal/{thermal_sys.c => thermal_core.c} |    0
> > >    2 files changed, 1 insertion(+)
> > >    rename drivers/thermal/{thermal_sys.c => thermal_core.c} (100%)
> > >
> > > diff --git a/drivers/thermal/Makefile b/drivers/thermal/Makefile
> > > index d3a2b38..b2009bd 100644
> > > --- a/drivers/thermal/Makefile
> > > +++ b/drivers/thermal/Makefile
> > > @@ -3,6 +3,7 @@
> > >    #
> > >
> > >    obj-$(CONFIG_THERMAL)		+= thermal_sys.o
> > > +thermal_sys-y			+= thermal_core.o
> > >
> > >    # governors
> > >    obj-$(CONFIG_THERMAL_GOV_FAIR_SHARE)	+= fair_share.o
> > > diff --git a/drivers/thermal/thermal_sys.c b/drivers/thermal/thermal_core.c
> > > similarity index 100%
> > > rename from drivers/thermal/thermal_sys.c
> > > rename to drivers/thermal/thermal_core.c

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/