On 2017-10-11 10:06 AM, Alex Deucher wrote: > dc uses this. Not sure how important it is. > > Signed-off-by: Alex Deucher <alexander.deucher at amd.com> > --- > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_services.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_services.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_services.c > index 56bc7cc..2a58f86 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_services.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_services.c > @@ -103,8 +103,7 @@ bool dm_read_persistent_data(struct dc_context *ctx, > void dm_delay_in_microseconds(struct dc_context *ctx, > unsigned int microSeconds) > { > - /*TODO implement*/ > - return; > + udelay(microSeconds); > } > We can drop this completely and replace all calls to it with udelay. Not sure why we have this in the first place. Harry > /**** power component interfaces ****/ >