Am 23.03.2018 um 12:10 schrieb Rex Zhu: > [SNIP] > -int amdgpu_device_ip_set_powergating_state(struct amdgpu_device *adev, > +int amdgpu_device_ip_set_powergating_state(void *dev, > enum amd_ip_block_type block_type, > enum amd_powergating_state state) > { > int i, r = 0; > + struct amdgpu_device *adev = dev; The usual coding style is longer lines first and commonly used variables like "i" and "r" last. Apart from that the series is Acked-by: Christian König <christian.koenig at amd.com>. Regards, Christian.