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

[Rob Herring]

On Mon, Sep 25, 2023 at 08:48:01AM +0000, Ming Qian wrote:
> >From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> >Sent: 2023年9月25日 16:33
> >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: Re: [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 10:20, Ming Qian wrote:
> >>> From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> >>> Sent: 2023年9月25日 16:14
> >>> 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: Re: [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 10:13, Ming Qian wrote:
> >>>>> From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> >>>>> Sent: 2023年9月25日 16:10
> >>>>> 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: Re: [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 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?
> >>>
> >>> So this is still ignored? Can you respond to all my comments?
> >>>
> >>>>>
> >>>>> Best regards,
> >>>>> Krzysztof
> >>>>
> >>>> The link content is as below:
> >>>>
> >>>>   - The body of the explanation, line wrapped at 75 columns, which will
> >>>>     be copied to the permanent changelog to describe this patch.
> >>>>
> >>>> I'm sorry that I didn’t get your point.
> >>>
> >>> Wrap
> >>> your
> >>> commit
> >>> according
> >>> to
> >>> coding
> >>> style.
> >>>
> >>> Best regards,
> >>> Krzysztof
> >>
> >> Hi Krzysztof,
> >>     I opened your link about the Linux coding style / submission process, but I
> >only find the following message:
> >>
> >>   - The body of the explanation, line wrapped at 75 columns, which will
> >>     be copied to the permanent changelog to describe this patch.
> >>
> >>      So I don't know what do you mean the coding style.
> >
> >
> >Linked document explains what is a patch, how it is formatted. Just read
> >entire doc, not one sentence.
> >
> >You keep ignoring my other comments all the time, so I will stop asking you to
> >respond to them. Instead I will be NAK-ing your patches till you decide to
> >respond.
> >
> >Best regards,
> >Krzysztof
> 
> Hi Krzysztof,
>     I'm sorry that I make you unhappy, as I didn't know how to improve the commit message.
> And your other comments about the change, I get your point, and prepare to improve in the next version.
> 
> Sorry again that I didn't response your every comments, I will avoid doing it again.
> 
> And can you give a example how to improve the commit message.

'git log'

There are 10K+ examples every 2 months.

Please talk to your NXP colleagues experienced with upsteam about how to 
write and submit patches upstream, and explain what Krzysztof has said 
if it is not clear.

Rob