diff mbox series

firmware: scmi: fix struct layout for scmi_clk_rate_set_in

Message ID 20211013140004.4122342-1-clement.leger@bootlin.com
State Accepted
Commit 32cc3929a173246bf55b2a602082cabd971fc71b
Delegated to: Tom Rini
Headers show
Series firmware: scmi: fix struct layout for scmi_clk_rate_set_in | expand

Commit Message

Clément Léger Oct. 13, 2021, 2 p.m. UTC
First two fields are reversed compared to what is expected by the SCMI
specification.

Signed-off-by: Clément Léger <clement.leger@bootlin.com>
Fixes: 60388844836 ("clk: add clock driver for SCMI agents")
---
 include/scmi_protocols.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Tom Rini Oct. 26, 2021, 10:41 p.m. UTC | #1
On Wed, Oct 13, 2021 at 04:00:04PM +0200, Clément Léger wrote:

> First two fields are reversed compared to what is expected by the SCMI
> specification.
> 
> Signed-off-by: Clément Léger <clement.leger@bootlin.com>
> Fixes: 60388844836 ("clk: add clock driver for SCMI agents")

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/include/scmi_protocols.h b/include/scmi_protocols.h
index ccab97c96c..653aa7ef30 100644
--- a/include/scmi_protocols.h
+++ b/include/scmi_protocols.h
@@ -96,14 +96,14 @@  struct scmi_clk_rate_get_out {
 
 /**
  * struct scmi_clk_state_in - Message payload for CLOCK_RATE_SET command
- * @clock_id:	SCMI clock ID
  * @flags:	Flags for the clock rate set request
+ * @clock_id:	SCMI clock ID
  * @rate_lsb:	32bit LSB of the clock rate in Hertz
  * @rate_msb:	32bit MSB of the clock rate in Hertz
  */
 struct scmi_clk_rate_set_in {
-	u32 clock_id;
 	u32 flags;
+	u32 clock_id;
 	u32 rate_lsb;
 	u32 rate_msb;
 };