From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Barnes Subject: Re: [PATCH 2/5] Dynamic ID addition doesn't need get_driver() Date: Fri, 27 Jan 2012 10:40:05 -0800 Message-ID: <20120127104005.6dde9416@jbarnes-desktop> References: Mime-Version: 1.0 Content-Type: multipart/signed; micalg=PGP-SHA1; boundary="Sig_/Ah0bVaxQFqREfpvCvj2HeRG"; protocol="application/pgp-signature" Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Alan Stern Cc: Greg KH , Dmitry Torokhov , Jiri Kosina , Dominik Brodowski , linux-input@vger.kernel.org, linux-pci@vger.kernel.org, linux-pcmcia@lists.infradead.org, USB list , Kernel development list List-Id: linux-input@vger.kernel.org --Sig_/Ah0bVaxQFqREfpvCvj2HeRG Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Tue, 24 Jan 2012 13:34:41 -0500 (EST) Alan Stern wrote: > As part of the removal of get_driver()/put_driver(), this patch > (as1511) changes all the places that add dynamic IDs for drivers. > Since these additions are done by writing to the drivers' sysfs > attribute files, and the attributes are removed when the drivers are > unregistered, there is no reason to take an extra reference to the > drivers. >=20 > The one exception is the pci-stub driver, which calls pci_add_dynid() > as part of its registration. But again, there's no reason to take an > extra reference here, because the driver can't be unloaded while it is > being registered. Looks fine, you can add my ack too. Thanks, --=20 Jesse Barnes, Intel Open Source Technology Center --Sig_/Ah0bVaxQFqREfpvCvj2HeRG Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iQIcBAEBAgAGBQJPIu+GAAoJEIEoDkX4Qk9hLD0P/jnz89Va5dr4iXEXhK5PwRvj ia2nWFmZA99R6bD9VaUno7L1GiOGkwQ9k+od/c0g6yQkh657BlL5xhwvYaYQDuOD bukzyrCJl8v0k12ZkbK7SeMaabyvab7hgSenAUtJdJ38qjuog82PMLSirwS7YYvA RFjNip1ewnxVFRfZkXSnHQDdBQtx/S16eHE1CZojNujhfwI7i4tt2O9Pt+iDi17t zS6zh+YnZYnU4UPfNcqjJcBvqNvxq8TQ3Sg+/RAZ07nlp13FTErcXRhipZ7WZa/T rSPXmxbFKLoViRBZ0DvpYS32hGSAjGVdUAJefF1HMlXUeSgLtwIBPJfA+6XL+0pi nE0bO+FMSnapsaZW0ne+A4IObwkMXFLouzYxK939E3n+c/w96HZmEh/FVq00jZeG uhTyepBcXeRJwlj2TOtoPJfAPJhflNBOYBBshG0hsL3QDIWgnM1xkkdX4KJr2yf0 rZ/bA3mcH/GL4BGiEe75zBmRhpk8LhhCAyqoNq+PwFgbgV4l6+FTtZO99WGmzTBe z6elvrGWyvJZu7ONQeVaj9F+1E2GtaJ+Lf6YKjc7YGTG9yGV5N7IIWh/K3eA+NRX yI7/xtscLRVmRIUjyVxrLV1wNmG7NvghAxeUNbqMtpH+voeMBc6KxgTwj4dZva5P ENCAn1Joaf47NCms+H4D =4RkF -----END PGP SIGNATURE----- --Sig_/Ah0bVaxQFqREfpvCvj2HeRG--