In commit 482812d56698e ("drm/amd/display: Set max TTU on
DPG enable"), "hubp.h" was added which caused the duplicate include.
To be on the safe side, remove the later duplicate include.
Signed-off-by: Wan Jiabing <[email protected]>
---
drivers/gpu/drm/amd/display/dc/core/dc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 8f0a13807d05..dcaa6b1e16af 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -58,7 +58,6 @@
#include "dc_link_ddc.h"
#include "dm_helpers.h"
#include "mem_input.h"
-#include "hubp.h"
#include "dc_link_dp.h"
#include "dc_dmub_srv.h"
--
2.25.1
Nice catch!
Are you using any tool to identify this problem?
Reviewed-by: Rodrigo Siqueira <[email protected]>
On 04/29, Wan Jiabing wrote:
> In commit 482812d56698e ("drm/amd/display: Set max TTU on
> DPG enable"), "hubp.h" was added which caused the duplicate include.
> To be on the safe side, remove the later duplicate include.
>
> Signed-off-by: Wan Jiabing <[email protected]>
> ---
> drivers/gpu/drm/amd/display/dc/core/dc.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
> index 8f0a13807d05..dcaa6b1e16af 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
> @@ -58,7 +58,6 @@
> #include "dc_link_ddc.h"
> #include "dm_helpers.h"
> #include "mem_input.h"
> -#include "hubp.h"
>
> #include "dc_link_dp.h"
> #include "dc_dmub_srv.h"
> --
> 2.25.1
>
--
Rodrigo Siqueira
https://siqueira.tech
>Nice catch!
>
>Are you using any tool to identify this problem?
Yes, I have a robot including many detecting scripts :)
>Reviewed-by: Rodrigo Siqueira <[email protected]>
>
>On 04/29, Wan Jiabing wrote:
>> In commit 482812d56698e ("drm/amd/display: Set max TTU on
>> DPG enable"), "hubp.h" was added which caused the duplicate include.
>> To be on the safe side, remove the later duplicate include.
>>
>> Signed-off-by: Wan Jiabing <[email protected]>
>> ---
>> drivers/gpu/drm/amd/display/dc/core/dc.c | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
>> index 8f0a13807d05..dcaa6b1e16af 100644
>> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
>> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
>> @@ -58,7 +58,6 @@
>> #include "dc_link_ddc.h"
>> #include "dm_helpers.h"
>> #include "mem_input.h"
>> -#include "hubp.h"
>>
>> #include "dc_link_dp.h"
>> #include "dc_dmub_srv.h"
>> --
>> 2.25.1
>>
>
>--
>Rodrigo Siqueira
>https://siqueira.tech
Yours,
Wan Jiabing
Is your robot public available?
btw, applied to amd-staging-drm-next.
Thanks
On 04/29, Jiabing Wan wrote:
>
> >Nice catch!
> >
> >Are you using any tool to identify this problem?
>
> Yes, I have a robot including many detecting scripts :)
>
> >Reviewed-by: Rodrigo Siqueira <[email protected]>
> >
> >On 04/29, Wan Jiabing wrote:
> >> In commit 482812d56698e ("drm/amd/display: Set max TTU on
> >> DPG enable"), "hubp.h" was added which caused the duplicate include.
> >> To be on the safe side, remove the later duplicate include.
> >>
> >> Signed-off-by: Wan Jiabing <[email protected]>
> >> ---
> >> drivers/gpu/drm/amd/display/dc/core/dc.c | 1 -
> >> 1 file changed, 1 deletion(-)
> >>
> >> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
> >> index 8f0a13807d05..dcaa6b1e16af 100644
> >> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
> >> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
> >> @@ -58,7 +58,6 @@
> >> #include "dc_link_ddc.h"
> >> #include "dm_helpers.h"
> >> #include "mem_input.h"
> >> -#include "hubp.h"
> >>
> >> #include "dc_link_dp.h"
> >> #include "dc_dmub_srv.h"
> >> --
> >> 2.25.1
> >>
> >
> >--
> >Rodrigo Siqueira
> >https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsiqueira.tech%2F&data=04%7C01%7CRodrigo.Siqueira%40amd.com%7Cb8ff161b3a904a49e59808d90b0991d9%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637552958013669698%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=I7AtEiOlzUuRQag7uAuArUvfd64l1b3gstq7ZTeucik%3D&reserved=0
>
> Yours,
> Wan Jiabing
>
--
Rodrigo Siqueira
https://siqueira.tech
>Is your robot public available?
Sorry, I started to write the robot just one week ago.
It is not strong and complete enough so it is not public available now.
But I am still working on it. If complete, it can be public available.
In fact, this patch is catched by the script called checkinclude.pl
in linux/script. The robot just run the script and catch it ;).
There will be more scripts in the robot and I think
it will be public available soon.
>btw, applied to amd-staging-drm-next.
Thank you very much.
>
>Thanks
>On 04/29, Jiabing Wan wrote:
>>
>> >Nice catch!
>> >
>> >Are you using any tool to identify this problem?
>>
>> Yes, I have a robot including many detecting scripts :)
>>
>> >Reviewed-by: Rodrigo Siqueira <[email protected]>
>> >
>> >On 04/29, Wan Jiabing wrote:
>> >> In commit 482812d56698e ("drm/amd/display: Set max TTU on
>> >> DPG enable"), "hubp.h" was added which caused the duplicate include.
>> >> To be on the safe side, remove the later duplicate include.
>> >>
>> >> Signed-off-by: Wan Jiabing <[email protected]>
>> >> ---
>> >> drivers/gpu/drm/amd/display/dc/core/dc.c | 1 -
>> >> 1 file changed, 1 deletion(-)
>> >>
>> >> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
>> >> index 8f0a13807d05..dcaa6b1e16af 100644
>> >> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
>> >> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
>> >> @@ -58,7 +58,6 @@
>> >> #include "dc_link_ddc.h"
>> >> #include "dm_helpers.h"
>> >> #include "mem_input.h"
>> >> -#include "hubp.h"
>> >>
>> >> #include "dc_link_dp.h"
>> >> #include "dc_dmub_srv.h"
>> >> --
>> >> 2.25.1
>> >>
>> >
>> >--
>> >Rodrigo Siqueira
>> >https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsiqueira.tech%2F&data=04%7C01%7CRodrigo.Siqueira%40amd.com%7Cb8ff161b3a904a49e59808d90b0991d9%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637552958013669698%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=I7AtEiOlzUuRQag7uAuArUvfd64l1b3gstq7ZTeucik%3D&reserved=0
>>
>> Yours,
>> Wan Jiabing
>>
>
>--
>Rodrigo Siqueira
>https://siqueira.tech
Yours,
Wan Jiabing
Well if you are bored we quite often have the problem of unused fields
in structures.
If your robot could find fields which are never read, that would be
quite helpful I think.
There probably will be a lot of false positives for things like padding,
but maybe that yields something.
Christian.
Am 29.04.21 um 17:18 schrieb Jiabing Wan:
>
>> Is your robot public available?
> Sorry, I started to write the robot just one week ago.
> It is not strong and complete enough so it is not public available now.
> But I am still working on it. If complete, it can be public available.
>
> In fact, this patch is catched by the script called checkinclude.pl
> in linux/script. The robot just run the script and catch it ;).
>
> There will be more scripts in the robot and I think
> it will be public available soon.
>
>> btw, applied to amd-staging-drm-next.
> Thank you very much.
>
>> Thanks
>> On 04/29, Jiabing Wan wrote:
>>>
>>>> Nice catch!
>>>>
>>>> Are you using any tool to identify this problem?
>>> Yes, I have a robot including many detecting scripts :)
>>>
>>>> Reviewed-by: Rodrigo Siqueira <[email protected]>
>>>>
>>>> On 04/29, Wan Jiabing wrote:
>>>>> In commit 482812d56698e ("drm/amd/display: Set max TTU on
>>>>> DPG enable"), "hubp.h" was added which caused the duplicate include.
>>>>> To be on the safe side, remove the later duplicate include.
>>>>>
>>>>> Signed-off-by: Wan Jiabing <[email protected]>
>>>>> ---
>>>>> drivers/gpu/drm/amd/display/dc/core/dc.c | 1 -
>>>>> 1 file changed, 1 deletion(-)
>>>>>
>>>>> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
>>>>> index 8f0a13807d05..dcaa6b1e16af 100644
>>>>> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
>>>>> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
>>>>> @@ -58,7 +58,6 @@
>>>>> #include "dc_link_ddc.h"
>>>>> #include "dm_helpers.h"
>>>>> #include "mem_input.h"
>>>>> -#include "hubp.h"
>>>>>
>>>>> #include "dc_link_dp.h"
>>>>> #include "dc_dmub_srv.h"
>>>>> --
>>>>> 2.25.1
>>>>>
>>>> --
>>>> Rodrigo Siqueira
>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsiqueira.tech%2F&data=04%7C01%7Cchristian.koenig%40amd.com%7C99075b6ba68a4c6d4f9108d90b220472%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637553063002111816%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=U5VbADSJ8HJ%2Bq%2BPCGYmCuwks61s5GZtCI4n%2BdjT0LXc%3D&reserved=0
>>> Yours,
>>> Wan Jiabing
>>>
>> --
>> Rodrigo Siqueira
>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsiqueira.tech%2F&data=04%7C01%7Cchristian.koenig%40amd.com%7C99075b6ba68a4c6d4f9108d90b220472%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637553063002111816%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=U5VbADSJ8HJ%2Bq%2BPCGYmCuwks61s5GZtCI4n%2BdjT0LXc%3D&reserved=0
> Yours,
> Wan Jiabing
>
>