u-boot.git/drivers/core, branch v2018.01

dm: core: Correct address cast in dev_read_addr_ptr()

2017-11-17T01:45:05+00:00

This currently causes a warning in sandbox and will not do the right
thing:

drivers/core/read.c: In function ‘dev_read_addr_ptr’:
drivers/core/read.c:64:44: warning: cast to pointer from integer of
	different size [-Wint-to-pointer-cast]
  return (addr == FDT_ADDR_T_NONE) ? NULL : (void *)addr;

Use map_sysmem() which is the correct way to convert an address to a
pointer.

Fixes: c131c8bca8 (dm: core: add dev_read_addr_ptr())
Signed-off-by: Simon Glass 
Reviewed-by: Philipp Tomsich 
Reviewed-by: Bin Meng 
Tested-by: Bin Meng 
Acked-by: Philipp Tomsich

simple-bus: remove DECLARE_GLOBAL_DATA_PTR

2017-10-23T21:25:40+00:00

No global pointer is used in this file.

Signed-off-by: Masahiro Yamada 
Reviewed-by: Bin Meng

dm: ofnode: query correct property in livetree ofnode_get_addr_size

2017-10-09T02:41:09+00:00

The livetree codepath of ofnode_get_addr_size always used the "reg"
property for of_get_property. Use the property parameter of the function
call instead and check the return value if the property exists.
Otherwise return FDT_ADDR_T_NONE.

This was discoverd while using SPI NOR with livetree.
spi_flash_decode_fdt checks for memory-map and will not fail with
livetree even if the property does not exist.

Signed-off-by: Klaus Goger 
Reviewed-by: Philipp Tomsich 
Reviewed-by: Simon Glass

dm: core: Round up size when allocating so that it is cache line aligned

2017-10-09T02:41:09+00:00

The size variable may not be always be a mulitple of
ARCH_DMA_MINALIGN and using it to flush cache leads to cache
misaligned warnings.

Therefore, round up the size to a multiple of ARCH_DMA_MINLAIGN
when allocating private data.

Signed-off-by: Faiz Abbas 
Reviewed-by: Simon Glass

dm: replace dm_dbg() with pr_debug()

2017-10-04T16:00:21+00:00

As we discussed before in ML, dm_dbg() causes undefined reference
error if #define DEBUG is added to users, but not drivers/core/util.c

We do not need this macro because we can use pr_debug() instead, and
it is pretty easy to enable it for the DM core by using ccflags-y.

Signed-off-by: Masahiro Yamada

dm: core: Add functions to get strings and the string count from a stringlist

2017-09-22T14:23:54+00:00

dev_read_string_count() is used to get the number of strings in a
stringlist.
dev_read_string_index() is used to get a string in the stringlist based on
its position in the list.

Signed-off-by: Jean-Jacques Hiblot

dm: core: add dev_read_addr_ptr()

2017-09-18T18:40:33+00:00

The dev_read_addr_ptr() mimics the behaviour of the devfdt_get_addr_ptr(),
retrieving the first address of the node's reg-property and returning
it as a pointer (or NULL on failure).

Signed-off-by: Philipp Tomsich 
Acked-by: Philipp Tomsich 
Reviewed-by: Simon Glass

dtoc: Add support for 32 or 64-bit addresses

2017-09-15T11:27:38+00:00

When using 32-bit addresses dtoc works correctly. For 64-bit addresses it
does not since it ignores the #address-cells and #size-cells properties.

Update the tool to use fdt64_t as the element type for reg properties when
either the address or size is larger than one cell. Use the correct value
so that C code can obtain the information from the device tree easily.

Alos create a new type, fdt_val_t, which is defined to either fdt32_t or
fdt64_t depending on the word size of the machine. This type corresponds
to fdt_addr_t and fdt_size_t. Unfortunately we cannot just use those types
since they are defined to phys_addr_t and phys_size_t which use
'unsigned long' in the 32-bit case, rather than 'unsigned int'.

Add tests for the four combinations of address and size values (32/32,
64/64, 32/64, 64/32). Also update existing uses for rk3399 and rk3368
which now need to use the new fdt_val_t type.

Signed-off-by: Simon Glass 

Suggested-by: Heiko Stuebner 
Reported-by: Kever Yang 
Reviewed-by: Philipp Tomsich 
Tested-by: Kever Yang

dm: core: add clocks node scan

2017-09-12T03:43:58+00:00

Currently, all fixed-clock declared in "clocks" node in device tree
can be binded by clk_fixed_rate.c driver only if each of them have
the "simple-bus" compatible string.
This constraint has been invoked here [1].

This patch offers a solution to avoid adding "simple-bus" compatible
string to nodes that are not busses.

[1] https://patchwork.ozlabs.org/patch/558837/

Signed-off-by: Patrice Chotard 
Reviewed-by: Simon Glass

dm: core: Add ofnode_for_each_subnode()

2017-09-12T03:43:58+00:00

Add a convenience macro to iterate over subnodes of a node. Make use of
this where appropriate in the code.

Signed-off-by: Simon Glass