Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <c0c7747d-582f-9e90-b664-94f25c0c227f@amd.com>
Date:   Wed, 7 Sep 2022 09:44:06 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCH] drm/ttm: Remove unnecessary '0' values from ret
Content-Language: en-US
To:     Li zeming <zeming@nfschina.com>, ray.huang@amd.com,
        airlied@linux.ie, daniel@ffwll.ch
Cc:     dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
References: <20220907032934.4490-1-zeming@nfschina.com>
From:   =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>
In-Reply-To: <20220907032934.4490-1-zeming@nfschina.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UDlxME9vZEN2cGp3WkFKUmEvMDYwL21OdjVrMVZhaWo3THY1dkEweEFnNk12?=
 =?utf-8?B?dkdIY05EN1QyUFczaklLVmJsaGkzQXR5cTlpZWZJQVNQYmNzdWVEaHR5Qmh3?=
 =?utf-8?B?RDdUOFoycFNMZWVTODFIZFljWWxyM25JMnV1NWZZeFR1cEhqNUJ2LytkdE9B?=
 =?utf-8?B?RkxnNHoxT2dpVWJzbUE0TXhaSXR5ZlYvR0xFWjlmeEpyREh4YmNIc0NBdnRR?=
 =?utf-8?B?NEFxUFBLcmZjaENBS2dmZEFwOXFwMDRrQmJWOHJFMVRYSXhpQkJ1S1J0SWtu?=
 =?utf-8?B?UFgwSlB3UUQyZTM0cWZHK2J4TVFpK1pMQjVmMHp0SVlrMVJpUzlBQTJiUFJR?=
 =?utf-8?B?KzFJNTVkTmticC9hNjE2MmJFUXAybk5TeUZsY2JlNEllSnBTcGpjcjQ5aXll?=
 =?utf-8?B?emttS2FmNjRETFhidkNqV0VBRWlkQzNVS1NwVmdHMzdaZkh2TUk0WC9lSits?=
 =?utf-8?B?endTano5R1FtNnFIalY4QnBOOVZxa1R2L0U0SDdYUWFPYkZkaFZGWWhxdGIz?=
 =?utf-8?B?alhSSDF1d2J4RzZyc3B4b1hMQ3JGUld3UVZvaktaZCtBVDRoSDNXak95aFJK?=
 =?utf-8?B?a0hQWHV4d2Z4TTM5c1FrWEZ5MU5QeHRmK3huZFBDTGQvOEpIaTAveUtDYUZ3?=
 =?utf-8?B?eXZHKy96YmZZR29DSHVWNmwwMjNPOE5zTmlYdFZpaGQ4M2MyakMyY0E0L3M3?=
 =?utf-8?B?cFRlV2lwU0JMVDZGMkkvSitDMjVoMlZKTVJkaFU0UFUxZFFRemowWDZBbFBo?=
 =?utf-8?B?Qm1sWEZ4WW1XNVhkeUFDaFY3MG1uMTNjUUJPRlFmam5RVHJrWmJYbVlFYTg4?=
 =?utf-8?B?dE1zVmxxZUpWOUF1UEpWQzBwTWhUR1VJTDBVRnA2VXBXU24raGRrcmZxRHpE?=
 =?utf-8?B?a3NUb3B4WVlmcFRXbHFTc0lJcUtOaUhDQW5HZFlOei9DTjBMdzE1VlljWDdW?=
 =?utf-8?B?WkprcXFWREIrZ2tWY1B1VEh6TDBoZnFFZ2RQcS9RVFNGY1gzMC9CNjlTUEJw?=
 =?utf-8?B?eE9rYVJHd3RNTXZzYVVVRXNyZGx3ektIeXNxWkViSzhKL3BWRlpnU2hMaDlS?=
 =?utf-8?B?cXJtTWgvczV1V2UwRVluVmZXZ2NqamR5NnhnWmthL1FuemtXc0JnRFZmMTFM?=
 =?utf-8?B?ZWd1VEErOUJTSEVheXFlRjVvSlF1MXJSSFplQkl0a3dhUC8vRm5XTGZNTkI5?=
 =?utf-8?B?Q3hWNm0yelBUbTR3b1pHVUdRMUc5K29HUE44MlE5Yk84ME9GbE5kUWpXR3g5?=
 =?utf-8?B?L0JCa3kyQ0JGNThVNi9LK2RZTjd4MHBsZGZpM0VWaHhudFpHVEhmZllEdzhp?=
 =?utf-8?B?dXg5YThYMUh1UnR6L0dCc25sOXFaYlpYdG1xYkJuOUhKMHJTZGh2MGpBQUZQ?=
 =?utf-8?B?SU9CbGIyWDBCUlIwMWsyR01WNUJnQzFNSERVdlhpSG9DbnI5RUpDMU43eEM5?=
 =?utf-8?B?dmxTNDZmaTdtNkhVWHR2TE90V2MrazlmVHlzd205RVpnMytZdEVjQjVNcUFq?=
 =?utf-8?B?NnBoMnMvcTVKdFJ0c1haVjdJU21nenhGWjdOaytIZENDNjB2by9ZUW1nZGNF?=
 =?utf-8?B?My9nNlp5R0FKQlJzOG92d2Q0QjNDbm9kdEMrRWFCS1B2bGhqbnVDNXg1MFF5?=
 =?utf-8?B?WjR1WUFMMTNXdHh2SExnWUhYbG9yRXZKM0pVQm0xRVdhcG5BZVhDNmMrVFho?=
 =?utf-8?B?NjBkcEM1SHJCY2FrOGFacTBiRW5iSmZVcXZvOTVQZFVEcUJZOU4raDNwa0NW?=
 =?utf-8?B?Y1VoSnlYbnBMSzRqY0JxbXkzU2FOeVhQcnRGMEc2YXZvQ2dOU3hZd3NMTGxM?=
 =?utf-8?B?OW9JaG9pWWlIMitEQ0lXYUpYZUpuQk9IcGZPZFBmS0dhUFJqc1hsRXpHb1Nn?=
 =?utf-8?B?UFVmVVhDU0NVTjE4Si9HRkZ2ZWhNWXVLZVJFYVAxcUsxcU02bkRuWkdMU2lS?=
 =?utf-8?B?eVEvQWJwQnlNN3k5MEd6dXBUZFN5YkRtcFplZXFWUHl1cThXMXBTTDhTVGFh?=
 =?utf-8?B?cEZNTEROWHI1S2RTd2grcU1ZQkVqMysrdTVsd2hPOHdYVmp4V1hFemNJTWN3?=
 =?utf-8?B?ZXlyWE9HcmZ0a1M0MFNVNVQwTTltWHJBS3RhMm9iTHo1UW9RYW9FalIrVEVU?=
 =?utf-8?Q?80ZGGPlGTglWg7W8M0/if983X?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f3372158-7dd1-424b-a62c-08da90a4c18f
X-MS-Exchange-CrossTenant-AuthSource: BN8PR12MB3587.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Sep 2022 07:44:12.5611
 (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: 6FwnRyGUwacHD5qHLoSUlyv+GRCopwjPF65FA3zjQ8wnMxU4cPdF48Dh9IhrgaZ8
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ1PR12MB6313
X-Spam-Status: No, score=-3.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Am 07.09.22 um 05:29 schrieb Li zeming:
> The variable ret is assigned in the judgment branch statement, he does
> not need to initialize the assignment.
>
> Signed-off-by: Li zeming <zeming@nfschina.com>

Reviewed and pushed to drm-misc-next.

Thanks,
Christian.

> ---
>   include/drm/ttm/ttm_bo_driver.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/drm/ttm/ttm_bo_driver.h b/include/drm/ttm/ttm_bo_driver.h
> index 897b88f0bd59..1afa891f488a 100644
> --- a/include/drm/ttm/ttm_bo_driver.h
> +++ b/include/drm/ttm/ttm_bo_driver.h
> @@ -106,7 +106,7 @@ static inline int ttm_bo_reserve(struct ttm_buffer_object *bo,
>   				 bool interruptible, bool no_wait,
>   				 struct ww_acquire_ctx *ticket)
>   {
> -	int ret = 0;
> +	int ret;
>   
>   	if (no_wait) {
>   		bool success;