Thread (39 messages) 39 messages, 8 authors, 2023-08-14

Re: [PATCH v3 3/5] dt-bindings: clock: add Intel Agilex5 clock manager

From: Dinh Nguyen <dinguyen@kernel.org>
Date: 2023-08-06 17:53:47
Also in: linux-clk, linux-devicetree, lkml


On 8/2/23 02:14, Rabara, Niravkumar L wrote:
quoted
-----Original Message-----
From: Conor Dooley <conor.dooley@microchip.com>
Sent: Wednesday, 2 August, 2023 3:02 PM
To: Rabara, Niravkumar L <redacted>
Cc: Ng, Adrian Ho Yin <redacted>; andrew@lunn.ch;
conor+dt@kernel.org; devicetree@vger.kernel.org; dinguyen@kernel.org;
krzysztof.kozlowski+dt@linaro.org; linux-clk@vger.kernel.org; linux-
kernel@vger.kernel.org; Turquette, Mike [off-list ref];
netdev@vger.kernel.org; p.zabel@pengutronix.de;
richardcochran@gmail.com; robh+dt@kernel.org; sboyd@kernel.org;
wen.ping.teh@intel.com
Subject: Re: [PATCH v3 3/5] dt-bindings: clock: add Intel Agilex5 clock
manager

On Wed, Aug 02, 2023 at 10:58:42AM +0800, niravkumar.l.rabara@intel.com
wrote:
quoted
From: Niravkumar L Rabara <redacted>

Add clock ID definitions for Intel Agilex5 SoCFPGA.
The registers in Agilex5 handling the clock is named as clock manager.

Signed-off-by: Teh Wen Ping <redacted>
Reviewed-by: Dinh Nguyen <dinguyen@kernel.org>
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Niravkumar L Rabara <redacted>
Damn, I was too late - you already sent a v3 :/

However, there only seems to be a v3 of this one patch and it was sent in
reply to the v2 series? The normal thing to do is resend the entire series, not
just one patch, as a new thread. Not using a new thread may make it harder
to apply & will also bury the email in people's mailboxes that use things like
mutt. A single patch as a reply is also confusing, as the rest of the v3 looks like
it is missing!

Thanks,
Conor.
Sorry I made a mistake.
Should I send out entire series with PATCH v3 subject? Or should I wait for review comment on remaining patches and then send entire series with rework and  subject prefix PATCH v3?
No need to send out a V3. I've applied patches 1-3 and 5. Will give a 
little more time for the clk patch.

Dinh
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help