Received: by 2002:a25:23cc:0:0:0:0:0 with SMTP id j195csp674241ybj; Tue, 5 May 2020 06:02:25 -0700 (PDT) X-Google-Smtp-Source: APiQypJydcth8jEvYOJt1mBejPxJCbjrJvCFBydZLEBtY6lMmt/XZQDYmpZWhngjDwCinZq0QtIW X-Received: by 2002:aa7:dcc3:: with SMTP id w3mr2314951edu.231.1588683744769; Tue, 05 May 2020 06:02:24 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1588683744; cv=none; d=google.com; s=arc-20160816; b=JTgjIjrcH6gO9d3nWuwObI1qawH97nBd202uX7ZgfPwTvQb80xgXsbwnedqEIufCA8 v5REjudEo/oO2TFoOSQ7rtB6RCCDJw5WBJdDx0zixKLyQBItRRSO2m/0XVz7EbiFp5gV LVpMs8hY7bAk2i4dkSGmWG4ZKOFYOlh8PMo7eOPOm05sdzjVnOS/Q3/ZGcIvGD9ELdQg r8t5U7sDDNZrRfiQqYCadc4NUu6miiRi4gNNZg9hp6h3tkCv5Ax/gANWCG8HrEQawOyY Za+gk3wFQfbOXJeYs1sFJ52I1Q6qy16ZyBFJF50CPV9bcg3/bQWg4rDQiE8UG0lRhQlO ge7g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-language :content-transfer-encoding:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=Hy8VkUtLdM41nbk1bxEaxImoR03VVCh7xDr3pWuRWtQ=; b=WxJMDI858csoOst/kdUaScRYU4mcL7GiS8lUNLKh9+c6cJgCRTRs7Z56vJOOEbktK3 TEWEb/hQA0BZTTHmX7sbcF8WYkOKBapFH2CLEDB17rTuQeEavipyL4PPKSIR74KEaiEt deEhz6VnQMIW45KzV2R43oOXjvqeFbOpCELi0zjjQXoeA3J7awPrv6/4e0S3ZygjF/hA PRpw5B1AvCi/03dpP5BQzfT8Y/u5jqZ7AM5w7F27KMQzab4+txnJ83aPlnIMfrh7Sans chqha1UKSlJRuBXfn1Hyq9mGG0U+BWz8v7ltXW264ihkPr26kYS491WXu1M9TqAT9thG 8v2w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id p16si1027056ejd.448.2020.05.05.06.01.58; Tue, 05 May 2020 06:02:24 -0700 (PDT) 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; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729012AbgEEM7u (ORCPT + 99 others); Tue, 5 May 2020 08:59:50 -0400 Received: from cmccmta1.chinamobile.com ([221.176.66.79]:43218 "EHLO cmccmta1.chinamobile.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728268AbgEEM7t (ORCPT ); Tue, 5 May 2020 08:59:49 -0400 Received: from spf.mail.chinamobile.com (unknown[172.16.121.17]) by rmmx-syy-dmz-app03-12003 (RichMail) with SMTP id 2ee35eb16318e0e-0d2cf; Tue, 05 May 2020 20:59:08 +0800 (CST) X-RM-TRANSID: 2ee35eb16318e0e-0d2cf X-RM-TagInfo: emlType=0 X-RM-SPAM-FLAG: 00000000 Received: from [192.168.0.102] (unknown[112.3.208.122]) by rmsmtp-syy-appsvr09-12009 (RichMail) with SMTP id 2ee95eb1631bfc1-6b8d7; Tue, 05 May 2020 20:59:07 +0800 (CST) X-RM-TRANSID: 2ee95eb1631bfc1-6b8d7 Subject: Re: [PATCH v3] ipmi:bt-bmc: Fix error handling and status check To: minyard@acm.org Cc: arnd@arndb.de, gregkh@linuxfoundation.org, openipmi-developer@lists.sourceforge.net, linux-kernel@vger.kernel.org, Shengju Zhang References: <20200505102906.17196-1-tangbin@cmss.chinamobile.com> <20200505114907.GK9902@minyard.net> From: Tang Bin Message-ID: <3610b0fb-25f5-1bf9-a3c1-1c7ca113a9c8@cmss.chinamobile.com> Date: Tue, 5 May 2020 20:59:51 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: <20200505114907.GK9902@minyard.net> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Corey: On 2020/5/5 19:49, Corey Minyard wrote: > On Tue, May 05, 2020 at 06:29:06PM +0800, Tang Bin wrote: >> If the function platform_get_irq() failed, the negative value >> returned will not be detected here. So fix error handling in >> bt_bmc_config_irq(). And in the function bt_bmc_probe(), >> when get irq failed, it will print error message. So use >> platform_get_irq_optional() to simplify code. Finally in the >> function bt_bmc_remove() should make the right status check >> if get irq failed. > Ok, this is included in my tree. I just want to say: thank you very much. After many days of communication with you, I have learned a lot, your patient teaching has inspired me a lot. It's my pleasure to meet you. Thanks, Tang Bin >