linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: defconfig: Do not override the MTK_PMIC_WRAP symbol
@ 2021-04-23  7:52 Enric Balletbo i Serra
  2021-05-14  7:54 ` Matthias Brugger
  0 siblings, 1 reply; 3+ messages in thread
From: Enric Balletbo i Serra @ 2021-04-23  7:52 UTC (permalink / raw)
  To: linux-kernel
  Cc: Collabora Kernel ML, linux-mediatek, matthias.bgg, drinkcat,
	hsinyi, Arnd Bergmann, Bjorn Andersson, Catalin Marinas,
	Dmitry Baryshkov, Guido Günther, Krzysztof Kozlowski,
	Lad Prabhakar, Max Krummenacher, Michael Walle, Nishanth Menon,
	Shawn Guo, Will Deacon, linux-arm-kernel

Commit 'fbbe38309d56 ("arm64: defconfig: Allow mt8173-based boards to boot
from usb")' added the MTK_PMIC_WRAP config built-in. It needs to be
built-in in order to be able to boot from USB or the MMC without needing
a ramdisk, but that symbol was already defined as a module so now we are
getting the following warning:

  arch/arm64/configs/defconfig:996:warning: override: reassigning to symbol MTK_PMIC_WRAP

Remove the MTK_PMIC_WRAP=m from the defconfig to remove the error.

Fixes: fbbe38309d56 ("arm64: defconfig: Allow mt8173-based boards to boot from usb")
Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
---

 arch/arm64/configs/defconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 08c6f769df9a..9907a431db0d 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -491,7 +491,6 @@ CONFIG_SPI_S3C64XX=y
 CONFIG_SPI_SH_MSIOF=m
 CONFIG_SPI_SUN6I=y
 CONFIG_SPI_SPIDEV=m
-CONFIG_MTK_PMIC_WRAP=m
 CONFIG_SPMI=y
 CONFIG_PINCTRL_SINGLE=y
 CONFIG_PINCTRL_MAX77620=y
-- 
2.30.2


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

* Re: [PATCH] arm64: defconfig: Do not override the MTK_PMIC_WRAP symbol
  2021-04-23  7:52 [PATCH] arm64: defconfig: Do not override the MTK_PMIC_WRAP symbol Enric Balletbo i Serra
@ 2021-05-14  7:54 ` Matthias Brugger
  2021-06-11 21:48   ` Petr Vorel
  0 siblings, 1 reply; 3+ messages in thread
From: Matthias Brugger @ 2021-05-14  7:54 UTC (permalink / raw)
  To: Enric Balletbo i Serra, linux-kernel
  Cc: Collabora Kernel ML, linux-mediatek, drinkcat, hsinyi,
	Arnd Bergmann, Bjorn Andersson, Catalin Marinas,
	Dmitry Baryshkov, Guido Günther, Krzysztof Kozlowski,
	Lad Prabhakar, Max Krummenacher, Michael Walle, Nishanth Menon,
	Shawn Guo, Will Deacon, linux-arm-kernel, Luo Longjun



On 23/04/2021 09:52, Enric Balletbo i Serra wrote:
> Commit 'fbbe38309d56 ("arm64: defconfig: Allow mt8173-based boards to boot
> from usb")' added the MTK_PMIC_WRAP config built-in. It needs to be
> built-in in order to be able to boot from USB or the MMC without needing
> a ramdisk, but that symbol was already defined as a module so now we are
> getting the following warning:
> 
>   arch/arm64/configs/defconfig:996:warning: override: reassigning to symbol MTK_PMIC_WRAP
> 
> Remove the MTK_PMIC_WRAP=m from the defconfig to remove the error.
> 
> Fixes: fbbe38309d56 ("arm64: defconfig: Allow mt8173-based boards to boot from usb")
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>

Applied to v5.13-next/defconfig

Thanks!

> ---
> 
>  arch/arm64/configs/defconfig | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 08c6f769df9a..9907a431db0d 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -491,7 +491,6 @@ CONFIG_SPI_S3C64XX=y
>  CONFIG_SPI_SH_MSIOF=m
>  CONFIG_SPI_SUN6I=y
>  CONFIG_SPI_SPIDEV=m
> -CONFIG_MTK_PMIC_WRAP=m
>  CONFIG_SPMI=y
>  CONFIG_PINCTRL_SINGLE=y
>  CONFIG_PINCTRL_MAX77620=y
> 

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

* Re: [PATCH] arm64: defconfig: Do not override the MTK_PMIC_WRAP symbol
  2021-05-14  7:54 ` Matthias Brugger
@ 2021-06-11 21:48   ` Petr Vorel
  0 siblings, 0 replies; 3+ messages in thread
From: Petr Vorel @ 2021-06-11 21:48 UTC (permalink / raw)
  To: Matthias Brugger
  Cc: Enric Balletbo i Serra, linux-kernel, Collabora Kernel ML,
	linux-mediatek, drinkcat, hsinyi, Arnd Bergmann, Bjorn Andersson,
	Catalin Marinas, Dmitry Baryshkov, Guido Günther,
	Krzysztof Kozlowski, Lad Prabhakar, Max Krummenacher,
	Michael Walle, Nishanth Menon, Shawn Guo, Will Deacon,
	linux-arm-kernel, Luo Longjun

Hi all,

> On 23/04/2021 09:52, Enric Balletbo i Serra wrote:
> > Commit 'fbbe38309d56 ("arm64: defconfig: Allow mt8173-based boards to boot
> > from usb")' added the MTK_PMIC_WRAP config built-in. It needs to be
> > built-in in order to be able to boot from USB or the MMC without needing
> > a ramdisk, but that symbol was already defined as a module so now we are
> > getting the following warning:

> >   arch/arm64/configs/defconfig:996:warning: override: reassigning to symbol MTK_PMIC_WRAP

> > Remove the MTK_PMIC_WRAP=m from the defconfig to remove the error.

> > Fixes: fbbe38309d56 ("arm64: defconfig: Allow mt8173-based boards to boot from usb")
> > Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>

> Applied to v5.13-next/defconfig
I'm sorry, please ignore my useless Reviewed-by: tag.

I was searching in arm64 git [1] and overlooked the date.
I see it was merged into Matthias' git tree [2] month ago.

Kind regards,
Petr

[1] https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
[2] https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/commit/?h=v5.13-next-defconfig&id=f0e70d4946332c681ceaba940652f30c7c33473d

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

end of thread, other threads:[~2021-06-11 21:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-23  7:52 [PATCH] arm64: defconfig: Do not override the MTK_PMIC_WRAP symbol Enric Balletbo i Serra
2021-05-14  7:54 ` Matthias Brugger
2021-06-11 21:48   ` Petr Vorel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).