Re: [PATCH v8 1/6] thermal: mediatek: Relocate driver to mediatek folder

Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> · Fri, 29 Jul 2022 17:35:57 +0200

Hi Balsam,

On 29/07/2022 17:21, Balsam CHIHI wrote:
On Fri, Jul 29, 2022 at 5:19 PM Balsam CHIHI <bchihi@xxxxxxxxxxxx> wrote:

[ ... ]

config MTK_SOC_THERMAL
         tristate "Temperature sensor driver for MediaTek SoCs"
         depends on HAS_IOMEM
         depends on NVMEM
         depends on RESET_CONTROLLER
         help
                 Enable this option if you want to get SoC temperature
                 information for MediaTek platforms. This driver
                 configures thermal controllers to collect temperature
                 via AUXADC interface.

drivers/thermal/mediatek/Makefile:

obj-$(MTK_SOC_THERMAL)  += soc_temp.o

However, rename 'soc_temp' to something more SoC explicit, eg.
mtxxx_thermal.c

Thanks

    -- Daniel

Hello Daniel,

Thank you for the feedback.
Changes have been made as you suggested.
Resubmitting seen.
*soon.

You may want to wait next week before sending a new version, I'm still 
reviewing your series (note you can send it anyway if you prefer)

--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog