Re: [PATHv11 26/43] configs/tbs2910_defconfig inc limit

2023-12-05 Thread Soeren Moch
On 05.12.23 21:00, Maxim Uvarov wrote: On Wed, 6 Dec 2023 at 00:25, Soeren Moch wrote: On 05.12.23 17:25, Maxim Uvarov wrote: On Tue, 5 Dec 2023 at 21:49, Soeren Moch wrote: On 05.12.23 14:15, Maxim Uvarov wrote: I think I solved the size issue on all the boards

Re: [PATHv11 26/43] configs/tbs2910_defconfig inc limit

2023-12-05 Thread Soeren Moch
On 05.12.23 17:25, Maxim Uvarov wrote: On Tue, 5 Dec 2023 at 21:49, Soeren Moch wrote: On 05.12.23 14:15, Maxim Uvarov wrote: I think I solved the size issue on all the boards. Key changes: 1. remove compilation of original ping.c and tftp.c (tftp had also server

Re: [PATHv11 26/43] configs/tbs2910_defconfig inc limit

2023-12-05 Thread Soeren Moch
@@ CONFIG_CMD_DHCP=y  CONFIG_CMD_MII=y  CONFIG_CMD_PING=y  CONFIG_CMD_CACHE=y -CONFIG_CMD_TIME=y +# CONFIG_CMD_DATE is not set  CONFIG_CMD_SYSBOOT=y  # CONFIG_CMD_VIDCONSOLE is not set  CONFIG_CMD_EXT2=y BR, Maxim. On Tue, 28 Nov 2023 at 13:09, Maxim Uvarov wrote: On Tue, 28 Nov 2023 at 03:20, So

Re: [PATHv11 26/43] configs/tbs2910_defconfig inc limit

2023-11-27 Thread Soeren Moch
On 27.11.23 14:11, Tom Rini wrote: On Mon, Nov 27, 2023 at 06:57:09PM +0600, Maxim Uvarov wrote: Increase allowed binary size to fit lwip code. Signed-off-by: Maxim Uvarov --- configs/tbs2910_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH v2 9/9] x86: Update qemu documentation

2023-07-30 Thread Soeren Moch
On 30.07.23 19:16, Simon Glass wrote: Add some hints and observations related to booting distros on QEMU on x86. Signed-off-by: Simon Glass Simon, you cc'd me on this patch (both versions), but I never used u-boot on x86. What do you expect from my side here? Thanks, Soeren --- (no

Re: [PATCH v2] doc: fix references to distro documentation

2023-01-09 Thread Soeren Moch
On 09.01.23 12:52, Dario Binacchi wrote: Commit 37c5195dfcd157 ("doc: Move distro boot doc to rST") renamed doc/README.distro to doc/develop/distro.rst. Signed-off-by: Dario Binacchi Reviewed-by: Patrice Chotard Reviewed-by: Soeren Moch --- Changes in v2: - Change 'distro.rs

Re: [PATCH] doc: fix references to distro documentation

2023-01-09 Thread Soeren Moch
Hi Dario, On 08.01.23 11:07, Dario Binacchi wrote: Commit 37c5195dfcd157 ("doc: Move distro boot doc to rST") renamed doc/README.distro to doc/develop/distro.rst. Signed-off-by: Dario Binacchi --- doc/README.gpt | 2 +- doc/README.uniphier | 2 +-

Re: [PATCH 2/2] board: tbs2910: Add serial rx buffer in defconfig

2022-09-20 Thread Soeren Moch
On 20.09.22 20:44, Fabio Estevam wrote: On Tue, Sep 20, 2022 at 1:05 PM Soeren Moch wrote: ... to avoid loosing characters when pasting text into the serial console. This allows to remove the workaround to disable the vidconsole output when no HDMI device is detected. This workaround only

[PATCH 2/2] board: tbs2910: Add serial rx buffer in defconfig

2022-09-20 Thread Soeren Moch
... to avoid loosing characters when pasting text into the serial console. This allows to remove the workaround to disable the vidconsole output when no HDMI device is detected. This workaround only was there to speed-up serial console processing. Signed-off-by: Soeren Moch --- Cc: Stefano Babic

[PATCH 1/2] board: tbs2910: Set all board dts files as maintained

2022-09-20 Thread Soeren Moch
This also includes the imx6q-tbs2910-u-boot.dtsi file now. Signed-off-by: Soeren Moch --- Cc: Stefano Babic Cc: Fabio Estevam Cc: "NXP i.MX U-Boot Team" Cc: u-boot@lists.denx.de --- board/tbs/tbs2910/MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

Re: [PATCH v3 1/3] Convert CONFIG_SYS_L2_PL310 to Kconfig

2022-08-12 Thread Soeren Moch
On 09.08.22 12:07, Philip Oberfichtner wrote: This converts CONFIG_SYS_L2_PL310 to Kconfig. Signed-off-by: Philip Oberfichtner We used common includes before to select L2_PL31, e.g. for tbs2910 (and many more similar boards) we had mx6_common.h . Now with this conversion lots of individual

Re: [PATCH v1 03/26] tbs2910: prepare to synchronise device trees with linux

2022-07-25 Thread Soeren Moch
On 25.07.22 18:42, sba...@denx.de wrote: From: Marcel Ziswiler As a preparatory step make sure to refer to absolute node paths where labels were removed in Linux upstream. This avoids the following error once synchronised: +Error: arch/arm/dts/imx6q-tbs2910-u-boot.dtsi:3.1-7 Label or path

Re: [PATCH v1 03/26] tbs2910: prepare to synchronise device trees with linux

2022-07-21 Thread Soeren Moch
On 21.07.22 15:27, Marcel Ziswiler wrote: From: Marcel Ziswiler As a preparatory step make sure to refer to absolute node paths where labels were removed in Linux upstream. Are you sure that this was removed upstream? It was just submitted for inclusion [1], and may not have landed yet.

Re: [PATCH 20/20] Convert CONFIG_SYS_BOOTM_LEN to Kconfig

2022-06-26 Thread Soeren Moch
On 26.06.22 00:44, Tom Rini wrote: On Sun, Jun 26, 2022 at 12:01:23AM +0200, Soeren Moch wrote: diff --git a/configs/tbs2910_defconfig b/configs/tbs2910_defconfig index 892d7c60d283..f0ecfd049d65 100644 --- a/configs/tbs2910_defconfig +++ b/configs/tbs2910_defconfig @@ -35,6 +35,7

Re: [PATCH 20/20] Convert CONFIG_SYS_BOOTM_LEN to Kconfig

2022-06-25 Thread Soeren Moch
diff --git a/configs/tbs2910_defconfig b/configs/tbs2910_defconfig index 892d7c60d283..f0ecfd049d65 100644 --- a/configs/tbs2910_defconfig +++ b/configs/tbs2910_defconfig @@ -35,6 +35,7 @@ CONFIG_CMD_BOOTZ=y # CONFIG_BOOTM_PLAN9 is not set # CONFIG_BOOTM_RTEMS is not set #

Re: [PATCH 20/20] Convert CONFIG_SYS_BOOTM_LEN to Kconfig

2022-06-25 Thread Soeren Moch
Hi Tom, On 25.06.22 17:02, Tom Rini wrote: This converts the following to Kconfig: CONFIG_SYS_BOOTM_LEN As part of this, rework error handling in boot/bootm.c so that we pass the buffer size to handle_decomp_error as CONFIG_SYS_BOOTM_LEN will not be available to host tools but we do know

Re: [PATCH] tbs2910: Convert to DM_SERIAL

2022-06-14 Thread Soeren Moch
On 14.06.22 18:27, Fabio Estevam wrote: On Sat, Mar 19, 2022 at 10:31 AM Fabio Estevam wrote: Fabio, will you sync the imx6qdl.dtsi changes to linux? Or how is this supposed to keep in sync? Yes, I plan to submit these imx6qdl.dtsi changes (and similar changes to the other i.MX dtsi

Re: [PATCH V2] imx: drop CONFIG_MXC_UART_BASE

2022-05-03 Thread Soeren Moch
On 03.05.22 06:14, Peng Fan (OSS) wrote: From: Peng Fan Since these boards has CONFIG_DM_SERIAL and/or CONFIG_SPL_DM_SERIAL, the legacy macro no need to be defined. Reviewed-by: Heiko Schocher Reviewed-by: Fabio Estevam Signed-off-by: Peng Fan For tbs2910: Acked-by: Soeren Moch

Re: [PATCH] imx: drop CONFIG_MXC_UART_BASE

2022-04-29 Thread Soeren Moch
On 29.04.22 15:10, Peng Fan (OSS) wrote: From: Peng Fan Since these boards has CONFIG_DM_SERIAL and/or CONFIG_SPL_DM_SERIAL, the legacy macro no need to be defined. Signed-off-by: Peng Fan --- include/configs/aristainetos2.h | 2 -- include/configs/imx8mm_evk.h | 1 -

Re: [PATCH] tbs2910: Convert to DM_SERIAL

2022-03-19 Thread Soeren Moch
Hi Fabio, On 19.03.22 14:31, Fabio Estevam wrote: Hi Soeren, On Sat, Mar 19, 2022 at 8:34 AM Soeren Moch wrote: Works great (on top of v2022.04-rc4), thanks! Tested-by: Soeren Moch Great, thanks for confirming it works. Fabio, will you sync the imx6qdl.dtsi changes to linux? Or how

Re: [PATCH 2/2] board: tbs2910: Convert to DM_SERIAL

2022-03-19 Thread Soeren Moch
Hi Fabio, On 15.03.22 21:56, Soeren Moch wrote: Hi Fabio, On 15.03.22 00:16, Fabio Estevam wrote: Hi Soeren, On Mon, Mar 14, 2022 at 7:38 PM Tom Rini wrote: Again, DM_SERIAL can be enabled on the board as he's already shown by setting two options, which silences the warning, increases

Re: [PATCH] tbs2910: Convert to DM_SERIAL

2022-03-19 Thread Soeren Moch
that describes the nodes that require dm-pre-reloc, which allows the DM model to configure the UART pinctrl early. Remove the now unneeded board UART initialization. Signed-off-by: Fabio Estevam Works great (on top of v2022.04-rc4), thanks! Tested-by: Soeren Moch Fabio, will you sync

Re: [PATCH] tbs2910: Convert to DM_SERIAL

2022-03-15 Thread Soeren Moch
On 15.03.22 21:47, Fabio Estevam wrote: From: Fabio Estevam Conversion to DM_SERIAL is mandatory. Select DM_SERIAL and add a imx6q-tbs2910-u-boot.dtsi file that describes the nodes that require dm-pre-reloc, which allows the DM model to configure the UART pinctrl early. Remove the now

Re: [PATCH 2/2] board: tbs2910: Convert to DM_SERIAL

2022-03-15 Thread Soeren Moch
Hi Fabio, On 15.03.22 00:16, Fabio Estevam wrote: Hi Soeren, On Mon, Mar 14, 2022 at 7:38 PM Tom Rini wrote: Again, DM_SERIAL can be enabled on the board as he's already shown by setting two options, which silences the warning, increases the size and doesn't make anything better. That's

Re: [PATCH 2/2] board: tbs2910: Convert to DM_SERIAL

2022-03-14 Thread Soeren Moch
On 14.03.22 19:28, Tom Rini wrote: On Mon, Mar 14, 2022 at 12:24:36PM -0600, Simon Glass wrote: Hi Soeren, On Mon, 14 Mar 2022 at 02:26, Soeren Moch wrote: ... to get rid of the build warning. Unfortunately we still need the board specific serial pin init code. Otherwise the first boot

Re: [PATCH v3 00/19] efi_loader: more tightly integrate UEFI disks to driver model

2022-03-14 Thread Soeren Moch
Hi Simon, On 14.03.22 18:08, Simon Glass wrote: Hi Soeren, [I think you sent your email with html or something so it is a big mangled. I'll just add one comment] On Mon, 14 Mar 2022 at 02:27, Soeren Moch wrote: Hi Simon, On 12.03.22 06:02, Simon Glass wrote: Hi Soeren, On Fri, 11 Mar

Re: [PATCH v3 00/19] efi_loader: more tightly integrate UEFI disks to driver model

2022-03-14 Thread Soeren Moch
Hi Simon, On 12.03.22 06:02, Simon Glass wrote: Hi Soeren, On Fri, 11 Mar 2022 at 15:43, Soeren Moch wrote: On 09.03.22 16:33, Simon Glass wrote: Hi Tom, On Wed, 9 Mar 2022 at 07:25, Tom Rini wrote: On Tue, Mar 08, 2022 at 08:10:38PM -0700, Simon Glass wrote: Hi Tom, On Tue, 8 Mar

[PATCH 2/2] board: tbs2910: Convert to DM_SERIAL

2022-03-14 Thread Soeren Moch
... to get rid of the build warning. Unfortunately we still need the board specific serial pin init code. Otherwise the first boot messages over the serial console are lost. Signed-off-by: Soeren Moch --- Cc: Stefano Babic Cc: Fabio Estevam Cc: Tom Rini Cc: Simon Glass Cc: u-boot

[PATCH 1/2] board: tbs2910: Enable Link Time Optimizations in defconfig

2022-03-14 Thread Soeren Moch
This saves about 12 kBytes image size and helps to stay within the size limit. Suggested-by: Tom Rini Signed-off-by: Soeren Moch --- Cc: Stefano Babic Cc: Fabio Estevam Cc: Simon Glass Cc: u-boot@lists.denx.de Unfortunately we already are at the size limit for U-Boot v2022.04-rc3. Maybe Tom

Re: [PATCH 2/5] tbs2910: Disable ext4 write

2022-03-11 Thread Soeren Moch
On 11.03.22 20:10, Simon Glass wrote: This board is right up against the size limit. Drop ext4 writing to give it some more space. Signed-off-by: Simon Glass NAK. At least for now. We should not randomly disable user visible functionality and not without investigating other possible

Re: [PATCH v3 00/19] efi_loader: more tightly integrate UEFI disks to driver model

2022-03-11 Thread Soeren Moch
, 8 Mar 2022 at 17:13, Tom Rini wrote: On Tue, Mar 08, 2022 at 02:20:15PM -0700, Simon Glass wrote: Hi Soeren, On Tue, 8 Mar 2022 at 12:15, Soeren Moch wrote: On 08.03.22 17:56, Simon Glass wrote: Hi, On Tue, 8 Mar 2022 at 09:49, Heinrich Schuchardt wrote: On 3/8/22 12:36, AKASHI Takahiro

Re: [PATCH v3 00/19] efi_loader: more tightly integrate UEFI disks to driver model

2022-03-08 Thread Soeren Moch
On 08.03.22 17:56, Simon Glass wrote: Hi, On Tue, 8 Mar 2022 at 09:49, Heinrich Schuchardt wrote: On 3/8/22 12:36, AKASHI Takahiro wrote: With this patch set[1] applied, UEFI subsystem maintains a list of its disk objects dynamically at runtime based on block device's probing. (See

Re: [PATCH] pci: imx: use reset-gpios if defined by device-tree

2021-07-03 Thread Soeren Moch
) > Cc: Sebastian Reichel (maintainer:GE BX50V3 > BOARD) > Cc: Fabio Estevam (maintainer:MX6SABRESD BOARD) > Cc: Marek Vasut (maintainer:NOVENA BOARD) > Cc: Soeren Moch (maintainer:TBS2910 BOARD) > Cc: Silvio Fricke (maintainer:VINING_2000 BOARD) > Signed-off-by: Tim H

Re: FW: [PATCH 0/6] video: remove VIDCONSOLE_AS_LCD and VIDCONSOLE_AS_NAME

2020-12-20 Thread Soeren Moch
On 08.12.20 17:16, Tom Rini wrote: > On Tue, Dec 08, 2020 at 03:30:43PM +0100, Patrick DELAUNAY wrote: >> Hi Soeren, >> >>> From: Soeren Moch Sent: dimanche 6 décembre 2020 20:59 On >>> 03.12.20 10:15, Patrick Delaunay wrote: >>> >>>> I p

Re: [PATCH 1/6] tbs2910: configs: remove VIDCONSOLE_AS_LCD

2020-12-06 Thread Soeren Moch
On 03.12.20 10:15, Patrick Delaunay wrote: > Remove the obsolete CONFIG_VIDCONSOLE_AS_LCD as vidconsole is used in > ./include/configs/tbs2910.h since commit 513acd04452f ("tbs2910: migrate > to DM_VIDEO"). I don't consider this workaround as obsolete, please see my response to the cover letter of

Re: [PATCH 0/6] video: remove VIDCONSOLE_AS_LCD and VIDCONSOLE_AS_NAME

2020-12-06 Thread Soeren Moch
On 03.12.20 10:15, Patrick Delaunay wrote: > I propose this serie to remove the vidconsole work-around, activated with > the 2 options VIDCONSOLE_AS_LCD and VIDCONSOLE_AS_NAME and cleanup > the associated code in console.c (under #ifdef CONFIG_VIDCONSOLE_AS_LCD) > > This options are now obsolete

Re: [PATCH 1/2] board: tbs2910: Use recommended distroboot addresses

2020-10-02 Thread Soeren Moch
-relocation of the kernel after starting linux. Thanks, Soeren On 27.08.20 21:52, Soeren Moch wrote: > According to doc/README.distro fdt_addr must not be set when DTB is not > available from hardware. So remove this entry. > > Use address 32MB above the start of DRAM for kernel_addr_r. This way

Re: [PATCH 2/2] board: tbs2910: Disable CONFIG_ENV_VARS_UBOOT_CONFIG in defconfig

2020-09-18 Thread Soeren Moch
On 18.09.20 18:21, Tom Rini wrote: > On Fri, Sep 18, 2020 at 05:56:03PM +0200, Soeren Moch wrote: >> >> On 18.09.20 17:51, Stefano Babic wrote: >>> On 18.09.20 17:48, Soeren Moch wrote: >>>> On 18.09.20 17:21, sba...@denx.de wrote: >>>>>>

Re: [PATCH 2/2] board: tbs2910: Disable CONFIG_ENV_VARS_UBOOT_CONFIG in defconfig

2020-09-18 Thread Soeren Moch
On 18.09.20 17:51, Stefano Babic wrote: > On 18.09.20 17:48, Soeren Moch wrote: >> On 18.09.20 17:21, sba...@denx.de wrote: >>>> This is not required for sysboot (we defined fdtfile), let's save a few >>>> bytes in the binary image without these variables

Re: [PATCH 2/2] board: tbs2910: Disable CONFIG_ENV_VARS_UBOOT_CONFIG in defconfig

2020-09-18 Thread Soeren Moch
On 18.09.20 17:21, sba...@denx.de wrote: >> This is not required for sysboot (we defined fdtfile), let's save a few >> bytes in the binary image without these variables. >> Signed-off-by: Soeren Moch >> Reviewed-by: Fabio Estevam > Applied to u-boot-imx, master,

[PATCH 2/2] board: tbs2910: Disable CONFIG_ENV_VARS_UBOOT_CONFIG in defconfig

2020-08-27 Thread Soeren Moch
This is not required for sysboot (we defined fdtfile), let's save a few bytes in the binary image without these variables. Signed-off-by: Soeren Moch --- Cc: Stefano Babic Cc: Fabio Estevam Cc: Denis 'GNUtoo' Carikli Cc: u-boot@lists.denx.de --- configs/tbs2910_defconfig | 1 - 1 file

[PATCH 1/2] board: tbs2910: Use recommended distroboot addresses

2020-08-27 Thread Soeren Moch
script / extlinux.conf at offset 0 / 1MB from start of DRAM. This space is available for processing in U-Boot. Signed-off-by: Soeren Moch --- Cc: Stefano Babic Cc: Fabio Estevam Cc: Denis 'GNUtoo' Carikli Cc: u-boot@lists.denx.de Recommendations taken from [1]. Distroboot support

Re: [PATCH] video: ipuv3: fix framebuffer base address init with multiple IPUs

2020-08-03 Thread Soeren Moch
allocated framebuffer is not correctly > programmed in the display controller. Fix it. > > Reported-by: Soeren Moch > Signed-off-by: Anatolij Gustschin Successfully tested on a tbs2910 board (i.MX6Q based). Tested-by: Soeren Moch Thanks, Soeren > --- > drivers/video/imx/mxc_ipuv3_fb.c

Re: [BUG] DM_VIDEO broken on imx6q

2020-08-03 Thread Soeren Moch
HiAnatolij, On 03.08.20 15:54, Anatolij Gustschin wrote: > Hi, > > On Sat, 1 Aug 2020 21:14:10 +0200 > Soeren Moch sm...@web.de wrote: > >> Hi, >> >> unfortunately the patch [1] is only the first part to get DM_VIDEO >> running properly on tbs2910 for u

[BUG] DM_VIDEO broken on imx6q

2020-08-01 Thread Soeren Moch
Hi, unfortunately the patch [1] is only the first part to get DM_VIDEO running properly on tbs2910 for u-boot-2020.10. With this, I get a valid video output signal, but no actual video console output, only a blank screen. After some debugging, no with access to a HDMI monitor, the problem seems

[PATCH] board: tbs2910: Fix video output with existing environments

2020-07-30 Thread Soeren Moch
output also with existing legacy environments. Signed-off-by: Soeren Moch --- Cc: Stefano Babic Cc: Fabio Estevam Cc: Anatolij Gustschin Cc: u-boot@lists.denx.de There was an automatic fix in the original DM_VIDEO conversion for this case. Apparently this fix was lost on the way to v2020.10-rc1

Re: [PATCH v2] ARM: dts: imx6q-tbs2910: Fix Ethernet regression

2020-06-25 Thread Soeren Moch
berger > > , the clock output setting for the AR803x driver is removed from being > hardcoded in the PHY driver and should be passed via device tree instead. > > Update the device tree with the "qca,clk-out-frequency" property so that > Ethernet can work agai

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-25 Thread Soeren Moch
Hi Fabio, On 25.06.20 01:19, Fabio Estevam wrote: > Hi Soeren, > > On Wed, Jun 24, 2020 at 7:29 PM Soeren Moch wrote: > >> Unfortunately I have no access to my usual development board (and will >> not have until the v2020.07 release). I only can use an old board &g

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Soeren Moch
Hi Fabio, On 24.06.20 23:35, Fabio Estevam wrote: > Hi Soeren, > > On Wed, Jun 24, 2020 at 5:45 PM Sören Moch wrote: > >> Still no luck: > > mx6cuboxi has an AR8035 too and it is working fine in U-Boot mainline. > > I am not sure why tbs2910 is failing. Could you try to bisect? > Unfortunately

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Soeren Moch
Hi Fabio On 24.06.20 21:39, Fabio Estevam wrote: > Hi Soeren, > > On Wed, Jun 24, 2020 at 4:20 PM Soeren Moch wrote: > >> I tested current master (v2020.07-rc5 + imx fixes) on tbs2910 (imx6q + >> atheros PHY). >> Unfortunately ethernet is broken now, while it

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Soeren Moch
On 20.06.20 04:14, Fabio Estevam wrote: > Hi Tom, > > On Thu, Jun 18, 2020 at 10:39 AM Tom Rini wrote: > >> It's a good question what else doesn't work. What has been actively >> verified at this point? > > After the fixes I sent yesterday, the few imx boards I have access at > the moment have

Re: [PATCH v1 00/15] add basic driver support for broadcom NS3 soc

2020-06-05 Thread Soeren Moch
On 04.06.20 05:16, Tom Rini wrote: > On Wed, Jun 03, 2020 at 08:59:58PM -0600, Simon Glass wrote: >> Hi Rayagonda, >> >> On Wed, 3 Jun 2020 at 03:10, Rayagonda Kokatanur >> wrote: >>> >>> Hi Simon, >>> >>> On Wed, May 20, 2020 at 7:50 PM Simon Glass wrote: Hi Rayagonda, On

Re: [PATCHv5][ 7/7] board: tbs2910: add documentation

2020-06-04 Thread Soeren Moch
On 04.06.20 06:39, Denis 'GNUtoo' Carikli wrote: > On Sat, 30 May 2020 19:18:33 +0200 > Soeren Moch wrote: > >>> +Links: >>> +-- >>> + - >>> https://www.tbsdtv.com/download/document/tbs2910/TBS2910-Matrix-ARM-mini-PC-SCH_rev2.1.pdf &

Re: [PATCHv5][ 4/7] board: tbs2910_defconfig: Add support for booting general purpose distributions

2020-06-04 Thread Soeren Moch
On 04.06.20 06:13, Denis 'GNUtoo' Carikli wrote: > On Sat, 30 May 2020 19:17:49 +0200 > Soeren Moch wrote: > > Since I need to fix the MAINTAINER file to add the documentation path > in it anyway, it's probably better to resend again a v6 serie. > > As I plan to resend

Re: [PATCHv5][ 4/7] board: tbs2910_defconfig: Add support for booting general purpose distributions

2020-05-30 Thread Soeren Moch
li It is quite uncommon to use "board: tbs2910_defconfig" instead of "board: tbs2910" in the patch title.But maybe not that important. Reviewed-by: Soeren Moch Thanks, Soeren > --- > Changelog since v4: > --- > - As requested, I Added in

Re: [PATCHv5][ 6/7] board: tbs2910: Enable distro_boot support.

2020-05-30 Thread Soeren Moch
bs2910.dtb instead. > > With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola > GNU/Linux distribution, we have the following size > differences: > - text: +2041 bytes > - data: 0 bytes > - bss: 0 bytes > - total: +2041 bytes > > Signed-off-by: Denis 'GNUtoo' Ca

Re: [PATCHv5][ 7/7] board: tbs2910: add documentation

2020-05-30 Thread Soeren Moch
On 30.05.20 05:24, Denis 'GNUtoo' Carikli wrote: > This documents the u-boot installation procedure and the > hardware in order to get started. > > Signed-off-by: Denis 'GNUtoo' Carikli > --- > Changelog since v4: > --- > - Added dded information about SYSBOOT boot support as

Re: [PATCHv5][ 5/7] board: tbs2910: remove CONFIG_DHCP

2020-05-30 Thread Soeren Moch
On 30.05.20 05:24, Denis 'GNUtoo' Carikli wrote: > As that the resulting image is already very close to the > size limit, and that CONFIG_DHCP is not strictly required, > removing it shouldn't hurt. You removed CONFIG_CMD_DHCP.  This is used to check the ethernet connection of the board. And since

Re: [PATCH] tbs2910: migrate to DM_VIDEO

2020-05-30 Thread Soeren Moch
On 28.05.20 10:07, Anatolij Gustschin wrote: > On Thu, 28 May 2020 09:54:42 +0200 > Soeren Moch sm...@web.de wrote: > ... >>> I'll merge this via u-boot-video tree. v2 patch depends on a few more >>> video patches which are under review/built-test currently. >>

Re: [PATCH] tbs2910: migrate to DM_VIDEO

2020-05-28 Thread Soeren Moch
On 26.05.20 22:13, Anatolij Gustschin wrote: > On Sun, 24 May 2020 17:46:22 +0200 > Soeren Moch sm...@web.de wrote: > >> On 23.05.20 01:24, Anatolij Gustschin wrote: >>> Migration to DM_VIDEO driver is long overdue, configure it in >>> board config files. To e

Re: [PATCH][v4 5/6] board: tbs2910: Enable distro_boot support.

2020-05-27 Thread Soeren Moch
On 27.05.20 02:39, Denis 'GNUtoo' Carikli wrote: > On Sun, 1 Mar 2020 14:59:04 +0100 > Soeren Moch wrote: > >> On 27.02.20 01:37, Denis 'GNUtoo' Carikli wrote: >>> This keeps the compatibility with the old bootcmd. > [...] >>> --- a/include/configs/tbs2910.h

Re: [RFC PATCH] imx: skip unused compatible strings in drivers

2020-05-26 Thread Soeren Moch
On 26.05.20 13:25, Marek Vasut wrote: > On 5/26/20 1:12 PM, Soeren Moch wrote: >> On 25.05.20 22:24, Anatolij Gustschin wrote: >>> Converting to DM increases binary size and breaks building some >>> boards (i.e. tbs2910, gcc 9.2). The approach to address this issue

Re: [RFC PATCH] imx: skip unused compatible strings in drivers

2020-05-26 Thread Soeren Moch
On 25.05.20 22:24, Anatolij Gustschin wrote: > Converting to DM increases binary size and breaks building some > boards (i.e. tbs2910, gcc 9.2). The approach to address this issue > via cutting off unused properties/nodes in device tree via custom > u-boot.dtsi was not welcome, even if the

Re: [PATCH] tbs2910: migrate to DM_VIDEO

2020-05-24 Thread Soeren Moch
On 23.05.20 01:24, Anatolij Gustschin wrote: > Migration to DM_VIDEO driver is long overdue, configure it in > board config files. To enable the display set stdout like: > > setenv stdout serial,vidconsole > > Signed-off-by: Anatolij Gustschin > --- > This is only build tested. I tested this on

Re: [PATCH] video: extend stdout video console work-around for 'vga'

2020-05-24 Thread Soeren Moch
stschin Tested on tbs2910, works as intended. Tested-by: Soeren Moch Thanks, Soeren > --- > arch/arm/mach-tegra/Kconfig | 1 - > common/console.c| 7 --- > drivers/video/Kconfig | 16 +--- > 3 files changed, 13 insertions(+), 11 deletions(-)

Re: [PATCH] tbs2910: migrate to DM_VIDEO

2020-05-23 Thread Soeren Moch
On 23.05.20 17:17, Anatolij Gustschin wrote: > On Sat, 23 May 2020 09:55:48 +0200 > Soeren Moch sm...@web.de wrote: > >> On 23.05.20 01:24, Anatolij Gustschin wrote: >>> Migration to DM_VIDEO driver is long overdue, configure it in >>> board config files. To e

Re: [PATCH] tbs2910: migrate to DM_VIDEO

2020-05-23 Thread Soeren Moch
On 23.05.20 01:24, Anatolij Gustschin wrote: > Migration to DM_VIDEO driver is long overdue, configure it in > board config files. To enable the display set stdout like: > > setenv stdout serial,vidconsole And this exactly is the problem. There are some users of this board without serial

Re: [PATCH][v4 6/6] board: tbs2910: add documentation

2020-05-12 Thread Soeren Moch
On 09.05.20 19:59, Denis 'GNUtoo' Carikli wrote: > On Sun, 1 Mar 2020 14:59:23 +0100 > Soeren Moch wrote: > >> To ease the review process it is good style to add a patch revision >> history below the sign-off after a "---" separator. This would look >> lik

Re: [PATCH][v4 6/6] board: tbs2910: add documentation

2020-05-10 Thread Soeren Moch
On 09.05.20 19:36, Denis 'GNUtoo' Carikli wrote: > On Thu, 7 May 2020 20:12:19 +0200 > Soeren Moch wrote: >> do you plan to re-send a fixed version of this series? > I definitely do. OK. I look forward to your new series. > > I'm really sorry for the delay. No problem, it

Re: [PATCH][v4 6/6] board: tbs2910: add documentation

2020-05-07 Thread Soeren Moch
Denis, do you plan to re-send a fixed version of this series? Thanks, Soeren On 01.03.20 14:59, Soeren Moch wrote: > On 27.02.20 01:37, Denis 'GNUtoo' Carikli wrote: >> This documents the u-boot installation procedure and the >> hardware in order to get started. >

Re: Fixing low-speed USB keyboard detection

2020-03-01 Thread Soeren Moch
atches to 2 different maintainers, and the u-boot mailing list, of course. And please provide your full name for sign-off. Regards, Soeren > > > Stefan > > >> Am 29.02.2020 um 13:04 schrieb Soeren Moch : >> >> >> >> On 29.02.20 00:46, Marek Vasut wrote: >>

Re: [PATCH][v4 5/6] board: tbs2910: Enable distro_boot support.

2020-03-01 Thread Soeren Moch
On 27.02.20 01:37, Denis 'GNUtoo' Carikli wrote: > This keeps the compatibility with the old bootcmd. > > The fdtfile environment variable also needed to be set to > imx6q-tbs2910.dtb to enable booting mainline kernels > otherwise with extlinux.conf it tries to load > mx6-tbs2910.dtb instead. > >

Re: [PATCH][v4 6/6] board: tbs2910: add documentation

2020-03-01 Thread Soeren Moch
On 27.02.20 01:37, Denis 'GNUtoo' Carikli wrote: > This documents the u-boot installation procedure and the > hardware in order to get started. Thanks for adding this documentation! Since you added SYSBOOT support in this series, it also might be good to mention this (and maybe how to use it) in

Re: [PATCH][v4 4/6] board: tbs2910: enable CONFIG_DISTRO_DEFAULTS

2020-03-01 Thread Soeren Moch
On 27.02.20 01:37, Denis 'GNUtoo' Carikli wrote: > The side effect is that it increase the size of the > resultimg image, which is already very close to the > size limit. Please also mention the main purpose, preparation for distro_boot support, not only the side effect of this patch. > > With

Re: [PATCH][v4 1/6] board: tbs2910: disable loadb and loads commands

2020-03-01 Thread Soeren Moch
already very close to the > size limit, removing the loadb and loads commands shouldn't > hurt. > > With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola > GNU/Linux distribution we have the following size > reduction: > - text: 6733 bytes > - data: 116 bytes > - bss: 1172 bytes &g

Re: [PATCH][v4 2/6] board: tbs2910: disable CONFIG_GZIP

2020-03-01 Thread Soeren Moch
stribution we have the following size > reduction: > - text: 9752 > - data: 0 > - bss: 16 > - total: 9768 > > Signed-off-by: Denis 'GNUtoo' Carikli Acked-by: Soeren Moch Thanks, Soeren

Re: [PATCH][v4 3/6] board: tbs2910: move CONFIG_BOOTCOMMAND from header to defconfig

2020-03-01 Thread Soeren Moch
On 27.02.20 01:36, Denis 'GNUtoo' Carikli wrote: > This doesn't affect the size of the image: with > arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola GNU/Linux > distribution, the text, data, bss and total sizes remain > unchanged. > > Signed-off-by: Denis 'GNUtoo' Carikli Review

Re: Fixing low-speed USB keyboard detection

2020-02-29 Thread Soeren Moch
On 29.02.20 00:46, Marek Vasut wrote: > On 2/26/20 12:04 PM, Soeren Moch wrote: >> Adding Marek as USB maintainer. Otherwise this non-patch-email may get >> lost when sent to the mailing list only. > Well, can you send these as proper patches ? Well, I can try to make a pr

Re: Fixing low-speed USB keyboard detection

2020-02-26 Thread Soeren Moch
Adding Marek as USB maintainer. Otherwise this non-patch-email may get lost when sent to the mailing list only. Soeren On 25.02.20 18:45, Stefan wrote: > Hello! > > I own a D-Link DBT-120 Bluetooth Adapter, which has a CSR firmware running in > a so called “HID proxy mode”. This firmware

Re: [PATCH v3][ 6/6] boards: tbs2910: add README

2020-02-08 Thread Soeren Moch
On 08.02.20 06:38, Denis 'GNUtoo' Carikli wrote: > The installation documentation make use of the > "BOOT/UPDATE" mechanism that is documented in the TBS2910 > OS installation manual[1]. > > This is to make sure that they work on the different PCB > revisions of this board. > > There is also a

Re: [PATCH v3][ 4/6] board: tbs2910: enable CONFIG_DISTRO_DEFAULTS

2020-02-08 Thread Soeren Moch
On 08.02.20 06:38, Denis 'GNUtoo' Carikli wrote: > The side effect is that it increase the size of the > resultimg image, which is already very close to the > size limit. > > With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola > GNU/Linux distribution, we have the following size > increase: > -

Re: [PATCH] distro_bootcmd: Add support for loading user environment

2020-01-31 Thread Soeren Moch
On 31.01.20 19:18, Stephen Warren wrote: > On 1/31/20 7:54 AM, Tom Rini wrote: >> On Fri, Jan 31, 2020 at 02:42:25PM +, Jon Hunter wrote: >>> >>> On 31/01/2020 14:05, Tom Rini wrote: >>>> On Fri, Jan 31, 2020 at 01:24:21PM +, Jon Hunter wrote:

Re: [PATCH] distro_bootcmd: Add support for loading user environment

2020-01-31 Thread Soeren Moch
On 31.01.20 11:59, Jon Hunter wrote: > U-Boot supports loading a user environment from a file in the > file-system. Therefore to make it easier for users to override the > default environment, add support to the 'distro_bootcmd' to look for > and load a user environment in a file called 'uEnv.txt'

Re: [PATCH v2][ 3/3] board: tbs2910: Add support for generic distro configuration

2020-01-30 Thread Soeren Moch
On 28.01.20 22:27, Denis 'GNUtoo' Carikli wrote: > On Tue, 28 Jan 2020 18:51:26 +0100 > Soeren Moch wrote: > >> As already discussed, the bootm_size environment variable is not >> necessary, otherwise  somewhat dangerous with this value. > Sorry, for the mistake, I've fix

Re: [PATCH v2][ 3/3] board: tbs2910: Add support for generic distro configuration

2020-01-29 Thread Soeren Moch
On 28.01.20 22:27, Denis 'GNUtoo' Carikli wrote: > On Tue, 28 Jan 2020 18:51:26 +0100 > Soeren Moch wrote: > >> As already discussed, the bootm_size environment variable is not >> necessary, otherwise  somewhat dangerous with this value. > Sorry, for the mistake, I've fix

Re: [PATCH v2][ 3/3] board: tbs2910: Add support for generic distro configuration

2020-01-28 Thread Soeren Moch
On 28.01.20 18:04, Denis 'GNUtoo' Carikli wrote: > This keeps the compatibility with the old bootcmd. > > The fdtfile environment variable also needed to be set to > imx6q-tbs2910.dtb to enable booting mainline kernels > otherwise with extlinux.conf it tries to load > mx6-tbs2910.dtb instead. >

Re: [PATCH v2][ 2/3] tbs2910: disable loadb and loads commands

2020-01-28 Thread Soeren Moch
NUtoo' Carikli I don't know any use case of these commands on tbs2910, so Acked-by: Soeren Moch

Re: [PATCH] board: tbs2910: Add support for generic distro configuration

2020-01-28 Thread Soeren Moch
On 28.01.20 18:02, Denis 'GNUtoo' Carikli wrote: > On Sun, 26 Jan 2020 01:40:13 +0100 > Soeren Moch wrote: >>> Ahh ok, now I understand. That probably explains the repeated size >>> issues. >> Why? With SPL+u-boot proper it would be even worse, since then there

Re: [PATCH v2][ 1/3] tbs2910: disable fuse command

2020-01-28 Thread Soeren Moch
Sorry, sent with wrong sender address. Please only use this address here. Soeren On 28.01.20 18:13, Soeren Moch wrote: > On 28.01.20 18:07, Fabio Estevam wrote: >> On Tue, Jan 28, 2020 at 2:04 PM Denis 'GNUtoo' Carikli >> wrote: >>> The fuse command is not needed for

Re: [PATCH] board: tbs2910: Add support for generic distro configuration

2020-01-25 Thread Soeren Moch
On 26.01.20 00:15, Denis 'GNUtoo' Carikli wrote: > On Sat, 25 Jan 2020 20:52:36 +0100 > Soeren Moch wrote: > >> Hi Denis, > Hi, > >> thanks for your patch. In general I think it could be a good idea to >> support distroboot on this board, especially if

Re: [PATCH] board: tbs2910: Add support for generic distro configuration

2020-01-25 Thread Soeren Moch
On 25.01.20 17:24, Tom Rini wrote: > On Sat, Jan 25, 2020 at 02:42:40AM +0100, Denis 'GNUtoo' Carikli wrote: > >> This keeps the compatibility with the old bootcmd. >> >> Signed-off-by: Denis 'GNUtoo' Carikli > First, I have concerns about this increasing the size of the board. > >> --- >>

Re: [PATCH] board: tbs2910: Add support for generic distro configuration

2020-01-25 Thread Soeren Moch
Hi Denis, thanks for your patch. In general I think it could be a good idea to support distroboot on this board, especially if we can maintain compatibility with the existing boot procedure. However, since this board repeatedly has size problems with the u-boot binary, we carefully need to check

Re: [PATCH v2 2/2] tbs2910: add custom CONFIG_OF_REMOVE_PROPS list to defconfig

2020-01-12 Thread Soeren Moch
On 12.01.20 15:57, Anatolij Gustschin wrote: > This shrinks the image size: all -3840.0 text -3840.0 > > Signed-off-by: Anatolij Gustschin Acked-by: Soeren Moch > --- > Changes in v2: > - add CONFIG_OF_DTB_PROPS_REMOVE=y > - update commit description > - gitla

Re: [PATCH 2/2] tbs2910: add custom CONFIG_OF_REMOVE_PROPS list to defconfig

2020-01-09 Thread Soeren Moch
On 08.01.20 22:38, Anatolij Gustschin wrote: > This shrinks the image size: all -1536.0 bss +16.0 text -1552.0 > > Signed-off-by: Anatolij Gustschin Acked-by: Soeren Moch Thanks, Soeren

Re: [PATCH] imx: dts: tbs2910: add u-boot dtsi to shrink image size

2020-01-09 Thread Soeren Moch
On 08.01.20 22:42, Anatolij Gustschin wrote: > On Fri, 6 Dec 2019 13:43:05 -0500 > Tom Rini tr...@konsulko.com wrote: > ... >> This gets an important conversation started. Perhaps we should expand >> OF_SPL_REMOVE_PROPS for full U-Boot, with some gating symbol for the >> platforms that do want to

Re: [PATCH 07/11] env: Enable SPI flash env for rockchip

2019-12-30 Thread Soeren Moch
On 28.12.19 13:08, Jagan Teki wrote: > On Fri, Dec 27, 2019 at 6:49 PM Soeren Moch wrote: >> On 27.12.19 13:04, Jagan Teki wrote: >>> On Fri, Dec 27, 2019 at 4:00 PM Soeren Moch wrote: >>>> Hi! >>>> >>>> On 27.12.19 07:50, Jagan Teki wrote:

Re: [PATCH 07/11] env: Enable SPI flash env for rockchip

2019-12-27 Thread Soeren Moch
On 27.12.19 13:04, Jagan Teki wrote: > On Fri, Dec 27, 2019 at 4:00 PM Soeren Moch wrote: >> Hi! >> >> On 27.12.19 07:50, Jagan Teki wrote: >>> Hi Kever, >>> >>> On Mon, Dec 23, 2019 at 8:04 AM Kever Yang >>> wrote: >>

Re: [PATCH 07/11] env: Enable SPI flash env for rockchip

2019-12-27 Thread Soeren Moch
Hi! On 27.12.19 07:50, Jagan Teki wrote: > Hi Kever, > > On Mon, Dec 23, 2019 at 8:04 AM Kever Yang wrote: >> >> Jagan, >> >> >> On 2019/12/21 下午3:54, Jagan Teki wrote: >>> Most of the SPI flash devices in rockchip are 16MiB size. >>> >>> So, keeping U-Boot proper offset start from 128MiB with

Re: [PATCH] imx: dts: tbs2910: add u-boot dtsi to shrink image size

2019-12-06 Thread Soeren Moch
On 06.12.19 19:43, Tom Rini wrote: > On Fri, Dec 06, 2019 at 07:30:47PM +0100, Anatolij Gustschin wrote: > >> Use U-Boot specific board dtsi to remove some not used nodes and >> properties. With this change applied considerable size reduction >> has been observed: >> >> $

Re: [U-Boot] Maximum size of u-boot.imx for TBS2910 board

2019-12-06 Thread Soeren Moch
On 06.12.19 17:18, Soeren Moch wrote: > On 06.12.19 15:02, Tom Rini wrote: >> On Fri, Dec 06, 2019 at 01:30:44PM +0100, Anatolij Gustschin wrote: >>> Hi Stefano, >>> >>> On Fri, 6 Dec 2019 12:41:47 +0100 >>> Stefano Babic sba...@denx.de wrote

  1   2   3   >