u-boot.git/lib/fdtdec.c, branch v2019.01

spl: fix build failure with !CONFIG_SPL_PCI_SUPPORT

2018-12-12T17:14:23+00:00

Building U-Boot with CONFIG_PCI and CONFIG_DM_PCI enabled, but
CONFIG_SPL_PCI_SUPPORT disabled, results in following linker
error:

lib/built-in.o: In function `fdtdec_get_pci_bar32':
lib/fdtdec.c:305: undefined reference to `dm_pci_read_bar32'
fdtdec.c:305:(.text.fdtdec_get_pci_bar32+0x24): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `dm_pci_read_bar32'

This is because reference to dm_pci_read_bar32() remains in lib/fdtdec.c
while SPL build does not descend into drivers/pci directory in
drivers/Makefile if CONFIG_SPL_PCI_SUPPORT is not enabled.

Fix this by applying appropriate #define guards in lib/fdtdec.c.
It looks like ns16550.c has the same problem, so fixed that too.

To simplify this, CONFIG_SPL_PCI_SUPPORT is renamed to CONFIG_SPL_PCI
(enables use of CONFIG_IS_ENABLED() macro).

Suggested-by: Vignesh R 
Signed-off-by: Sekhar Nori 
Reviewed-by: Lokesh Vutla 
Reviewed-by: Tom Rini

lib: fdtdec: Add function re-setup the fdt more effeciently

2018-12-10T06:21:21+00:00

In some cases it may be useful to be able to change the fdt we have been
using and use another one instead. For example, the TI platforms uses an
EEPROM to store board information and, based on the type of board,
different dtbs are used by the SPL. When DM_I2C is used, a first dtb must
be used before the I2C is initialized and only then the final dtb can be
selected.
To speed up the process and reduce memory usage, introduce a new function
fdtdec_setup_best_match() that re-use the DTBs loaded in memory by
fdtdec_setup() to select the best match.

Signed-off-by: Jean-Jacques Hiblot 
Reviewed-by: Heiko Schocher

fdt: sandbox: correct use of ${fdtcontroladdr}

2018-12-02T20:59:37+00:00

The sandbox uses a virtual address space that is neither the physical nor
the virtual address space of the operating system. All address used on the
command line live in this address space. So also the environment variable
${fdtcontroladdr} has to be in this address space.

Commands like bootefi and booti receive the fdt address as parameter.
Without the patch ${fdtcontroladdr} cannot be used as parameter value on
the sandbox.

Signed-off-by: Heinrich Schuchardt 
Signed-off-by: Alexander Graf

fdt: Fix uncompress_blob() for U-Boot proper

2018-11-21T02:14:22+00:00

When U-Boot proper is compiled with CONFIG_MULTI_DTB_FIT and tries
to call uncompress_blob(), it fails with -ENOTSUPP. This is because
the full implementation of this function which includes compression
is available only in SPL. In U-Boot proper or if the compression is
not enabled, the blob is not compressed and thus can be passed to
locate_dtb_in_fit() in fdtdec_setup() without any changes. Pass the
blob without any changes if compression is not enabled instead of
failing.

Signed-off-by: Marek Vasut 
Cc: Michal Simek 
Cc: Tom Rini 
Reviewed-by: Simon Glass

Revert "fdt: fdtdec_setup_memory_banksize() use livetree"

2018-10-15T20:22:49+00:00

This reverts commit c35a7d375ec8f0a8ee343ae4868be3242172632e.
This commit is breaking SPL on zc706.

Signed-off-by: Michal Simek 
Reviewed-by: Simon Glass 
Tested-by: Patrice Chotard  [on STM32F746-disco]

fdt: Remove fdtdec_decode_region() function

2018-10-09T10:40:27+00:00

This function is not used in U-Boot now. Remove it along with its 'memory'
version.

Signed-off-by: Simon Glass

fdt: fdtdec_setup_memory_banksize() use livetree

2018-09-29T17:49:35+00:00

Converts fdtdec_setup_memory_banksize() to use ofnode functions instead.

Reviewed-by: Simon Glass 
Signed-off-by: Jens Wiklander

Remove includes and PRI* usages in printf() entirely

2018-09-11T00:48:17+00:00

In int-ll64.h, we always use the following typedefs:

  typedef unsigned int         u32;
  typedef unsigned long        uintptr_t;
  typedef unsigned long long   u64;

This does not need to match to the compiler's .
Do not include it.

The use of PRI* makes the code super-ugly.  You can simply use
"l" for printing uintptr_t, "ll" for u64, and no modifier for u32.

Signed-off-by: Masahiro Yamada

fdt: fix get_next_memory_node()

2018-09-10T13:05:41+00:00

The get_next_memory_node() always sets mem to -1 , which is incorrect,
because then every iteration of memory bank parsing will start from the
first memory bank instead of the previous one.

On systems with 1 memory bank defined in DT and CONFIG_NR_DRAM_BANKS=4 ,
like ie. r8a77965-salvator-x , this will result in U-Boot incorrectly
reporting four identical memory banks with the same memory configuration.

Fix this by setting mem to startoffset value, which restores the behavior
before the fixed patch was applied.

Signed-off-by: Marek Vasut 
Cc: Jens Wiklander 
Cc: Simon Glass 
Cc: Tom Rini 
Fixes: 452bc121027d ("fdt: fix fdtdec_setup_memory_banksize()")
Tested-by: Michal Simek  [on ZynqMP}

fdt: fix fdtdec_setup_memory_banksize()

2018-07-26T20:19:38+00:00

Prior to this patch is fdtdec_setup_memory_banksize() incorrectly
ignoring the "status" field. This patch fixes that by testing the status
with fdtdec_get_is_enabled() before using a memory node.

Signed-off-by: Jens Wiklander 
Reviewed-by: Simon Glass