Re: [Patch v7] dt-bindings: media: s5p-mfc: convert bindings to json-schema

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

 



On 13/05/2023 16:44, Alim Akhtar wrote:
> Hi Krzysztof,
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>> Sent: Saturday, May 13, 2023 3:03 PM
>> To: Aakarsh Jain <aakarsh.jain@xxxxxxxxxxx>; linux-arm-
>> kernel@xxxxxxxxxxxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; linux-
>> kernel@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx
>> Cc: m.szyprowski@xxxxxxxxxxx; andrzej.hajda@xxxxxxxxx;
>> mchehab@xxxxxxxxxx; hverkuil-cisco@xxxxxxxxx;
>> ezequiel@xxxxxxxxxxxxxxxxxxxx; jernej.skrabec@xxxxxxxxx;
>> benjamin.gaignard@xxxxxxxxxxxxx; krzysztof.kozlowski+dt@xxxxxxxxxx;
>> stanimir.varbanov@xxxxxxxxxx; dillon.minfei@xxxxxxxxx;
>> david.plowman@xxxxxxxxxxxxxxx; mark.rutland@xxxxxxx;
>> robh+dt@xxxxxxxxxx; krzk+dt@xxxxxxxxxx; andi@xxxxxxxxxxx;
>> alim.akhtar@xxxxxxxxxxx; aswani.reddy@xxxxxxxxxxx;
>> pankaj.dubey@xxxxxxxxxxx
>> Subject: Re: [Patch v7] dt-bindings: media: s5p-mfc: convert bindings to json-
>> schema
>>
>> On 11/05/2023 11:34, Aakarsh Jain wrote:
>>>
>>>
>>>> -----Original Message-----
>>>> From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@xxxxxxxxxx]
>>>> Sent: 28 March 2023 20:06
>>>> To: Aakarsh Jain <aakarsh.jain@xxxxxxxxxxx>; linux-arm-
>>>> kernel@xxxxxxxxxxxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; linux-
>>>> kernel@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx
>>>> Cc: m.szyprowski@xxxxxxxxxxx; andrzej.hajda@xxxxxxxxx;
>>>> mchehab@xxxxxxxxxx; hverkuil-cisco@xxxxxxxxx;
>>>> ezequiel@xxxxxxxxxxxxxxxxxxxx; jernej.skrabec@xxxxxxxxx;
>>>> benjamin.gaignard@xxxxxxxxxxxxx; krzysztof.kozlowski+dt@xxxxxxxxxx;
>>>> stanimir.varbanov@xxxxxxxxxx; dillon.minfei@xxxxxxxxx;
>>>> david.plowman@xxxxxxxxxxxxxxx; mark.rutland@xxxxxxx;
>>>> robh+dt@xxxxxxxxxx; krzk+dt@xxxxxxxxxx; andi@xxxxxxxxxxx;
>>>> alim.akhtar@xxxxxxxxxxx; aswani.reddy@xxxxxxxxxxx;
>>>> pankaj.dubey@xxxxxxxxxxx
>>>> Subject: Re: [Patch v7] dt-bindings: media: s5p-mfc: convert bindings
>>>> to json- schema
>>>>
>>>> On 28/03/2023 13:47, Aakarsh Jain wrote:
>>>>> Convert s5p-mfc bindings to DT schema format using json-schema.
>>>>>
>>>>> Signed-off-by: Aakarsh Jain <aakarsh.jain@xxxxxxxxxxx>
>>>>> ---
>>>>> changes since v6:
>>>>
>>>> This is a friendly reminder during the review process.
>>>>
>>>> It looks like you received a tag and forgot to add it.
>>>>
>>>> If you do not know the process, here is a short explanation:
>>>> Please add Acked-by/Reviewed-by/Tested-by tags when posting new
>>>> versions. However, there's no need to repost patches *only* to add
>>>> the tags. The upstream maintainer will do that for acks received on
>>>> the version they apply.
>>>>
>>>> https://protect2.fireeye.com/v1/url?k=03601d03-62eb0848-0361964c-
>>>> 74fe485fb305-ca0023b5279dd925&q=1&e=9490b51d-9547-4566-bb76-
>>>>
>> 8c1401745ae1&u=https%3A%2F%2Felixir.bootlin.com%2Flinux%2Fv5.17%2Fs
>>>> ource%2FDocumentation%2Fprocess%2Fsubmitting-patches.rst%23L540
>>>>
>>>> If a tag was not added on purpose, please state why and what changed.
>>>>
>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>>>>
>>>> Best regards,
>>>> Krzysztof
>>>
>>>
>>> Hi Krzysztof,
>>>
>>> I don’t see this patch in linux-next. It's been more than a month now.
>>> Please let me know if any other changes required.
>>
>> I already provided you review tag.
> Thanks for your review.
> The question was more on, if you will pick up this patch via your tree or this will go via Rob's or Media tree.
> I will prefer if this goes via your tree.

Generic recommendation is that all device-related bindings go via their
subsystem, so in this case via media tree. I see in patchwork it is
still in new state, so maybe was forgotten. OK, let me grab it.

Best regards,
Krzysztof




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux