[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.20.1710131938400.6944@hadrien>
Date: Fri, 13 Oct 2017 19:40:05 +0200 (CEST)
From: Julia Lawall <julia.lawall@...6.fr>
To: Harsha Sharma <harshasharmaiitr@...il.com>
cc: alexander.deucher@....com, christian.koenig@....com,
airlied@...ux.ie, amd-gfx@...ts.freedesktop.org,
dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
outreachy-kernel@...glegroups.com
Subject: Re: [Outreachy kernel] [PATCH v2] drm/amd/powerplay: Remove unnecessary
cast on void pointer
> @@ -3400,7 +3400,7 @@ static int smu7_read_sensor(struct pp_hwmgr *hwmgr, int idx,
> static int smu7_find_dpm_states_clocks_in_dpm_table(struct pp_hwmgr *hwmgr, const void *input)
> {
> const struct phm_set_power_state_input *states =
> - (const struct phm_set_power_state_input *)input;
> + input;
Actually, there are some more cleanup opportunities heer and in some other
cases. There is no need for the newline before input.
julia
Powered by blists - more mailing lists