Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH][next] drm/amd/display: Fix spelling mistake of function
 name
To:     Colin King <colin.king@canonical.com>, Leo Li <sunpeng.li@amd.com>,
        Alex Deucher <alexander.deucher@amd.com>,
        =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>,
        amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Cc:     kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20210120092624.13061-1-colin.king@canonical.com>
From:   Harry Wentland <harry.wentland@amd.com>
Message-ID: <a06aeb18-b02d-41cd-f717-6ff30ea48bb4@amd.com>
Date:   Wed, 20 Jan 2021 09:46:45 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.6.1
In-Reply-To: <20210120092624.13061-1-colin.king@canonical.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from [172.31.7.88] (165.204.84.11) by BN3PR03CA0078.namprd03.prod.outlook.com (2a01:111:e400:7a4d::38) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3763.10 via Frontend Transport; Wed, 20 Jan 2021 14:46:48 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-HT: Tenant
X-MS-Office365-Filtering-Correlation-Id: 3b6eb747-5503-449c-404d-08d8bd52381d
X-MS-TrafficTypeDiagnostic: MWHPR1201MB0205:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <MWHPR1201MB02059EE029C2FA462C06A4728CA20@MWHPR1201MB0205.namprd12.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:8;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: aNYc6/5eTwHtS4MIJ/C6TGlj83ssXqaqatD3sSMRTtcVTx2XKROSN6ZOeLigp3dzZepUbCmQYYYTsiBBV1L6MkQ1jz53vHI0v8jCtSfhQtkdja5UzVbdmCvkSNOjOoYrazfhakowinFaCMS0ss/oXs+Z1LKbyG9DhqWT+f+shAqKwcuoRqkqDoyafZqokvlx8cs+bpJWKZ7XcIe1QBKKHgjCN64Zthx+Zg/IaF5Ht74kDysatj32mAkyaWFb3RzDIrNvHuySK34WbVf+cwAVZhxCdwSfmZsb0l4G1XOiEHJHmEnxTL4m2uDYsZa5C+9ItHOLAs1Z/C6Stm1eVdIOAr9YdMbT+5GkAG7xGm9fnmLYXtFjKXLzXi2KNObTu2qFoxwMAEMeM4C+EiUwy9MN1FM9vcJWWedtTUqdSTr9lXzSS9BYMypwEwVtJ6rO+3FgPc8c1eC0Im2ZZPHTn733WhM0um3EvccOcXBUF/r2Dk0=
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MW3PR12MB4379.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(396003)(136003)(346002)(366004)(39860400002)(376002)(44832011)(956004)(6666004)(2616005)(2906002)(316002)(478600001)(8936002)(4326008)(26005)(52116002)(8676002)(16526019)(110136005)(186003)(53546011)(6486002)(31686004)(16576012)(5660300002)(86362001)(66946007)(31696002)(66556008)(83380400001)(36756003)(66476007)(45980500001)(43740500002);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData: =?utf-8?B?Y2k5YzJiVU5QdWUvbHBsUjlVWGpqS2c4SXpaUTdJYlR6dHNvU2hYZzF4N3FS?=
 =?utf-8?B?UnFxTzJRU1hTbk14cEg5TDFCM09VdzFIdW0ydjJjK3ZqeDltM2JlVnAxR0V6?=
 =?utf-8?B?ZUdWVjJPSEVUYmVXcGpIYXEwMmdYWUZSQ1NrQU9wNGFmRGhvQzEzYzZwL2lC?=
 =?utf-8?B?VVhtOFlUd1RLLzFKQkRUZXNheDZhYkFKdkVSYkN4dVpkWjBSTFJzZnBUdFk4?=
 =?utf-8?B?RzdpYVBMZEQvSCt4WnVPdkQ4dGxiSUkwVkl1Um9wQ0gwVzE3eGpsclQ3MWRF?=
 =?utf-8?B?OGRXNFZlanVGbHcycDN0ek1BanBRL3hBbWFaS21HOHJTWXZMSFRtQ1lxdy9i?=
 =?utf-8?B?bDVvblVYdUFKM1lZQ3poMEduSDNaU1IyRk9CSlJpNWFrMlZ4ditSNXlQL0Fo?=
 =?utf-8?B?dTNiWVVjeVBmTkNMTUFvdXF4Sm9Kc29qWDZQem5oMnJ6eWtGUFNHTXFWZVUz?=
 =?utf-8?B?c2lyak1vbmhPQ0JHaGlhaTBDcFhQVUNsM3VYTEZXbzR5VmlaMjI0bVpqSTYx?=
 =?utf-8?B?V1VsNFd3ejNPdnZrbDlQYjh6cFpIREVjUytsdFVrY3JxNnZMNUNyS3F6bVYv?=
 =?utf-8?B?R0llL2JiVWl1OS9lOEJITUt3N2wxNjNhelFsbFdsVHFQQmtJTGE2WWxRT05k?=
 =?utf-8?B?QU01VTlEY01haUo2dWNIa2xRQlRLL3ZqVm9ZdXA5Mng2VjIzOEl2TmEzUVAr?=
 =?utf-8?B?eTM5NU12di9jTHFJVFJST09jeGZ0VlZ5R1NkaHlTeXpIWVpNZThnelVMaGor?=
 =?utf-8?B?WmF2YXBiRVNRUGk4aUlCVERnUDBydEF1NzZVdVZEL0RpdVlYb3BaUVlqS0Yx?=
 =?utf-8?B?TzNpUG42alUrZDU0Qy9RVm9xUjhEQkpDQVRCWGwyOWRRZEZoWlVDQWV1TEVW?=
 =?utf-8?B?ZVJaZjlscjVsNm51Q2k1enlnUGxCeFRuSUtzQmFpcUYrWnFoVTVvdzFiV1c5?=
 =?utf-8?B?SzNEUGhNUkExc3gvMmtxclZoOXNqdG1ybzM2K2UxTVhuN1JwTlZwTDRSODMx?=
 =?utf-8?B?YTloRnlFSWs5SmFqUXgzMW50bTJXY2QwMTE0OThTL3JnQTNqeWlDSWMvSnBI?=
 =?utf-8?B?SE10MnhzUWllODEzenRpRWpJNnNUOXdETTdQdi9qb3E2YVpra2lKMitwNy9N?=
 =?utf-8?B?NnJ0SmxGLzJQcFZ6c29sdnExVC9EZmlWUG1rTTVobEVia25sd2xWSUtnUXhr?=
 =?utf-8?B?NnFzcHhCcnRVN0xuSnhUbkcvSlFJLzhMZytGc2U5alg0OTNjeUZOc3YvV1N2?=
 =?utf-8?B?RkIybUxTK3pCOVZLN1VHQll2VytJOFllZjZyUjZ3V3hFSmszYW5lMU1TQU0y?=
 =?utf-8?Q?MuJJzYj7KZs+vn+P+jL7VoaIGhT660PvCr?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3b6eb747-5503-449c-404d-08d8bd52381d
X-MS-Exchange-CrossTenant-AuthSource: MW3PR12MB4379.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Jan 2021 14:46:50.0827
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: cCWPFY5yw8PfqY/D1EHfHh/GFRV5GIPPFKZ6CYuDJIJ2qcNYg4FGVQbTb+JdjoHQTf0OWywoSnjoh0inihSONg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR1201MB0205
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2021-01-20 4:26 a.m., Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
> 

Reviewed-by: Harry Wentland <harry.wentland@amd.com>

Harry

> There are two spelling mistakes of the function name, fix this
> by using __func__ instead of a hard coded name string.
> 
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>   drivers/gpu/drm/amd/display/dc/core/dc_link.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link.c b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
> index c16af3983fdb..91d4130cd2cb 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc_link.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
> @@ -1602,7 +1602,7 @@ static bool dc_link_construct(struct dc_link *link,
>   
>   	link->psr_settings.psr_version = DC_PSR_VERSION_UNSUPPORTED;
>   
> -	DC_LOG_DC("BIOS object table - dc_link_contruct finished successfully.\n");
> +	DC_LOG_DC("BIOS object table - %s finished successfully.\n", __func__);
>   	return true;
>   device_tag_fail:
>   	link->link_enc->funcs->destroy(&link->link_enc);
> @@ -1619,7 +1619,7 @@ static bool dc_link_construct(struct dc_link *link,
>   		link->hpd_gpio = NULL;
>   	}
>   
> -	DC_LOG_DC("BIOS object table - dc_link_contruct failed.\n");
> +	DC_LOG_DC("BIOS object table - %s failed.\n", __func__);
>   	kfree(info);
>   
>   	return false;
>