Linux-mediatek Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Crystal Guo (郭晶)" <Crystal.Guo@mediatek.com>
To: "robh@kernel.org" <robh@kernel.org>,
	"matthias.bgg@gmail.com" <matthias.bgg@gmail.com>,
	"conor+dt@kernel.org" <conor+dt@kernel.org>,
	"krzk@kernel.org" <krzk@kernel.org>,
	AngeloGioacchino Del Regno
	<angelogioacchino.delregno@collabora.com>
Cc: "linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-mediatek@lists.infradead.org"
	<linux-mediatek@lists.infradead.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Project_Global_Chrome_Upstream_Group
	<Project_Global_Chrome_Upstream_Group@mediatek.com>
Subject: Re: [v2,2/2] dt-bindings: memory-controllers: Add MediaTek DRAM controller interface
Date: Wed, 12 Feb 2025 10:07:47 +0000	[thread overview]
Message-ID: <468d079e89a5459dfe894402af10605c62676412.camel@mediatek.com> (raw)
In-Reply-To: <43c6bc3b-5dac-4925-99de-76764ea77424@kernel.org>

On Wed, 2025-02-12 at 06:34 +0100, Krzysztof Kozlowski wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> On 12/02/2025 04:50, Crystal Guo (郭晶) wrote:
> > On Sun, 2025-02-09 at 11:51 +0100, Krzysztof Kozlowski wrote:
> > > External email : Please do not click links or open attachments
> > > until
> > > you have verified the sender or the content.
> > > 
> > > 
> > > On 07/02/2025 02:42, Crystal Guo wrote:
> > > > A MediaTek DRAM controller interface to provide the current DDR
> > > > data rate.
> > 
> > Hi Krzysztof,
> > 
> > Regarding this commit message, you previously commented as follows:
> > "Please wrap commit message according to Linux coding style /
> > submission process (neither too early nor over the limit):
> > 
https://urldefense.com/v3/__https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst*L5__;Iw!!CTRNKA9wMg0ARbw!jY0i3mRY0mA3yLhvxm42NZiFnheY6L5Q9dKVJkv911eQ9Z03SBi3-Mgslg_fOzkwAfkJJODcjD6Ka5_I$
> > "
> > 
> > May I ask if this means that the description "A MediaTek DRAM
> > controller interface to provide the current DDR data rate." needs
> > to be
> > split into two lines? The total number of characters in this
> > description is 74.
> 
> And what is the wrapping limit?
> 
> 
> Best regards,
> Krzysztof

The wrapping limit for the commit message body is 75 columns.
I will update the commit message body in next version:
A MediaTek DRAM controller interface to provide the current DDR
data rate.
Is this Okay?

Best regards,
Crystal

  reply	other threads:[~2025-02-12 10:28 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-07  1:42 [v2,0/2] Add an interface to get current DDR data rate Crystal Guo
2025-02-07  1:42 ` [v2,1/2] memory/mediatek: " Crystal Guo
2025-02-09 10:50   ` Krzysztof Kozlowski
2025-02-11 11:33     ` Crystal Guo (郭晶)
2025-03-04 14:59   ` AngeloGioacchino Del Regno
2025-03-17  2:30     ` Crystal Guo (郭晶)
2025-02-07  1:42 ` [v2,2/2] dt-bindings: memory-controllers: Add MediaTek DRAM controller interface Crystal Guo
2025-02-09 10:51   ` Krzysztof Kozlowski
2025-02-12  3:50     ` Crystal Guo (郭晶)
2025-02-12  5:34       ` Krzysztof Kozlowski
2025-02-12 10:07         ` Crystal Guo (郭晶) [this message]
  -- strict thread matches above, loose matches on Subject: below --
2025-02-06 12:16 [v2,0/2] Add an interface to get current DDR data rate Crystal Guo
2025-02-06 12:16 ` [v2,2/2] dt-bindings: memory-controllers: Add MediaTek DRAM controller interface Crystal Guo
2025-02-09 10:48   ` Krzysztof Kozlowski
2025-02-11 11:25     ` Crystal Guo (郭晶)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=468d079e89a5459dfe894402af10605c62676412.camel@mediatek.com \
    --to=crystal.guo@mediatek.com \
    --cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzk@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=robh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox