On 06/02/2017 14:37, Tom Rini wrote:
On Mon, Feb 06, 2017 at 02:27:42PM +0100, Jean-Jacques Hiblot wrote:
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 ?
Based on what I see in v4.10-rc7, yes, we should adapt the driver to
know about the new property and things are working as expected wrt DT
compatibility.
Ok. I have prepared a new series to do this
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot
