Re: [PATCH v2 3/6] ARM: dts: NSP: Fix SP804 compatible node

From: Florian Fainelli
Date: Tue Sep 01 2020 - 19:04:23 EST




On 8/28/2020 10:12 AM, Florian Fainelli wrote:
On 8/28/20 7:20 AM, Andre Przywara wrote:
The DT binding for SP804 requires to have an "arm,primecell" compatible
string.
Add this string so that the Linux primecell bus driver picks the device
up and activates the clock.

Fixes: a0efb0d28b77 ("ARM: dts: NSP: Add SP804 Support to DT")
Tested-by: Florian Fainelli <f.fainelli@xxxxxxxxx>
Signed-off-by: Andre Przywara <andre.przywara@xxxxxxx>

This looks fine, however there is a ccbtimer1 instance that you missed,
can you resubmit with it included?

With that:

Acked-by: Florian Fainelli <f.fainelli@xxxxxxxxx>

Andre are you going to resubmit a patch with the second instance (ccbtimer1) fixed as well, or should I take care of that while applying the patch? Either way is fine, just let me know.
--
Florian