Re: [PATCH v2 1/5] ARM: memory: da8xx-ddrctl: new driver

From: Sudeep Holla
Date: Mon Nov 21 2016 - 12:57:04 EST


Hi Robin,

On 21/11/16 17:47, Robin Murphy wrote:
Hi Bartosz, Sekhar,

On 21/11/16 16:48, Bartosz Golaszewski wrote:

[...]

Hi Sekhar,

thanks for spotting that.

I think we should introduce this function right away, rather than
having two static functions doing the same thing. If you don't mind,
I'll try to find a good spot for it and send a follow-up series fixing
the issue.

As it happens, that was already proposed last week, for much the same
reason:

http://www.mail-archive.com/linuxppc-dev@xxxxxxxxxxxxxxxx/msg111395.html


Thanks for pointing this out, yet another platform to move to the new
API after v4.10.

Hi Shekar, Bartosz,

For v4.10, please continue with the open coding as proposed in this
thread in order to avoid cross tree dependencies. I will rework on the
above patch once v4.10 merge window closes to include all these
occurrence and replace them.

--
Regards,
Sudeep