Re: [PATCH V2 1/3] Thermal: rename thermal_sys.c to thermal_core.c
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 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH V2 1/3] Thermal: rename thermal_sys.c to thermal_core.c
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 > > 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. 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? 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH V2 1/3] Thermal: rename thermal_sys.c to thermal_core.c
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 Durga and me have already reviewed and acked this patch. Can you please add our Acked-by on this patch? --- 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH V2 1/3] Thermal: rename thermal_sys.c to thermal_core.c
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 --- 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 -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/