[PATCH v8 07/12] mach-snapdragon: Kconfig: changes / additions to support SPL

[Michael Srba]

Hi,

On 5/18/26 22:58, Tom Rini wrote:
> On Mon, May 18, 2026 at 10:25:05PM +0200, Michael Srba wrote:
>> Hi,
>>
>> On 5/18/26 10:40, Casey Connolly wrote:
>>> Hi Michael,
>>>
>>> On 16/05/2026 20:19, michael.srba at seznam.cz wrote:
>>>> From: Michael Srba <Michael.Srba at seznam.cz>
>>>>
>>>> Select SUPPORT_SPL so SPL build can be enabled, disable SYSRESET_PSCI in SPL.
>>>> (SPL runs in EL3, so if SPL itself doesn't provide PSCI, nothing else will.)
>>>>
>>>> Also select (SPL_)OF_LIVE and DM_EVENT/SPL_EVENT, which are needed to fix up
>>>> upstream dt to make usb work.
>>>>
>>>> Mirror u-boot proper selections like GPIO and pinctrl to ensure consistent
>>>> behavior, and select SPL_SPRINTF, SPL_LIBCOMMON_SUPPORT etc for similar
>>>> reasons.
>>>>
>>>> Signed-off-by: Michael Srba <Michael.Srba at seznam.cz>
>>>> Reviewed-by: Simon Glass <sjg at chromium.org>
>>>> ---
>>>>    arch/arm/Kconfig                 | 33 ++++++++++++++++++++++++++++++++-
>>>>    arch/arm/mach-snapdragon/Kconfig | 10 ++++++++++
>>>>    2 files changed, 42 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
>>>> index 850768282ce..255d2035cdc 100644
>>>> --- a/arch/arm/Kconfig
>>>> +++ b/arch/arm/Kconfig
>>>> @@ -1136,6 +1136,7 @@ config ARCH_SNAPDRAGON
>>>>    	bool "Qualcomm Snapdragon SoCs"
>>>>    	select ARM64
>>>>    	select DM
>>>> +	select DM_EVENT if USB_DWC3_GENERIC
>>> Is there any reason not to make this uncoditional?
>> Well, my understanding is that it's only relevant if something registers
>> to receive an event. Maybe some stuff just has a slower path but would
>> benefit from this? I don't know, all I know is that the fixup has a hard
>> dependency on it.
> This seems like a USB_DWC3_GENERIC bug (missing select DM_EVENT), but
> not something you need to tackle as this is just following the existing
> way.
>
>
I added it because of the fixup that mach-snapdragon code does, which only
needs to actually work if dwc3 is enabled (because otherwise clearly usb
support is not a concern), I'm not sure if USB_DWC3_GENERIC itself needs it.