Re: [PATCH 1/4] common/board_f: remove XTRN_DECLARE_GLOBAL_DATA_PTR dead code

2022-09-12 Thread Wolfgang Denk
e reservation of a specific register for it), but that the implementation uses a register nevertheless because this results in smaller code? Maybe add such a comment instead ? Thanks! Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munic

Re: [PATCH 2/3] examples: hello_world: Drop inclusion of common header

2022-08-29 Thread Wolfgang Denk
a standalone app can be built without > it. > > Signed-off-by: Paul Barker > --- > examples/standalone/hello_world.c | 1 - > 1 file changed, 1 deletion(-) Acked-off-by: Wolfgang Denk Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgan

Re: [PATCH 3/3] exports: Fix export of SPI access functions

2022-08-26 Thread Wolfgang Denk
inclluded. I2C was there because it was needed for reading the environment from an EEPROM, but then the interface was frozen ano no more new interfices / drivers should be allowed. Viele Grüße, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich,

Re: [PATCH 1/3] Licenses: Clarify exceptions for standalone apps

2022-08-26 Thread Wolfgang Denk
Dear Paul Barker, In message <20220505153242.1598807-2-paul.bar...@sancloud.com> you wrote: > On 2010-01-27, an email [1] was sent to the mailing list by Wolfgang > Denk which clarified the intended licensing exceptions for standalone > applications. As the "export.h"

Re: [PATCH] image: Ensure image header name is null terminated

2022-08-23 Thread Wolfgang Denk
*hdr, const char *name) > { > - strncpy(image_get_name(hdr), name, IH_NMLEN); > + char *hdr_name = image_get_name(hdr); > + > + strncpy(hdr_name, name, IH_NMLEN - 1); > + hdr_name[IH_NMLEN - 1] = '\0'; > } Why don't you use strlcpy() instead? This covers

Re: [RFC 0/2] A/B firmware update based in eMMC boot partition.

2022-01-25 Thread Wolfgang Denk
u should definitely look into it. I strongly recommend not to try to reinvent that wheel - poorly - in U-Boot. We have been there before, and got rid of this approach for a pretty long list of good reasons. [1] https://sbabic.github.io/swupdate/ Best regards, Wolfgang Denk -- DENX Soft

Re: [ANN] U-Boot v2022.01 released

2022-01-14 Thread Wolfgang Denk
Armstrong2449 (2.6%) Michael Walle 2312 (2.4%) Jan Kiszka2125 (2.2%) ... Developers with the most lines removed Tom Rini 11969 (26.1%) Anatolij Gustschin 917 (2.0%) Patrick Delaunay 813 (1.8%) Eugen Hristev 526 (1.1%)

Re: [PATCH v2] tools/mxsimage: Remove fclose on empty FILE pointer

2021-11-24 Thread Wolfgang Denk
ge` to segfault. > > This patch removes the label for error handling and instead returns > immediately which skips the `fclose()` and prevents the segfault. The > errno is also described in the error message to guide users. > > Signed-off-by: Mattias Hansson Reviewed-by: Wolfgang De

Re: [PATCH] tools/mxsimage: Remove fclose on empty FILE pointer

2021-11-24 Thread Wolfgang Denk
set(&cmd, 0, sizeof(cmd)); @@ -1616,12 +1619,6 @@ static int sb_load_cmdfile(struct sb_image_ctx *ictx) fclose(fp); return 0; - -err_file: - fclose(fp); - fprintf(stderr, "ERR: Failed to load file \"%s\"\n", - ictx->cfg_filename);

Re: [PATCH v2 1/2] lib/circbuf: Make circbuf selectable symbol

2021-11-22 Thread Wolfgang Denk
_ADDR > the size is too small then the message which says the amount of early > data being coped will the the same as the > > +config CIRCBUF > +bool > + > source lib/dhry/Kconfig Please add a description what this symbos is doing. Thanks. Wolfgang Denk

Re: [PATCHv2] net: uclass: Save ethernet MAC address when generated

2021-11-22 Thread Wolfgang Denk
tting the env var. Suggestion: Selecting this will allow the Ethernet interface to function even when the ethaddr variable for that interface is unset. In this case, a random MAC address in the locally administered address space is generated. It will be saved to

Re: [PATCH v2 0/2] env: setenv add resolve value option

2021-11-22 Thread Wolfgang Denk
this as: => setenv a hello => setenv b world => setenv c '${a} ${b}' => setenv a goodbye => setenv b sunshine => setenv foo "setenv d ${c}! ${a}..." => run foo => printenv d d=goodbye sunshine! goodbye... And yes, here you have

Re: [PATCH v2 0/2] env: setenv add resolve value option

2021-11-20 Thread Wolfgang Denk
ct. Yes, this is neither perfect nor elegant. But you can use that in C code as well. > will be easy get useful features via simple solution ( deep resolve > all vars by one line ) I understand what you want, but this is not a good way to solve the problem. I'd really rather see su

Re: [PATCH v2 0/2] env: setenv add resolve value option

2021-11-18 Thread Wolfgang Denk
nt verion of the hush shell. As far as I understand, Francis (added to Cc:) has started working on an update of hush to a current version. We should rather help him with that instead of implementing non-standard workarounds. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH,

Re: [PATCH] net: uclass: Save ethernet MAC address when generated

2021-11-18 Thread Wolfgang Denk
thaddr" environment variable which is mandatory for correct funktion (query by the user, passing to Linux). > So, yes, OK, this is a bug fix, in that NET_RANDOM_ETHADDR was incorrect > at introduction. That means we do need a v2 of this patch that updates > the Kconfig help t

Re: [PATCH] net: uclass: Save ethernet MAC address when generated

2021-11-18 Thread Wolfgang Denk
ioned in the specifications, and certainly not a mandatory feature. That's all copy & paste garbage. Viele Grüße, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-81

Re: [PATCH] net: uclass: Save ethernet MAC address when generated

2021-11-17 Thread Wolfgang Denk
to introduce another change that we won't found out messed > up a bunch of people on until some new work-around is accepted in to > Linux or something. Well, this work-around in Linux is because there have been incnsistent ways how to store the MAC address in the device tree. This has n

Re: [PATCH] net: uclass: Save ethernet MAC address when generated

2021-11-17 Thread Wolfgang Denk
thaddr fallback in eth.c"; from the changes then I'm not sure if this was even USB related. > > If we use some MAC address, it shall be possible to read it using > > "printenv ethaddr" and to set it using "setenv ethaddr". Anything > > else is inconsistent crap.

Re: [PATCH] net: uclass: Save ethernet MAC address when generated

2021-11-17 Thread Wolfgang Denk
it invisible to the user who uses documented APIs ("printenv ethaddr"). If we use some MAC address, it shall be possible to read it using "printenv ethaddr" and to set it using "setenv ethaddr". Anything else is inconsistent crap. Best regards, Wolfgang Denk -- DENX

Re: [PATCH] net: uclass: Save ethernet MAC address when generated

2021-11-16 Thread Wolfgang Denk
the environment already. If the MAC address is not placed in the environment, then how can a user query the currently used MAC address? All documentation says basically: run "printenv ethaddr". Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director

Re: [PATCH] net: uclass: Save ethernet MAC address when generated

2021-11-16 Thread Wolfgang Denk
-going > thread, perhaps "ethmacskip" should play a role in preserving existing > behavior? We have way too many ways to do the same thing - nearly, just a little different :-( Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk

Re: [PATCH RFC] cmd: fix net list command

2021-11-16 Thread Wolfgang Denk
befoire it would not have been read by Linux, so the same should happen here. [Here it is even less problematic as U-Boot has the very same MAC address in it's environment.] Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Mu

Re: [PATCH RFC] cmd: fix net list command

2021-11-16 Thread Wolfgang Denk
OTP > driver in U-Boot. As Marek pointed out, in this specific case it would be best to provide such a driver. > From my end, I think we can conclude the discussion. ACK. Thanks. Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich,

Re: [PATCH RFC] cmd: fix net list command

2021-11-16 Thread Wolfgang Denk
environment. Agreed. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de A girl with

Re: [PATCH RFC] cmd: fix net list command

2021-11-16 Thread Wolfgang Denk
ore or less a management decision how board bringup / inital commissioning is organized. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-

Re: [PATCH RFC] cmd: fix net list command

2021-11-15 Thread Wolfgang Denk
lid produt data like serial number, MAC address etc. Yes, this can be done over serial consone as well, but 1) this is slower than network and 2) many commecial products don't have the connector etc. populated. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Directo

Re: [PATCH RFC] cmd: fix net list command

2021-11-15 Thread Wolfgang Denk
is indeed obvious abuse of a feature which has been designed for a few specific use cases. It shpould be made clear that this is nothing to select without thinking. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenst

Re: [PATCH RFC] cmd: fix net list command

2021-11-15 Thread Wolfgang Denk
o use the address I want to have used. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Perfection is re

Re: [PATCH RFC] cmd: fix net list command

2021-11-15 Thread Wolfgang Denk
. And it may be intentional to use a random MAC address. There is this old rule: "UNIX was not designed to stop you from doing stupid things, because that would also stop you from doing clever things." - Doug Gwyn If you don't like it, then disable the feature. It is nonstand

Re: [PATCH RFC] cmd: fix net list command

2021-11-15 Thread Wolfgang Denk
andom MAC address is used, then this _is_ the correct MAC address, as normally (*) U-Boot is just a tool that does what the user requests. (*) Exceptions are situations where U-Boot's user interfaces are locked down for example for security reasons. Best regards, Wolfgang Denk -- D

Re: [RFC 0/2] Do not stop with an error when mkimage fails

2021-11-11 Thread Wolfgang Denk
ates the needed data, at least dummy versions of it? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de :

Re: [RFC 0/2] Do not stop with an error when mkimage fails

2021-11-11 Thread Wolfgang Denk
me other idea? I'm just thinking out loud here. Well, why not fix the root cause? Heiko writes that "an external HDMI firmware" is needed - so the fix is to provide one, or at least a dummy file which is good enough for the build to succeed. It should be trivial to cre

Re: [RFC 0/2] Do not stop with an error when mkimage fails

2021-11-11 Thread Wolfgang Denk
ithout doing this kind of workaround for the U-Boot CI. It's not only not a perfect solution, it is the intentional introduction of a bug, and thus totally unacceptable. If there is a file missing in the CI, then add/create it there. But do remove necessary error handling which would cause har

Re: [RFC 0/2] Do not stop with an error when mkimage fails

2021-11-07 Thread Wolfgang Denk
u expect when you ignore an error and just continue with garbage data as if nothing happened? Sorry, but this makes no sense to me. If there is an error, it should be reported and - if possible - handled. If this is not possible, then the correct thing is to abort. Ignoring errors and trying to

Re: [RFC 0/2] Do not stop with an error when mkimage fails

2021-11-04 Thread Wolfgang Denk
if mkimage fails and you continue, the next step is tools.ReadFile(output_fname) trying to read that file. How is this possible? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Re: [PATCH v9 3/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-26 Thread Wolfgang Denk
like: foo += bar foo += \ bar foo += ' bar' Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80

Re: [PATCH v9 3/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-25 Thread Wolfgang Denk
f something else can parse it (libubootenv?) it can > be done. Actually I have an even more low-level approach in mind, like the capability to include (or rather import) binary U-Boot environment data given in the usual =\0...=\0\0 form. This might come in handy if your data comes from export

Re: [PATCH v9 3/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-24 Thread Wolfgang Denk
e commit message and in the related documentation/readme. 2) There should be another, more primitive way to generate environment settings without these restrictions.. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich,

Re: [PATCH v10 3/9] env: Allow U-Boot scripts to be placed in a .env file

2021-10-24 Thread Wolfgang Denk
200 ... You don't like it? Don't do it, then. Yes, robust programming is something different, but at that time we were fighting for 10 or 20 byte code size - there were so many systems where U-Boot, Linux, and root file system had to fit into 4 MB flash or so. Best regar

Re: [PATCH v10 3/9] env: Allow U-Boot scripts to be placed in a .env file

2021-10-22 Thread Wolfgang Denk
legal, too), so above line should actually set the environment variable "maximum\+" to "value". Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)

Re: [PATCH v9 3/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-22 Thread Wolfgang Denk
l name character. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Q: How many IBM CPU's does

Re: [PATCH v9 3/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-22 Thread Wolfgang Denk
> We could update it to skip any leading space after the = I think. So what if you need a leading space? > I don't like spaces before the = though. It doesn't match the 'print' > output (which has no space) and it is confusing: env print also does not add any spaces af

Re: [PATCH v9 3/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-21 Thread Wolfgang Denk
ot;7". Why not... In the end it boils down that the file format should be properly defined and have a clear syntax description. Apparently the "=\0" format as used in U-Boot persistent storage should not be taken literally here. Best regards, Wolfgang Denk -- DENX Software Engineeri

Re: [PATCH v9 3/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-21 Thread Wolfgang Denk
d of your variables". In which way is it worse? For esthetic reasons? I confirm that '+=' looks better. But '+=" is technically broken. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office:

Re: [PATCH v9 3/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-21 Thread Wolfgang Denk
"=+" instead of "+=" as append operator do not require such restictions. Thus, and only for the restictions on variable names: Naked-by: Wolfgang Denk Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Of

Re: [PATCH v8 4/8] env: Allow U-Boot scripts to be placed in a .env file

2021-10-21 Thread Wolfgang Denk
' maxim > at this point. In which way is "+=" easier to parse than "=+"? The only extension you have to make is for the "=[^+]" case to check the first charater of the value: if it is a backslash, then ignore it. This has zero influence on doing this manuall

Re: [PATCH v8 4/8] env: Allow U-Boot scripts to be placed in a .env file

2021-10-21 Thread Wolfgang Denk
cases / side-effects / position in parsing. I fully agree that "+=" would be nicer. But it has issues, which "=+" does not have. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82

Re: [PATCH v8 4/8] env: Allow U-Boot scripts to be placed in a .env file

2021-10-21 Thread Wolfgang Denk
tuations I can think of in a somewhat reasonable way, and it does not need to place new restrictions on variable names. > is very confusing. Is it? > Since people will be converting their out-of-tree scripts anyway, they > can check for this sort of madness at the time. There should b

Re: [PATCH v8 4/8] env: Allow U-Boot scripts to be placed in a .env file

2021-10-19 Thread Wolfgang Denk
r own escape rules, for example: var=fredassigns var=+fred appends "fred" var=\+fred assignes the value "+fred" var=++fred appends "+fred" Best regards, Wolfgang Denk -- DENX Software Engineering GmbH,

Re: [PATCH v6 4/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-19 Thread Wolfgang Denk
bar.c fred\+=aaa -> gcc -E bar.c # 0 "bar.c" # 0 "" # 0 "" # 1 "/usr/include/stdc-predef.h" 1 3 4 # 0 "" 2 # 1 "bar.c" fred\+=aaa I do not see this problem... Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Mana

Re: [PATCH v8 4/8] env: Allow U-Boot scripts to be placed in a .env file

2021-10-19 Thread Wolfgang Denk
tually this would be also more allowing for variations in white space, so that "=" could also be written as " = ". which in turn would allow for foo+ = bar vs. foo += bar . Just my 2¢... Best regards, Wolfgang Denk -- DENX Software Engineering Gm

Re: [PATCH v8 4/8] env: Allow U-Boot scripts to be placed in a .env file

2021-10-19 Thread Wolfgang Denk
sly valid > environment variables, if this was written in Python say, would we have > this problem? A classic tool would be lex ... Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Gro

Re: [PATCH v8 4/8] env: Allow U-Boot scripts to be placed in a .env file

2021-10-19 Thread Wolfgang Denk
lash. I would omit this as it does not provide any relevant information here, and in addition it is misleading as it could be interpreted that such characters are only legal at the start of the variable name. But you can do: => setenv .-^-. foo => printenv .-^-. .-^-.=foo Best regard

Re: [PATCH v8 0/8] env: Allow environment in text files

2021-10-19 Thread Wolfgang Denk
g. This should be board//.env , right? But that's only the cover letter... Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)

Re: [PATCH v6 4/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-19 Thread Wolfgang Denk
of getting the env you want > using #define > > My current motivation is to complete the CONFIG migration, now in its 8th > year. I fully understand your motivation and appreciate your efforts. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: W

Re: [PATCH v6 4/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-19 Thread Wolfgang Denk
any good reason for it - yes, your awk script cannot handle this situation, but I tend to belive the fix is in a better parser than in placing restrictions on the input data. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, O

Re: [PATCH v6 4/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-19 Thread Wolfgang Denk
1 years no one has been using > foo+bar, baz+, etc, in their variable names, maybe we just document > that's not valid and move on? We cannot know what people have been using in their environemnts. Even for those boards that are in mainline, the environment settings used in real life are often tota

Re: [PATCH v6 4/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-18 Thread Wolfgang Denk
m... I can't find it right now but did I not also read about other restrictions to variable names, like they must noch begin with '_' when using this new tool? I feel it is wrong to place new restrictions on something that was constant for 21 years, just because our parser cannot pa

Re: [PATCH v7 5/7] doc: Mention CONFIG_DEFAULT_ENV_FILE

2021-10-18 Thread Wolfgang Denk
uot;No prams or dogs or food being eaten!" I mean eating dogs is bad enough, but eating prams? ? ? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-669

Re: [PATCH v7 3/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-18 Thread Wolfgang Denk
ed in the commit message, not only in the code. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Command,

Re: [PATCH v7 0/7] env: Allow environment in text files

2021-10-18 Thread Wolfgang Denk
oard//.env , right? Reviewed-by: Wolfgang Denk Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de READ T

Re: [PATCH v6 4/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-18 Thread Wolfgang Denk
foo\+=bar or foo+ = bar ? And please see also my comments about changing the autostart functionality for the user. > I haven't seen any positive comments to this series yet... Maybe many long-term users of U-Boot don't see the current situation as such a big problem? I

Re: [PATCH v6 7/7] bootm: Tidy up use of autostart env var

2021-10-15 Thread Wolfgang Denk
d-off-by: Simon Glass > Suggested-by: Wolfgang Denk It should be noted that this commit changes the behaviour of U-Boot for "autostart" users, thus it has the potential of breaking existent systems. The problematic cases are in do_bootelf() [cmd/elf.c] and do_bootm_standalone() [common/boo

Re: [PATCH v6 4/7] env: Allow U-Boot scripts to be placed in a .env file

2021-10-15 Thread Wolfgang Denk
le in a board//env > directory called .env (or common.env if you want the same > environment for all boards). Argh... did you bother to read my comments? Apparently not. Thus: NAKed by: Wolfgang Denk I really think your fixed filename proposal does not work well in reality. The file name

Re: [PATCH v5 08/10] include/configs: apalis-imx8/verdin-imx8mm: rename kernel image variable

2021-10-09 Thread Wolfgang Denk
ting with modern scripts and does not break working > of earlier versions of boot.scr. Same comment as before: please document all changes, i. e. the FDT related changes. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, O

Re: [PATCH v4 08/10] include/configs: apalis-imx8/verdin-imx8mm: rename kernel image variable

2021-10-09 Thread Wolfgang Denk
ting with modern scripts and does not break working > of earlier versions of boot.scr. The patch also changes FST handling without mentioning it in the description. Please fix. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235

Re: [PATCH v5 3/5] env: Allow U-Boot scripts to be placed in a .env file

2021-10-06 Thread Wolfgang Denk
".env" scheme, and "common.env" does not fit either, as we have two "common" files. I wonder if there should rather be a Kconfig option so each board can select it's env file name; default would be ".env". what do you think? Best regards, Wolfgang Denk --

Re: [ANN] U-Boot v2021.10 released

2021-10-05 Thread Wolfgang Denk
source.denx.de all te time, I just didn't look at the right place. See https://source.denx.de/u-boot/gitdm Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49

Re: [PATCH v5 3/5] env: Allow U-Boot scripts to be placed in a .env file

2021-10-05 Thread Wolfgang Denk
board//env/common.env" in my .env files, your logic would trigger on the existence of the common.env file and ignore the .env files. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groe

Re: [ANN] U-Boot v2021.10 released

2021-10-05 Thread Wolfgang Denk
.de server. It was not moved to source.denx.de I will fix this ASAP... Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email

Re: [ANN] U-Boot v2021.10 released

2021-10-05 Thread Wolfgang Denk
ARM 3 (1.6%) Intel 3 (1.6%) ... Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142

Re: [PATCH v5 4/5] env: Allow environment files to use the C preprocessor

2021-10-04 Thread Wolfgang Denk
x27; and the NUL characters. Adding artificial restrictions now just to enable your custom notation for appending seems not acceptable to me. You might want to chose a different notation or implement a proper parser instead. Thanks. Wolfgang Denk -- DENX Software Engineering GmbH, Managing

Re: [PATCH v5 3/5] env: Allow U-Boot scripts to be placed in a .env file

2021-10-04 Thread Wolfgang Denk
nv if you want the same > environment for all boards). This should be no exclusive "or" here. If a common.env exists, it should be used for all boards, and if additionally one ore more .env files exist, these should ALSO be applied to the respective boards. Best regards, W

Re: [PATCH v5 2/5] doc: Move environment documentation to rST

2021-10-04 Thread Wolfgang Denk
n > +When set to a value < 4095 the traffic over > +Ethernet is encapsulated/received over 802.1q > +VLAN tagged frames. Really? I can't see where "vlan" is actually used in the code. [I looked for it because I could not remember if the values is interpreted

Re: [PATCH v2] cmd: ubi: add a command to swap volumes

2021-09-28 Thread Wolfgang Denk
ate. So if you think to use this for example for reliable software updates, then you have a big problem here. Or am I missing something? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell

Re: [PATCH] ARM: omap3_logic: Cleanup usage of MUX_VAL

2021-09-27 Thread Wolfgang Denk
ther lines. You might want to fix that, too, while we are at it? Reviewd-by: Wolfgang Denk Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10

Re: Bug in board/logicpd/omap3som/omap3logic.h ???

2021-09-27 Thread Wolfgang Denk
#x27;t trip anything up. Most > > of the time it's not used with one either, from a quick grep. > > I am guessing it was left out by accident. I would not have > intentionally done that. I can submit a patch to make it consistent > if you want. Yes, it should be made xonsis

[PATCH 1/4] WS cleanup: remove trailing empty lines

2021-09-27 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- arch/arm/cpu/arm926ejs/cache.c| 1 - arch/arm/cpu/armv7/psci-common.c | 1 - arch/arm/cpu/armv8/hisilicon/pinmux.c | 2 -- arch/arm/cpu/armv8/xen/hypercall.S

[PATCH 3/4] WS cleanup: remove trailing white space

2021-09-27 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- Licenses/lgpl-2.0.txt | 2 +- Makefile | 4 ++-- arch/arm/include/asm/arch-am33xx/cpu.h| 2 +- arch/arm/lib/lib1funcs.S | 6 +++--- arch/xtensa/include/asm/arch-de212

[PATCH 2/4] WS cleanup: remove excessive empty lines

2021-09-27 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- README| 1 - arch/arm/mach-mvebu/serdes/a38x/high_speed_env_spec.c | 2 -- arch/xtensa/include/asm/arch-dc232b/core.h| 2 -- arch/xtensa/include/asm/arch-dc232b/tie-asm.h | 4 arch/xtensa

[PATCH 0/4] White Space cleanup

2021-09-27 Thread Wolfgang Denk
code changes result from this. I am aware that such a global cosmetic cleanup is always kind of disruptive, but we are at -rc5, so the code should be pretty stable for now, and I've done a few rebases before, with little and easy to fix conflicts. Wolfgang Denk (4): WS cleanup: r

Bug in board/logicpd/omap3som/omap3logic.h ???

2021-09-27 Thread Wolfgang Denk
I reckon this is a mistake and should be fixed? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de

Re: [PATCH v4 4/5] env: Allow environment files to use the C preprocessor

2021-09-20 Thread Wolfgang Denk
} Hm... this is (so far) a legal command: => setenv foobar+ foo+=bar Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 E

Re: [PATCH v4 3/5] env: Allow U-Boot scripts to be placed in a .env file

2021-09-20 Thread Wolfgang Denk
ncorrect arsing. > + # Is this the start of a new environment variable? > + if (match($0, "^([^ =][^ =]*)=(.*)", arr)) { This is not what you document above. > + if (length(env) != 0) { > + # Record the value of the variable now comple

Re: [PATCH v4 2/5] doc: Move environment documentation to rST

2021-09-20 Thread Wolfgang Denk
> +gatewayip IP address of the Gateway (Router) to use > +hostname Target hostname > +ipaddr See above > +netmaskSubnet Mask > +rootpath Pathname of the root filesystem on the NFS server > +serverip see above > +=

Re: U-BOOT missing Security support

2021-09-15 Thread Wolfgang Denk
rd). This algorithm uses a symetric key 1982 and is widely used as a streaming cipher. Different key lengths are 1983 supported by the algorithm but this command only supports 128 bits 1984 at present. 1985 Best regards, Wolfgang Denk -- DENX Software Engi

Re: [PATCH 1/2] xyz-modem: Wait infinitely for initial y-modem packet

2021-09-13 Thread Wolfgang Denk
ll possible > combinations. I would recommend to define a timeout (in seconds), this is easier to understand for the end user - without looking at the source code they cannot know how a retry count translates into a time interval. And yes, setting the timeout to 0 could mean it waits forever.

Re: [PATCH 1/2] xyz-modem: Wait infinitely for initial y-modem packet

2021-09-13 Thread Wolfgang Denk
; changes UI interaction. We can do both, and still solve your problem: make the timeout adjustable so you can set it to something you can conveniently work with. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchens

Re: [PATCH 2/2] xyz-modem: Wait infinitely for initial x-modem packet

2021-09-13 Thread Wolfgang Denk
; common/xyzModem.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) Ditto here. Please don't. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-814

Re: [PATCH 1/2] xyz-modem: Wait infinitely for initial y-modem packet

2021-09-13 Thread Wolfgang Denk
ary, so I recommend NOT to change the behaviour here, especially as it does not hurt. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)

Re: [PATCH v2] powerpc: mpc: Put U-Boot version string at correct place by linker script

2021-08-24 Thread Wolfgang Denk
ashes are no longer found in new designs, nor does anybody care to keep U-Boot small and tidy. As a result, knowledge about such optimizations is disappearing. Hope this helps. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Muni

Re: Remove the gitlab-ci-runner on source.denx.de?

2021-08-23 Thread Wolfgang Denk
(say, a month or two), I will finally remove it. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de How much net work could a network work, if a network could net work?

Re: How should we deal with actual hush odd behavior?

2021-08-23 Thread Wolfgang Denk
ot, but > what if fixing these bugs lead to a board script failing and so a device not > booting... The new version should be optional in any case, at least for some longer migration period to give users time to test their scripts. If someone selects the new version in his board co

Re: [PATCH v6 10/12] watchdog: add gpio watchdog driver

2021-08-23 Thread Wolfgang Denk
- Simon Glass in > > > > Well you've taken it out of context :-) It would make more sense if > you mention the need for tests Testing can show the presense of bugs, but not their absence. -- Edsger Dijkstr Best regards, Wol

Re: [PATCH v6 10/12] watchdog: add gpio watchdog driver

2021-08-19 Thread Wolfgang Denk
users. Wow. I think I'll add this to my signature database: | "Trying to catch [programming errors] in the field at runtime is not | very kind to your users." | | - Simon Glass in Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wol

Re: [PATCH v6 10/12] watchdog: add gpio watchdog driver

2021-08-19 Thread Wolfgang Denk
rors can never happen" does not seem to be a clever approach to software engineering to me. I stop here. You know my opinion. You are the maintainer. Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groe

Re: [PATCH v6 10/12] watchdog: add gpio watchdog driver

2021-08-19 Thread Wolfgang Denk
we have now a policy to wave through code, and ask others to clean it up later? That's ... sad. Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-

Re: [PATCH v6 10/12] watchdog: add gpio watchdog driver

2021-08-19 Thread Wolfgang Denk
doing a > > git grep -10 -E 'dev_get(_uclass)?_priv' > > and see how many cases you can find where that is followed by a NULL check? The existence of bad code is not a justification to add more of it. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Manag

Re: [PATCH v6 10/12] watchdog: add gpio watchdog driver

2021-08-19 Thread Wolfgang Denk
t;always_running = dev_read_bool(dev, "always-running"); Ditto. > + ret = gpio_request_by_name(dev, "gpios", 0, &priv->gpio, GPIOD_IS_OUT); > + if (ret < 0) { > + dev_err(dev, "Request for wdt gpio failed: %d\n", ret); > +

Re: [PATCH v6 09/12] watchdog: wdt-uclass.c: handle all DM watchdogs in watchdog_reset()

2021-08-19 Thread Wolfgang Denk
turn; Do I see this crrectly that you remove here the code which you just added in patch 02 of this series? Why not doing it right from the beginning? > + uclass_foreach_dev(dev, uc) { > + if (!device_active(dev)) > + continue; > + priv =

  1   2   3   4   5   6   7   8   9   10   >