On Monday, October 24, 2016 12:36:52 PM CEST Alex Deucher wrote: > On Sat, Oct 22, 2016 at 4:56 AM, Baoyou Xie <baoyou.xie@xxxxxxxxxx> wrote: > > We get a few warnings when building kernel with W=1: > > drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/fiji_smumgr.c:162:5: warning: no previous prototype for 'fiji_setup_pwr_virus' [-Wmissing-prototypes] > > drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/fiji_smc.c:2052:5: warning: no previous prototype for 'fiji_program_mem_timing_parameters' [-Wmissing-prototypes] > > drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/polaris10_smumgr.c:175:5: warning: no previous prototype for 'polaris10_avfs_event_mgr' [-Wmissing-prototypes] > > drivers/gpu/drm/amd/amdgpu/../powerplay/hwmgr/cz_hwmgr.c:1020:5: warning: no previous prototype for 'cz_tf_reset_acp_boot_level' [-Wmissing-prototypes] > > drivers/gpu/drm/amd/amdgpu/../powerplay/hwmgr/smu7_hwmgr.c:92:26: warning: no previous prototype for 'cast_phw_smu7_power_state' [-Wmissing-prototypes] > > .... > > > > In fact, these functions are only used in the file in which they are > > declared and don't need a declaration, but can be made static. > > So this patch marks these functions with 'static'. > > > > Signed-off-by: Baoyou Xie <baoyou.xie@xxxxxxxxxx> > > This was already applied the last time you sent it out. Sorry if I > didn't mention that previously. For some reason the patch hasn't made it into linux-next, so I can see why Baoyou was getting confused here. Can you clarify what the timeline is for the AMD DRM driver patches from between they get applied to the AMD tree to when they make it into linux-next? I've occasionally had a hard time with DRM (and a few other subsystems) with bugfix patches trying to find out whether they got lost or whether they just haven't made it into -next but are in some other tree. Baoyou, when you resend a patch, please try to list explicitly why you are resending it, when it was last sent, and what kind of reply you got (integrating any Ack, listing what changes you did, and if there are no other changes, why you think you have to resend it). Arnd _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
