Re: [PATCH v2 5/6] dt-bindings: dma: qcom,bam-dma: Increase iommus maxItems to seven

[Krzysztof Kozlowski <krzk@kernel.org>]

On 03/07/2026 11:01, Kuldeep Singh wrote:
> 
> 
> On 03-07-2026 14:27, Konrad Dybcio wrote:
>> On 7/3/26 10:38 AM, Kuldeep Singh wrote:
>>> On 03-07-2026 12:24, Krzysztof Kozlowski wrote:
>>>> On Thu, Jul 02, 2026 at 01:47:15AM +0530, Kuldeep Singh wrote:
>>>>> Upcoming Shikra BAM DMA uses 7 IOMMU entries and not 6, so increase the
>>>>> `iommus` maxItems constraint.
>>>>>
>>>>> Fix below error:
>>>>> dma-controller@1b04000 (qcom,bam-v1.7.4): iommus: [[25, 132, 17], [25,
>>>>
>>>> There is no dma-controller@1b04000 in DTS. Please drop all the warnings
>>>> which do not exist.
>>>
>>> Kindly check patch 6/6, it is introducing bam node with 7iommus which IP
>>> describes and hence, updated bindings before to accustom this which also
>>> helps in avoiding rob's dt-schema bot error.
>>
>> Krzysztof is saying that the error doesn't exist in the tree (because
>> the offending DTS hunk is not merged), so you shouldn't claim this fixes
>> an error, rather that Shikra simply needs it
> 
> In previous patchset, krzysztof ask was to add error in commit log and
> hence updated that.
> Maybe some misunderstanding!

From that answer you should have understood there are no warnings to be
fixed, no warnings to be mentioned, so that commit msg should have been
fixed.

Solution is not to add fake warnings in such case, like you did here.

Best regards,
Krzysztof