On 12/02/2025 07:24, Satya Priya Kakitapalli wrote: > > On 2/11/2025 5:20 PM, Krzysztof Kozlowski wrote: >> On 11/02/2025 12:46, Krzysztof Kozlowski wrote: >>> On 11/02/2025 12:15, Satya Priya Kakitapalli wrote: >>>> On 12/13/2024 2:08 PM, Krzysztof Kozlowski wrote: >>>>> On Thu, Dec 12, 2024 at 09:41:20PM +0530, Satya Priya Kakitapalli wrote: >>>>>> + >>>>>> +required: >>>>>> + - compatible >>>>>> + - reg >>>>>> + - interrupts >>>>>> + - io-channels >>>>>> + - io-channel-names >>>>> Binding looks ok, but this wasn't tested due to unneeded dependency. >>>>> Please decouple from dependency, so automation can properly test it. >>>> >>>> The dependency is needed because this mbg peripheral is present on only >>>> targets which have GEN3 ADC5, for which the bindings support is added in >>>> the series [1] >>>> >>>> >>>> [1] >>>> https://lore.kernel.org/linux-arm-msm/c4ca0a4c-e421-4cf6-b073-8e9019400f4c@xxxxxxxxxxx/ >>> Sure. Then this cannot be merged due to resulting test failure. >>> >>> Please don't post new versions before this can be actually tested and >>> applied. >> Heh, you responded *after two months*, to an old email so even previous >> discussion is gone from my inbox. > > > Sorry about that, I initially misunderstood it was regarding the b4 deps > I created. I wanted to confirm before posting new version yesterday, > hence clarified about the dependency. Please correct me if I understood: After two months you responded dependency is needed. Dependency means this cannot be merged. You want to confirm if you can post new version. In that case depends on the dependency. If it is still there, it won't be possible to merge it, right? Or maybe it is not there, but it is not us who should guess it but you should tell us. Best regards, Krzysztof
