Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-12-13 Thread Fabio Estevam
On Tue, Nov 7, 2023 at 7:16 PM wrote: > > From: Benjamin Szőke > > Technexion PICO-IMX7 SoM is supporting USDHC3 (eMMC or micro SD on SoM) > and USDHC1 (SD on carrier board) to use on any carrier board like > PICO-NYMPH. Based on the U-Boot version from Technexion it adds > baseboard SD card

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-29 Thread Dragan Simic
://lore.kernel.org/u-boot/ Eredeti levél Feladó: Dragan Simic Dátum: 2023 november 29 22:34:25 Tárgy: Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect Címzett: Szőke Kálmán Benjamin On 2023-11-29 22:12, Szőke Kálmán Benjamin wrote: https

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-29 Thread Szőke Kálmán Benjamin
tem. I saw no benefits, though. >> Eredeti levél >> >> Feladó: Dragan Simic >> >> Dátum: 2023 november 29 05:47:55 >> >> Tárgy: Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot >> detect >> >> Címzett:

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-29 Thread Dragan Simic
: add baseboard SD card boot detect Címzett: Szőke Kálmán Benjamin On 2023-11-29 00:35, Szőke Kálmán Benjamin wrote: Sorry i can not fix it, this is an average e-mail service provider that provides the expected tech level in 2023.Please upgrade your u-boot list website engine to able

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-29 Thread Szőke Kálmán Benjamin
https://lists.yoctoproject.org/g/main  Eredeti levél Feladó: Dragan Simic Dátum: 2023 november 29 05:47:55Tárgy: Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detectCímzett: Szőke Kálmán Benjamin On 2023-11-29 00:35, Szőke Kálmán Benjamin wrote: > Sorry i can not fix

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-28 Thread Dragan Simic
On 2023-11-29 00:35, Szőke Kálmán Benjamin wrote: Sorry i can not fix it, this is an average e-mail service provider that provides the expected tech level in 2023.Please upgrade your u-boot list website engine to able to support e-mail formats/technologies from 2020s ages. I suggest to use

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-28 Thread Szőke Kálmán Benjamin
Sorry i can not fix it, this is an average e-mail service provider that provides the expected tech level in 2023.Please upgrade your u-boot list website engine to able to support e-mail formats/technologies from 2020s ages. I suggest to use similar what Yocto project has: 

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-28 Thread Hugo Villeneuve
On Mon, 27 Nov 2023 23:26:08 + (GMT) Szőke Kálmán Benjamin wrote: > It was absolutely conform, i supressed the potentional warning. Once again i > say, i will not remove it, if you do not liket it you can remoce as a > maintainer. My patch is ready and final. Let's do something to improve

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-27 Thread Szőke Kálmán Benjamin
It was absolutely conform, i supressed the potentional warning. Once again i say, i will not remove it, if you do not liket it you can remoce as a maintainer. My patch is ready and final. Let's do something to improve i.MX family as a maintainer. I am realy sad to see in last 2-3 years there

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-27 Thread Fabio Estevam
On Mon, Nov 27, 2023 at 4:55 PM Szőke Kálmán Benjamin wrote: > > Unused function parameters should be removed > https://rules.sonarsource.com/c/tag/based-on-misra/RSPEC-1172/ >From this same URL: "Exceptions There are some cases when you want to have an unused parameter (usually because the

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-27 Thread Szőke Kálmán Benjamin
Unused function parameters should be removedhttps://rules.sonarsource.com/c/tag/based-on-misra/RSPEC-1172/Quaility means the full SW has no any functions with unused paramteres. Unfortunatelly it seems u-boot and kernel source code are not best, it is time to improve it. You can see, i did it,

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-26 Thread Fabio Estevam
Hi Benjamin, On Sun, Nov 26, 2023 at 8:17 PM Szőke Kálmán Benjamin wrote: > > It can happen when will -Wall be enabled as in others SW projects. My patches > are final, ready and works. I will not remove this "supression", it must be a > part of it if the target is to provide code in good

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-26 Thread Szőke Kálmán Benjamin
It can happen when will -Wall be enabled as in others SW projects. My patches are final, ready and works. I will not remove this "supression", it must be a part of it if the target is to provide code in good quality. If you have any further whishes you can edit my patch and you can be a

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-19 Thread Fabio Estevam
On Sun, Nov 19, 2023 at 8:47 PM Szőke Kálmán Benjamin wrote: > > It will warn, if warning levels is set correctly. I will do not remove it. How can it happen if U-Boot uses warning-1 := -Wextra -Wunused -Wno-unused-parameter ?

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-19 Thread Szőke Kálmán Benjamin
It will warn, if warning levels is set correctly. I will do not remove it.https://stackoverflow.com/questions/3599160/how-can-i-suppress-unused-parameter-warnings-in-cbtwSorry for "top-post" i am using an e-mail servicer which is not prepared to following the 1990s styles which is need for

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-19 Thread Fabio Estevam
Hi Benjamin, [Please do not top-post] On Sun, Nov 19, 2023 at 6:08 PM Szőke Kálmán Benjamin wrote: > (void)(devno); is need because devno arguments is an unused variable, need to > use this solution to suppress any potential warnings. No, devno is a function parameter. There is no warning if

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-19 Thread Szőke Kálmán Benjamin
Hi,All previouse stage/patches was a "blind alley" solution because finally i found, many code parts are ready from freesacle sources. So needed to refactor the full codes, which seems now a totally new code/solution for it.So, I don't see the point in describing them in a changelog, useless to

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-15 Thread Fabio Estevam
Hi Benjamin, You should have indicated that this is v7. The subject should be: [PATCH v7] pico-imx7d: add baseboard SD card boot detect No need for adding 1/1 for a single patch. On Tue, Nov 7, 2023 at 7:16 PM wrote: > > From: Benjamin Szőke > > Technexion PICO-IMX7 SoM is supporting USDHC3