Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp2796627imm; Mon, 16 Jul 2018 14:29:18 -0700 (PDT) X-Google-Smtp-Source: AAOMgpfr9WGBNIOvkCQr2EUbD3BjIx80p+CF6f1NTYkq2n8MzM+TbVZTfTQ5/rjFOzNwJc7s72UI X-Received: by 2002:a17:902:bd42:: with SMTP id b2-v6mr18023908plx.23.1531776558161; Mon, 16 Jul 2018 14:29:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531776558; cv=none; d=google.com; s=arc-20160816; b=XqSY8ID8aLwC1OGaf5d86H9q1LdvMVEpGtZgDztJ3Lfwfsl/Iji+92VCvsIKF/aTnM z/Ew9bMlRXedLlSFYeZt3U0YEY1P+d1WMH4ZFYTPMgyVr1X+qymiB7LB5crOPNPUOBSF dcI2uC9hdhHMr7jq7EBLcPeFYh4cKbrpwWAjhIBctv9WF4Dr9TiSE/klgIGHrh9beq0i Vg6e1xyi7wXxyQa6ZTOwml2fppZSe0p+xJwvXfqcSWfPYeMmsEi0FHfYwlzNOapTx8rk V74J2ciowFT94yOgCzkqgtmvYKzuTT2bT7gZLBMa8Y2+7Rc2pKJun/Zlq4fSsm3GXa09 9HjQ== 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 :references:in-reply-to:mime-version:dkim-signature :arc-authentication-results; bh=d53myzn0QbaJyHytaN8vLBqiiC4WT1n54JmZmVJ/TVo=; b=PV72jqoPGJ6XT4KJVcDcp+FKENB/XDDaVEFHWPUSJNUxMoDv79uo4DG08tsPrG1D4x v4XMVIL40gc/UTKI4aqh0sFIwbd99WMvPIV4XdyoIzKHqsQz2A6AW3fRsFvVSewdAL2P HFxrKI4fmIHIxBaIz042Fo+NBvfl8GLdFPVFDXAxLMWsvGyG2nqRajEX81F6+v5KhNIk RzFRMidqg5EoGSv2dLv/Nkjq4nx619FSsYjmNP2lbhkD5ctFegjbcdICq57ytEIm9QTG VQhV2N+NHX0K/TJSoHN5Gse2+cOH+I3Fhr2cACqszs8bH8CS24deF8s0t9gtiH7KvgRk OyAA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=o3vRzYjs; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 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. [209.132.180.67]) by mx.google.com with ESMTP id s82-v6si33683224pfj.338.2018.07.16.14.29.03; Mon, 16 Jul 2018 14:29:18 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=o3vRzYjs; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 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 S1730052AbeGPV5m (ORCPT + 99 others); Mon, 16 Jul 2018 17:57:42 -0400 Received: from mail-yw0-f195.google.com ([209.85.161.195]:36720 "EHLO mail-yw0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728413AbeGPV5l (ORCPT ); Mon, 16 Jul 2018 17:57:41 -0400 Received: by mail-yw0-f195.google.com with SMTP id v197-v6so2304044ywg.3 for ; Mon, 16 Jul 2018 14:28:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=d53myzn0QbaJyHytaN8vLBqiiC4WT1n54JmZmVJ/TVo=; b=o3vRzYjsU+58h1Ulw+ZghlW6gMPsj9TG6SPTdwauFc116l8L+OotC614SQIoNKbvem fwL17SMemboqAy/T8M2zHxeTknqY68uPhB0it5WjFSQ8xEUPZKBus9hoIjXoq2rb2Qbx tLKKeqUfBXt2tFf7NrESu4nvQ7RiacZmYxGV9VMhdISS21obD5DsjalOCO/+rV5U6MWs LGLnCaCKDCa2qN9PbJNbpbZldHVeB+yUJWapSWM87l+Pk941u93wXrzhcRWg0rj/PZP3 QxQ9xQgoxpnM1gRLSSgt2l71K02Ej3GGM6DNr8sfGv84f/BKIPMLWl7ZPZCUXLer3v48 GDhQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=d53myzn0QbaJyHytaN8vLBqiiC4WT1n54JmZmVJ/TVo=; b=Mj9bPMgTO9/whDHJ6fjkko5FTitvxJOO3ezKkFqEG+bQXlT88GRyEX3SkF7O5IHO87 v6mnPDi/QHKCcjSLMjS+s3xo2lRzNeqMnrU23A/xHjVUaF7a/tPynLehrb0I/rzkzHiS TOfaEN7wq3JKuChC5s9RJs0tXWbtqSkuRRIRGhz9rX7vRMaExEMkGFetVEs+FV6j6sRV XG8c3ipvCOTXg8cqsC/xNc3+cwESxVQtOnfl5fqr9xGn4Jr2py56oJm6q9Q6jVOzlkRG DXNBFDK/8Bb2NIc7IehxDi3+Iz0NiXa4FwuavC5CR++OSxxjDeXIUey99Ksd6QwUVb5u y23A== X-Gm-Message-State: AOUpUlFr2zPaV1LWJzoYHqLczbFlb4p1VYNfba3+PDSP7fcxWgqAMgCY 9ZtD56k+vxraKDLLETK7qLNeR9VrDEsbHoZt7gU= X-Received: by 2002:a81:8c1:: with SMTP id 184-v6mr8955333ywi.126.1531776505445; Mon, 16 Jul 2018 14:28:25 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:a25:5e42:0:0:0:0:0 with HTTP; Mon, 16 Jul 2018 14:28:24 -0700 (PDT) In-Reply-To: <20180714063904.GH20451@kroah.com> References: <20180714055816.223754-1-toddpoynor@gmail.com> <20180714055816.223754-14-toddpoynor@gmail.com> <20180714063904.GH20451@kroah.com> From: Todd Poynor Date: Mon, 16 Jul 2018 14:28:24 -0700 Message-ID: Subject: Re: [PATCH 13/18] staging: gasket: gasket_enable_dev fixups To: Greg Kroah-Hartman Cc: Rob Springer , John Joseph , Ben Chan , devel@driverdev.osuosl.org, Zhongze Hu , lkml , Simon Que , Guenter Roeck , Todd Poynor , Dmitry Torokhov 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, Jul 13, 2018 at 11:39 PM, Greg Kroah-Hartman wrote: > On Fri, Jul 13, 2018 at 10:58:11PM -0700, Todd Poynor wrote: >> From: Todd Poynor >> >> Remove unnecessary variable. >> >> Bail out if no physical device. >> >> Signed-off-by: Todd Poynor >> --- >> drivers/staging/gasket/gasket_core.c | 19 ++++++------------- >> 1 file changed, 6 insertions(+), 13 deletions(-) >> >> diff --git a/drivers/staging/gasket/gasket_core.c b/drivers/staging/gasket/gasket_core.c >> index 65aa7cf454fb..f7d8f66e8746 100644 >> --- a/drivers/staging/gasket/gasket_core.c >> +++ b/drivers/staging/gasket/gasket_core.c >> @@ -915,8 +915,6 @@ static int gasket_enable_dev( >> { >> int tbl_idx; >> int ret; >> - bool has_dma_ops; >> - struct device *ddev; >> const struct gasket_driver_desc *driver_desc = >> internal_desc->driver_desc; >> >> @@ -934,26 +932,21 @@ static int gasket_enable_dev( >> return ret; >> } >> >> - has_dma_ops = true; >> + if (!gasket_dev->pci_dev) { >> + gasket_log_error(gasket_dev, >> + "%s: no physical device", __func__); >> + return -ENODEV; >> + } > > How can this ever be true? I don't see where this can be removed to > enable this error check to be hit. What am I missing? Agree, I'll split this into two patches for the two things it does, and just remove the unnecessary check. Thanks, > > thanks, > > greg k-h -- Todd