* [U-Boot] [PATCH v2 1/3] arm: exynos: arndale: Remove unused CONFIG_POWER and CONFIG_POWER_I2C
@ 2019-05-11 11:24 ` Krzysztof Kozlowski
2019-05-11 11:24 ` [U-Boot] [PATCH v2 2/3] arm: exynos: arndale: Remove unused CONFIG_DM_I2C_COMPAT Krzysztof Kozlowski
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2019-05-11 11:24 UTC (permalink / raw)
To: u-boot
The CONFIG_POWER and CONFIG_POWER_I2C were introduced in
include/configs/exynos5-common.h in commit 19bd3aaa5991 ("exynos5: fix
build break by adding CONFIG_POWER") and then it propagated up to
include/configs/arndale.h. However before that commit, there was no
build break at all on Arndale and SMDK5250 boards. It seems the commit
fixed nothing and just added unused defines.
In fact, the Arndale board is not configuring its PMIC (S5M8767) which
uses I2C bus.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
Not tested on Arndale board. Testing is welcomed.
Changes since v1:
1. Add Simon's tag.
2. Reorder patches - first remove CONFIG_POWER_I2C, then
CONFIG_DM_I2C_COMPAT.
---
include/configs/arndale.h | 8 --------
1 file changed, 8 deletions(-)
diff --git a/include/configs/arndale.h b/include/configs/arndale.h
index dd321c4748d0..841f3616482b 100644
--- a/include/configs/arndale.h
+++ b/include/configs/arndale.h
@@ -29,10 +29,6 @@
#define CONFIG_SYS_INIT_SP_ADDR CONFIG_IRAM_STACK
-/* PMIC */
-#define CONFIG_POWER
-#define CONFIG_POWER_I2C
-
#define CONFIG_PREBOOT
#define CONFIG_S5P_PA_SYSRAM 0x02020000
@@ -41,8 +37,4 @@
/* The PERIPHBASE in the CBAR register is wrong on the Arndale, so override it */
#define CONFIG_ARM_GIC_BASE_ADDRESS 0x10480000
-/* Power */
-#define CONFIG_POWER
-#define CONFIG_POWER_I2C
-
#endif /* __CONFIG_H */
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH v2 2/3] arm: exynos: arndale: Remove unused CONFIG_DM_I2C_COMPAT
2019-05-11 11:24 ` [U-Boot] [PATCH v2 1/3] arm: exynos: arndale: Remove unused CONFIG_POWER and CONFIG_POWER_I2C Krzysztof Kozlowski
@ 2019-05-11 11:24 ` Krzysztof Kozlowski
2019-05-16 7:33 ` Minkyu Kang
2019-05-11 11:24 ` [U-Boot] [PATCH v2 3/3] configs: arndale: Use appropriate driver for Asix AX88760 Krzysztof Kozlowski
2019-05-16 7:33 ` [U-Boot] [PATCH v2 1/3] arm: exynos: arndale: Remove unused CONFIG_POWER and CONFIG_POWER_I2C Minkyu Kang
2 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2019-05-11 11:24 UTC (permalink / raw)
To: u-boot
The CONFIG_DM_I2C_COMPAT was introduced in
include/configs/exynos5-common.h in commit 189d80166b31 ("exynos5:
enable dm i2c") and then it propagated up to configs/arndale_defconfig.
However since beginning the Arndale board (Exynos5250) was not using
I2C.
In fact, the Arndale board is not configuring its PMIC (S5M8767) which
uses I2C bus. This setting can be thus safely removed to fix build
warning:
This board uses CONFIG_DM_I2C_COMPAT. Please remove
(possibly in a subsequent patch in your series)
before sending patches to the mailing list.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
Not tested on Arndale board. Testing is welcomed.
Changes since v1:
1. Add Simon's tag.
2. Reorder patches - first remove CONFIG_POWER_I2C, then
CONFIG_DM_I2C_COMPAT.
---
configs/arndale_defconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/configs/arndale_defconfig b/configs/arndale_defconfig
index 9727d28c1241..e90d670f6813 100644
--- a/configs/arndale_defconfig
+++ b/configs/arndale_defconfig
@@ -25,7 +25,6 @@ CONFIG_CMD_SOUND=y
CONFIG_CMD_EXT4_WRITE=y
CONFIG_DEFAULT_DEVICE_TREE="exynos5250-arndale"
CONFIG_ENV_IS_IN_MMC=y
-CONFIG_DM_I2C_COMPAT=y
CONFIG_SUPPORT_EMMC_BOOT=y
CONFIG_MMC_DW=y
CONFIG_MMC_SDHCI=y
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH v2 3/3] configs: arndale: Use appropriate driver for Asix AX88760
2019-05-11 11:24 ` [U-Boot] [PATCH v2 1/3] arm: exynos: arndale: Remove unused CONFIG_POWER and CONFIG_POWER_I2C Krzysztof Kozlowski
2019-05-11 11:24 ` [U-Boot] [PATCH v2 2/3] arm: exynos: arndale: Remove unused CONFIG_DM_I2C_COMPAT Krzysztof Kozlowski
@ 2019-05-11 11:24 ` Krzysztof Kozlowski
2019-05-16 7:33 ` Minkyu Kang
2019-05-16 7:33 ` [U-Boot] [PATCH v2 1/3] arm: exynos: arndale: Remove unused CONFIG_POWER and CONFIG_POWER_I2C Minkyu Kang
2 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2019-05-11 11:24 UTC (permalink / raw)
To: u-boot
Arndale board has an Asix AX88760 USB 2.0 Hub and Fast Ethernet combo.
The appropriate driver for it is USB_ETHER_ASIX.
The mistake probably came from misinterpretation of commit e9954b867ce0
("usb: eth: add ASIX AX88179 DRIVER") which was tested on RECS5250 COM
module. This module indeed has Exynos5250 and some similarities with
Arndale 5250 board but the USB/Ethernet chip used there is apparently
different.
Fixes: f58ad98a621c ("usb: net: migrate USB Ethernet adapters to Kconfig")
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Lukasz Majewski <lukma@denx.de>
---
Not tested.
Changes since v1:
1. Add Lukasz's tag.
---
configs/arndale_defconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configs/arndale_defconfig b/configs/arndale_defconfig
index e90d670f6813..37ad6accc396 100644
--- a/configs/arndale_defconfig
+++ b/configs/arndale_defconfig
@@ -42,4 +42,4 @@ CONFIG_USB_XHCI_HCD=y
CONFIG_USB_XHCI_DWC3=y
CONFIG_USB_EHCI_HCD=y
CONFIG_USB_HOST_ETHER=y
-CONFIG_USB_ETHER_ASIX88179=y
+CONFIG_USB_ETHER_ASIX=y
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH v2 2/3] arm: exynos: arndale: Remove unused CONFIG_DM_I2C_COMPAT
2019-05-11 11:24 ` [U-Boot] [PATCH v2 2/3] arm: exynos: arndale: Remove unused CONFIG_DM_I2C_COMPAT Krzysztof Kozlowski
@ 2019-05-16 7:33 ` Minkyu Kang
0 siblings, 0 replies; 6+ messages in thread
From: Minkyu Kang @ 2019-05-16 7:33 UTC (permalink / raw)
To: u-boot
Hi,
On 11/05/2019 20:24, Krzysztof Kozlowski wrote:
> The CONFIG_DM_I2C_COMPAT was introduced in
> include/configs/exynos5-common.h in commit 189d80166b31 ("exynos5:
> enable dm i2c") and then it propagated up to configs/arndale_defconfig.
> However since beginning the Arndale board (Exynos5250) was not using
> I2C.
>
> In fact, the Arndale board is not configuring its PMIC (S5M8767) which
> uses I2C bus. This setting can be thus safely removed to fix build
> warning:
>
> This board uses CONFIG_DM_I2C_COMPAT. Please remove
> (possibly in a subsequent patch in your series)
> before sending patches to the mailing list.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> Reviewed-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Not tested on Arndale board. Testing is welcomed.
>
> Changes since v1:
> 1. Add Simon's tag.
> 2. Reorder patches - first remove CONFIG_POWER_I2C, then
> CONFIG_DM_I2C_COMPAT.
> ---
> configs/arndale_defconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/configs/arndale_defconfig b/configs/arndale_defconfig
> index 9727d28c1241..e90d670f6813 100644
> --- a/configs/arndale_defconfig
> +++ b/configs/arndale_defconfig
> @@ -25,7 +25,6 @@ CONFIG_CMD_SOUND=y
> CONFIG_CMD_EXT4_WRITE=y
> CONFIG_DEFAULT_DEVICE_TREE="exynos5250-arndale"
> CONFIG_ENV_IS_IN_MMC=y
> -CONFIG_DM_I2C_COMPAT=y
> CONFIG_SUPPORT_EMMC_BOOT=y
> CONFIG_MMC_DW=y
> CONFIG_MMC_SDHCI=y
>
applied to u-boot-samsung.
Thanks,
Minkyu Kang.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH v2 1/3] arm: exynos: arndale: Remove unused CONFIG_POWER and CONFIG_POWER_I2C
2019-05-11 11:24 ` [U-Boot] [PATCH v2 1/3] arm: exynos: arndale: Remove unused CONFIG_POWER and CONFIG_POWER_I2C Krzysztof Kozlowski
2019-05-11 11:24 ` [U-Boot] [PATCH v2 2/3] arm: exynos: arndale: Remove unused CONFIG_DM_I2C_COMPAT Krzysztof Kozlowski
2019-05-11 11:24 ` [U-Boot] [PATCH v2 3/3] configs: arndale: Use appropriate driver for Asix AX88760 Krzysztof Kozlowski
@ 2019-05-16 7:33 ` Minkyu Kang
2 siblings, 0 replies; 6+ messages in thread
From: Minkyu Kang @ 2019-05-16 7:33 UTC (permalink / raw)
To: u-boot
Hi,
On 11/05/2019 20:24, Krzysztof Kozlowski wrote:
> The CONFIG_POWER and CONFIG_POWER_I2C were introduced in
> include/configs/exynos5-common.h in commit 19bd3aaa5991 ("exynos5: fix
> build break by adding CONFIG_POWER") and then it propagated up to
> include/configs/arndale.h. However before that commit, there was no
> build break at all on Arndale and SMDK5250 boards. It seems the commit
> fixed nothing and just added unused defines.
>
> In fact, the Arndale board is not configuring its PMIC (S5M8767) which
> uses I2C bus.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> Reviewed-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Not tested on Arndale board. Testing is welcomed.
>
> Changes since v1:
> 1. Add Simon's tag.
> 2. Reorder patches - first remove CONFIG_POWER_I2C, then
> CONFIG_DM_I2C_COMPAT.
> ---
> include/configs/arndale.h | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/include/configs/arndale.h b/include/configs/arndale.h
> index dd321c4748d0..841f3616482b 100644
> --- a/include/configs/arndale.h
> +++ b/include/configs/arndale.h
> @@ -29,10 +29,6 @@
>
> #define CONFIG_SYS_INIT_SP_ADDR CONFIG_IRAM_STACK
>
> -/* PMIC */
> -#define CONFIG_POWER
> -#define CONFIG_POWER_I2C
> -
> #define CONFIG_PREBOOT
>
> #define CONFIG_S5P_PA_SYSRAM 0x02020000
> @@ -41,8 +37,4 @@
> /* The PERIPHBASE in the CBAR register is wrong on the Arndale, so override it */
> #define CONFIG_ARM_GIC_BASE_ADDRESS 0x10480000
>
> -/* Power */
> -#define CONFIG_POWER
> -#define CONFIG_POWER_I2C
> -
> #endif /* __CONFIG_H */
>
applied to u-boot-samsung.
Thanks,
Minkyu Kang.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH v2 3/3] configs: arndale: Use appropriate driver for Asix AX88760
2019-05-11 11:24 ` [U-Boot] [PATCH v2 3/3] configs: arndale: Use appropriate driver for Asix AX88760 Krzysztof Kozlowski
@ 2019-05-16 7:33 ` Minkyu Kang
0 siblings, 0 replies; 6+ messages in thread
From: Minkyu Kang @ 2019-05-16 7:33 UTC (permalink / raw)
To: u-boot
Hi,
On 11/05/2019 20:24, Krzysztof Kozlowski wrote:
> Arndale board has an Asix AX88760 USB 2.0 Hub and Fast Ethernet combo.
> The appropriate driver for it is USB_ETHER_ASIX.
>
> The mistake probably came from misinterpretation of commit e9954b867ce0
> ("usb: eth: add ASIX AX88179 DRIVER") which was tested on RECS5250 COM
> module. This module indeed has Exynos5250 and some similarities with
> Arndale 5250 board but the USB/Ethernet chip used there is apparently
> different.
>
> Fixes: f58ad98a621c ("usb: net: migrate USB Ethernet adapters to Kconfig")
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> Reviewed-by: Lukasz Majewski <lukma@denx.de>
>
> ---
>
> Not tested.
>
> Changes since v1:
> 1. Add Lukasz's tag.
> ---
> configs/arndale_defconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configs/arndale_defconfig b/configs/arndale_defconfig
> index e90d670f6813..37ad6accc396 100644
> --- a/configs/arndale_defconfig
> +++ b/configs/arndale_defconfig
> @@ -42,4 +42,4 @@ CONFIG_USB_XHCI_HCD=y
> CONFIG_USB_XHCI_DWC3=y
> CONFIG_USB_EHCI_HCD=y
> CONFIG_USB_HOST_ETHER=y
> -CONFIG_USB_ETHER_ASIX88179=y
> +CONFIG_USB_ETHER_ASIX=y
>
applied to u-boot-samsung.
Thanks,
Minkyu Kang.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-05-16 7:33 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20190511112446epcas3p20c09b77c3b0044bac615ec4d4d848149@epcas3p2.samsung.com>
2019-05-11 11:24 ` [U-Boot] [PATCH v2 1/3] arm: exynos: arndale: Remove unused CONFIG_POWER and CONFIG_POWER_I2C Krzysztof Kozlowski
2019-05-11 11:24 ` [U-Boot] [PATCH v2 2/3] arm: exynos: arndale: Remove unused CONFIG_DM_I2C_COMPAT Krzysztof Kozlowski
2019-05-16 7:33 ` Minkyu Kang
2019-05-11 11:24 ` [U-Boot] [PATCH v2 3/3] configs: arndale: Use appropriate driver for Asix AX88760 Krzysztof Kozlowski
2019-05-16 7:33 ` Minkyu Kang
2019-05-16 7:33 ` [U-Boot] [PATCH v2 1/3] arm: exynos: arndale: Remove unused CONFIG_POWER and CONFIG_POWER_I2C Minkyu Kang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox