mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] soc: mediatek: mtk-socinfo: select CONFIG_SOC_BUS
@ 2024-04-08 12:55 Arnd Bergmann
  2024-04-08 12:57 ` AngeloGioacchino Del Regno
  2024-04-08 12:59 ` Daniel Golle
  0 siblings, 2 replies; 4+ messages in thread
From: Arnd Bergmann @ 2024-04-08 12:55 UTC (permalink / raw)
  To: Matthias Brugger, AngeloGioacchino Del Regno, William-tw Lin
  Cc: Arnd Bergmann, Ulf Hansson, linux-kernel, linux-arm-kernel,
	linux-mediatek

From: Arnd Bergmann <arnd@arndb.de>

This is needed to avoid a link failure:

ERROR: modpost: "soc_device_register" [drivers/soc/mediatek/mtk-socinfo.ko] undefined!
ERROR: modpost: "soc_device_unregister" [drivers/soc/mediatek/mtk-socinfo.ko] undefined!

Fixes: 423a54da3c7e ("soc: mediatek: mtk-socinfo: Add driver for getting chip information")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/soc/mediatek/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/soc/mediatek/Kconfig b/drivers/soc/mediatek/Kconfig
index 50c664b65f4d..1b7afb19ccd6 100644
--- a/drivers/soc/mediatek/Kconfig
+++ b/drivers/soc/mediatek/Kconfig
@@ -72,6 +72,7 @@ config MTK_SOCINFO
 	tristate "MediaTek SoC Information"
 	default y
 	depends on NVMEM_MTK_EFUSE
+	select SOC_BUS
 	help
 	  The MediaTek SoC Information (mtk-socinfo) driver provides
 	  information about the SoC to the userspace including the
-- 
2.39.2


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] soc: mediatek: mtk-socinfo: select CONFIG_SOC_BUS
  2024-04-08 12:55 [PATCH] soc: mediatek: mtk-socinfo: select CONFIG_SOC_BUS Arnd Bergmann
@ 2024-04-08 12:57 ` AngeloGioacchino Del Regno
  2024-04-08 12:59 ` Daniel Golle
  1 sibling, 0 replies; 4+ messages in thread
From: AngeloGioacchino Del Regno @ 2024-04-08 12:57 UTC (permalink / raw)
  To: Arnd Bergmann, Matthias Brugger, William-tw Lin
  Cc: Arnd Bergmann, Ulf Hansson, linux-kernel, linux-arm-kernel,
	linux-mediatek

Il 08/04/24 14:55, Arnd Bergmann ha scritto:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> This is needed to avoid a link failure:
> 
> ERROR: modpost: "soc_device_register" [drivers/soc/mediatek/mtk-socinfo.ko] undefined!
> ERROR: modpost: "soc_device_unregister" [drivers/soc/mediatek/mtk-socinfo.ko] undefined!
> 
> Fixes: 423a54da3c7e ("soc: mediatek: mtk-socinfo: Add driver for getting chip information")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] soc: mediatek: mtk-socinfo: select CONFIG_SOC_BUS
  2024-04-08 12:55 [PATCH] soc: mediatek: mtk-socinfo: select CONFIG_SOC_BUS Arnd Bergmann
  2024-04-08 12:57 ` AngeloGioacchino Del Regno
@ 2024-04-08 12:59 ` Daniel Golle
  2024-04-08 14:55   ` AngeloGioacchino Del Regno
  1 sibling, 1 reply; 4+ messages in thread
From: Daniel Golle @ 2024-04-08 12:59 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Matthias Brugger, AngeloGioacchino Del Regno, William-tw Lin,
	Arnd Bergmann, Ulf Hansson, linux-kernel, linux-arm-kernel,
	linux-mediatek

Hi,

On Mon, Apr 08, 2024 at 02:55:00PM +0200, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> This is needed to avoid a link failure:
> 
> ERROR: modpost: "soc_device_register" [drivers/soc/mediatek/mtk-socinfo.ko] undefined!
> ERROR: modpost: "soc_device_unregister" [drivers/soc/mediatek/mtk-socinfo.ko] undefined!
> 

I've tried to take care of this in the exact same way a long time
ago. Don't ask me why it didn't find it's way into stable yet

https://patchwork.kernel.org/project/linux-mediatek/patch/cc8f7f7da5bdccce514a320e0ae7468659cf7346.1707327680.git.daniel@makrotopia.org/


Cheers


Daniel

> Fixes: 423a54da3c7e ("soc: mediatek: mtk-socinfo: Add driver for getting chip information")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
>  drivers/soc/mediatek/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/soc/mediatek/Kconfig b/drivers/soc/mediatek/Kconfig
> index 50c664b65f4d..1b7afb19ccd6 100644
> --- a/drivers/soc/mediatek/Kconfig
> +++ b/drivers/soc/mediatek/Kconfig
> @@ -72,6 +72,7 @@ config MTK_SOCINFO
>  	tristate "MediaTek SoC Information"
>  	default y
>  	depends on NVMEM_MTK_EFUSE
> +	select SOC_BUS
>  	help
>  	  The MediaTek SoC Information (mtk-socinfo) driver provides
>  	  information about the SoC to the userspace including the
> -- 
> 2.39.2
> 
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] soc: mediatek: mtk-socinfo: select CONFIG_SOC_BUS
  2024-04-08 12:59 ` Daniel Golle
@ 2024-04-08 14:55   ` AngeloGioacchino Del Regno
  0 siblings, 0 replies; 4+ messages in thread
From: AngeloGioacchino Del Regno @ 2024-04-08 14:55 UTC (permalink / raw)
  To: Daniel Golle, Arnd Bergmann
  Cc: Matthias Brugger, William-tw Lin, Arnd Bergmann, Ulf Hansson,
	linux-kernel, linux-arm-kernel, linux-mediatek

Il 08/04/24 14:59, Daniel Golle ha scritto:
> Hi,
> 
> On Mon, Apr 08, 2024 at 02:55:00PM +0200, Arnd Bergmann wrote:
>> From: Arnd Bergmann <arnd@arndb.de>
>>
>> This is needed to avoid a link failure:
>>
>> ERROR: modpost: "soc_device_register" [drivers/soc/mediatek/mtk-socinfo.ko] undefined!
>> ERROR: modpost: "soc_device_unregister" [drivers/soc/mediatek/mtk-socinfo.ko] undefined!
>>
> 
> I've tried to take care of this in the exact same way a long time
> ago. Don't ask me why it didn't find it's way into stable yet
> 
> https://patchwork.kernel.org/project/linux-mediatek/patch/cc8f7f7da5bdccce514a320e0ae7468659cf7346.1707327680.git.daniel@makrotopia.org/
> 
> 

Sorry, went out of my radar for reasons.
Your commit should take precedence.

Angelo

> Cheers
> 
> 
> Daniel
> 
>> Fixes: 423a54da3c7e ("soc: mediatek: mtk-socinfo: Add driver for getting chip information")
>> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>> ---
>>   drivers/soc/mediatek/Kconfig | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/soc/mediatek/Kconfig b/drivers/soc/mediatek/Kconfig
>> index 50c664b65f4d..1b7afb19ccd6 100644
>> --- a/drivers/soc/mediatek/Kconfig
>> +++ b/drivers/soc/mediatek/Kconfig
>> @@ -72,6 +72,7 @@ config MTK_SOCINFO
>>   	tristate "MediaTek SoC Information"
>>   	default y
>>   	depends on NVMEM_MTK_EFUSE
>> +	select SOC_BUS
>>   	help
>>   	  The MediaTek SoC Information (mtk-socinfo) driver provides
>>   	  information about the SoC to the userspace including the
>> -- 
>> 2.39.2
>>
>>


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-04-08 14:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-08 12:55 [PATCH] soc: mediatek: mtk-socinfo: select CONFIG_SOC_BUS Arnd Bergmann
2024-04-08 12:57 ` AngeloGioacchino Del Regno
2024-04-08 12:59 ` Daniel Golle
2024-04-08 14:55   ` AngeloGioacchino Del Regno

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®