Received: by 2002:a5b:505:0:0:0:0:0 with SMTP id o5csp1907124ybp; Wed, 9 Oct 2019 22:43:58 -0700 (PDT) X-Google-Smtp-Source: APXvYqzQGDvSHdyzBMZcoeoRp0ZQuFQkA5gjI6chXHImpFeXVMSUjJds94aXNrBgHEA7lbxQ0kSZ X-Received: by 2002:a17:906:1390:: with SMTP id f16mr6297352ejc.295.1570686238827; Wed, 09 Oct 2019 22:43:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1570686238; cv=none; d=google.com; s=arc-20160816; b=Ds06HuNxfcY3TFEte7dZn7b6AL3c8s+IpQnnS4sr4nJUW4ecFFpMvy0ACD2xMsg7Hu REvM6IutviWtep64STnIIZNH1cKw6do52EilSCkuHRNMvTfLQdq9bTTnT7bPFNcopZcv +Oz82GzBn6Cc8uqUrpFbxPVfQk81BSuhOVGUzuJZIO6zn/c29ZgE2LVl+Tt/4uE/BA+/ LglKVlILnOS04cjPhG6K191Cz00HT5uKkrZyzetik9uk5mrZItOodHPwe/Bv75UwMloL iRcPrvQiq/jwoHnrv2Tabqz5SpgFWZN7teh1rdCVOtTBEUH6YlotKHO7D47JK2jjx99K gINA== 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=fS3zhEs2E2HrZ3PHJX+bOppih8Vd+cHD1zJ+LmOQ0gU=; b=F0ZgTdM+5RkIWB6/GcHtarwoTU7ORnVmyIhRkgjrib1WnUnU4t7Lvy/YtD6GddTt5u uMha0kkkJZdnEwp2EvmxhKBmEkrXI8aV0+cbgjIRhMN0sRpEtezkH4fjaI2r8IsHVMHb G2aNSDDhZN5jOTNgJJlVfC5Pjqk5YOVKblmvGb4ht69ueNZcwXJIgspvz6XtkXcabm1f Hz4LY5CDJV/JPtHrLGY8Ed9fL+Vh4tYaQhNpEc7vwoQFMsYyAvTAGZwaipkPgv4EMLzQ 0+M2fufNJkrLBVxxzmHAeaf4sbB3Pt546gBH1d1ku/IPcdxHKCb0NsV3+1qF/u9sEQQ6 l23w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=SEOhnRg6; 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 w14si2455151ejv.112.2019.10.09.22.43.35; Wed, 09 Oct 2019 22:43:58 -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=SEOhnRg6; 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 S1732817AbfJJFkp (ORCPT + 99 others); Thu, 10 Oct 2019 01:40:45 -0400 Received: from mail-pg1-f193.google.com ([209.85.215.193]:33432 "EHLO mail-pg1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728912AbfJJFkp (ORCPT ); Thu, 10 Oct 2019 01:40:45 -0400 Received: by mail-pg1-f193.google.com with SMTP id i76so2936824pgc.0; Wed, 09 Oct 2019 22:40:43 -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=fS3zhEs2E2HrZ3PHJX+bOppih8Vd+cHD1zJ+LmOQ0gU=; b=SEOhnRg6TMQTlLecTCBzAHS19lqshY915BrKmvEtHDm+98+lubpBO59C6vhBcA5rcF wHwioY7uZbURd7hlfm3ghfPpQvv0q8zDufXoOekV5oHDkQMbZ2D5Y6TRS4a2bbdZNXFv qHQVvSJvVvBA/gs76ePGS+BNszCyfb7iBmFl9MkG6exctb5ID35QWHpgqdSsO/dRsC2H xXfmLY1OriFIy6r+9Mt1rH6ytl0jY14sqVk1xGXZ0SLPqE3ay2+EiEQSZvo9PIq75o8t EmTijihzNo2bsQWtVb6rGXrKusBnAKpau6tmsKY0lzDFCbVv8N6tUD+1SqmY524+4mQO b6FA== 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=fS3zhEs2E2HrZ3PHJX+bOppih8Vd+cHD1zJ+LmOQ0gU=; b=j6/Ht0eYx3WbmpvW4r3gmZQtunv5ksgPCwZVbtW8FH9z6bXhn+ePhvtlYd2COYcp8g WjERCkuWDwX0wPRc6s8p8K2edzPtCqRgSxK65+qTG0pomaaGq/HOHBcNgzsj+zz3Y2Tw 7aB3mJGEgAigQow4MTnvCwE77FaZjkaCFBVVIdL2wDv/EqC14Oqw2KzoguR2V1l5Vpf6 1+dw7dYM/BptMq0caJk0U/Dqzx63FcM2aYO49sRLA4tOZAn99iDWCT5zoUgG93entCty 9t6CoCkeK9DKvvLXntOHU3ZwY3V9rjvG+2v+LhqPzXfKIxTe1St3FxXcXpZiXxkqjh6F dYLQ== X-Gm-Message-State: APjAAAU2T7iJjYlx7J+9f55HyWjv4EQ5aJms8NBj11/qY7oRM4zOFpeH uZdVlokFeYJA7oBuE+EGSjR1xf3MVQ/ma+f7yik= X-Received: by 2002:a17:90a:9416:: with SMTP id r22mr9177676pjo.20.1570686042755; Wed, 09 Oct 2019 22:40:42 -0700 (PDT) MIME-Version: 1.0 References: <20191009200523.8436-1-stuart.w.hayes@gmail.com> <20191009200523.8436-3-stuart.w.hayes@gmail.com> In-Reply-To: From: Andy Shevchenko Date: Thu, 10 Oct 2019 08:40:29 +0300 Message-ID: Subject: Re: [PATCH 2/3] PCI: pciehp: Wait for PDS if in-band presence is disabled To: Stuart Hayes Cc: Bjorn Helgaas , Austin Bolen , Keith Busch , Alexandru Gagniuc , "Rafael J . Wysocki" , Mika Westerberg , "Gustavo A . R . Silva" , Sinan Kaya , Oza Pawandeep , linux-pci@vger.kernel.org, Linux Kernel Mailing List , Lukas Wunner 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 Thu, Oct 10, 2019 at 8:37 AM Andy Shevchenko wrote: > On Wed, Oct 9, 2019 at 11:05 PM Stuart Hayes wrote: > > +static void pcie_wait_for_presence(struct pci_dev *pdev) > > +{ > > + int timeout = 1250; > > + bool pds; Also this is redundant. Just use the following outside the loop if (!retries) pc_info(...); . > > + u16 slot_status; > > + > > + while (true) { > > + pcie_capability_read_word(pdev, PCI_EXP_SLTSTA, &slot_status); > > + pds = !!(slot_status & PCI_EXP_SLTSTA_PDS); > > + if (pds || timeout <= 0) > > + break; > > + msleep(10); > > + timeout -= 10; > > + } > > Can we avoid infinite loops? They are hard to parse (in most cases, > and especially when it's a timeout loop) > > unsigned int retries = 125; // 1250 ms > > do { > ... > } while (--retries); > > > + > > + if (!pds) > > + pci_info(pdev, "Presence Detect state not set in 1250 msec\n"); > > +} > > -- > With Best Regards, > Andy Shevchenko -- With Best Regards, Andy Shevchenko