Received: by 2002:a05:6902:102b:0:0:0:0 with SMTP id x11csp1687677ybt; Thu, 9 Jul 2020 12:58:14 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzAgyYMRhto47P9kJvbWkFxabSHlG6IfH8u967AHb0nu9pUVIm1WkZl2lAHUOUBpos3u+Mj X-Received: by 2002:a50:d55b:: with SMTP id f27mr36516585edj.312.1594324694103; Thu, 09 Jul 2020 12:58:14 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1594324694; cv=none; d=google.com; s=arc-20160816; b=NPTbnAxOTj52irX/Z5PlKQMguZw2hKU0+caB0K85FHrynsseI0qzgarjrvec+10rjO /AVXs3pHGqglO3JFKLk7CCd8dI4ljDzykr85fuyzhtRyxpyyUAAyxSWLUjj0VNh8aiN8 GjyF/9+DaLfoBl/ADFVNptf3ays2WzIPS9YeTanbR+GUlDf2qv1aJ05XhO2SHdio0WkR G04Gyu40AXGeJ348KPIg5UlraN70OM/Uw21o8P9hSHgS/LQqYIBDR9nWOl+Dc1fvP/W7 uDyNE0Za10mtMABkffKo/nUlgRGxZNinR1tHct/wdPqC3a5rgfA5jMnAvF3+z7qGmzIu ebXg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=f87FobS5tphZ53TIg74HP41DCxPjF/CnQ052D76Odw8=; b=isHIcK3V3gSAUH4ZVjZH4eLF3J61V2qzi4ZYHVEJrSZC6mFERJj7GmOjtqC2ZqdYkp 2KRBsfkhRASL5w3Q8sItpLszIOOhog4Wjnqqpd2dypqFMJuFIOi7xyulUUZD5uBadjoO PHJufSDGBz/mOgO/m8LmVB6xKU9LhmMtaY/SqRRLzpwpEYyLIXEaL+0tvG1txYqIeSM9 M0HjO3LBS7BseLDH2oh68VnqtNnL5iSEaEEAMSegC1H1Uu/4rJOgFGTnji60+5/Oep9b 4thFCq6xTZyvaAZucM/Yh9AS76Gm9ueezOBoyZQUWM75spKiDMAqZtHxK9EvSepFabOL Y5Bg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b="A/WomoT7"; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id qo11si2568311ejb.566.2020.07.09.12.57.50; Thu, 09 Jul 2020 12:58:14 -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; dkim=pass header.i=@gmail.com header.s=20161025 header.b="A/WomoT7"; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726323AbgGIT4A (ORCPT + 99 others); Thu, 9 Jul 2020 15:56:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40988 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726220AbgGITzN (ORCPT ); Thu, 9 Jul 2020 15:55:13 -0400 Received: from mail-pl1-x641.google.com (mail-pl1-x641.google.com [IPv6:2607:f8b0:4864:20::641]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C6FFFC08C5CE; Thu, 9 Jul 2020 12:55:11 -0700 (PDT) Received: by mail-pl1-x641.google.com with SMTP id x11so1250507plo.7; Thu, 09 Jul 2020 12:55:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=f87FobS5tphZ53TIg74HP41DCxPjF/CnQ052D76Odw8=; b=A/WomoT7zM/utL6mitFJEdY5DidT6+KDI5BDIeU4+tvRX22K48TRyogqwhkY2bMMT3 5g1DJy4N/9IcSZRX5mqraUxJfzV49gjBdzptVGAeyNPzoq/OF9LEnAFzpZo0Q6YkraOs 2gxrnlm3zGD21YB0jkL0vtwIoc2hXezDHT9ksvWSmUFFq9khY4KuwX+y/73ghuKsRr4o qKjqrqlxk2hjR90GYCdp95CdfyYwV1vqA7zxgjV3+vUcXYfW0ti+sUAvzjj0yphbXQm1 MnCuklsI3U2FxXdBZZ+0ShsAMZ1jJ1IxTbe0PssGYr3hD4K+SKtobrGbpgszXA93EZuK +z+g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=f87FobS5tphZ53TIg74HP41DCxPjF/CnQ052D76Odw8=; b=JcU6JRSdhiu76xcSneLCUGvHAmykejd/MjQHpZWs0MHn1FI+gJLphrR2oVAGvkTTUQ 7RvXrcjiWIjDiwvb4FYG8HnBpzxyLN/gK+QSqq7cc0zAFi9CNOoxDsXa/emCe8mzs1f6 WuO00QE67WMVwMIguEGTiQ/TQ03uxjnB4G/m9ftf1N50HZ8TLbtzgJl++rWlEzUHjjEs f8ykRN/RPHV3NuVPdOUu+ReKsSUaxE2OUS5OmJ/ffjfHlBKkK+b75lpEhf2qRXSyBYf9 JmqPQrQE4q3iAdpMm/ac88arDJXfwQqNON1aGheXMi830i951eO5P6ZlqeVcSJBpYp5K ALNQ== X-Gm-Message-State: AOAM532hPWSLzb9sujvln8RoD7mycLp1ZsPjnVmo9SA6O88yYNQhzbWN O+1v2bfy2BN47UyCOPxARt/VcZZ0X+GiS7LcnBA= X-Received: by 2002:a17:90b:1b52:: with SMTP id nv18mr1862373pjb.129.1594324511348; Thu, 09 Jul 2020 12:55:11 -0700 (PDT) MIME-Version: 1.0 References: <1591357094-39850-1-git-send-email-luwei32@huawei.com> In-Reply-To: <1591357094-39850-1-git-send-email-luwei32@huawei.com> From: Andy Shevchenko Date: Thu, 9 Jul 2020 22:54:55 +0300 Message-ID: Subject: Re: [PATCH v2] ACPI / platform: Fix return value check in check_acpi_dev() To: Lu Wei Cc: Alex Hung , Darren Hart , Andy Shevchenko , AceLan Kao , Platform Driver , Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jun 5, 2020 at 2:42 PM Lu Wei wrote: > > In the function check_acpi_dev(), if it fails to create > platform device, the return value is ERR_PTR() or NULL. Thus it must > use IS_ERR_OR_NULL to check return value. > Thanks! > Fixes: ecc83e52b28c (intel-hid: new hid event driver for hotkeys) This is not the correct format. Please, fix it in the next version, thanks! > drivers/platform/x86/intel-hid.c | 2 +- > drivers/platform/x86/intel-vbtn.c | 2 +- Please, split per driver. -- With Best Regards, Andy Shevchenko