From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261211AbVDQAFd (ORCPT ); Sat, 16 Apr 2005 20:05:33 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261212AbVDQAFd (ORCPT ); Sat, 16 Apr 2005 20:05:33 -0400 Received: from rproxy.gmail.com ([64.233.170.205]:37900 "EHLO rproxy.gmail.com") by vger.kernel.org with ESMTP id S261211AbVDQAF3 convert rfc822-to-8bit (ORCPT ); Sat, 16 Apr 2005 20:05:29 -0400 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=jms4k0Yf+bLa0F9a2gorbs1jiqsRh2aVwW58KndcoBuxHaXqKAANHAEBRMj5ineoYqKb1N+3tNCPOTdWKpAP715DKAg6LAhBkfyVv3y84NvbBSz/f70Q1Vfmkt521M5pA4KUVLC3to5qdtvRyn2xIucdsQGbc0i5qnMiWRg1tr4= Message-ID: <21d7e9970504161705a129893@mail.gmail.com> Date: Sun, 17 Apr 2005 10:05:27 +1000 From: Dave Airlie Reply-To: Dave Airlie To: Greg KH Subject: Re: [PATCH] tpm: Stop taking over the non-unique lpc bus PCI ID, Also timer, stack and enum fixes Cc: Kylene Hall , linux-kernel@vger.kernel.org, jgarzik@pobox.com In-Reply-To: <20050415235250.GA24204@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT Content-Disposition: inline References: <20050415235250.GA24204@kroah.com> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org > NO! DO NOT use pci_find_device(). It is broken for systems with pci > hotplug (which means any pci system). Please use the way the driver > currently works, that is correct. But its not an LPC driver, it only uses a small piece of the LPC, we really do need some sort of bridge driver layer or something for these, then other drivers can sit on top of that, The DRM still uses pci_find_device for the exact same reason, the fb drivers take the PCI device and we have been told we can't use the proper interface, hence one of the needs to merge fb and DRM.. Dave.