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

Szőke Kálmán Benjamin egyszeregy at freemail.hu
Mon Nov 27 20:55:06 CET 2023


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, so if you do not like it and you would like to keep the bad and historical solution you can change it in my patch yourself as a maintaner. I am not planning to submit any new patch version and my patch is done in my point of view.-------- Eredeti levél --------Feladó: Fabio Estevam <festevam at gmail.com>Dátum: 2023 november 27 01:44:16Tárgy: Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detectCímzett: Szőke Kálmán Benjamin <egyszeregy at freemail.hu>Hi Benjamin,  On Sun, Nov 26, 2023 at 8:17 PM Szőke Kálmán Benjamin <egyszeregy at freemail.hu> 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 quality. If you have any further whishes you can edit my patch and you can be a co-author.  Sorry, but this makes no sense.  You are going in the opposite direction of good code quality.  Both U-Boot and kernel pass -Wno-unused-parameter.  It is common to write codes for stub functions like this:  https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/include/linux/regulator/of_regulator.h?h=v6.6.2#n29  No (void)parameter is passed as you did in your patch. Like I said, it is unnecessary. 


More information about the U-Boot mailing list