On 2019-07-18 10:49 a.m., Michel Dänzer wrote: > On 2019-07-15 11:19 p.m., [email protected] wrote: >> From: Eric Yang <[email protected]> >> >> [Why] >> For better readability and reusability >> >> [How] >> Move snippets of BW calculation code into helpers. >> >> Signed-off-by: Eric Yang <[email protected]> >> Reviewed-by: Fatemeh Darbehani <[email protected]> >> Acked-by: Leo Li <[email protected]> >> --- >> [...] >> >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> index 086d2f3b0db3..e9ccf41e31ee 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> @@ -126,6 +126,7 @@ static DEVICE_ATTR(pcie_replay_count, S_IRUGO, >> amdgpu_device_get_pcie_replay_count, NULL); >> >> static void amdgpu_device_get_pcie_info(struct amdgpu_device *adev); >> +static void amdgpu_device_parse_faked_did(struct amdgpu_device *adev); > > This function isn't defined anywhere, producing a warning: > > drivers/gpu/drm//amd/amdgpu/amdgpu_device.c:131:13: warning: > ‘amdgpu_device_parse_faked_did’ declared ‘static’ but never defined > [-Wunused-function] > static void amdgpu_device_parse_faked_did(struct amdgpu_device *adev); > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Oops, I don't know how that managed to get in there. It's not even in the display dir... Sent a fix. Leo _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
