[PATCH 0/3] drm/amd/display: fix codestyle for some dmub files

[PATCH 0/3] drm/amd/display: fix codestyle for some dmub files

[Marcelo Mendes Spessoto Junior]

Marcelo Mendes Spessoto Junior (3):
  drm/amd/display: clean codestyle errors
  drm/amd/display: clean codestyle errors
  drm/amd/display: clean codestyle errors

 drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h   | 6 +++---
 drivers/gpu/drm/amd/display/dmub/src/dmub_dcn32.c | 2 +-
 drivers/gpu/drm/amd/display/dmub/src/dmub_dcn35.c | 5 +++--
 3 files changed, 7 insertions(+), 6 deletions(-)

-- 
2.39.2

[PATCH 1/3] drm/amd/display: clean codestyle errors

[Marcelo Mendes Spessoto Junior]

Use flexible array members in dmub/inc/dmub_cmd.h

Signed-off-by: Marcelo Mendes Spessoto Junior <marcelomspessoto@gmail.com>
---
 drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h b/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h
index a529e369b..d748bedda 100644
--- a/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h
+++ b/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h
@@ -1588,7 +1588,7 @@ struct dmub_rb_cmd_idle_opt_dcn_restore {
  */
 struct dmub_dcn_notify_idle_cntl_data {
 	uint8_t driver_idle;
-	uint8_t pad[1];
+	uint8_t pad[];
 };
 
 /**
@@ -2421,7 +2421,7 @@ struct dmub_cmd_psr_copy_settings_data {
 	/**
 	 * @ rate_control_caps : Indicate FreeSync PSR Sink Capabilities
 	 */
-	uint8_t rate_control_caps ;
+	uint8_t rate_control_caps;
 	/*
 	 * Force PSRSU always doing full frame update
 	 */
@@ -3921,7 +3921,7 @@ struct dmub_cmd_abm_pause_data {
 	/**
 	 * Explicit padding to 4 byte boundary.
 	 */
-	uint8_t pad[1];
+	uint8_t pad[];
 };
 
 /**
-- 
2.39.2

[PATCH 2/3] drm/amd/display: clean codestyle error

[Marcelo Mendes Spessoto Junior]

Use () for complex macro in dmub/src/dmub_dcn32.c

Signed-off-by: Marcelo Mendes Spessoto Junior <marcelomspessoto@gmail.com>
---
 drivers/gpu/drm/amd/display/dmub/src/dmub_dcn32.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn32.c b/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn32.c
index 0d521eeda..db9bf55f5 100644
--- a/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn32.c
+++ b/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn32.c
@@ -35,7 +35,7 @@
 #define BASE_INNER(seg) ctx->dcn_reg_offsets[seg]
 #define CTX dmub
 #define REGS dmub->regs_dcn32
-#define REG_OFFSET_EXP(reg_name) BASE(reg##reg_name##_BASE_IDX) + reg##reg_name
+#define REG_OFFSET_EXP(reg_name) (BASE(reg##reg_name##_BASE_IDX) + reg##reg_name)
 
 void dmub_srv_dcn32_regs_init(struct dmub_srv *dmub,  struct dc_context *ctx)
 {
-- 
2.39.2

[PATCH 3/3] drm/amd/display: clean codestyle errors

[Marcelo Mendes Spessoto Junior]

Use () for macro and adjust initial brace for dmub/src/dmub_dcn35.c

Signed-off-by: Marcelo Mendes Spessoto Junior <marcelomspessoto@gmail.com>
---
 drivers/gpu/drm/amd/display/dmub/src/dmub_dcn35.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn35.c b/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn35.c
index 60223efc6..fa38a5ae6 100644
--- a/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn35.c
+++ b/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn35.c
@@ -35,9 +35,10 @@
 #define BASE_INNER(seg) ctx->dcn_reg_offsets[seg]
 #define CTX dmub
 #define REGS dmub->regs_dcn35
-#define REG_OFFSET_EXP(reg_name) BASE(reg##reg_name##_BASE_IDX) + reg##reg_name
+#define REG_OFFSET_EXP(reg_name) (BASE(reg##reg_name##_BASE_IDX) + reg##reg_name)
 
-void dmub_srv_dcn35_regs_init(struct dmub_srv *dmub, struct dc_context *ctx) {
+void dmub_srv_dcn35_regs_init(struct dmub_srv *dmub, struct dc_context *ctx)
+{
 	struct dmub_srv_dcn35_regs *regs = dmub->regs_dcn35;
 #define REG_STRUCT regs
 
-- 
2.39.2