[Public]
Acked-by: Alex Deucher <alexander.deuc...@amd.com>
________________________________
From: SHANMUGAM, SRINIVASAN <srinivasan.shanmu...@amd.com>
Sent: Monday, June 12, 2023 6:28 AM
To: Deucher, Alexander <alexander.deuc...@amd.com>; Koenig, Christian
<christian.koe...@amd.com>
Cc: amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>; SHANMUGAM,
SRINIVASAN <srinivasan.shanmu...@amd.com>
Subject: [PATCH] drm/radeon: Fix missing prototypes in radeon_atpx_handler.c
Fixes the following gcc with W=1:
drivers/gpu/drm/radeon/radeon_atpx_handler.c:64:6: warning: no previous
prototype for ‘radeon_has_atpx’ [-Wmissing-prototypes]
64 | bool 4(void) {
| ^~~~~~~~~~~~~~~
drivers/gpu/drm/radeon/radeon_atpx_handler.c:68:6: warning: no previous
prototype for ‘radeon_has_atpx_dgpu_power_cntl’ [-Wmissing-prototypes]
68 | bool radeon_has_atpx_dgpu_power_cntl(void) {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/radeon/radeon_atpx_handler.c:72:6: warning: no previous
prototype for ‘radeon_is_atpx_hybrid’ [-Wmissing-prototypes]
72 | bool radeon_is_atpx_hybrid(void) {
| ^~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/radeon/radeon_atpx_handler.c:77:6: warning: no previous
prototype for ‘radeon_atpx_dgpu_req_power_for_displays’ [-Wmissing-prototypes]
77 | bool radeon_atpx_dgpu_req_power_for_displays(void) {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/radeon/radeon_atpx_handler.c:596:6: warning: no previous
prototype for ‘radeon_register_atpx_handler’ [-Wmissing-prototypes]
596 | void radeon_register_atpx_handler(void)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/radeon/radeon_atpx_handler.c:614:6: warning: no previous
prototype for ‘radeon_unregister_atpx_handler’ [-Wmissing-prototypes]
614 | void radeon_unregister_atpx_handler(void)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/radeon/radeon_atpx_handler.c:159: warning: expecting prototype
for radeon_atpx_validate_functions(). Prototype was for radeon_atpx_validate()
instead
Cc: Christian König <christian.koe...@amd.com>
Cc: Alex Deucher <alexander.deuc...@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmu...@amd.com>
---
drivers/gpu/drm/radeon/radeon_acpi.h | 9 +++++++++
drivers/gpu/drm/radeon/radeon_atpx_handler.c | 2 +-
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/radeon/radeon_acpi.h
b/drivers/gpu/drm/radeon/radeon_acpi.h
index 35202a453e66..974fbb4ce2c2 100644
--- a/drivers/gpu/drm/radeon/radeon_acpi.h
+++ b/drivers/gpu/drm/radeon/radeon_acpi.h
@@ -453,4 +453,13 @@ struct acpi_bus_event;
* BYTE - number of active lanes
*/
+#if defined(CONFIG_VGA_SWITCHEROO)
+void radeon_register_atpx_handler(void);
+void radeon_unregister_atpx_handler(void);
+bool radeon_has_atpx_dgpu_power_cntl(void);
+bool radeon_is_atpx_hybrid(void);
+bool radeon_has_atpx(void);
+bool radeon_atpx_dgpu_req_power_for_displays(void);
+#endif
+
#endif
diff --git a/drivers/gpu/drm/radeon/radeon_atpx_handler.c
b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
index 6f93f54bf651..d0b450a06506 100644
--- a/drivers/gpu/drm/radeon/radeon_atpx_handler.c
+++ b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
@@ -147,7 +147,7 @@ static void radeon_atpx_parse_functions(struct
radeon_atpx_functions *f, u32 mas
}
/**
- * radeon_atpx_validate_functions - validate ATPX functions
+ * radeon_atpx_validate() - validate ATPX functions
*
* @atpx: radeon atpx struct
*
--
2.25.1
