[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <e243a285-156f-4947-8932-45f2a8e3ec93@linaro.org>
Date: Thu, 28 Sep 2023 07:06:56 +0200
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Ming Qian <ming.qian@....com>,
"Mirela Rabulea (OSS)" <mirela.rabulea@....nxp.com>,
"robh+dt@...nel.org" <robh+dt@...nel.org>,
"shawnguo@...nel.org" <shawnguo@...nel.org>
Cc: "krzysztof.kozlowski+dt@...aro.org"
<krzysztof.kozlowski+dt@...aro.org>,
"conor+dt@...nel.org" <conor+dt@...nel.org>,
"mchehab@...nel.org" <mchehab@...nel.org>,
"hverkuil-cisco@...all.nl" <hverkuil-cisco@...all.nl>,
"s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
"kernel@...gutronix.de" <kernel@...gutronix.de>,
"festevam@...il.com" <festevam@...il.com>,
"X.H. Bao" <xiahong.bao@....com>, Eagle Zhou <eagle.zhou@....com>,
Tao Jiang <tao.jiang_2@....com>,
dl-linux-imx <linux-imx@....com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [EXT] Re: [PATCH v4 2/2] dt-bindings: media: imx-jpeg: Assign
slot for imx jpeg encoder/decoder
On 27/09/2023 11:10, Ming Qian wrote:
>> From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>> Sent: 2023年9月27日 16:06
>> To: Ming Qian <ming.qian@....com>; Mirela Rabulea (OSS)
>> <mirela.rabulea@....nxp.com>; robh+dt@...nel.org; shawnguo@...nel.org
>> Cc: krzysztof.kozlowski+dt@...aro.org; conor+dt@...nel.org;
>> mchehab@...nel.org; hverkuil-cisco@...all.nl; s.hauer@...gutronix.de;
>> kernel@...gutronix.de; festevam@...il.com; X.H. Bao
>> <xiahong.bao@....com>; Eagle Zhou <eagle.zhou@....com>; Tao Jiang
>> <tao.jiang_2@....com>; dl-linux-imx <linux-imx@....com>;
>> devicetree@...r.kernel.org; linux-media@...r.kernel.org; linux-
>> kernel@...r.kernel.org; linux-arm-kernel@...ts.infradead.org
>> Subject: [EXT] Re: [PATCH v4 2/2] dt-bindings: media: imx-jpeg: Assign slot for
>> imx jpeg encoder/decoder
>>
>> Caution: This is an external email. Please take care when clicking links or
>> opening attachments. When in doubt, report the message using the 'Report
>> this email' button
>>
>>
>> On 26/09/2023 12:10, Ming Qian wrote:
>>> There are total 4 slots available in the IP, and we only need to use
>>> one slot in one os, assign a single slot, configure interrupt and
>>> power domain only for 1 slot, not for the all 4 slots.
>>>
>>> Signed-off-by: Ming Qian <ming.qian@....com>
>>> ---
>>> v4
>>> - improve commit message
>>> - don't make an ABI break
>>> v3
>>> - add vender prefix, change property slot to nxp,slot
>>> - add type for property slot
>>>
>>> .../bindings/media/nxp,imx8-jpeg.yaml | 45 +++++++++----------
>>> 1 file changed, 21 insertions(+), 24 deletions(-)
>>>
>>> diff --git
>>> a/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
>>> b/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
>>> index 3d9d1db37040..4bcfc815c894 100644
>>> --- a/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
>>> +++ b/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
>>> @@ -32,19 +32,26 @@ properties:
>>> maxItems: 1
>>>
>>> interrupts:
>>> - description: |
>>> - There are 4 slots available in the IP, which the driver may use
>>> - If a certain slot is used, it should have an associated interrupt
>>> - The interrupt with index i is assumed to be for slot i
>>> - minItems: 1 # At least one slot is needed by the driver
>>> - maxItems: 4 # The IP has 4 slots available for use
>>> + description:
>>> + Interrupt number for slot
>>> + maxItems: 1
>>>
>>> power-domains:
>>> description:
>>> List of phandle and PM domain specifier as documented in
>>> Documentation/devicetree/bindings/power/power_domain.txt
>>> - minItems: 2 # Wrapper and 1 slot
>>> - maxItems: 5 # Wrapper and 4 slots
>>> + minItems: 1 # VPUMIX
>>> + maxItems: 2 # Wrapper and 1 slot
>>> +
>>> + nxp,slot:
>>> + $ref: /schemas/types.yaml#/definitions/uint32
>>> + description:
>>> + Integer number of slot index used. There are 4 slots available in the IP,
>>> + and driver can use a certain slot, it should have an associated interrupt
>>> + and power-domain. In theory, it supports 4 os or vm. If not specified, 0
>>> + is used by default.
>>> + minimum: 0
>>> + maximum: 3
>>
>> NAK, you still did not answer concerns why this is needed and justified.
>> I keep asking and you keep ignoring...
>>
>> Best regards,
>> Krzysztof
>
> Hi Krzysztof,
>
>> Nothing explains what is a slot and nothing explains why do you need this property.
>
> I thought I had answered that question, but seems you don't agree with that explanation.
> Would the following description be any better?
No, you didn't. I don't see any explanation in the commit msg. You just
say you have "4 slots".
Best regards,
Krzysztof
Powered by blists - more mailing lists