Re: Re: [PATCH 2/7] dt-bindings: add binding for the Allwinner A64 DE2 bus

于 2018年3月27日 GMT+08:00 上午6:22:58, Rob Herring [off-list ref] 写到:

On Thu, Mar 22, 2018 at 02:08:43PM +0800, Icenowy Zheng wrote:

quoted

于 2018年3月22日 GMT+08:00 上午1:13:42, "Jernej Škrabec"

[off-list ref] 写到:

quoted

quoted

Hi all,

Dne sreda, 21. marec 2018 ob 03:18:13 CET je Icenowy Zheng

napisal(a):

quoted

quoted

quoted

于 2018年3月21日 GMT+08:00 上午2:46:46, Maxime Ripard

[off-list ref] 
写到:

quoted

quoted

On Sat, Mar 17, 2018 at 01:53:49AM +0800, Icenowy Zheng wrote:

quoted

All the sub-blocks of Allwinner A64 DE2 needs the SRAM C on A64

SoC

quoted

quoted

to

quoted

be claimed, otherwise the whole DE2 space is inaccessible.

Add a device tree binding of the DE2 part as a sub-bus.

Where did you get the info that it was a bus?

There's no direct evidence, just some guess.

The DE2 is a whole part that is just allocated a memory
space at the user manual, and the SRAM controls the
access to all modules in the DE2.

So it might be a bus.

Implement it as a bus is a clear representation on A64.

Since there is already syscon for same mmio region, we migh as well

use

quoted

quoted

it 
when loading ccu-sun8i-de2 driver on A64.

Other options, like SRAM driver or bus driver, might better

represent

quoted

quoted

HW, but 

I think the device tree should properly represent the HW,
it's a basic requirment.

quoted

then we would have two DT nodes covering same mmio region, which I
think is 
not really acceptable.

It's acceptable, and DE2 is not the only user of SRAM controller so

far.

No, it's not acceptable. Don't create overlapping mmio regions in DT.

Then should the SRAM controller driver be configured to take the syscon?

quoted

VE will also need a SRAM region to be claimed.

quoted

Any suggestions?

BTW, H6 has same design in this regard.

Best regards,
Jernej

--
To unsubscribe from this list: send the line "unsubscribe devicetree"

in

quoted

the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

-- 
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.