* [PATCH 3/3] drm/amd/powerplay: Make two functions static
@ 2019-10-28 13:36 YueHaibing
2019-10-28 14:06 ` Alex Deucher
0 siblings, 1 reply; 3+ messages in thread
From: YueHaibing @ 2019-10-28 13:36 UTC (permalink / raw)
To: rex.zhu, evan.quan, alexander.deucher, christian.koenig,
David1.Zhou, airlied, daniel
Cc: amd-gfx, dri-devel, linux-kernel, YueHaibing
Fix sparse warnings:
drivers/gpu/drm/amd/amdgpu/../powerplay/arcturus_ppt.c:2050:5:
warning: symbol 'arcturus_i2c_eeprom_control_init' was not declared. Should it be static?
drivers/gpu/drm/amd/amdgpu/../powerplay/arcturus_ppt.c:2068:6:
warning: symbol 'arcturus_i2c_eeprom_control_fini' was not declared. Should it be static?
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
drivers/gpu/drm/amd/powerplay/arcturus_ppt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
index d48a49d..3099ac2 100644
--- a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
+++ b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
@@ -2047,7 +2047,7 @@ static const struct i2c_algorithm arcturus_i2c_eeprom_i2c_algo = {
.functionality = arcturus_i2c_eeprom_i2c_func,
};
-int arcturus_i2c_eeprom_control_init(struct i2c_adapter *control)
+static int arcturus_i2c_eeprom_control_init(struct i2c_adapter *control)
{
struct amdgpu_device *adev = to_amdgpu_device(control);
int res;
@@ -2065,7 +2065,7 @@ int arcturus_i2c_eeprom_control_init(struct i2c_adapter *control)
return res;
}
-void arcturus_i2c_eeprom_control_fini(struct i2c_adapter *control)
+static void arcturus_i2c_eeprom_control_fini(struct i2c_adapter *control)
{
i2c_del_adapter(control);
}
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 3/3] drm/amd/powerplay: Make two functions static
2019-10-28 13:36 [PATCH 3/3] drm/amd/powerplay: Make two functions static YueHaibing
@ 2019-10-28 14:06 ` Alex Deucher
2019-10-28 14:29 ` Yuehaibing
0 siblings, 1 reply; 3+ messages in thread
From: Alex Deucher @ 2019-10-28 14:06 UTC (permalink / raw)
To: YueHaibing
Cc: Rex Zhu, Quan, Evan, Deucher, Alexander, Christian Koenig,
Chunming Zhou, Dave Airlie, Daniel Vetter,
Maling list - DRI developers, amd-gfx list, LKML
On Mon, Oct 28, 2019 at 9:37 AM YueHaibing <yuehaibing@huawei.com> wrote:
>
> Fix sparse warnings:
>
> drivers/gpu/drm/amd/amdgpu/../powerplay/arcturus_ppt.c:2050:5:
> warning: symbol 'arcturus_i2c_eeprom_control_init' was not declared. Should it be static?
> drivers/gpu/drm/amd/amdgpu/../powerplay/arcturus_ppt.c:2068:6:
> warning: symbol 'arcturus_i2c_eeprom_control_fini' was not declared. Should it be static?
>
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Applied. Thanks. Is there more to this series? I don't see patches 1 or 2.
Alex
> ---
> drivers/gpu/drm/amd/powerplay/arcturus_ppt.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
> index d48a49d..3099ac2 100644
> --- a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
> +++ b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
> @@ -2047,7 +2047,7 @@ static const struct i2c_algorithm arcturus_i2c_eeprom_i2c_algo = {
> .functionality = arcturus_i2c_eeprom_i2c_func,
> };
>
> -int arcturus_i2c_eeprom_control_init(struct i2c_adapter *control)
> +static int arcturus_i2c_eeprom_control_init(struct i2c_adapter *control)
> {
> struct amdgpu_device *adev = to_amdgpu_device(control);
> int res;
> @@ -2065,7 +2065,7 @@ int arcturus_i2c_eeprom_control_init(struct i2c_adapter *control)
> return res;
> }
>
> -void arcturus_i2c_eeprom_control_fini(struct i2c_adapter *control)
> +static void arcturus_i2c_eeprom_control_fini(struct i2c_adapter *control)
> {
> i2c_del_adapter(control);
> }
> --
> 2.7.4
>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 3/3] drm/amd/powerplay: Make two functions static
2019-10-28 14:06 ` Alex Deucher
@ 2019-10-28 14:29 ` Yuehaibing
0 siblings, 0 replies; 3+ messages in thread
From: Yuehaibing @ 2019-10-28 14:29 UTC (permalink / raw)
To: Alex Deucher
Cc: Rex Zhu, Quan, Evan, Deucher, Alexander, Christian Koenig,
Chunming Zhou, Dave Airlie, Daniel Vetter,
Maling list - DRI developers, amd-gfx list, LKML
On 2019/10/28 22:06, Alex Deucher wrote:
> On Mon, Oct 28, 2019 at 9:37 AM YueHaibing <yuehaibing@huawei.com> wrote:
>>
>> Fix sparse warnings:
>>
>> drivers/gpu/drm/amd/amdgpu/../powerplay/arcturus_ppt.c:2050:5:
>> warning: symbol 'arcturus_i2c_eeprom_control_init' was not declared. Should it be static?
>> drivers/gpu/drm/amd/amdgpu/../powerplay/arcturus_ppt.c:2068:6:
>> warning: symbol 'arcturus_i2c_eeprom_control_fini' was not declared. Should it be static?
>>
>> Reported-by: Hulk Robot <hulkci@huawei.com>
>> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
>
> Applied. Thanks. Is there more to this series? I don't see patches 1 or 2.
No, only this one. I forget to change the patch title. Sorry for confusion.
>
> Alex
>
>> ---
>> drivers/gpu/drm/amd/powerplay/arcturus_ppt.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
>> index d48a49d..3099ac2 100644
>> --- a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
>> +++ b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
>> @@ -2047,7 +2047,7 @@ static const struct i2c_algorithm arcturus_i2c_eeprom_i2c_algo = {
>> .functionality = arcturus_i2c_eeprom_i2c_func,
>> };
>>
>> -int arcturus_i2c_eeprom_control_init(struct i2c_adapter *control)
>> +static int arcturus_i2c_eeprom_control_init(struct i2c_adapter *control)
>> {
>> struct amdgpu_device *adev = to_amdgpu_device(control);
>> int res;
>> @@ -2065,7 +2065,7 @@ int arcturus_i2c_eeprom_control_init(struct i2c_adapter *control)
>> return res;
>> }
>>
>> -void arcturus_i2c_eeprom_control_fini(struct i2c_adapter *control)
>> +static void arcturus_i2c_eeprom_control_fini(struct i2c_adapter *control)
>> {
>> i2c_del_adapter(control);
>> }
>> --
>> 2.7.4
>>
>>
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>
> .
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-10-28 14:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-28 13:36 [PATCH 3/3] drm/amd/powerplay: Make two functions static YueHaibing
2019-10-28 14:06 ` Alex Deucher
2019-10-28 14:29 ` Yuehaibing
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).