Adding new DH electronics mailing list and myself as a maintainer.
Signed-off-by: Christoph Niedermaier
---
board/dhelectronics/dh_imx6/MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/board/dhelectronics/dh_imx6/MAINTAINERS
b/board/dhelectronics/dh_imx6/MAINTAINERS
index
Adding new DH electronics mailing list.
Signed-off-by: Christoph Niedermaier
---
board/dhelectronics/dh_stm32mp1/MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/board/dhelectronics/dh_stm32mp1/MAINTAINERS
b/board/dhelectronics/dh_stm32mp1/MAINTAINERS
index fd70131f9e
Adding new DH electronics mailing list and update list of maintainers.
Signed-off-by: Christoph Niedermaier
---
V2: Correct message tag and update list of maintainers
---
board/dhelectronics/dh_imx6/MAINTAINERS | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/board
Adding new DH electronics mailing list.
Signed-off-by: Christoph Niedermaier
---
V2: Correct message tag
---
board/dhelectronics/dh_stm32mp1/MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/board/dhelectronics/dh_stm32mp1/MAINTAINERS
b/board/dhelectronics/dh_stm32mp1/MAINTAINERS
bin
> and the necessary DTs. Therefore, use u-boot.img instead of u-boot.uim
> to generate u-boot-with-spl.imx when OF_SEPARATE is enabled.
>
> Tested-by: Fabio Estevam
> Signed-off-by: Marek Vasut
> Cc: Christoph Niedermaier
> Cc: Fabio Estevam
> Cc: Peng Fan
> Cc: Simo
From: Marek Vasut [mailto:ma...@denx.de]
Sent: Thursday, February 4, 2021 7:00 PM
> To: u-boot@lists.denx.de
> Cc: Marek Vasut ; Christoph Niedermaier electronics.com>; Fabio Estevam ; Peng Fan
> ; Simon Glass ; Stefano Babic
> ; Ye Li ; uboot-imx
> Subject: [PATCH V3] A
From: Stefano Babic
Sent: Tuesday 5th November 2019 17:26
>On 05/11/19 17:05, Anatolij Gustschin wrote:
>> Hi Stefano, Christoph,
>>
>> On Tue, 5 Nov 2019 16:57:18 +0100
>> Stefano Babic sba...@denx.de wrote:
>> ...
-static inline u8 imx6_is_bmode_from_gpr9(void)
+static inline u32 im
A return type of u08 cuts the return value always to 0,
because IMX6_SRC_GPR10_BMODE = bit 28 is above u08.
Changing the return type to u32 gives
the function the expected operation.
Fixes: cba586b400 ("imx6: Add imx6_src_get_boot_mode")
Signed-off-by: Christoph Niedermaier
---
arch/a
: Christoph Niedermaier
Cc: Stefano Babic
Cc: Simon Glass
Cc: Marek Behún
To: u-boot@lists.denx.de
---
Makefile | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index 4b152249ca..67ed4edc1a 100644
--- a/Makefile
+++ b/Makefile
@@ -2439,7 +2439,8 @@ endif
After sorting, unnoticed blank lines at the end move to the top.
Avoid this by removing it before sorting.
Signed-off-by: Christoph Niedermaier
Cc: Lukasz Majewski
Cc: Rasmus Villemoes
Cc: Tom Rini
To: u-boot@lists.denx.de
---
scripts/get_default_envs.sh | 4 ++--
1 file changed, 2
From: Simon Glass [mailto:s...@chromium.org]
Sent: Wednesday, February 23, 2022 11:59 PM
>
> Hi Christoph,
>
Hi Simon,
> On Wed, 23 Feb 2022 at 02:33, Christoph Niedermaier
> wrote:
>>
>> For a better overview and better comparison sort the initial
>> envi
erves the order of equal
variable names by sorting only the variable name, and
disable the last-resort comparison. After sorting,
unnoticed blank lines at the end move to the top. Avoid
that by removing it before sorting.
[1] https://github.com/sbabic/libubootenv
Signed-off-by: Christoph Niedermai
From: Simon Glass [mailto:s...@chromium.org]
Sent: Friday, February 25, 2022 12:54 AM
>
> Hi Christoph,
>
Hi Simon,
> On Thu, 24 Feb 2022 at 02:59, Christoph Niedermaier
> wrote:
>>
>> From: Simon Glass [mailto:s...@chromium.org]
>> Sent: Wednesday, Febr
erves the order of equal
variable names by sorting only the variable name, and
disable the last-resort comparison. After sorting,
unnoticed blank lines at the end move to the top. Avoid
that by removing it before sorting.
[1] https://github.com/sbabic/libubootenv
Signed-off-by: Christoph Niedermai
. If
so, use the MAC address. There may be multiple EEPROMs with an ID page
on this platform, always use the first one.
Signed-off-by: Christoph Niedermaier
---
Cc: "NXP i.MX U-Boot Team"
Cc: Marek Vasut
Cc: Fabio Estevam
Cc: Stefano Babic
Cc: Tom Rini
Cc: u-b...@dh-electronic
item and serial number to the U-Boot
environment if the aren't there. If the environment is already
there compare it with the one from the EEPROM ID page and output
a warning if it differs.
Signed-off-by: Christoph Niedermaier
---
Cc: "NXP i.MX U-Boot Team"
Cc: Marek Vasut
Cc: Fa
From: Marek Vasut
Sent: Saturday, October 12, 2024 10:56 PM
> On 10/10/24 3:23 PM, Christoph Niedermaier wrote:
>> The new i.MX8M Plus DHCOM rev.200 is populated with M24C32-D EEPROM
>> which contains additional write-lockable page called ID page, which
>> is populated with a
From: Marek Vasut
Sent: Saturday, October 12, 2024 10:43 PM
> On 10/10/24 3:23 PM, Christoph Niedermaier wrote:
>> The i.MX8M Plus DHCOM currently supports parsing ethernet MAC address
>> from multiple sources in the following priority order:
>>
>> 1) U-Boot envi
From: Marek Vasut
Sent: Thursday, October 17, 2024 4:01 PM
> On 10/17/24 1:09 PM, Christoph Niedermaier wrote:
>> From: Marek Vasut
>> Sent: Wednesday, October 16, 2024 2:16 PM
>>> On 10/16/24 1:57 PM, Christoph Niedermaier wrote:
>>>> From: Marek Vasut
>
From: Marek Vasut
Sent: Thursday, October 17, 2024 8:35 PM
> On 10/17/24 1:55 PM, Christoph Niedermaier wrote:
>
> [...]
>
>>>>>> +__func__, ret);
>>>>>
>>>>> This will be printed on every device, even th
From: Marek Vasut
Sent: Thursday, October 17, 2024 2:22 AM
> On 10/16/24 2:31 PM, Christoph Niedermaier wrote:
>
> [...]
>
>>>> diff --git a/board/dhelectronics/common/dh_common.h
>>>> b/board/dhelectronics/common/dh_common.h
>>>> inde
From: Marek Vasut
Sent: Wednesday, October 16, 2024 2:16 PM
> On 10/16/24 1:57 PM, Christoph Niedermaier wrote:
>> From: Marek Vasut
>> Sent: Saturday, October 12, 2024 10:43 PM
>>> On 10/10/24 3:23 PM, Christoph Niedermaier wrote:
>>>> The i.MX8M Plus DHCOM
From: Marek Vasut
Sent: Tuesday, October 22, 2024 1:57 PM
> On 10/22/24 11:31 AM, Christoph Niedermaier wrote:
>> From: Marek Vasut
>> Sent: Tuesday, October 22, 2024 1:01 AM
>>> On 10/21/24 5:38 PM, Christoph Niedermaier wrote:
>>>
>>> [...]
>&
From: Marek Vasut
Sent: Tuesday, October 22, 2024 1:01 AM
> On 10/21/24 5:38 PM, Christoph Niedermaier wrote:
>
> [...]
>
>>> If yes, then there is no need for any static variable:
>>>
>>> board_init() {
>>>u8 eeprom[32];
>>&g
From: Marek Vasut
Sent: Wednesday, October 23, 2024 4:05 PM
> On 10/23/24 3:20 PM, Christoph Niedermaier wrote:
>> From: Marek Vasut
>> Sent: Wednesday, October 23, 2024 2:41 PM
>>> On 10/23/24 2:18 PM, Christoph Niedermaier wrote:
>>>
>>> [...]
>>
From: Marek Vasut
Sent: Wednesday, October 23, 2024 2:41 PM
> On 10/23/24 2:18 PM, Christoph Niedermaier wrote:
>
> [...]
>
>>> You do not, the following automatically reserves space on stack when
>>> called and frees it up upon function return:
>>>
&
page.
Signed-off-by: Christoph Niedermaier
---
Cc: "NXP i.MX U-Boot Team"
Cc: Marek Vasut
Cc: Fabio Estevam
Cc: Stefano Babic
Cc: Tom Rini
Cc: u-b...@dh-electronics.com
---
V2: - Merging the two previous patches
"arm64: imx8mp: Read MAC address from M24C32-D write-lockable pag
The new i.MX8M Plus DHCOM rev.200 is populated with M24C32-D EEPROM
that contains an additional write-lockable page called ID page. Add
aliases eeprom0wl and eeprom1wl for the access to the EEPROM ID
page node.
Signed-off-by: Christoph Niedermaier
---
Cc: "NXP i.MX U-Boot Team"
Cc: M
/'eth1addr' environment variable
2) SoC OTP fuses
3) On-SoM EEPROM
Add support for parsing the content of this new EEPROM ID page and place
it between 2) and 3) on the priority list. The new entry is 2.5) On-SoM
EEPROM write-lockable page.
Signed-off-by: Christoph Niedermaier
---
Cc: "
patch
series depends on patch [1].
[1]
https://patchwork.ozlabs.org/project/uboot/patch/20241120160135.28262-1-cniederma...@dh-electronics.com/
Christoph Niedermaier (4):
arm64: dts: imx8mp: Add aliases for the access to the EEPROM ID page
node
arm64: imx8mp: Read values from M24C32-D
("bar", 0); ... }
static int on_bar(...) { env_set("foo", 0); ... }
U_BOOT_ENV_CALLBACK(foo, on_foo);
U_BOOT_ENV_CALLBACK(bar, on_bar);
Signed-off-by: Christoph Niedermaier
Suggested-by: Marek Vasut
---
Cc: Tom Rini
Cc: Simon Glass
Cc: Marek Vasut
Cc: u-b...@dh-electroni
The env variable "SN" is used to store the serial number on DH electronics
SoMs. New SoMs will use the variable "dh_som_serial_number". To ensure
compatibility, these env variables are synchronized. This is achieved
using callback functions.
Signed-off-by: Christoph Niedermaie
From: Marek Vasut
Sent: Sunday, November 24, 2024 11:11 PM
> On 11/24/24 10:10 PM, Marek Vasut wrote:
>> On 11/21/24 6:21 PM, Christoph Niedermaier wrote:
>>> The env variable "SN" is used to store the serial number on DH
>>> electronics
>
Switch the callback static list from the board configuration varibale
CFG_ENV_CALLBACK_LIST_STATIC to Kconfig CONFIG_ENV_CALLBACK_LIST_STATIC.
Signed-off-by: Christoph Niedermaier
---
Cc: Joe Hershberger
Cc: Marek Vasut
Cc: Tom Rini
Cc: u-b...@dh-electronics.com
---
doc/develop
Switch the callback static list from the board configuration variable
CFG_ENV_CALLBACK_LIST_STATIC to Kconfig CONFIG_ENV_CALLBACK_LIST_STATIC.
Signed-off-by: Christoph Niedermaier
Reviewed-by: Marek Vasut
---
Cc: Joe Hershberger
Cc: Marek Vasut
Cc: Tom Rini
Cc: u-b...@dh-electronics.com
From: Marek Vasut
Sent: Tuesday, November 19, 2024 3:22 PM
> On 11/19/24 11:57 AM, Christoph Niedermaier wrote:
>> Switch the callback static list from the board configuration varibale
>
> variable , typo
>
>> CFG_ENV_CALLBACK_LIST_STATIC to Kconfig CONFIG_ENV_CALLBACK_L
From: Marek Vasut
Sent: Sunday, November 24, 2024 10:02 PM
> On 11/21/24 6:21 PM, Christoph Niedermaier wrote:
>
> [...]
>
>> +struct eeprom_id_page {
>> +u8 id[3]; /* Identifier 'D', 'H', 'E' - 'D' is at index
From: Marek Vasut
Sent: Sunday, December 1, 2024 7:44 PM
> On 11/28/24 3:20 PM, Christoph Niedermaier wrote:
>
> [...]
>
>> +++ b/board/dhelectronics/common/dh_common.c
>> @@ -7,6 +7,7 @@
>> #include
>> #include
>> #include
>> +#includ
/'eth1addr' environment variable
2) SoC OTP fuses
3) On-SoM EEPROM
Add support for parsing the content of this new EEPROM ID page and place
it between 2) and 3) on the priority list. The new entry is 2.5) On-SoM
EEPROM write-lockable page.
Signed-off-by: Christoph Niedermaier
---
Cc: "
patch
series depends on patch [1].
[1]
https://patchwork.ozlabs.org/project/uboot/patch/20241120160135.28262-1-cniederma...@dh-electronics.com/
Christoph Niedermaier (4):
arm64: dts: imx8mp: Add aliases for the access to the EEPROM ID page
node
arm64: imx8mp: Read values from M24C32-D
The env variable "SN" is used to store the serial number on DH electronics
SoMs. New SoMs will use the variable "dh_som_serial_number". To ensure
compatibility, these env variables are synchronized. This is achieved
using callback functions.
Signed-off-by: Christoph Niedermaie
The new i.MX8M Plus DHCOM rev.200 is populated with M24C32-D EEPROM
that contains an additional write-lockable page called ID page. Add
aliases eeprom0wl and eeprom1wl for the access to the EEPROM ID
page node.
Signed-off-by: Christoph Niedermaier
Reviewed-by: Marek Vasut
---
Cc: "NXP i
From: Marek Vasut
Sent: Sunday, December 1, 2024 7:21 PM
> On 11/28/24 3:20 PM, Christoph Niedermaier wrote:
>> The env variable "SN" is used to store the serial number on DH electronics
>> SoMs. New SoMs will use the variable "dh_som_serial_number". To
From: Marek Vasut
Sent: Friday, November 8, 2024 8:27 PM
> On 11/8/24 6:06 PM, Christoph Niedermaier wrote:
>
> [...]
>
>> +int dh_read_eeprom_id_page(u8 *eeprom_buffer, const char *alias)
>> +{
>> + u8 buffer[DH_EEPROM_ID_PAGE_MAX_SIZE] = { 0 };
>&g
("bar", 0); ... }
static int on_bar(...) { env_set("foo", 0); ... }
U_BOOT_ENV_CALLBACK(foo, on_foo);
U_BOOT_ENV_CALLBACK(bar, on_bar);
Signed-off-by: Christoph Niedermaier
Suggested-by: Marek Vasut
---
Cc: Tom Rini
Cc: Simon Glass
Cc: Marek Vasut
Cc: u-b...@dh-electroni
From: Marek Vasut
Sent: Thursday, November 21, 2024 6:37 PM
> On 11/21/24 6:21 PM, Christoph Niedermaier wrote:
>> The new i.MX8M Plus DHCOM rev.200 is populated with M24C32-D EEPROM
>> that contains an additional write-lockable page called ID page. Add
>> aliases eeprom0wl
The new i.MX8M Plus DHCOM rev.200 is populated with M24C32-D EEPROM
that contains an additional write-lockable page called ID page. Add
aliases eeprom0wl and eeprom1wl for the access to the EEMPROM ID
page node.
Signed-off-by: Christoph Niedermaier
---
Cc: "NXP i.MX U-Boot Team"
Cc: M
/'eth1addr' environment variable
2) SoC OTP fuses
3) On-SoM EEPROM
Add support for parsing the content of this new EEPROM ID page and place
it between 2) and 3) on the priority list. The new entry is 2.5) On-SoM
EEPROM write-lockable page.
Signed-off-by: Christoph Niedermaier
---
Cc: "
locked
against each other.
Signed-off-by: Christoph Niedermaier
---
Cc: "NXP i.MX U-Boot Team"
Cc: Marek Vasut
Cc: Fabio Estevam
Cc: Stefano Babic
Cc: Tom Rini
Cc: u-b...@dh-electronics.com
---
V3: - Add this patch to the series
---
board/dhelectronics/common/dh_common.c | 26 +
/'eth1addr' environment variable
2) SoC OTP fuses
3) On-SoM EEPROM
Add support for parsing the content of this new EEPROM ID page and place
it between 2) and 3) on the priority list. The new entry is 2.5) On-SoM
EEPROM write-lockable page.
Signed-off-by: Christoph Niedermaier
Reviewed-by: Marek
The env variable "SN" is used to store the serial number on DH electronics
SoMs. New SoMs will use the variable "dh_som_serial_number". To ensure
compatibility, these env variables are synchronized. This is achieved
using callback functions.
Signed-off-by: Christoph Nied
patch
series depends on patch [1].
[1]
https://patchwork.ozlabs.org/project/uboot/patch/20241120160135.28262-1-cniederma...@dh-electronics.com/
Christoph Niedermaier (4):
arm64: dts: imx8mp: Add aliases for the access to the EEPROM ID page
node
arm64: imx8mp: Read values from M24C32-D
("bar", 0); ... }
static int on_bar(...) { env_set("foo", 0); ... }
U_BOOT_ENV_CALLBACK(foo, on_foo);
U_BOOT_ENV_CALLBACK(bar, on_bar);
Signed-off-by: Christoph Niedermaier
Suggested-by: Marek Vasut
---
Cc: Tom Rini
Cc: Simon Glass
Cc: Marek Vasut
Cc: u-b...@dh-electroni
The new i.MX8M Plus DHCOM rev.200 is populated with M24C32-D EEPROM
that contains an additional write-lockable page called ID page. Add
aliases eeprom0wl and eeprom1wl for the access to the EEPROM ID
page node.
Signed-off-by: Christoph Niedermaier
Reviewed-by: Marek Vasut
---
Cc: "NXP i
From: Tom Rini
Sent: Thursday, March 20, 2025 3:19 PM
> On Thu, Mar 20, 2025 at 03:14:03PM +0100, Marek Vasut wrote:
> > On 3/20/25 3:00 PM, Quentin Schulz wrote:
> > > Hi Marek,
> > >
> > > On 3/20/25 12:49 PM, Marek Vasut wrote:
> > > >
From: Tom Rini
Sent: Thursday, March 20, 2025 7:28 PM
> On Thu, Mar 20, 2025 at 05:58:56PM +0000, Christoph Niedermaier wrote:
> > From: Tom Rini
> > Sent: Thursday, March 20, 2025 3:19 PM
> > > On Thu, Mar 20, 2025 at 03:14:03PM +0100, Marek Vasut wrote:
> > &
with %X.
Signed-off-by: Christoph Niedermaier
---
Cc: Marek Vasut
Cc: Tom Rini
Cc: Benedikt Spranger
Cc: Simon Glass
Cc: John Ogness
Cc: Jerome Forissier
Cc: Ilias Apalodimas
---
lib/tiny-printf.c | 31 +--
1 file changed, 25 insertions(+), 6 deletions(-)
diff
From: Marek Vasut
Sent: Thursday, March 20, 2025 12:56 PM
> On 3/20/25 11:32 AM, Christoph Niedermaier wrote:
>> If "env select" is called without a target parameter the result is
>> "Select Environment on : driver not found".
>
> Why is this not the cor
If "env select" is called without a target parameter the result is
"Select Environment on : driver not found". Improve the output
by showing a list of available evironment targets, where the active
target is marked with an asterisk.
Signed-off-by: Christoph Niedermaier
From: Michael Walle
Sent: Monday, April 7, 2025 7:28 AM
> Hi,
> > > > > I wouldn't mind printing the pointer for %p[mMI], but %pa prints the
> > > > > *content* of the pointer which is really confusing. I.e. in
> > > > > pinctrl-single.c the reg value pairs are printed like
> > > > >
> > > > > d
From: Marek Vasut
Sent: Friday, March 21, 2025 8:23 PM
> On 3/21/25 7:43 PM, Christoph Niedermaier wrote:
>> Add parameter "-l" for printing available environment targets. The
>> active target is marked with an asterisk. This is done by adding
>> the function en
s enabling pointer formatting only
> > > > >> when needed. For SPL_NET and NET_LWIP it is selected by default.
> > > > >> Then it also supports the formatting with %pm, %pM and %pI4.
> > > > >>
> > > > >> Signed-off-by: Chri
From: Michael Walle
Sent: Tuesday, April 1, 2025 10:56 AM
>
> Hi,
>
>> The issue is that disabling TINY_PRINTF may not be possible (size
>> constraints) and some code is compiled for different stages and people
>> typically don't check whether the format used in printf is valid with
>> tiny_prin
4.
Signed-off-by: Christoph Niedermaier
---
Cc: Tom Rini
Cc: Simon Glass
Cc: Michael Walle
Cc: Quentin Schulz
Cc: Marek Vasut
Cc: Benedikt Spranger
Cc: Jerome Forissier
Cc: John Ogness
Cc: Ilias Apalodimas
---
Kconfig| 1 +
common/spl/Kconfig | 1 +
lib/Kconfig
From: Tom Rini
Sent: Tuesday, April 8, 2025 10:58 PM
> To: Christoph Niedermaier
> Cc: u-boot@lists.denx.de; Simon Glass ; Michael Walle
> ; Quentin Schulz ; Marek Vasut
> ; Benedikt Spranger ; Jerome
> Forissier
> ; John Ogness ; Ilias
> Apalodimas
>
> Subje
From: Christoph Niedermaier
Sent: Tuesday, April 29, 2025 6:49 PM
> From: Marek Vasut
> Sent: Tuesday, April 29, 2025 5:28 PM
>> On 4/29/25 4:47 PM, Christoph Niedermaier wrote:
>>> The formatting with %pa / %pap behaves like %x, which results in an
>>> incorrect
From: Francesco Dolcini
Sent: Wednesday, April 16, 2025 11:27 AM
> On Tue, Apr 15, 2025 at 02:13:30PM -0300, Fabio Estevam wrote:
>> On Tue, Apr 15, 2025 at 1:55 PM Marek Vasut wrote:
>>> Do you have af9cdd1ccd2d ("Revert "arm64: dts: imx8mn: Include 32kHz
>>> oscillator clock in SPL DTs"") in pl
From: Marek Vasut
Sent: Tuesday, April 29, 2025 5:28 PM
> On 4/29/25 4:47 PM, Christoph Niedermaier wrote:
>> The formatting with %pa / %pap behaves like %x, which results in an
>> incorrect value being output. To improve this, a new fine-tuning
>> Kconfig SPL_USE_TINY_PRINT
From: Christoph Niedermaier
Sent: Thursday, April 10, 2025 2:17 PM
> From: Michael Walle
> Sent: Thursday, April 10, 2025 12:44 PM
> > On Wed Apr 9, 2025 at 5:22 PM CEST, Tom Rini wrote:
> > > On Wed, Apr 09, 2025 at 02:33:08PM +0200, Michael Walle wrote:
> > >
From: Tom Rini
Sent: Wednesday, May 7, 2025 7:43 PM
> On Wed, Apr 30, 2025 at 12:35:48PM +0200, Christoph Niedermaier wrote:
>
>> The formatting with %pa / %pap behaves like %x, which results in an
>> incorrect value being output. To improve this, a new fine
From: Michael Walle
Sent: Wednesday, April 2, 2025 9:02 AM
The issue is that disabling TINY_PRINTF may not be possible (size
constraints) and some code is compiled for different stages and people
typically don't check whether the format used in printf is valid with
tiny_printf.
From: Andy Shevchenko
Sent: Monday, May 12, 2025 9:38 AM
> On Fri, May 09, 2025 at 09:00:40PM +0200, Christoph Niedermaier wrote:
>> Add the "-l" parameter to the "env select" command to print the available
>> environment targets, convert all argument parsin
NY_PRINTF_POINTER_SUPPORT is enabled or DEBUG is defined
tiny printf supports formatting %p and %pa / %pap.
3) If SPL_NET is enabled the support of pointers is extended
for %pm, %pM and %pI4.
Signed-off-by: Christoph Niedermaier
---
Cc: Tom Rini
Cc: Simon Glass
Cc: Michael Walle
The entire parsing of the arguments is done manually. Improve this
by converting all parsing of arguments to getopt().
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Christian Marangi
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Jerome Forissier
Cc: Joe Hershberger
Cc
Add basic unit tests for argument checking of the env command.
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Christian Marangi
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Jerome Forissier
Cc: Joe Hershberger
Cc: Marek Vasut
Cc: Michal Simek
Cc: Patrick Delaunay
found". Replace this not
so useful output by showing the env usage message instead.
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Christian Marangi
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Jerome Forissier
Cc: Joe Hershberger
Cc: Marek Vasut
Cc: Michal S
second one adds unit tests for checking the env command.
These were used to check the env commands before and after the conversion.
The third patch adds the actual new parameter "-l" for the "env select"
command.
Christoph Niedermaier (3):
cmd: nvedit: Convert the parsing of a
From: Andy Shevchenko
Sent: Monday, May 12, 2025 10:38 AM
> On Mon, May 12, 2025 at 08:22:19AM +0000, Christoph Niedermaier wrote:
>> From: Andy Shevchenko
>> Sent: Monday, May 12, 2025 9:38 AM
>>> On Fri, May 09, 2025 at 09:00:40PM +0200, Christoph Niedermaier wrote:
>
From: U-Boot On Behalf Of Michael Trimarchi
Sent: Monday, May 26, 2025 9:48 PM
Hello,
> Subject: [PATCH v2 1/4] clk: imx: add i.MX6UL clk driver
>
> Add i.MX6UL clk driver for i.MX6UL CLK driver model usage
>
> Reviewed-by: Peng Fan
> Signed-off-by: Michael Trimarchi
>
> ---
>
> Changes in
From: Tom Rini
Sent: Tuesday, June 3, 2025 10:13 PM
> On Fri, May 09, 2025 at 09:00:42PM +0200, Christoph Niedermaier wrote:
>
>> Add basic unit tests for argument checking of the env command.
>>
>> Signed-off-by: Christoph Niedermaier
> [snip]
>> +/* Bin
g", "ipg", base + 0x80, 14));
> + clk_dm(IMX6UL_CLK_UART8_SERIAL,
> + imx_clk_gate2(dev, "uart8_serial", "uart_podf", base + 0x80,
> 14));
Thanks for adding the UARTs. Now my Console is working.
> +
> +#if CONFIG_IS_ENABLED(NAND_MXS)
> + clk_dm(IMX6UL_CLK_PER_BCH,
> + imx_clk_gate2(dev, "per_bch", "bch_podf", base + 0x78, 12));
> + clk_dm(IMX6UL_CLK_GPMI_BCH_APB,
> + imx_clk_gate2(dev, "gpmi_bch_apb", "bch_podf", base + 0x78,
> 24));
> + clk_dm(IMX6UL_CLK_GPMI_BCH,
> + imx_clk_gate2(dev, "gpmi_bch", "gpmi_podf", base + 0x78, 26));
> + clk_dm(IMX6UL_CLK_GPMI_IO,
> + imx_clk_gate2(dev, "gpmi_io", "enfc_podf", base + 0x78, 28));
> + clk_dm(IMX6UL_CLK_GPMI_APB,
> + imx_clk_gate2(dev, "gpmi_apb", "bch_podf", base + 0x78, 30));
> +#endif
> + clk_dm(IMX6UL_CLK_IPG,
> + imx_clk_divider(dev, "ipg", "ahb", base + 0x14, 8, 2));
Sorry again, but the line here with IMX6UL_CLK_IPG is a duplicate of the one
above.
It should be removed.
> + clk_dm(IMX6UL_CLK_I2C1,
> + imx_clk_gate2(dev, "i2c1", "perclk", base + 0x70, 6));
> + clk_dm(IMX6UL_CLK_I2C2,
> + imx_clk_gate2(dev, "i2c2", "perclk", base + 0x70, 8));
> + clk_dm(IMX6UL_CLK_I2C3,
> + imx_clk_gate2(dev, "i2c3", "perclk", base + 0x70, 10));
> + clk_dm(IMX6UL_CLK_PWM1,
> + imx_clk_gate2(dev, "pwm1", "perclk", base + 0x78, 16));
> +
> + clk_dm(IMX6UL_CLK_ENET,
> + imx_clk_gate2(dev, "enet", "ipg", base + 0x6c, 10));
> + clk_dm(IMX6UL_CLK_ENET_REF,
> + imx_clk_fixed_factor(dev, "enet_ref", "pll6_enet", 1, 1));
> +
> + struct clk *clk, *clk1;
> +
> + clk_get_by_id(IMX6UL_CLK_ENFC_SEL, &clk);
> + clk_get_by_id(IMX6UL_CLK_PLL2_PFD2, &clk1);
> +
> + clk_set_parent(clk, clk1);
> +
> + return 0;
> +}
> +
> +static const struct udevice_id imx6ul_clk_ids[] = {
> + {.compatible = "fsl,imx6ul-ccm" },
^
|
If you want you could add a space here for consistency with the other drivers.
> + { },
> +};
> +
> +U_BOOT_DRIVER(imx6ul_clk) = {
> + .name = "clk_imx6ul",
> + .id = UCLASS_CLK,
> + .of_match = imx6ul_clk_ids,
> + .ops = &imx6ul_clk_ops,
> + .probe = imx6ul_clk_probe,
> + .flags = DM_FLAG_PRE_RELOC,
> +};
I have only an imx6ull for testing. At the moment it looks good, although
the Ethernet on my device is not yet working with this clock driver. I need
to find out why it's not working and will probably do a follow up patch.
With that changes above
Reviewed-by: Christoph Niedermaier
Regards
Christoph
ck driver. I need
>> to find out why it's not working and will probably do a follow up patch.
>>
>> With that changes above
>> Reviewed-by: Christoph Niedermaier
>>
>
> I will add you review and tested-by: during respin. For the ethernet I
> can take a
device is not yet working with this clock driver. I need
>>> to find out why it's not working and will probably do a follow up patch.
>>>
>>> With that changes above
>>> Reviewed-by: Christoph Niedermaier
>>>
>>
>> I will add you review
From: Tom Rini
Sent: Friday, July 11, 2025 7:41 PM
> On Fri, Jul 11, 2025 at 05:37:55PM +0000, Christoph Niedermaier wrote:
>> From: Tom Rini
>> Sent: Thursday, July 10, 2025 3:58 AM
>>> Subject: Re: [PATCH V4 0/3] cmd: env: select: Add output for available
>>&g
From: Tom Rini
Sent: Thursday, July 10, 2025 3:58 AM
> Subject: Re: [PATCH V4 0/3] cmd: env: select: Add output for available
> environment targets
>
> On Mon, Jun 30, 2025 at 05:12:29PM +0200, Christoph Niedermaier wrote:
>
>> Add the "-l" parameter to the &q
second one adds unit tests for checking the env command.
These were used to check the env commands before and after the conversion.
The third patch adds the actual new parameter "-l" for the "env select"
command.
Christoph Niedermaier (3):
cmd: nvedit: Convert the parsing of a
Add basic unit tests for argument checking of the env command.
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Christian Marangi
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Jerome Forissier
Cc: Joe Hershberger
Cc: Marek Vasut
Cc: Michal Simek
Cc: Patrick Delaunay
The entire parsing of the arguments is done manually. Improve this
by converting all parsing of arguments to getopt().
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Christian Marangi
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Jerome Forissier
Cc: Joe Hershberger
Cc
found". Replace this not
so useful output by showing the env usage message instead.
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Christian Marangi
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Jerome Forissier
Cc: Joe Hershberger
Cc: Marek Vasut
Cc: Michal S
From: Christoph Niedermaier
Sent: Friday, July 11, 2025 10:57 PM
> From: Tom Rini
> Sent: Friday, July 11, 2025 7:41 PM
>> On Fri, Jul 11, 2025 at 05:37:55PM +0000, Christoph Niedermaier wrote:
>>> From: Tom Rini
>>> Sent: Thursday, July 10, 2025 3:58 AM
>>>
Add basic unit tests for argument checking of the env command.
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Casey Connolly
Cc: Christian Marangi
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Jerome Forissier
Cc: Joe Hershberger
Cc: Marek Vasut
Cc: Mattijs
found". Replace this not
so useful output by showing the env usage message instead.
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Casey Connolly
Cc: Christian Marangi
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Jerome Forissier
Cc: Joe Hershberger
Cc: Marek
he second one avoids stopping unit test on console output
"## Error: ". The third one adds unit tests for checking the env command.
These were used to check the env commands before and after the conversion.
The fourth patch adds the actual new parameter "-l" for the "env selec
The entire parsing of the arguments is done manually. Improve this
by converting all parsing of arguments to getopt().
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Casey Connolly
Cc: Christian Marangi
Cc: Heinrich Schuchardt
Cc: Ilias Apalodimas
Cc: Jerome Forissier
Cc
Some tests deliberately provoke an error, but if the console output is
"## Error: " pytest catches it as an error and cancels the test.
Remove this to enable this type of tests.
Signed-off-by: Christoph Niedermaier
---
Cc: Andy Shevchenko
Cc: Casey Connolly
Cc: Christian Marangi
Cc
From: Tom Rini
Sent: Monday, July 28, 2025 9:03 PM
> On Fri, Jul 18, 2025 at 07:05:58PM +0200, Christoph Niedermaier wrote:
>
>> Add the "-l" parameter to the "env select" command to print the available
>> environment targets, convert all argument parsing i
pull
> up via DT pinctrl node entry.
>
> Signed-off-by: Marek Vasut
> ---
> Cc: Andreas Geisreiter
> Cc: Christoph Niedermaier
> Cc: Tom Rini
> Cc: u-b...@dh-electronics.com
> Cc: u-boot@lists.denx.de
> ---
> board/dhelectronics/dh_imx6/dh_imx6_spl.c | 6 ++
97 matches
Mail list logo
