From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10on2075.outbound.protection.outlook.com [40.107.93.75]) by gabe.freedesktop.org (Postfix) with ESMTPS id 73C1D10E347 for ; Tue, 29 Aug 2023 14:37:55 +0000 (UTC) Message-ID: Date: Tue, 29 Aug 2023 08:37:52 -0600 Content-Language: en-US To: Aurabindo Pillai , igt-dev@lists.freedesktop.org References: <20230825135053.196364-1-aurabindo.pillai@amd.com> <20230825135053.196364-3-aurabindo.pillai@amd.com> From: Rodrigo Siqueira Jordao In-Reply-To: <20230825135053.196364-3-aurabindo.pillai@amd.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Subject: Re: [igt-dev] [PATCH v2 3/3] lib/amdgpu: checkpatch.pl fixes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: hersenxs.wu@amd.com Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: On 8/25/23 07:50, Aurabindo Pillai wrote: > fix checkpatch.pl identified issue: typo and braces > > Signed-off-by: Aurabindo Pillai > --- > lib/igt_amd.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/lib/igt_amd.c b/lib/igt_amd.c > index b67cc9be0..84c6d7545 100644 > --- a/lib/igt_amd.c > +++ b/lib/igt_amd.c > @@ -1154,7 +1154,8 @@ void igt_amd_allow_edp_hotplug_detect(int drm_fd, char *connector_name, bool ena > close(hpd_fd); > } > > -static bool get_dm_capabilites(int drm_fd, char *buf, size_t size) { > +static bool get_dm_capabilities(int drm_fd, char *buf, size_t size) > +{ > int ret, fd; > bool has_capablities = amd_has_debugfs(drm_fd, DEBUGFS_DM_CAPABILITIES); > > @@ -1183,7 +1184,7 @@ static bool get_dm_capabilites(int drm_fd, char *buf, size_t size) { > * > * @param drm_fd DRM file descriptor > * @return true if dm capabilities interface exists and MALL is supported > - * @return false if capabilites could not be read. > + * @return false if capabilities could not be read. > */ > void igt_amd_get_mall_status(int drm_fd, bool *supported, bool *enabled) > { > @@ -1193,7 +1194,7 @@ void igt_amd_get_mall_status(int drm_fd, bool *supported, bool *enabled) > *supported = false; > *enabled = false; > > - if (!get_dm_capabilites(drm_fd, buf, 1024)) > + if (!get_dm_capabilities(drm_fd, buf, 1024)) > return; > > mall_loc = strstr(buf, "mall supported: yes"); Reviewed-by: Rodrigo Siqueira