On 06/02/2017 13:47, Tom Rini wrote:
On Mon, Feb 06, 2017 at 12:09:12PM +0100, Jean-Jacques Hiblot wrote:

On 03/02/2017 17:52, Tom Rini wrote:
On Wed, Feb 01, 2017 at 11:53:42AM +0100, Jean-Jacques Hiblot wrote:

Add qspi control module register maps to device tree. This register map had
been removed by error when syncing the DTS with the latest linux at the
time (commit 7aa1a4087 "ARM: dts: dra7xx: sync DT with latest Linux").
Without this entry the driver do not support memory-mapped access and the
data read from the SF is invalid.

Signed-off-by: Jean-Jacques Hiblot <[email protected]>
Why is this not in the upstream DTS file?
In Linux, a syscon/regmap is  used to map this register while the
QSPI driver in u-boot uses a direct memory access.
OK, but it's a valid part of the upstream kernel binding, yes?
syscon-chipselects is part of the mainlined binding. Using a 3rd memory range used to be the way to do it, but it is not supported anymore (I think it has been removed removed in 4.6).
Using a third memory range has been supported in u-boot for some time now.
So you think we should move to using the syscon ?
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to