Re: [PATCH V7 3/5] dt-bindings: iio: adc: Add support for QCOM PMIC5 Gen3 ADC

[Krzysztof Kozlowski <krzk@kernel.org>]

On 20/10/2025 14:51, Konrad Dybcio wrote:
> On 10/17/25 3:40 PM, Krzysztof Kozlowski wrote:
>> On 17/10/2025 13:18, Jishnu Prakash wrote:
>>> Hi Krzysztof,
>>>
>>> On 10/9/2025 5:22 AM, Krzysztof Kozlowski wrote:
>>>> On 08/10/2025 23:20, Jishnu Prakash wrote:
>>>>> Hi Krzysztof,
>>>>>
>>>>> On 10/4/2025 12:22 PM, Krzysztof Kozlowski wrote:
>>>>>> On Sat, 4 Oct 2025 at 11:42, Jishnu Prakash
>>>>>> <jishnu.prakash@oss.qualcomm.com> wrote:
>>>>>>>
>>>>>>> Hi Jonathan,
>>>>>>>
>>>>>>> On 9/27/2025 7:17 PM, Jonathan Cameron wrote:
>>>>>>>> On Fri, 19 Sep 2025 20:17:43 +0530
>>>>>>>> Jishnu Prakash <jishnu.prakash@oss.qualcomm.com> wrote:
> 
> [...]
> 
>>> Can you please provide your suggestions on changes we can make
>>> in the above points ?
>>
>> You just pasted DT. I asked about SW, software. Please read carefully
>> previous comments.
> 
> Is the problem that Jishnu included some indices in dt-bindings without
> also adding them in the driver's adc5_gen3_chans_pmic[] array?
> 
> As in, would the resolution to this thread be simply handling all of
> them in the driver correctly?

The solution is to remove them from the bindings, just like we do with
many other hardware constants. Of course if these are not hardware
constants, but part of ABI, then solution would be different but no one
provided proof or argument that this is any binding. All proofs were
"but I want to use it in my DTS", which proofs nothing. Not a binding.

While this issue is not that important, we keep discussing it because
author does not try to understand the problem or even keep up the
discussion. Instead repeats the same without really reading my
messages... and then disappears for month or more.

Best regards,
Krzysztof