[U-Boot] [PATCH v1 1/4] arm: socfpga: imply SPL config instead of select

[Simon Goldschmidt]

Am 07.01.2019 um 23:53 schrieb Marek Vasut:
> On 1/7/19 10:14 PM, Simon Goldschmidt wrote:
>> In order to build a smaller SPL, let's imply SPL_DM_RESET and
>> SPL_WATCHDOG_SUPPORT instead of selecting them, so they can be disabled
>> via defconfig.
>>
>> This also seems to be required to use OF_PLATDATA, as the reset drivers
>> don't seem to work with it.
> 
> How do you un-reset IP blocks if you disable the reset controller ?

I found that out just now: there's the function 
'reset_deassert_peripherals_handoff()' in spl_gen5.c that should 
"De-assert reset for peripherals and bridges based on handoff". However, 
at least for Gen5, it just writes a 0 to rstmgr->permodrst. By doing 
that, it enables *ALL* peripherals on the SoC (except for some DMA 
channels that aren't really used) :-)

I guess that needs some cleaning up as well ;-)

I think the proper thing to do here would be to remove this function and 
convert all drivers to provide appropriate 'resets' properties in the dts?

Regards,
Simon