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;
IronPort-SDR: B853POwWx+C/LctXjItTXIXyz4MJo0OqgSgiiuYMWaQ0VjYYX8/7yJt6n+1TFmpBZzTKup3BG3
 v3wF5TS/FquA==
IronPort-SDR: N3vv3ND+OnFJU9V0jEIRGgw0vxCUTShGBSE6hTDap1fDoKWpNoKTUhnT+137SEhsp507I8dbry
 7ovP33xCmHgw==
Date:   Thu, 16 Jul 2020 11:52:42 -0700
From:   "Luck, Tony" <tony.luck@intel.com>
To:     Jason Baron <jbaron@akamai.com>
Cc:     bp@suse.de, linux-kernel@vger.kernel.org,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        linux-edac <linux-edac@vger.kernel.org>
Subject: Re: [PATCH] EDAC/ie31200: fallback if host bridge device is already
 initialized
Message-ID: <20200716185242.GA7045@agluck-desk2.amr.corp.intel.com>
References: <1594923911-10885-1-git-send-email-jbaron@akamai.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1594923911-10885-1-git-send-email-jbaron@akamai.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu, Jul 16, 2020 at 02:25:11PM -0400, Jason Baron wrote:
> The Intel uncore driver may claim some of the pci ids from ie31200 which
> means that the ie31200 edac driver will not initialize them as part of
> pci_register_driver().
> 
> Let's add a fallback for this case to 'pci_get_device()' to get a
> reference on the device such that it can still be configured. This is
> similar in approach to other edac drivers.

What functionality is lost when this happens?

Does the user see some message in the console
log to let them know?

-Tony