mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH 0/8] Misc fixes part III
@ 2022-06-27 10:24 Alexander Shiyan
  2022-06-27 10:24 ` [PATCH 1/8] ARM: vexpress: Simplify sysctl initialization Alexander Shiyan
                   ` (8 more replies)
  0 siblings, 9 replies; 17+ messages in thread
From: Alexander Shiyan @ 2022-06-27 10:24 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Shiyan

Unsorted various fixes for the barebox.

Alexander Shiyan (8):
  ARM: vexpress: Simplify sysctl initialization
  ARM: bcm2835: Drop uneeded headers
  of: Use appropriate header for of_clk_init()
  usb: gadget: Fix Kconfig dependencies
  logo: Fix Kconfig dependencies
  pbl: Fix Kconfig dependencies
  ARM: mxs: Fix Kconfig dependencies
  ARM: boards: ccmx51: Remove unused KConfig option

 arch/arm/mach-bcm283x/core.c |  9 ---------
 arch/arm/mach-imx/Kconfig    |  1 -
 arch/arm/mach-mxs/Kconfig    |  1 +
 arch/arm/mach-vexpress/v2m.c | 13 +------------
 drivers/of/base.c            |  2 +-
 drivers/usb/gadget/Kconfig   |  8 +++++---
 lib/logo/Kconfig             |  1 +
 pbl/Kconfig                  | 24 +++++++++++-------------
 8 files changed, 20 insertions(+), 39 deletions(-)

-- 
2.32.0




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

* [PATCH 1/8] ARM: vexpress: Simplify sysctl initialization
  2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
@ 2022-06-27 10:24 ` Alexander Shiyan
  2022-06-27 10:24 ` [PATCH 2/8] ARM: bcm2835: Drop uneeded headers Alexander Shiyan
                   ` (7 subsequent siblings)
  8 siblings, 0 replies; 17+ messages in thread
From: Alexander Shiyan @ 2022-06-27 10:24 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Shiyan

Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
 arch/arm/mach-vexpress/v2m.c | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)

diff --git a/arch/arm/mach-vexpress/v2m.c b/arch/arm/mach-vexpress/v2m.c
index 3535262848..c60e9dbd75 100644
--- a/arch/arm/mach-vexpress/v2m.c
+++ b/arch/arm/mach-vexpress/v2m.c
@@ -5,16 +5,8 @@
  */
 
 #include <common.h>
-#include <init.h>
 #include <io.h>
-
-#include <linux/clk.h>
-#include <linux/clkdev.h>
-#include <linux/amba/bus.h>
-
-#include <asm/hardware/arm_timer.h>
 #include <asm/hardware/sp810.h>
-
 #include <mach/devices.h>
 
 void __iomem *v2m_sysreg_base;
@@ -23,8 +15,7 @@ static void v2m_sysctl_init(void __iomem *base)
 {
 	u32 scctrl;
 
-	if (WARN_ON(!base))
-		return;
+	v2m_sysreg_base = base;
 
 	/* Select 1MHz TIMCLK as the reference clock for SP804 timers */
 	scctrl = readl(base + SCCTRL);
@@ -36,13 +27,11 @@ static void v2m_sysctl_init(void __iomem *base)
 void vexpress_a9_legacy_init(void)
 {
 	v2m_wdt_base = IOMEM(0x1000f000);
-	v2m_sysreg_base = IOMEM(0x10001000);
 	v2m_sysctl_init(IOMEM(0x10001000));
 }
 
 void vexpress_init(void)
 {
 	v2m_wdt_base = IOMEM(0x1c0f0000);
-	v2m_sysreg_base = IOMEM(0x1c020000);
 	v2m_sysctl_init(IOMEM(0x1c020000));
 }
-- 
2.32.0




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

* [PATCH 2/8] ARM: bcm2835: Drop uneeded headers
  2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
  2022-06-27 10:24 ` [PATCH 1/8] ARM: vexpress: Simplify sysctl initialization Alexander Shiyan
@ 2022-06-27 10:24 ` Alexander Shiyan
  2022-06-29  7:10   ` Sascha Hauer
  2022-06-27 10:24 ` [PATCH 3/8] of: Use appropriate header for of_clk_init() Alexander Shiyan
                   ` (6 subsequent siblings)
  8 siblings, 1 reply; 17+ messages in thread
From: Alexander Shiyan @ 2022-06-27 10:24 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Shiyan

Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
 arch/arm/mach-bcm283x/core.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/arch/arm/mach-bcm283x/core.c b/arch/arm/mach-bcm283x/core.c
index f2528cf1f1..bcf2ecb204 100644
--- a/arch/arm/mach-bcm283x/core.c
+++ b/arch/arm/mach-bcm283x/core.c
@@ -17,19 +17,10 @@
  */
 
 #include <common.h>
-#include <init.h>
-
-#include <linux/clk.h>
-#include <linux/clkdev.h>
-#include <linux/err.h>
-
 #include <io.h>
 #include <asm/armlinux.h>
 #include <linux/sizes.h>
-
-#include <mach/platform.h>
 #include <mach/core.h>
-#include <linux/amba/bus.h>
 
 void bcm2835_add_device_sdram(u32 size)
 {
-- 
2.32.0




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

* [PATCH 3/8] of: Use appropriate header for of_clk_init()
  2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
  2022-06-27 10:24 ` [PATCH 1/8] ARM: vexpress: Simplify sysctl initialization Alexander Shiyan
  2022-06-27 10:24 ` [PATCH 2/8] ARM: bcm2835: Drop uneeded headers Alexander Shiyan
@ 2022-06-27 10:24 ` Alexander Shiyan
  2022-06-27 10:24 ` [PATCH 4/8] usb: gadget: Fix Kconfig dependencies Alexander Shiyan
                   ` (5 subsequent siblings)
  8 siblings, 0 replies; 17+ messages in thread
From: Alexander Shiyan @ 2022-06-27 10:24 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Shiyan

Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
 drivers/of/base.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/of/base.c b/drivers/of/base.c
index 83291c4785..101b2f74c7 100644
--- a/drivers/of/base.c
+++ b/drivers/of/base.c
@@ -17,8 +17,8 @@
 #include <linux/sizes.h>
 #include <of_graph.h>
 #include <string.h>
+#include <linux/clk.h>
 #include <linux/ctype.h>
-#include <linux/amba/bus.h>
 #include <linux/err.h>
 
 static struct device_node *root_node;
-- 
2.32.0




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

* [PATCH 4/8] usb: gadget: Fix Kconfig dependencies
  2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
                   ` (2 preceding siblings ...)
  2022-06-27 10:24 ` [PATCH 3/8] of: Use appropriate header for of_clk_init() Alexander Shiyan
@ 2022-06-27 10:24 ` Alexander Shiyan
  2022-06-29  6:52   ` Sascha Hauer
  2022-06-27 10:24 ` [PATCH 5/8] logo: " Alexander Shiyan
                   ` (4 subsequent siblings)
  8 siblings, 1 reply; 17+ messages in thread
From: Alexander Shiyan @ 2022-06-27 10:24 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Shiyan

WARNING: unmet direct dependencies detected for SYSTEM_PARTITIONS
Depends on [n]: GLOBALVAR [=n]
Selected by [y]:
- USB_GADGET_AUTOSTART [=y] && USB_GADGET [=y] && USB_GADGET_MASS_STORAGE [=y]

WARNING: unmet direct dependencies detected for USB_GADGET_DRIVER_ARC_PBL
Depends on [n]: USB_GADGET [=n]
Selected by [y]:
- MACH_PROTONIC_IMX8M [=y] && ARCH_IMX [=y] && IMX_MULTI_BOARDS [=y]

Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
 drivers/usb/gadget/Kconfig | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
index 10c713e833..f2ca3695a2 100644
--- a/drivers/usb/gadget/Kconfig
+++ b/drivers/usb/gadget/Kconfig
@@ -4,6 +4,10 @@ menuconfig USB_GADGET
 	select POLLER
 	bool "USB gadget support"
 
+config USB_GADGET_DRIVER_ARC_PBL
+	select USB_GADGET
+	bool
+
 if USB_GADGET
 
 config USB_GADGET_DUALSPEED
@@ -16,9 +20,6 @@ config USB_GADGET_DRIVER_ARC
 	default y
 	select USB_GADGET_DUALSPEED
 
-config USB_GADGET_DRIVER_ARC_PBL
-	bool
-
 config USB_GADGET_DRIVER_AT91
 	bool
 	prompt "at91 gadget driver"
@@ -37,6 +38,7 @@ config USB_GADGET_AUTOSTART
 	bool
 	default y
 	prompt "Automatically start usbgadget on boot"
+	depends on GLOBALVAR
 	select SYSTEM_PARTITIONS if USB_GADGET_MASS_STORAGE
 	help
 	  Enabling this option allows to automatically start a dfu or
-- 
2.32.0




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

* [PATCH 5/8] logo: Fix Kconfig dependencies
  2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
                   ` (3 preceding siblings ...)
  2022-06-27 10:24 ` [PATCH 4/8] usb: gadget: Fix Kconfig dependencies Alexander Shiyan
@ 2022-06-27 10:24 ` Alexander Shiyan
  2022-06-27 10:24 ` [PATCH 6/8] pbl: " Alexander Shiyan
                   ` (3 subsequent siblings)
  8 siblings, 0 replies; 17+ messages in thread
From: Alexander Shiyan @ 2022-06-27 10:24 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Shiyan

WARNING: unmet direct dependencies detected for BMP
Depends on [n]: IMAGE_RENDERER [=n]
Selected by [y]:
- BAREBOX_LOGO_BMP [=y] && <choice>

Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
 lib/logo/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/logo/Kconfig b/lib/logo/Kconfig
index de3494eccf..7e5a6fcb63 100644
--- a/lib/logo/Kconfig
+++ b/lib/logo/Kconfig
@@ -2,6 +2,7 @@
 
 menuconfig BAREBOX_LOGO
 	bool "include barebox logos in build"
+	depends on IMAGE_RENDERER
 	help
 	  Say yes here to build the barebox logos. This adds ImageMagick's
 	  convert tool to the build dependencies. The logo can be found under
-- 
2.32.0




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

* [PATCH 6/8] pbl: Fix Kconfig dependencies
  2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
                   ` (4 preceding siblings ...)
  2022-06-27 10:24 ` [PATCH 5/8] logo: " Alexander Shiyan
@ 2022-06-27 10:24 ` Alexander Shiyan
  2022-06-30 11:57   ` Sascha Hauer
  2022-06-27 10:24 ` [PATCH 7/8] ARM: mxs: " Alexander Shiyan
                   ` (2 subsequent siblings)
  8 siblings, 1 reply; 17+ messages in thread
From: Alexander Shiyan @ 2022-06-27 10:24 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Shiyan

WARNING: unmet direct dependencies detected for PBL_IMAGE
Depends on [n]: HAVE_PBL_IMAGE [=n]
Selected by [y]:
- PBL_MULTI_IMAGES [=y] && HAVE_PBL_MULTI_IMAGES [=y]

WARNING: unmet direct dependencies detected for PBL_VERIFY_PIGGY
Depends on [n]: PBL_IMAGE [=n] && ARM [=y]
Selected by [y]:
- ARCH_IMX8M [=y] && ARCH_IMX [=y] && HABV4 [=y]

WARNING: unmet direct dependencies detected for BOARD_GENERIC_DT
Depends on [n]: PBL_IMAGE [=n]
Selected by [y]:
- BOARD_ARM_GENERIC_DT [=y] && HAVE_PBL_MULTI_IMAGES [=y] && OFDEVICE [=y]

Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
 pbl/Kconfig | 24 +++++++++++-------------
 1 file changed, 11 insertions(+), 13 deletions(-)

diff --git a/pbl/Kconfig b/pbl/Kconfig
index ce0acbb646..90eeafe29f 100644
--- a/pbl/Kconfig
+++ b/pbl/Kconfig
@@ -11,23 +11,29 @@ config HAVE_IMAGE_COMPRESSION
 
 config PBL_IMAGE
 	bool "Pre-Bootloader image"
-	depends on HAVE_PBL_IMAGE
+	depends on HAVE_PBL_IMAGE || HAVE_PBL_MULTI_IMAGES
+
+config PBL_VERIFY_PIGGY
+	depends on ARM
+	bool
+
+config BOARD_GENERIC_DT
+	bool
+	select LIBFDT
+
+if PBL_IMAGE
 
 config PBL_MULTI_IMAGES
 	bool
-	select PBL_IMAGE
 	select PBL_RELOCATABLE
 	depends on HAVE_PBL_MULTI_IMAGES
 	default y
 
 config PBL_SINGLE_IMAGE
 	bool
-	depends on PBL_IMAGE
 	depends on !HAVE_PBL_MULTI_IMAGES
 	default y
 
-if PBL_IMAGE
-
 config PBL_RELOCATABLE
 	depends on ARM || MIPS || RISCV
 	bool "relocatable pbl image"
@@ -38,14 +44,6 @@ config PBL_RELOCATABLE
 	  This option only influences the PBL image. See RELOCATABLE to also make
 	  the real image relocatable.
 
-config PBL_VERIFY_PIGGY
-	depends on ARM
-	bool
-
-config BOARD_GENERIC_DT
-	bool
-	select LIBFDT
-
 config IMAGE_COMPRESSION
 	bool
 	depends on HAVE_IMAGE_COMPRESSION
-- 
2.32.0




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

* [PATCH 7/8] ARM: mxs: Fix Kconfig dependencies
  2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
                   ` (5 preceding siblings ...)
  2022-06-27 10:24 ` [PATCH 6/8] pbl: " Alexander Shiyan
@ 2022-06-27 10:24 ` Alexander Shiyan
  2022-06-27 10:25 ` [PATCH 8/8] ARM: boards: ccmx51: Remove unused KConfig option Alexander Shiyan
  2022-06-29  7:10 ` [PATCH 0/8] Misc fixes part III Sascha Hauer
  8 siblings, 0 replies; 17+ messages in thread
From: Alexander Shiyan @ 2022-06-27 10:24 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Shiyan

WARNING: unmet direct dependencies detected for EEPROM_AT24
Depends on [n]: I2C [=n]
Selected by [y]:
- MACH_CFA10036 [=y] && ARCH_MXS [=y] && ARCH_IMX28 [=y]

WARNING: unmet direct dependencies detected for I2C_GPIO
Depends on [n]: I2C [=n] && GENERIC_GPIO [=y]
Selected by [y]:
- MACH_CFA10036 [=y] && ARCH_MXS [=y] && ARCH_IMX28 [=y]

Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
 arch/arm/mach-mxs/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-mxs/Kconfig b/arch/arm/mach-mxs/Kconfig
index 691c159e36..c8ef2c62af 100644
--- a/arch/arm/mach-mxs/Kconfig
+++ b/arch/arm/mach-mxs/Kconfig
@@ -79,6 +79,7 @@ config MACH_DUCKBILL
 config MACH_CFA10036
 	bool "cfa-10036"
 	select MXS_OCOTP
+	select I2C
 	select I2C_GPIO
 	select EEPROM_AT24
 	help
-- 
2.32.0




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

* [PATCH 8/8] ARM: boards: ccmx51: Remove unused KConfig option
  2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
                   ` (6 preceding siblings ...)
  2022-06-27 10:24 ` [PATCH 7/8] ARM: mxs: " Alexander Shiyan
@ 2022-06-27 10:25 ` Alexander Shiyan
  2022-06-29  7:10 ` [PATCH 0/8] Misc fixes part III Sascha Hauer
  8 siblings, 0 replies; 17+ messages in thread
From: Alexander Shiyan @ 2022-06-27 10:25 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Shiyan

RELOCATABLE is already selected by IMX_MULTI_BOARDS

Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
 arch/arm/mach-imx/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
index 6b962dcf7e..4283764a17 100644
--- a/arch/arm/mach-imx/Kconfig
+++ b/arch/arm/mach-imx/Kconfig
@@ -277,7 +277,6 @@ config MACH_CCMX51
 	select SPI
 	select DRIVER_SPI_IMX
 	select MFD_MC13XXX
-	select RELOCATABLE
 	help
 	  Say Y here if you are using Digi ConnectCore (W)i-i.MX51
 	  equipped with a Freescale i.MX51 Processor
-- 
2.32.0




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

* Re: [PATCH 4/8] usb: gadget: Fix Kconfig dependencies
  2022-06-27 10:24 ` [PATCH 4/8] usb: gadget: Fix Kconfig dependencies Alexander Shiyan
@ 2022-06-29  6:52   ` Sascha Hauer
  2022-06-29  6:55     ` Ahmad Fatoum
  0 siblings, 1 reply; 17+ messages in thread
From: Sascha Hauer @ 2022-06-29  6:52 UTC (permalink / raw)
  To: Alexander Shiyan; +Cc: barebox, Ahmad Fatoum

On Mon, Jun 27, 2022 at 01:24:56PM +0300, Alexander Shiyan wrote:
> WARNING: unmet direct dependencies detected for SYSTEM_PARTITIONS
> Depends on [n]: GLOBALVAR [=n]
> Selected by [y]:
> - USB_GADGET_AUTOSTART [=y] && USB_GADGET [=y] && USB_GADGET_MASS_STORAGE [=y]
> 
> WARNING: unmet direct dependencies detected for USB_GADGET_DRIVER_ARC_PBL
> Depends on [n]: USB_GADGET [=n]
> Selected by [y]:
> - MACH_PROTONIC_IMX8M [=y] && ARCH_IMX [=y] && IMX_MULTI_BOARDS [=y]
> 
> Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
> ---
>  drivers/usb/gadget/Kconfig | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
> index 10c713e833..f2ca3695a2 100644
> --- a/drivers/usb/gadget/Kconfig
> +++ b/drivers/usb/gadget/Kconfig
> @@ -4,6 +4,10 @@ menuconfig USB_GADGET
>  	select POLLER
>  	bool "USB gadget support"
>  
> +config USB_GADGET_DRIVER_ARC_PBL
> +	select USB_GADGET
> +	bool

The "select USB_GADGET" is not necessary. The PBL part of that driver
doesn't use anything from USB_GADGET.

> +
>  if USB_GADGET
>  
>  config USB_GADGET_DUALSPEED
> @@ -16,9 +20,6 @@ config USB_GADGET_DRIVER_ARC
>  	default y
>  	select USB_GADGET_DUALSPEED
>  
> -config USB_GADGET_DRIVER_ARC_PBL
> -	bool
> -
>  config USB_GADGET_DRIVER_AT91
>  	bool
>  	prompt "at91 gadget driver"
> @@ -37,6 +38,7 @@ config USB_GADGET_AUTOSTART
>  	bool
>  	default y
>  	prompt "Automatically start usbgadget on boot"
> +	depends on GLOBALVAR

Without GLOBALVAR selected the USB gadget autostart stuff can be
compiled, but without any functionality, as all functionality is
behind globalvars, so adding this dependency mykes sense.

>  	select SYSTEM_PARTITIONS if USB_GADGET_MASS_STORAGE

I am not sure why SYSTEM_PARTITIONS is selected here. Maybe Ahmad can
comment on that. From what I can see with USB_GADGET_AUTOSTART selected
without SYSTEM_PARTITIONS we still get autostart functionality, but
without global.system.partitions support which seems ok.

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



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

* Re: [PATCH 4/8] usb: gadget: Fix Kconfig dependencies
  2022-06-29  6:52   ` Sascha Hauer
@ 2022-06-29  6:55     ` Ahmad Fatoum
  2022-06-29  6:59       ` Sascha Hauer
  0 siblings, 1 reply; 17+ messages in thread
From: Ahmad Fatoum @ 2022-06-29  6:55 UTC (permalink / raw)
  To: Sascha Hauer, Alexander Shiyan; +Cc: barebox, Ahmad Fatoum

Hello Sascha,

On 29.06.22 08:52, Sascha Hauer wrote:
> Without GLOBALVAR selected the USB gadget autostart stuff can be
> compiled, but without any functionality, as all functionality is
> behind globalvars, so adding this dependency mykes sense.
> 
>>  	select SYSTEM_PARTITIONS if USB_GADGET_MASS_STORAGE
> 
> I am not sure why SYSTEM_PARTITIONS is selected here. Maybe Ahmad can
> comment on that. From what I can see with USB_GADGET_AUTOSTART selected
> without SYSTEM_PARTITIONS we still get autostart functionality, but
> without global.system.partitions support which seems ok.

Instead of adding yet another gadget-specific global variable,
the mass storage gadget only uses system partitions, so this
select makes sense IMO.

Cheers,
Ahmad

> 
> Sascha
> 


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



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

* Re: [PATCH 4/8] usb: gadget: Fix Kconfig dependencies
  2022-06-29  6:55     ` Ahmad Fatoum
@ 2022-06-29  6:59       ` Sascha Hauer
  2022-06-29  7:03         ` Ahmad Fatoum
  0 siblings, 1 reply; 17+ messages in thread
From: Sascha Hauer @ 2022-06-29  6:59 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: Alexander Shiyan, barebox, Ahmad Fatoum

On Wed, Jun 29, 2022 at 08:55:46AM +0200, Ahmad Fatoum wrote:
> Hello Sascha,
> 
> On 29.06.22 08:52, Sascha Hauer wrote:
> > Without GLOBALVAR selected the USB gadget autostart stuff can be
> > compiled, but without any functionality, as all functionality is
> > behind globalvars, so adding this dependency mykes sense.
> > 
> >>  	select SYSTEM_PARTITIONS if USB_GADGET_MASS_STORAGE
> > 
> > I am not sure why SYSTEM_PARTITIONS is selected here. Maybe Ahmad can
> > comment on that. From what I can see with USB_GADGET_AUTOSTART selected
> > without SYSTEM_PARTITIONS we still get autostart functionality, but
> > without global.system.partitions support which seems ok.
> 
> Instead of adding yet another gadget-specific global variable,
> the mass storage gadget only uses system partitions, so this
> select makes sense IMO.

I would argument that USB_GADGET_AUTOSTART is functional without system
partitions, and the additional support for system partitions can be
manually selected by the user.

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



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

* Re: [PATCH 4/8] usb: gadget: Fix Kconfig dependencies
  2022-06-29  6:59       ` Sascha Hauer
@ 2022-06-29  7:03         ` Ahmad Fatoum
  2022-06-29  7:08           ` Sascha Hauer
  0 siblings, 1 reply; 17+ messages in thread
From: Ahmad Fatoum @ 2022-06-29  7:03 UTC (permalink / raw)
  To: Sascha Hauer; +Cc: Alexander Shiyan, barebox, Ahmad Fatoum

Hello Sascha,

On 29.06.22 08:59, Sascha Hauer wrote:
> On Wed, Jun 29, 2022 at 08:55:46AM +0200, Ahmad Fatoum wrote:
>> Hello Sascha,
>>
>> On 29.06.22 08:52, Sascha Hauer wrote:
>>> Without GLOBALVAR selected the USB gadget autostart stuff can be
>>> compiled, but without any functionality, as all functionality is
>>> behind globalvars, so adding this dependency mykes sense.
>>>
>>>>  	select SYSTEM_PARTITIONS if USB_GADGET_MASS_STORAGE
>>>
>>> I am not sure why SYSTEM_PARTITIONS is selected here. Maybe Ahmad can
>>> comment on that. From what I can see with USB_GADGET_AUTOSTART selected
>>> without SYSTEM_PARTITIONS we still get autostart functionality, but
>>> without global.system.partitions support which seems ok.
>>
>> Instead of adding yet another gadget-specific global variable,
>> the mass storage gadget only uses system partitions, so this
>> select makes sense IMO.
> 
> I would argument that USB_GADGET_AUTOSTART is functional without system
> partitions, and the additional support for system partitions can be
> manually selected by the user.

I think it's a bit surprising to enable USB_GADGET_AUTOSTART and then
see DFU and fastboot working, but not USB_GADGET_MASS_STORAGE.

If you have both USB_GADGET_MASS_STORAGE and USB_GADGET_AUTOSTART enabled,
the user implicitly agrees to waste some space on them working together.

Cheers,
Ahmad

> 
> Sascha
> 


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



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

* Re: [PATCH 4/8] usb: gadget: Fix Kconfig dependencies
  2022-06-29  7:03         ` Ahmad Fatoum
@ 2022-06-29  7:08           ` Sascha Hauer
  0 siblings, 0 replies; 17+ messages in thread
From: Sascha Hauer @ 2022-06-29  7:08 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: Alexander Shiyan, barebox, Ahmad Fatoum

On Wed, Jun 29, 2022 at 09:03:41AM +0200, Ahmad Fatoum wrote:
> Hello Sascha,
> 
> On 29.06.22 08:59, Sascha Hauer wrote:
> > On Wed, Jun 29, 2022 at 08:55:46AM +0200, Ahmad Fatoum wrote:
> >> Hello Sascha,
> >>
> >> On 29.06.22 08:52, Sascha Hauer wrote:
> >>> Without GLOBALVAR selected the USB gadget autostart stuff can be
> >>> compiled, but without any functionality, as all functionality is
> >>> behind globalvars, so adding this dependency mykes sense.
> >>>
> >>>>  	select SYSTEM_PARTITIONS if USB_GADGET_MASS_STORAGE
> >>>
> >>> I am not sure why SYSTEM_PARTITIONS is selected here. Maybe Ahmad can
> >>> comment on that. From what I can see with USB_GADGET_AUTOSTART selected
> >>> without SYSTEM_PARTITIONS we still get autostart functionality, but
> >>> without global.system.partitions support which seems ok.
> >>
> >> Instead of adding yet another gadget-specific global variable,
> >> the mass storage gadget only uses system partitions, so this
> >> select makes sense IMO.
> > 
> > I would argument that USB_GADGET_AUTOSTART is functional without system
> > partitions, and the additional support for system partitions can be
> > manually selected by the user.
> 
> I think it's a bit surprising to enable USB_GADGET_AUTOSTART and then
> see DFU and fastboot working, but not USB_GADGET_MASS_STORAGE.
> 
> If you have both USB_GADGET_MASS_STORAGE and USB_GADGET_AUTOSTART enabled,
> the user implicitly agrees to waste some space on them working together.

I was mistaken. I didn't realize that system partitions is the only way
to activate USB mass storage support automatically. Ok then, applied
this one with just the "select USB_GADGET" removed.

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



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

* Re: [PATCH 2/8] ARM: bcm2835: Drop uneeded headers
  2022-06-27 10:24 ` [PATCH 2/8] ARM: bcm2835: Drop uneeded headers Alexander Shiyan
@ 2022-06-29  7:10   ` Sascha Hauer
  0 siblings, 0 replies; 17+ messages in thread
From: Sascha Hauer @ 2022-06-29  7:10 UTC (permalink / raw)
  To: Alexander Shiyan; +Cc: barebox

On Mon, Jun 27, 2022 at 01:24:54PM +0300, Alexander Shiyan wrote:
> Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
> ---
>  arch/arm/mach-bcm283x/core.c | 9 ---------
>  1 file changed, 9 deletions(-)
> 
> diff --git a/arch/arm/mach-bcm283x/core.c b/arch/arm/mach-bcm283x/core.c
> index f2528cf1f1..bcf2ecb204 100644
> --- a/arch/arm/mach-bcm283x/core.c
> +++ b/arch/arm/mach-bcm283x/core.c
> @@ -17,19 +17,10 @@
>   */
>  
>  #include <common.h>
> -#include <init.h>
> -
> -#include <linux/clk.h>
> -#include <linux/clkdev.h>
> -#include <linux/err.h>
> -
>  #include <io.h>
>  #include <asm/armlinux.h>
>  #include <linux/sizes.h>
> -
> -#include <mach/platform.h>
>  #include <mach/core.h>
> -#include <linux/amba/bus.h>

This one no longer applies, the file looks completely different now. I
didn't track if the content went somewhere else and the patch should be
applied on another file no. If so, please resend.

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



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

* Re: [PATCH 0/8] Misc fixes part III
  2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
                   ` (7 preceding siblings ...)
  2022-06-27 10:25 ` [PATCH 8/8] ARM: boards: ccmx51: Remove unused KConfig option Alexander Shiyan
@ 2022-06-29  7:10 ` Sascha Hauer
  8 siblings, 0 replies; 17+ messages in thread
From: Sascha Hauer @ 2022-06-29  7:10 UTC (permalink / raw)
  To: Alexander Shiyan; +Cc: barebox

On Mon, Jun 27, 2022 at 01:24:52PM +0300, Alexander Shiyan wrote:
> Unsorted various fixes for the barebox.
> 
> Alexander Shiyan (8):
>   ARM: vexpress: Simplify sysctl initialization
>   ARM: bcm2835: Drop uneeded headers
>   of: Use appropriate header for of_clk_init()
>   usb: gadget: Fix Kconfig dependencies
>   logo: Fix Kconfig dependencies
>   pbl: Fix Kconfig dependencies
>   ARM: mxs: Fix Kconfig dependencies
>   ARM: boards: ccmx51: Remove unused KConfig option

Applied, thanks

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



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

* Re: [PATCH 6/8] pbl: Fix Kconfig dependencies
  2022-06-27 10:24 ` [PATCH 6/8] pbl: " Alexander Shiyan
@ 2022-06-30 11:57   ` Sascha Hauer
  0 siblings, 0 replies; 17+ messages in thread
From: Sascha Hauer @ 2022-06-30 11:57 UTC (permalink / raw)
  To: Alexander Shiyan; +Cc: barebox

On Mon, Jun 27, 2022 at 01:24:58PM +0300, Alexander Shiyan wrote:
> WARNING: unmet direct dependencies detected for PBL_IMAGE
> Depends on [n]: HAVE_PBL_IMAGE [=n]
> Selected by [y]:
> - PBL_MULTI_IMAGES [=y] && HAVE_PBL_MULTI_IMAGES [=y]
> 
> WARNING: unmet direct dependencies detected for PBL_VERIFY_PIGGY
> Depends on [n]: PBL_IMAGE [=n] && ARM [=y]
> Selected by [y]:
> - ARCH_IMX8M [=y] && ARCH_IMX [=y] && HABV4 [=y]
> 
> WARNING: unmet direct dependencies detected for BOARD_GENERIC_DT
> Depends on [n]: PBL_IMAGE [=n]
> Selected by [y]:
> - BOARD_ARM_GENERIC_DT [=y] && HAVE_PBL_MULTI_IMAGES [=y] && OFDEVICE [=y]

I had to revert this one as it breaks for example am335x_mlo_defconfig.
It breaks because HAVE_PBL_MULTI_IMAGES is set. With this
PBL_MULTI_IMAGES becomes enabled because it is default y. That option
in turn selects PBL_IMAGE. Now with this patch applied we end up
with PBL_IMAGE disabled and the config doesn't compile anymore.

I think we have to split this patch up further. What I trapped in was:

> WARNING: unmet direct dependencies detected for PBL_IMAGE
> Depends on [n]: HAVE_PBL_IMAGE [=n]
> Selected by [y]:
> - PBL_MULTI_IMAGES [=y] && HAVE_PBL_MULTI_IMAGES [=y]

And I think that one could be fixed with:

config HAVE_PBL_MULTI_IMAGES
	select HAVE_PBL_IMAGE
	bool

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



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

end of thread, other threads:[~2022-06-30 11:59 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-27 10:24 [PATCH 0/8] Misc fixes part III Alexander Shiyan
2022-06-27 10:24 ` [PATCH 1/8] ARM: vexpress: Simplify sysctl initialization Alexander Shiyan
2022-06-27 10:24 ` [PATCH 2/8] ARM: bcm2835: Drop uneeded headers Alexander Shiyan
2022-06-29  7:10   ` Sascha Hauer
2022-06-27 10:24 ` [PATCH 3/8] of: Use appropriate header for of_clk_init() Alexander Shiyan
2022-06-27 10:24 ` [PATCH 4/8] usb: gadget: Fix Kconfig dependencies Alexander Shiyan
2022-06-29  6:52   ` Sascha Hauer
2022-06-29  6:55     ` Ahmad Fatoum
2022-06-29  6:59       ` Sascha Hauer
2022-06-29  7:03         ` Ahmad Fatoum
2022-06-29  7:08           ` Sascha Hauer
2022-06-27 10:24 ` [PATCH 5/8] logo: " Alexander Shiyan
2022-06-27 10:24 ` [PATCH 6/8] pbl: " Alexander Shiyan
2022-06-30 11:57   ` Sascha Hauer
2022-06-27 10:24 ` [PATCH 7/8] ARM: mxs: " Alexander Shiyan
2022-06-27 10:25 ` [PATCH 8/8] ARM: boards: ccmx51: Remove unused KConfig option Alexander Shiyan
2022-06-29  7:10 ` [PATCH 0/8] Misc fixes part III Sascha Hauer

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