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@xxxxxxxxxxx/ >>> [2] https://lore.kernel.org/all/174100756283.1496281.12555971793250210333.robh@xxxxxxxxxx/ >> >> 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
