[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <97d9ccf9-71f8-c97d-ce56-8aeb1a3db194@linaro.org>
Date: Tue, 5 Jul 2022 14:08:52 +0200
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Smitha T Murthy <smitha.t@...sung.com>,
linux-arm-kernel@...ts.infradead.org, linux-media@...r.kernel.org,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org
Cc: m.szyprowski@...sung.com, andrzej.hajda@...el.com,
mchehab@...nel.org, hverkuil-cisco@...all.nl,
ezequiel@...guardiasur.com.ar, jernej.skrabec@...il.com,
benjamin.gaignard@...labora.com, stanimir.varbanov@...aro.org,
dillon.minfei@...il.com, david.plowman@...pberrypi.com,
mark.rutland@....com, robh+dt@...nel.org, krzk+dt@...nel.org,
andi@...zian.org, alim.akhtar@...sung.com,
aswani.reddy@...sung.com, pankaj.dubey@...sung.com,
linux-fsd@...la.com
Subject: Re: [PATCH 02/20] dt-bindings: media: s5p-mfc: Convert s5p-mfc.txt to
new DT schema
On 05/07/2022 13:44, Smitha T Murthy wrote:
>
>
>> -----Original Message-----
>> From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@...aro.org]
>> Sent: Tuesday, May 17, 2022 7:26 PM
>> To: Smitha T Murthy <smitha.t@...sung.com>; linux-arm-
>> kernel@...ts.infradead.org; linux-media@...r.kernel.org; linux-
>> kernel@...r.kernel.org; devicetree@...r.kernel.org
>> Cc: m.szyprowski@...sung.com; andrzej.hajda@...el.com;
>> mchehab@...nel.org; hverkuil-cisco@...all.nl;
>> ezequiel@...guardiasur.com.ar; jernej.skrabec@...il.com;
>> benjamin.gaignard@...labora.com; stanimir.varbanov@...aro.org;
>> dillon.minfei@...il.com; david.plowman@...pberrypi.com;
>> mark.rutland@....com; robh+dt@...nel.org; krzk+dt@...nel.org;
>> andi@...zian.org; alim.akhtar@...sung.com; aswani.reddy@...sung.com;
>> pankaj.dubey@...sung.com; linux-fsd@...la.com
>> Subject: Re: [PATCH 02/20] dt-bindings: media: s5p-mfc: Convert s5p-mfc.txt
>> to new DT schema
>>
>> On 17/05/2022 14:55, Smitha T Murthy wrote:
>>> Adds DT schema for s5p-mfc in yaml format.
>>>
>>
>> Thank you for your patch. There is something to discuss/improve.
>>
>
> Thank you for the review.
>
You responded after two months, I don't remember what I reviewed... Two
months periods between resends do not really help to usptream.
>
>>> + compatible = "samsung,mfc-v12";
>>
>> Does not look like you tested the bindings. Please run `make
>> dt_binding_check` (see Documentation/devicetree/bindings/writing-
>> schema.rst for instructions).
>> Be sure to test your bindings before sending them.
>>
>
> I did do make dtbs and dt_binding_check using v2022.3, I will recheck post these changes.
>
>>> + reg = <0x12880000 0x10000>;
>>> + clock-names = "mfc";
>>> + interrupts = <0 137 4>;
>>
>> Use interrupt defines.
>>
>
> When I use interrupt defines I get errors as "1. Syntax error: This was due to interrupts field has some macro reference and needed to give absolute value.", hence I gave absolute values.
Look at other DT schema files...
Best regards,
Krzysztof
Powered by blists - more mailing lists