03 Dec, 2019
40 commits
-
Currently, we have two places where we list files that should always be
cleaned if they exist. One, the CLEAN_FILES variable, is from the Linux
build system and is the proper place to list files.The other, a set of extra arguments passed in the xargs template used to
remove files with certain extensions, was introduced by 8f06f0cee3d3
("Makefile: clean image.map") and is clearly wrong: by extending the
xargs template, we attempt to remove the files once for each batch of
arguments that xargs produces and we reduce the number of files from
stdin that xargs can include in each of its rm commands.To fix this, put all hardcoded files into CLEAN_FILES.
Signed-off-by: Thomas Hebb
-
All of the clean targets (clean, dist-clean, mrproper) are run without
loading a configuration. That means that any conditional modification of
CLEAN_FILES will either always apply or never apply and so shouldn't be
in a conditional in the first place.Since CLEAN_FILES is allowed to list nonexistent files, just add
everything to it unconditionally to fix the issue.Signed-off-by: Thomas Hebb
-
This missing dependency seems like an oversight, since all other
TPL_DM_* options have it.Signed-off-by: Thomas Hebb
-
27084c03d36a ("spl: Allow tiny printf() to be controlled in SPL and
TPL") split this option in two for TPL and SPL, but didn't change the
Kconfig names, making them hard to set quickly.Signed-off-by: Thomas Hebb
-
Signed-off-by: Tom Rini
-
- A large series of clean-ups to reduce common.h contents
-
These functions do not use driver model but are still used. Move them to a
new eeprom.h header file.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This is not used in U-Boot. Drop it.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
Move this function into the init.h header file.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
Reviewed-by: Daniel Schwierzeck -
This function can be dropped when all boards use driver model for PCI. For
now, move it into init.h with a comment.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
A number of board function belong in init.h with the others. Move them.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
Move this function into init.h which seems to be designed for this sort
of thing. Also update the header to declare struct global_data so that it
can be included without global_data.h being needed.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This function is not defined by any boards so the feature is not used.
Drop it.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
Move these functions into the command.h header file which is a better fit.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
Move these two functions into the irq_funcs.h header file. Also move
interrupt_handler_t as this is used by the irq_install_handler() function.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
These functions do not use driver model but are fairly widely used in
U-Boot. But it is not clear that they will use driver model anytime soon,
so we don't want to label them as 'legacy'.Move them to a new irq_func.h header file. Avoid the name 'irq.h' since it
is widely used in U-Boot already.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
Remove the unwanted space before the bracket.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
These functions are CPU-related and do not use driver model. Move them to
cpu_func.hSigned-off-by: Simon Glass
Reviewed-by: Daniel Schwierzeck
Reviewed-by: Tom Rini -
These are used by only one arch and only within a single file. Drop the
declarations from the common file.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
These functions belong in cpu_func.h. Another option would be cache.h
but that code uses driver model and we have not moved these cache
functions to use driver model. Since they are CPU-related it seems
reasonable to put them here.Move them over.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
Remove the unwanted space before the bracket.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
These functions belong in cpu_func.h so move them over.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This function belongs in cpu_func.h so move it over.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This function is not defined anywhere. Drop it.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
These functions belong in cpu_func.h since they do not use driver model.
Move them over. Don't bother adding comments since these functions should
be deleted.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This function belongs in mii.h so move it over.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This function belongs in time.h so move it over and add a comment.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This function belongs in time.h so move it over and update the comment
style.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This function has a similar name to the common wait_ticks(). It is only
used in one place and seems small enough to drop.Inline it.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This function belongs in time.h so move it over and add a comment.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
These functions belong in time.h so move them over and add comments.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
Add a header file to house the lz4 compression function. Add a comment
while we are here, since it not even clear from the name what the function
actuall does.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
These functions belong in serial.h so move them over.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
Move this function header to serial.h since this function is clearly
related to serial. The function itself stays in console.c since we don't
have a single serial file. DM and non-DM each has a separate file so we
would have to either create a new common serial file, or repeat the
function in both serial.c and serial-uclass.c, neither of which seem
worthwhile.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
The function declarations in serial.h are not in sync with what is
currently used in usbtty. Fix this by updating the header and including
it, to help catch future such problems.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
This function relates to networking, so move it out of the common.h
header file.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
Acked-by: Joe Hershberger -
At present this function sits in its own file but it does not really
justify it. There are similar string functions in vsprintf.h, so move it
there. Also add the missing function comment.Use the vsprintf.h include file explicitly where needed.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
These don't need to be in common.h so move them out into a new header.
Also add some missing comments.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini -
These don't need to be in common.h so move them out.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
Reviewed-by: Tom Rini -
It is not good practice to write code in a header file. If it is included
multiple times then the code can cause duplicate functions.Move the bootcount_store() and bootcount_load() functions into SPL.
Note: bootcount is a bit strange in that it uses driver model but does not
define proper drivers. This should be fixed.Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
