* [PATCH] accel/ivpu: Make function parameter names consistent
@ 2025-08-08 11:10 Jacek Lawrynowicz
2025-08-08 16:58 ` Lizhi Hou
2025-09-01 10:36 ` Jacek Lawrynowicz
0 siblings, 2 replies; 3+ messages in thread
From: Jacek Lawrynowicz @ 2025-08-08 11:10 UTC (permalink / raw)
To: dri-devel; +Cc: jeff.hugo, lizhi.hou, Jacek Lawrynowicz
Make ivpu_hw_btrs_dct_set_status() and ivpu_fw_boot_params_setup()
declaration and definition parameter names consistent.
Signed-off-by: Jacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>
---
drivers/accel/ivpu/ivpu_fw.h | 2 +-
drivers/accel/ivpu/ivpu_hw_btrs.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/accel/ivpu/ivpu_fw.h b/drivers/accel/ivpu/ivpu_fw.h
index 9a3935be1c057..7081913fb0dde 100644
--- a/drivers/accel/ivpu/ivpu_fw.h
+++ b/drivers/accel/ivpu/ivpu_fw.h
@@ -45,7 +45,7 @@ struct ivpu_fw_info {
int ivpu_fw_init(struct ivpu_device *vdev);
void ivpu_fw_fini(struct ivpu_device *vdev);
void ivpu_fw_load(struct ivpu_device *vdev);
-void ivpu_fw_boot_params_setup(struct ivpu_device *vdev, struct vpu_boot_params *bp);
+void ivpu_fw_boot_params_setup(struct ivpu_device *vdev, struct vpu_boot_params *boot_params);
static inline bool ivpu_fw_is_cold_boot(struct ivpu_device *vdev)
{
diff --git a/drivers/accel/ivpu/ivpu_hw_btrs.h b/drivers/accel/ivpu/ivpu_hw_btrs.h
index d2d82651976d1..032c384ac3d4d 100644
--- a/drivers/accel/ivpu/ivpu_hw_btrs.h
+++ b/drivers/accel/ivpu/ivpu_hw_btrs.h
@@ -36,7 +36,7 @@ u32 ivpu_hw_btrs_dpu_freq_get(struct ivpu_device *vdev);
bool ivpu_hw_btrs_irq_handler_mtl(struct ivpu_device *vdev, int irq);
bool ivpu_hw_btrs_irq_handler_lnl(struct ivpu_device *vdev, int irq);
int ivpu_hw_btrs_dct_get_request(struct ivpu_device *vdev, bool *enable);
-void ivpu_hw_btrs_dct_set_status(struct ivpu_device *vdev, bool enable, u32 dct_percent);
+void ivpu_hw_btrs_dct_set_status(struct ivpu_device *vdev, bool enable, u32 active_percent);
u32 ivpu_hw_btrs_telemetry_offset_get(struct ivpu_device *vdev);
u32 ivpu_hw_btrs_telemetry_size_get(struct ivpu_device *vdev);
u32 ivpu_hw_btrs_telemetry_enable_get(struct ivpu_device *vdev);
--
2.45.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] accel/ivpu: Make function parameter names consistent
2025-08-08 11:10 [PATCH] accel/ivpu: Make function parameter names consistent Jacek Lawrynowicz
@ 2025-08-08 16:58 ` Lizhi Hou
2025-09-01 10:36 ` Jacek Lawrynowicz
1 sibling, 0 replies; 3+ messages in thread
From: Lizhi Hou @ 2025-08-08 16:58 UTC (permalink / raw)
To: Jacek Lawrynowicz, dri-devel; +Cc: jeff.hugo
On 8/8/25 04:10, Jacek Lawrynowicz wrote:
> Make ivpu_hw_btrs_dct_set_status() and ivpu_fw_boot_params_setup()
> declaration and definition parameter names consistent.
>
> Signed-off-by: Jacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>
> ---
> drivers/accel/ivpu/ivpu_fw.h | 2 +-
> drivers/accel/ivpu/ivpu_hw_btrs.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/accel/ivpu/ivpu_fw.h b/drivers/accel/ivpu/ivpu_fw.h
> index 9a3935be1c057..7081913fb0dde 100644
> --- a/drivers/accel/ivpu/ivpu_fw.h
> +++ b/drivers/accel/ivpu/ivpu_fw.h
> @@ -45,7 +45,7 @@ struct ivpu_fw_info {
> int ivpu_fw_init(struct ivpu_device *vdev);
> void ivpu_fw_fini(struct ivpu_device *vdev);
> void ivpu_fw_load(struct ivpu_device *vdev);
> -void ivpu_fw_boot_params_setup(struct ivpu_device *vdev, struct vpu_boot_params *bp);
> +void ivpu_fw_boot_params_setup(struct ivpu_device *vdev, struct vpu_boot_params *boot_params);
>
> static inline bool ivpu_fw_is_cold_boot(struct ivpu_device *vdev)
> {
> diff --git a/drivers/accel/ivpu/ivpu_hw_btrs.h b/drivers/accel/ivpu/ivpu_hw_btrs.h
> index d2d82651976d1..032c384ac3d4d 100644
> --- a/drivers/accel/ivpu/ivpu_hw_btrs.h
> +++ b/drivers/accel/ivpu/ivpu_hw_btrs.h
> @@ -36,7 +36,7 @@ u32 ivpu_hw_btrs_dpu_freq_get(struct ivpu_device *vdev);
> bool ivpu_hw_btrs_irq_handler_mtl(struct ivpu_device *vdev, int irq);
> bool ivpu_hw_btrs_irq_handler_lnl(struct ivpu_device *vdev, int irq);
> int ivpu_hw_btrs_dct_get_request(struct ivpu_device *vdev, bool *enable);
> -void ivpu_hw_btrs_dct_set_status(struct ivpu_device *vdev, bool enable, u32 dct_percent);
> +void ivpu_hw_btrs_dct_set_status(struct ivpu_device *vdev, bool enable, u32 active_percent);
Reviewed-by: Lizhi Hou <lizhi.hou@amd.com>
> u32 ivpu_hw_btrs_telemetry_offset_get(struct ivpu_device *vdev);
> u32 ivpu_hw_btrs_telemetry_size_get(struct ivpu_device *vdev);
> u32 ivpu_hw_btrs_telemetry_enable_get(struct ivpu_device *vdev);
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] accel/ivpu: Make function parameter names consistent
2025-08-08 11:10 [PATCH] accel/ivpu: Make function parameter names consistent Jacek Lawrynowicz
2025-08-08 16:58 ` Lizhi Hou
@ 2025-09-01 10:36 ` Jacek Lawrynowicz
1 sibling, 0 replies; 3+ messages in thread
From: Jacek Lawrynowicz @ 2025-09-01 10:36 UTC (permalink / raw)
To: dri-devel; +Cc: jeff.hugo, lizhi.hou
Applied to drm-misc-next
On 8/8/2025 1:10 PM, Jacek Lawrynowicz wrote:
> Make ivpu_hw_btrs_dct_set_status() and ivpu_fw_boot_params_setup()
> declaration and definition parameter names consistent.
>
> Signed-off-by: Jacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>
> ---
> drivers/accel/ivpu/ivpu_fw.h | 2 +-
> drivers/accel/ivpu/ivpu_hw_btrs.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/accel/ivpu/ivpu_fw.h b/drivers/accel/ivpu/ivpu_fw.h
> index 9a3935be1c057..7081913fb0dde 100644
> --- a/drivers/accel/ivpu/ivpu_fw.h
> +++ b/drivers/accel/ivpu/ivpu_fw.h
> @@ -45,7 +45,7 @@ struct ivpu_fw_info {
> int ivpu_fw_init(struct ivpu_device *vdev);
> void ivpu_fw_fini(struct ivpu_device *vdev);
> void ivpu_fw_load(struct ivpu_device *vdev);
> -void ivpu_fw_boot_params_setup(struct ivpu_device *vdev, struct vpu_boot_params *bp);
> +void ivpu_fw_boot_params_setup(struct ivpu_device *vdev, struct vpu_boot_params *boot_params);
>
> static inline bool ivpu_fw_is_cold_boot(struct ivpu_device *vdev)
> {
> diff --git a/drivers/accel/ivpu/ivpu_hw_btrs.h b/drivers/accel/ivpu/ivpu_hw_btrs.h
> index d2d82651976d1..032c384ac3d4d 100644
> --- a/drivers/accel/ivpu/ivpu_hw_btrs.h
> +++ b/drivers/accel/ivpu/ivpu_hw_btrs.h
> @@ -36,7 +36,7 @@ u32 ivpu_hw_btrs_dpu_freq_get(struct ivpu_device *vdev);
> bool ivpu_hw_btrs_irq_handler_mtl(struct ivpu_device *vdev, int irq);
> bool ivpu_hw_btrs_irq_handler_lnl(struct ivpu_device *vdev, int irq);
> int ivpu_hw_btrs_dct_get_request(struct ivpu_device *vdev, bool *enable);
> -void ivpu_hw_btrs_dct_set_status(struct ivpu_device *vdev, bool enable, u32 dct_percent);
> +void ivpu_hw_btrs_dct_set_status(struct ivpu_device *vdev, bool enable, u32 active_percent);
> u32 ivpu_hw_btrs_telemetry_offset_get(struct ivpu_device *vdev);
> u32 ivpu_hw_btrs_telemetry_size_get(struct ivpu_device *vdev);
> u32 ivpu_hw_btrs_telemetry_enable_get(struct ivpu_device *vdev);
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-09-01 10:36 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-08 11:10 [PATCH] accel/ivpu: Make function parameter names consistent Jacek Lawrynowicz
2025-08-08 16:58 ` Lizhi Hou
2025-09-01 10:36 ` Jacek Lawrynowicz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).