On Fri, Jul 28, 2017 at 4:47 AM, Quan, Evan <Evan.Quan at amd.com> wrote: > Oops, the changes were made by a remote windows editor which added the executable bit wrongly. > Thanks for the remind. Will pay attention to it. With the mode fixed, the patch is: Reviewed-by: Alex Deucher <alexander.deucher at amd.com> > > Regards, > Evan >>-----Original Message----- >>From: Michel Dänzer [mailto:michel at daenzer.net] >>Sent: Friday, July 28, 2017 4:38 PM >>To: Quan, Evan <Evan.Quan at amd.com> >>Cc: amd-gfx at lists.freedesktop.org >>Subject: Re: [PATCH 2/3] drm/amd/powerplay: updated vega10 fan control >> >>On 28/07/17 05:00 PM, Evan Quan wrote: >>> Change-Id: Ifbeb1f0c8e195cc9cb1e9cff975284d96d49b193 >>> Signed-off-by: Evan Quan <evan.quan at amd.com> >>> --- >>> drivers/gpu/drm/amd/powerplay/hwmgr/vega10_thermal.c | 6 +----- >>> 1 file changed, 1 insertion(+), 5 deletions(-) >>> mode change 100644 => 100755 >>drivers/gpu/drm/amd/powerplay/hwmgr/vega10_thermal.c >> >>Please fix your setup not to set executable bits for source code files. >> >> >>-- >>Earthling Michel Dänzer | http://www.amd.com >>Libre software enthusiast | Mesa and X developer > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx
