* [PATCH 2/2] mvebu: uDPU: disable non-present peripherals
2022-06-03 10:53 [PATCH 1/2] mvebu: eDPU: disable SCSI support Robert Marko
@ 2022-06-03 10:53 ` Robert Marko
2022-06-03 12:43 ` Pali Rohár
` (2 more replies)
2022-06-03 12:41 ` [PATCH 1/2] mvebu: eDPU: disable SCSI support Pali Rohár
` (2 subsequent siblings)
3 siblings, 3 replies; 8+ messages in thread
From: Robert Marko @ 2022-06-03 10:53 UTC (permalink / raw)
To: sr, pali, marek.behun, rfried.dev, u-boot; +Cc: Robert Marko
uDPU like eDPU does not expose SCSI based peripherals like SATA nor PCI
and for sure it does not have the Intel E1000 PCI card.
So, like for eDPU remove those from the defconfig.
Signed-off-by: Robert Marko <robert.marko@sartura.hr>
---
configs/uDPU_defconfig | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig
index f7cd1a0589..ec7cda6a29 100644
--- a/configs/uDPU_defconfig
+++ b/configs/uDPU_defconfig
@@ -12,7 +12,6 @@ CONFIG_DEFAULT_DEVICE_TREE="armada-3720-uDPU"
CONFIG_DEBUG_UART_BASE=0xd0012000
CONFIG_SYS_LOAD_ADDR=0x6000000
CONFIG_DEBUG_UART=y
-CONFIG_AHCI=y
CONFIG_DISTRO_DEFAULTS=y
# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
CONFIG_FIT=y
@@ -33,7 +32,6 @@ CONFIG_CMD_GPIO=y
CONFIG_CMD_I2C=y
CONFIG_CMD_MMC=y
CONFIG_CMD_MTD=y
-CONFIG_CMD_PCI=y
CONFIG_CMD_SPI=y
CONFIG_CMD_USB=y
# CONFIG_CMD_SETEXPR is not set
@@ -49,7 +47,6 @@ CONFIG_SYS_RELOC_GD_ENV_ADDR=y
CONFIG_ARP_TIMEOUT=200
CONFIG_NET_RETRY_COUNT=50
CONFIG_NET_RANDOM_ETHADDR=y
-CONFIG_AHCI_MVEBU=y
CONFIG_CLK=y
CONFIG_CLK_MVEBU=y
CONFIG_DM_I2C=y
@@ -72,17 +69,15 @@ CONFIG_PHYLIB_10G=y
CONFIG_PHY_MARVELL=y
CONFIG_PHY_FIXED=y
CONFIG_PHY_GIGE=y
-CONFIG_E1000=y
CONFIG_MVNETA=y
CONFIG_MVMDIO=y
-CONFIG_PCI=y
-CONFIG_PCI_AARDVARK=y
CONFIG_PHY=y
CONFIG_MVEBU_COMPHY_SUPPORT=y
CONFIG_PINCTRL=y
CONFIG_PINCTRL_ARMADA_37XX=y
CONFIG_DM_REGULATOR_FIXED=y
CONFIG_DM_REGULATOR_GPIO=y
+# CONFIG_SCSI is not set
CONFIG_MVEBU_A3700_UART=y
CONFIG_MVEBU_A3700_SPI=y
CONFIG_SYSINFO=y
--
2.36.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 2/2] mvebu: uDPU: disable non-present peripherals
2022-06-03 10:53 ` [PATCH 2/2] mvebu: uDPU: disable non-present peripherals Robert Marko
@ 2022-06-03 12:43 ` Pali Rohár
2022-06-07 6:47 ` Stefan Roese
2022-07-21 12:27 ` Stefan Roese
2 siblings, 0 replies; 8+ messages in thread
From: Pali Rohár @ 2022-06-03 12:43 UTC (permalink / raw)
To: Robert Marko; +Cc: sr, marek.behun, rfried.dev, u-boot
On Friday 03 June 2022 12:53:05 Robert Marko wrote:
> uDPU like eDPU does not expose SCSI based peripherals like SATA nor PCI
> and for sure it does not have the Intel E1000 PCI card.
>
> So, like for eDPU remove those from the defconfig.
>
> Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Reviewed-by: Pali Rohár <pali@kernel.org>
> ---
> configs/uDPU_defconfig | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig
> index f7cd1a0589..ec7cda6a29 100644
> --- a/configs/uDPU_defconfig
> +++ b/configs/uDPU_defconfig
> @@ -12,7 +12,6 @@ CONFIG_DEFAULT_DEVICE_TREE="armada-3720-uDPU"
> CONFIG_DEBUG_UART_BASE=0xd0012000
> CONFIG_SYS_LOAD_ADDR=0x6000000
> CONFIG_DEBUG_UART=y
> -CONFIG_AHCI=y
> CONFIG_DISTRO_DEFAULTS=y
> # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
> CONFIG_FIT=y
> @@ -33,7 +32,6 @@ CONFIG_CMD_GPIO=y
> CONFIG_CMD_I2C=y
> CONFIG_CMD_MMC=y
> CONFIG_CMD_MTD=y
> -CONFIG_CMD_PCI=y
> CONFIG_CMD_SPI=y
> CONFIG_CMD_USB=y
> # CONFIG_CMD_SETEXPR is not set
> @@ -49,7 +47,6 @@ CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> CONFIG_ARP_TIMEOUT=200
> CONFIG_NET_RETRY_COUNT=50
> CONFIG_NET_RANDOM_ETHADDR=y
> -CONFIG_AHCI_MVEBU=y
> CONFIG_CLK=y
> CONFIG_CLK_MVEBU=y
> CONFIG_DM_I2C=y
> @@ -72,17 +69,15 @@ CONFIG_PHYLIB_10G=y
> CONFIG_PHY_MARVELL=y
> CONFIG_PHY_FIXED=y
> CONFIG_PHY_GIGE=y
> -CONFIG_E1000=y
> CONFIG_MVNETA=y
> CONFIG_MVMDIO=y
> -CONFIG_PCI=y
> -CONFIG_PCI_AARDVARK=y
> CONFIG_PHY=y
> CONFIG_MVEBU_COMPHY_SUPPORT=y
> CONFIG_PINCTRL=y
> CONFIG_PINCTRL_ARMADA_37XX=y
> CONFIG_DM_REGULATOR_FIXED=y
> CONFIG_DM_REGULATOR_GPIO=y
> +# CONFIG_SCSI is not set
> CONFIG_MVEBU_A3700_UART=y
> CONFIG_MVEBU_A3700_SPI=y
> CONFIG_SYSINFO=y
> --
> 2.36.1
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] mvebu: uDPU: disable non-present peripherals
2022-06-03 10:53 ` [PATCH 2/2] mvebu: uDPU: disable non-present peripherals Robert Marko
2022-06-03 12:43 ` Pali Rohár
@ 2022-06-07 6:47 ` Stefan Roese
2022-07-21 12:27 ` Stefan Roese
2 siblings, 0 replies; 8+ messages in thread
From: Stefan Roese @ 2022-06-07 6:47 UTC (permalink / raw)
To: Robert Marko, pali, marek.behun, rfried.dev, u-boot
On 03.06.22 12:53, Robert Marko wrote:
> uDPU like eDPU does not expose SCSI based peripherals like SATA nor PCI
> and for sure it does not have the Intel E1000 PCI card.
>
> So, like for eDPU remove those from the defconfig.
>
> Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Reviewed-by: Stefan Roese <sr@denx.de>
Thanks,
Stefan
> ---
> configs/uDPU_defconfig | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig
> index f7cd1a0589..ec7cda6a29 100644
> --- a/configs/uDPU_defconfig
> +++ b/configs/uDPU_defconfig
> @@ -12,7 +12,6 @@ CONFIG_DEFAULT_DEVICE_TREE="armada-3720-uDPU"
> CONFIG_DEBUG_UART_BASE=0xd0012000
> CONFIG_SYS_LOAD_ADDR=0x6000000
> CONFIG_DEBUG_UART=y
> -CONFIG_AHCI=y
> CONFIG_DISTRO_DEFAULTS=y
> # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
> CONFIG_FIT=y
> @@ -33,7 +32,6 @@ CONFIG_CMD_GPIO=y
> CONFIG_CMD_I2C=y
> CONFIG_CMD_MMC=y
> CONFIG_CMD_MTD=y
> -CONFIG_CMD_PCI=y
> CONFIG_CMD_SPI=y
> CONFIG_CMD_USB=y
> # CONFIG_CMD_SETEXPR is not set
> @@ -49,7 +47,6 @@ CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> CONFIG_ARP_TIMEOUT=200
> CONFIG_NET_RETRY_COUNT=50
> CONFIG_NET_RANDOM_ETHADDR=y
> -CONFIG_AHCI_MVEBU=y
> CONFIG_CLK=y
> CONFIG_CLK_MVEBU=y
> CONFIG_DM_I2C=y
> @@ -72,17 +69,15 @@ CONFIG_PHYLIB_10G=y
> CONFIG_PHY_MARVELL=y
> CONFIG_PHY_FIXED=y
> CONFIG_PHY_GIGE=y
> -CONFIG_E1000=y
> CONFIG_MVNETA=y
> CONFIG_MVMDIO=y
> -CONFIG_PCI=y
> -CONFIG_PCI_AARDVARK=y
> CONFIG_PHY=y
> CONFIG_MVEBU_COMPHY_SUPPORT=y
> CONFIG_PINCTRL=y
> CONFIG_PINCTRL_ARMADA_37XX=y
> CONFIG_DM_REGULATOR_FIXED=y
> CONFIG_DM_REGULATOR_GPIO=y
> +# CONFIG_SCSI is not set
> CONFIG_MVEBU_A3700_UART=y
> CONFIG_MVEBU_A3700_SPI=y
> CONFIG_SYSINFO=y
Viele Grüße,
Stefan Roese
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] mvebu: uDPU: disable non-present peripherals
2022-06-03 10:53 ` [PATCH 2/2] mvebu: uDPU: disable non-present peripherals Robert Marko
2022-06-03 12:43 ` Pali Rohár
2022-06-07 6:47 ` Stefan Roese
@ 2022-07-21 12:27 ` Stefan Roese
2 siblings, 0 replies; 8+ messages in thread
From: Stefan Roese @ 2022-07-21 12:27 UTC (permalink / raw)
To: Robert Marko, pali, marek.behun, rfried.dev, u-boot
On 03.06.22 12:53, Robert Marko wrote:
> uDPU like eDPU does not expose SCSI based peripherals like SATA nor PCI
> and for sure it does not have the Intel E1000 PCI card.
>
> So, like for eDPU remove those from the defconfig.
>
> Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Applied to u-boot-marvell/master
Thanks,
Stefan
> ---
> configs/uDPU_defconfig | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig
> index f7cd1a0589..ec7cda6a29 100644
> --- a/configs/uDPU_defconfig
> +++ b/configs/uDPU_defconfig
> @@ -12,7 +12,6 @@ CONFIG_DEFAULT_DEVICE_TREE="armada-3720-uDPU"
> CONFIG_DEBUG_UART_BASE=0xd0012000
> CONFIG_SYS_LOAD_ADDR=0x6000000
> CONFIG_DEBUG_UART=y
> -CONFIG_AHCI=y
> CONFIG_DISTRO_DEFAULTS=y
> # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
> CONFIG_FIT=y
> @@ -33,7 +32,6 @@ CONFIG_CMD_GPIO=y
> CONFIG_CMD_I2C=y
> CONFIG_CMD_MMC=y
> CONFIG_CMD_MTD=y
> -CONFIG_CMD_PCI=y
> CONFIG_CMD_SPI=y
> CONFIG_CMD_USB=y
> # CONFIG_CMD_SETEXPR is not set
> @@ -49,7 +47,6 @@ CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> CONFIG_ARP_TIMEOUT=200
> CONFIG_NET_RETRY_COUNT=50
> CONFIG_NET_RANDOM_ETHADDR=y
> -CONFIG_AHCI_MVEBU=y
> CONFIG_CLK=y
> CONFIG_CLK_MVEBU=y
> CONFIG_DM_I2C=y
> @@ -72,17 +69,15 @@ CONFIG_PHYLIB_10G=y
> CONFIG_PHY_MARVELL=y
> CONFIG_PHY_FIXED=y
> CONFIG_PHY_GIGE=y
> -CONFIG_E1000=y
> CONFIG_MVNETA=y
> CONFIG_MVMDIO=y
> -CONFIG_PCI=y
> -CONFIG_PCI_AARDVARK=y
> CONFIG_PHY=y
> CONFIG_MVEBU_COMPHY_SUPPORT=y
> CONFIG_PINCTRL=y
> CONFIG_PINCTRL_ARMADA_37XX=y
> CONFIG_DM_REGULATOR_FIXED=y
> CONFIG_DM_REGULATOR_GPIO=y
> +# CONFIG_SCSI is not set
> CONFIG_MVEBU_A3700_UART=y
> CONFIG_MVEBU_A3700_SPI=y
> CONFIG_SYSINFO=y
Viele Grüße,
Stefan Roese
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] mvebu: eDPU: disable SCSI support
2022-06-03 10:53 [PATCH 1/2] mvebu: eDPU: disable SCSI support Robert Marko
2022-06-03 10:53 ` [PATCH 2/2] mvebu: uDPU: disable non-present peripherals Robert Marko
@ 2022-06-03 12:41 ` Pali Rohár
2022-06-07 6:47 ` Stefan Roese
2022-07-21 12:27 ` Stefan Roese
3 siblings, 0 replies; 8+ messages in thread
From: Pali Rohár @ 2022-06-03 12:41 UTC (permalink / raw)
To: Robert Marko; +Cc: sr, marek.behun, rfried.dev, u-boot
On Friday 03 June 2022 12:53:04 Robert Marko wrote:
> eDPU does not use SCSI nor it has SATA exposed, and commit
> arm: mvebu: a3720: Set BOOT_TARGET_DEVICES list to enabled peripherals
> now allows compiling U-boot wihout all of the BOOT_TARGET_DEVICES since
> not all boards have all of the listed peripherals exposed.
>
> So, disable SCSI support in defconfig for eDPU.
>
> Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Reviewed-by: Pali Rohár <pali@kernel.org>
> ---
> configs/eDPU_defconfig | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/configs/eDPU_defconfig b/configs/eDPU_defconfig
> index e2e593ad21..9c124bf147 100644
> --- a/configs/eDPU_defconfig
> +++ b/configs/eDPU_defconfig
> @@ -32,7 +32,6 @@ CONFIG_CMD_GPIO=y
> CONFIG_CMD_I2C=y
> CONFIG_CMD_MMC=y
> CONFIG_CMD_MTD=y
> -# CONFIG_CMD_SCSI is not set
> CONFIG_CMD_SPI=y
> CONFIG_CMD_USB=y
> # CONFIG_CMD_SETEXPR is not set
> @@ -78,7 +77,7 @@ CONFIG_PINCTRL=y
> CONFIG_PINCTRL_ARMADA_37XX=y
> CONFIG_DM_REGULATOR_FIXED=y
> CONFIG_DM_REGULATOR_GPIO=y
> -CONFIG_DM_SCSI=y
> +# CONFIG_SCSI is not set
> CONFIG_MVEBU_A3700_UART=y
> CONFIG_MVEBU_A3700_SPI=y
> CONFIG_SYSINFO=y
> --
> 2.36.1
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] mvebu: eDPU: disable SCSI support
2022-06-03 10:53 [PATCH 1/2] mvebu: eDPU: disable SCSI support Robert Marko
2022-06-03 10:53 ` [PATCH 2/2] mvebu: uDPU: disable non-present peripherals Robert Marko
2022-06-03 12:41 ` [PATCH 1/2] mvebu: eDPU: disable SCSI support Pali Rohár
@ 2022-06-07 6:47 ` Stefan Roese
2022-07-21 12:27 ` Stefan Roese
3 siblings, 0 replies; 8+ messages in thread
From: Stefan Roese @ 2022-06-07 6:47 UTC (permalink / raw)
To: Robert Marko, pali, marek.behun, rfried.dev, u-boot
On 03.06.22 12:53, Robert Marko wrote:
> eDPU does not use SCSI nor it has SATA exposed, and commit
> arm: mvebu: a3720: Set BOOT_TARGET_DEVICES list to enabled peripherals
> now allows compiling U-boot wihout all of the BOOT_TARGET_DEVICES since
> not all boards have all of the listed peripherals exposed.
>
> So, disable SCSI support in defconfig for eDPU.
>
> Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Reviewed-by: Stefan Roese <sr@denx.de>
Thanks,
Stefan
> ---
> configs/eDPU_defconfig | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/configs/eDPU_defconfig b/configs/eDPU_defconfig
> index e2e593ad21..9c124bf147 100644
> --- a/configs/eDPU_defconfig
> +++ b/configs/eDPU_defconfig
> @@ -32,7 +32,6 @@ CONFIG_CMD_GPIO=y
> CONFIG_CMD_I2C=y
> CONFIG_CMD_MMC=y
> CONFIG_CMD_MTD=y
> -# CONFIG_CMD_SCSI is not set
> CONFIG_CMD_SPI=y
> CONFIG_CMD_USB=y
> # CONFIG_CMD_SETEXPR is not set
> @@ -78,7 +77,7 @@ CONFIG_PINCTRL=y
> CONFIG_PINCTRL_ARMADA_37XX=y
> CONFIG_DM_REGULATOR_FIXED=y
> CONFIG_DM_REGULATOR_GPIO=y
> -CONFIG_DM_SCSI=y
> +# CONFIG_SCSI is not set
> CONFIG_MVEBU_A3700_UART=y
> CONFIG_MVEBU_A3700_SPI=y
> CONFIG_SYSINFO=y
Viele Grüße,
Stefan Roese
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] mvebu: eDPU: disable SCSI support
2022-06-03 10:53 [PATCH 1/2] mvebu: eDPU: disable SCSI support Robert Marko
` (2 preceding siblings ...)
2022-06-07 6:47 ` Stefan Roese
@ 2022-07-21 12:27 ` Stefan Roese
3 siblings, 0 replies; 8+ messages in thread
From: Stefan Roese @ 2022-07-21 12:27 UTC (permalink / raw)
To: Robert Marko, pali, marek.behun, rfried.dev, u-boot
On 03.06.22 12:53, Robert Marko wrote:
> eDPU does not use SCSI nor it has SATA exposed, and commit
> arm: mvebu: a3720: Set BOOT_TARGET_DEVICES list to enabled peripherals
> now allows compiling U-boot wihout all of the BOOT_TARGET_DEVICES since
> not all boards have all of the listed peripherals exposed.
>
> So, disable SCSI support in defconfig for eDPU.
>
> Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Applied to u-boot-marvell/master
Thanks,
Stefan
> ---
> configs/eDPU_defconfig | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/configs/eDPU_defconfig b/configs/eDPU_defconfig
> index e2e593ad21..9c124bf147 100644
> --- a/configs/eDPU_defconfig
> +++ b/configs/eDPU_defconfig
> @@ -32,7 +32,6 @@ CONFIG_CMD_GPIO=y
> CONFIG_CMD_I2C=y
> CONFIG_CMD_MMC=y
> CONFIG_CMD_MTD=y
> -# CONFIG_CMD_SCSI is not set
> CONFIG_CMD_SPI=y
> CONFIG_CMD_USB=y
> # CONFIG_CMD_SETEXPR is not set
> @@ -78,7 +77,7 @@ CONFIG_PINCTRL=y
> CONFIG_PINCTRL_ARMADA_37XX=y
> CONFIG_DM_REGULATOR_FIXED=y
> CONFIG_DM_REGULATOR_GPIO=y
> -CONFIG_DM_SCSI=y
> +# CONFIG_SCSI is not set
> CONFIG_MVEBU_A3700_UART=y
> CONFIG_MVEBU_A3700_SPI=y
> CONFIG_SYSINFO=y
Viele Grüße,
Stefan Roese
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de
^ permalink raw reply [flat|nested] 8+ messages in thread