Hi Bin,
On Fri, Nov 12, 2021 at 6:58 AM Bin Meng bmeng.cn@gmail.com wrote:
On Thu, Nov 11, 2021 at 9:17 PM Ivan.Griffin@microchip.com wrote:
I agree with Bin here. You shouldn't introduce a new compatible just
for
u-boot. If you need one, please to it first in linux and get an ACK
there.
Or at least there should be patches for it pending in linux and it
should
be likely, that they will be accepted.
Please work towards having one binding for u-boot and linux.
-michael
I think both Michael and Bin are right, but that maybe this has gone
circular.
IIRC, Linux *doesn't need* any extra bindings because its driver already supports 64-bit DMA.
Padmarao's original patch added equivalent 64-bit functionality to the driver in U-Boot, but this was rejected.
I am not sure why it was rejected. Is that because it breaks some other platforms?
No
Instead I think the suggestion was to add a device-tree binding to
choose 32 or
64-bit DMA... however, there is no reasonably way of upstreaming this
into
the Linux device-tree, as Linux doesn't need it... so he is left in a
Catch-22.
A way forward may be to go back to his original approach and get the
U-Boot
driver functionality updated so that it works similarly to the Linux
driver
(and thus can use the same device-tree stanza)?
Let's go back to the original approach and see what happens.
Ok, I will do that and submit it in Patch v2.
Regards Padmarao
Regards, Bin