[EXT] Re: [PATCH v3 2/2] dt-bindings: media: imx-jpeg: Assign slot for imx jpeg encoder/decoder

Krzysztof Kozlowski krzysztof.kozlowski at linaro.org
Mon Sep 25 01:09:55 PDT 2023


On 25/09/2023 10:07, Ming Qian wrote:
>> From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
>> Sent: 2023年9月25日 16:02
>> To: Ming Qian <ming.qian at nxp.com>; Mirela Rabulea (OSS)
>> <mirela.rabulea at oss.nxp.com>; robh+dt at kernel.org; shawnguo at kernel.org
>> Cc: krzysztof.kozlowski+dt at linaro.org; conor+dt at kernel.org;
>> mchehab at kernel.org; hverkuil-cisco at xs4all.nl; s.hauer at pengutronix.de;
>> kernel at pengutronix.de; festevam at gmail.com; X.H. Bao
>> <xiahong.bao at nxp.com>; Eagle Zhou <eagle.zhou at nxp.com>; Tao Jiang
>> <tao.jiang_2 at nxp.com>; dl-linux-imx <linux-imx at nxp.com>;
>> devicetree at vger.kernel.org; linux-media at vger.kernel.org; linux-
>> kernel at vger.kernel.org; linux-arm-kernel at lists.infradead.org
>> Subject: [EXT] Re: [PATCH v3 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 25/09/2023 09:41, 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.
>>
>> No changelog and I do not see much of improvements. Do not ignore the
>> feedback but respond to it or fix the patch. Really, what happened here?
>>
> 
> Hi Krzysztof,
>    What do you mean wrap commit message? Do you mean line wrapped at 75 columns?
> 
>   - The body of the explanation, line wrapped at 75 columns, which will
>     be copied to the permanent changelog to describe this patch.

I gave you clear link to the doc explaining this. What is unclear there?

And you decide to ignore all the rest of the comments here as well?

Best regards,
Krzysztof




More information about the linux-arm-kernel mailing list