Re: [PATCH] Revert "Revert "dt-bindings: i2c: qcom-cci: Document sc8280xp compatible""

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/04/2024 12:16, Krzysztof Kozlowski wrote:
> On 11/04/2024 12:12, Bryan O'Donoghue wrote:
>> On 11/04/2024 10:36, Krzysztof Kozlowski wrote:
>>> On 11/04/2024 10:52, Vladimir Zapolskiy wrote:
>>>> This reverts commit 3e383dce513f426b7d79c0e6f8afe5d22a581f58.
>>>>
>>>> The commit ae2a1f0f2cb5 ("dt-bindings: i2c: qcom-cci: Document sc8280xp compatible")
>>>> was correct apparently, it is required to describe the sc8280xp-cci
>>>> controller properly, as well it eliminates dtbs_check warnings.
>>>>
>>>> Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@xxxxxxxxxx>
>>>
>>> I am lost. Not on your patch, because it looks reasonable, but on entire
>>> history.
>>>
>>> Can anyone explain me why original commit was reverted?
>>>
>>> https://lore.kernel.org/all/767bc246-a0a0-4dad-badc-81ed50573832@xxxxxxxxxx/
>>>
>>> Best regards,
>>> Krzysztof
>>>
>>
>> https://patchwork.ozlabs.org/project/devicetree-bindings/cover/20231006120159.3413789-1-bryan.odonoghue@xxxxxxxxxx/#3195094
>>
>> We can you sm8250-cci instead, so dropped the additional compat.
> 
> I am sorry, but that links point to cover letter and actually the same
> thread as I linked. What does it prove?
> 

And just to remind because you bring some discussions from driver: we
talk here *only* about bindings patch. Not driver.

Best regards,
Krzysztof





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux