On 25/09/2023 10:13, Ming Qian wrote: >> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> >> Sent: 2023年9月25日 16:10 >> To: Ming Qian <ming.qian@xxxxxxx>; Mirela Rabulea (OSS) >> <mirela.rabulea@xxxxxxxxxxx>; robh+dt@xxxxxxxxxx; shawnguo@xxxxxxxxxx >> Cc: krzysztof.kozlowski+dt@xxxxxxxxxx; conor+dt@xxxxxxxxxx; >> mchehab@xxxxxxxxxx; hverkuil-cisco@xxxxxxxxx; s.hauer@xxxxxxxxxxxxxx; >> kernel@xxxxxxxxxxxxxx; festevam@xxxxxxxxx; X.H. Bao >> <xiahong.bao@xxxxxxx>; Eagle Zhou <eagle.zhou@xxxxxxx>; Tao Jiang >> <tao.jiang_2@xxxxxxx>; dl-linux-imx <linux-imx@xxxxxxx>; >> devicetree@xxxxxxxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; linux- >> kernel@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx >> 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@xxxxxxxxxx> >>>> Sent: 2023年9月25日 16:02 >>>> To: Ming Qian <ming.qian@xxxxxxx>; Mirela Rabulea (OSS) >>>> <mirela.rabulea@xxxxxxxxxxx>; robh+dt@xxxxxxxxxx; >> shawnguo@xxxxxxxxxx >>>> Cc: krzysztof.kozlowski+dt@xxxxxxxxxx; conor+dt@xxxxxxxxxx; >>>> mchehab@xxxxxxxxxx; hverkuil-cisco@xxxxxxxxx; s.hauer@xxxxxxxxxxxxxx; >>>> kernel@xxxxxxxxxxxxxx; festevam@xxxxxxxxx; X.H. Bao >>>> <xiahong.bao@xxxxxxx>; Eagle Zhou <eagle.zhou@xxxxxxx>; Tao Jiang >>>> <tao.jiang_2@xxxxxxx>; dl-linux-imx <linux-imx@xxxxxxx>; >>>> devicetree@xxxxxxxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; linux- >>>> kernel@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx >>>> 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