Hi Pali,

On 1/13/23 09:31, Pali Rohár wrote:

<snip>

Side note: Does your patch /sync only touch the currently not supported
DDR4 handling? If changes also include DDR3 then this should be tested
on the boards currently using this code very intensive. All A38x board
maintainers should be pointed to such new changes in this case.

I guess that new code only adds ifdefs for DDR4 which should not be
enabled nor compiled for existing boards. This should be safe.

Best would be, if we could make sure that this patch does not change
anything for non-DDR4 platforms. And if, then we should extract this
into a separate patch for the DDR3 stuff, that can be more easily
reviewed.

Thanks,
Stefan

Reply via email to