Re: Any pending patches on me for Samsung SoC? Closing applying cycle

[Krzysztof Kozlowski <krzk@kernel.org>]

On 06/03/2025 19:37, Kaustabh Chakraborty wrote:
> On 2025-03-06 18:07, Krzysztof Kozlowski wrote:
>> On 06/03/2025 15:03, Kaustabh Chakraborty wrote:
>>> On 2025-03-05 19:45, Krzysztof Kozlowski wrote:
>>>> Hey,
>>>>
>>>> I went through my inbox and cleared my queue because I am approaching
>>>> end of applying cycle. I think I dealt with everything and pending
>>>> things are:
>>>>
>>>> 1. Exynos7870 SoC: waits for bindings,
>>>
>>> Is this possible to be merged? [1]
>>> It's over a month old at this point, seemingly left untouched.
>>
>> I don't have it in my inbox for some reason. Regardless, you want it to
>> be a binding, so is it going to be used also by U-Boot?
> 
> Don't have any plans with u-boot at the moment, though I'll likely consider
> it later. The macros are used with syscon-reboot-mode which helps switching
> modes from userland.

Exynos Auto added first binding for that, but I am not sure if we need
to keep going. Binding describes the interface, so for whom do you
describe it here? I can imagine downstream bootloader, but that is I
don't believe it is worth adding bindings for downstream.

If you it is not for downstream then it has to be for some other
upstream project. Which one?


> 
>>
>>>
>>> This one's also a build dependency for [2], as mentioned in its cover
>>> letter.
>>>
>>> [1] https://lore.kernel.org/all/20250204-exynos7870-bootmode-v1-1-0f17b3033c2d@disroot.org/
>>> [2] https://lore.kernel.org/all/174100756283.1496281.12555971793250210333.robh@kernel.org/
>>
>> Then it should not have been sent separately. Patchset should be split
>> by subsystem, so this belongs to soc with the rest.
> 
> Hmm, getmaintainer didn't report it as part of the soc subsystem, although
> the file path does have 'soc' in it. And thus I decided to kept it
> separate.
Yes, that's a missing path.

Best regards,
Krzysztof