<div dir="ltr">Thanks for your reply Alex,<div><br></div><div>I just realize with your comment that <i>drivers/gpu/drm/amd/display/dc/gpio/hw_gpio.c</i> import <i>dal_hw_gpio_get_value </i>and <i>dal_hw_gpio_set_value</i>.</div><div>So to make those functions inside <i>drivers/gpu/drm/amd/display/dc/gpio/hw_hpd.c</i>  more clear what do you think to change them to <i>dal_hw_<b>hpd</b>_get_config</i> and <i>dal_hw_<b>hpd</b>_set_config</i>, making clear that this is a function from the file hw_hpd?</div><div><br></div><div>Leonardo Gomes</div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">Em qua., 14 de mai. de 2025 às 18:08, Alex Deucher <<a href="mailto:alexdeucher@gmail.com">alexdeucher@gmail.com</a>> escreveu:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, May 14, 2025 at 4:48 PM Leonardo Gomes<br>
<<a href="mailto:leonardodasigomes@gmail.com" target="_blank">leonardodasigomes@gmail.com</a>> wrote:<br>
><br>
> Adjust get_value function in hw_hpd.c file to have<br>
> prefix to help in ftrace, the name change from<br>
> 'get_value' to 'dal_hw_gpio_get_value'<br>
<br>
This won't compile.  dal_hw_gpio_get_value is already defined in<br>
drivers/gpu/drm/amd/display/dc/gpio/hw_gpio.c.<br>
<br>
Alex<br>
<br>
><br>
> Signed-off-by: Leonardo da Silva Gomes <<a href="mailto:leonardodasigomes@gmail.com" target="_blank">leonardodasigomes@gmail.com</a>><br>
> Co-developed-by: Derick Frias <<a href="mailto:derick.william.moraes@gmail.com" target="_blank">derick.william.moraes@gmail.com</a>><br>
> Signed-off-by: Derick Frias <<a href="mailto:derick.william.moraes@gmail.com" target="_blank">derick.william.moraes@gmail.com</a>><br>
> ---<br>
>  drivers/gpu/drm/amd/display/dc/gpio/hw_hpd.c | 4 ++--<br>
>  1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/drivers/gpu/drm/amd/display/dc/gpio/hw_hpd.c b/drivers/gpu/drm/amd/display/dc/gpio/hw_hpd.c<br>
> index 3f13a744d07d..b11ed1089589 100644<br>
> --- a/drivers/gpu/drm/amd/display/dc/gpio/hw_hpd.c<br>
> +++ b/drivers/gpu/drm/amd/display/dc/gpio/hw_hpd.c<br>
> @@ -62,7 +62,7 @@ static void dal_hw_hpd_destroy(<br>
>         *ptr = NULL;<br>
>  }<br>
><br>
> -static enum gpio_result get_value(<br>
> +static enum gpio_result dal_hw_gpio_get_value(<br>
>         const struct hw_gpio_pin *ptr,<br>
>         uint32_t *value)<br>
>  {<br>
> @@ -104,7 +104,7 @@ static enum gpio_result set_config(<br>
>  static const struct hw_gpio_pin_funcs funcs = {<br>
>         .destroy = dal_hw_hpd_destroy,<br>
>         .open = dal_hw_gpio_open,<br>
> -       .get_value = get_value,<br>
> +       .get_value = dal_hw_gpio_get_value,<br>
>         .set_value = dal_hw_gpio_set_value,<br>
>         .set_config = set_config,<br>
>         .change_mode = dal_hw_gpio_change_mode,<br>
> --<br>
> 2.43.0<br>
><br>
</blockquote></div>