From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [PATCH v1 1/2] usb: phy: Add Kconfig entry for Tegra's PHY driver Date: Mon, 11 Dec 2017 11:02:48 +0100 Message-ID: <20171211100248.GD10671@ulmo> References: Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="LKTjZJSUETSlgu2t" Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Dmitry Osipenko Cc: Felipe Balbi , Alan Stern , Greg Kroah-Hartman , Jonathan Hunter , Peter Chen , linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-tegra@vger.kernel.org --LKTjZJSUETSlgu2t Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Dec 11, 2017 at 02:09:59AM +0300, Dmitry Osipenko wrote: > Add Kconfig entry so that other drivers other than ehci-tegra > (like ChipIdea) could add Tegra's PHY to build dependencies. >=20 > Signed-off-by: Dmitry Osipenko > --- > drivers/usb/host/Kconfig | 2 +- > drivers/usb/phy/Kconfig | 8 ++++++++ > drivers/usb/phy/Makefile | 2 +- > 3 files changed, 10 insertions(+), 2 deletions(-) I don't think we actually build-depend on the PHY driver from the ChipIdea driver. In the past, we've refrained from modelling runtime dependencies using Kconfig because in some cases (such as this) it'll include more than necessary (ChipIdea will automatically pull in the USB PHY driver irrespective of whether or not Tegra is enabled). Thierry --LKTjZJSUETSlgu2t Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAlouV8gACgkQ3SOs138+ s6EX4Q/9G8MlT8qwMzotfpafj8jVbSIbGCe3+SfcCIRKmodJ1VWEYnRGhsAmfyVg GLfmNMFeGtMlte5J8t2oKYAgvv2Fd/GWztofNmlyhm142zDe6KEuVPXk5kiVCG0o 7xLFQ6PdzGFT9UysuiIL3U5YCN2Nfb4BUXgis0GoeZcqaPNaIZ72o9VIHGlbkC5u fDe1C0ODONd2UwBuwqCWcxxZ6Pf4SjjDnAELnKj9ayetWYpxNFQhX19t1lloGG3p 2nCYdyuDdQy7/HZ8ElI96W65akxcWjA7CFJa8wVCEmsSgiDsEnJNUCooKp9XEAVg QAAZm1OXA/njdLX3aAfaXTJsOKseSOkhMSk6wu77Y63o24PZpW0EwqyhjODgUpME OZAOPk6ncJieVx+VQKHOcGC/OWM8hLnP8tdfXLRIx7cyotExvLaspNmOw285A2uK Oraa/eSETKLzXc8yhZR1eD+YZvywpOoIJWv8LCVXyTgLs33rQmdw8a9McVjOFAqa nUOKzo3/bPOrIpLiMaRFyP+ahA1hXwUIwP6GxwOunXXB/tsAilFZ+C9utGaJ6uaN O6rGhNjB3a6moMi/lxtlN4SvbjpbqWpPMlw+JanezfED0poixEY2a0TrrI8e+vuv ogW0p/YlGLOQOT2G9FQ+AePSGTUAeQWGqeoQYDFtJQ7nMUlI6YU= =mUHF -----END PGP SIGNATURE----- --LKTjZJSUETSlgu2t-- From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [v1,1/2] usb: phy: Add Kconfig entry for Tegra's PHY driver From: Thierry Reding Message-Id: <20171211100248.GD10671@ulmo> Date: Mon, 11 Dec 2017 11:02:48 +0100 To: Dmitry Osipenko Cc: Felipe Balbi , Alan Stern , Greg Kroah-Hartman , Jonathan Hunter , Peter Chen , linux-usb@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org List-ID: T24gTW9uLCBEZWMgMTEsIDIwMTcgYXQgMDI6MDk6NTlBTSArMDMwMCwgRG1pdHJ5IE9zaXBlbmtv IHdyb3RlOgo+IEFkZCBLY29uZmlnIGVudHJ5IHNvIHRoYXQgb3RoZXIgZHJpdmVycyBvdGhlciB0 aGFuIGVoY2ktdGVncmEKPiAobGlrZSBDaGlwSWRlYSkgY291bGQgYWRkIFRlZ3JhJ3MgUEhZIHRv IGJ1aWxkIGRlcGVuZGVuY2llcy4KPiAKPiBTaWduZWQtb2ZmLWJ5OiBEbWl0cnkgT3NpcGVua28g PGRpZ2V0eEBnbWFpbC5jb20+Cj4gLS0tCj4gIGRyaXZlcnMvdXNiL2hvc3QvS2NvbmZpZyB8IDIg Ky0KPiAgZHJpdmVycy91c2IvcGh5L0tjb25maWcgIHwgOCArKysrKysrKwo+ICBkcml2ZXJzL3Vz Yi9waHkvTWFrZWZpbGUgfCAyICstCj4gIDMgZmlsZXMgY2hhbmdlZCwgMTAgaW5zZXJ0aW9ucygr KSwgMiBkZWxldGlvbnMoLSkKCkkgZG9uJ3QgdGhpbmsgd2UgYWN0dWFsbHkgYnVpbGQtZGVwZW5k IG9uIHRoZSBQSFkgZHJpdmVyIGZyb20gdGhlCkNoaXBJZGVhIGRyaXZlci4gSW4gdGhlIHBhc3Qs IHdlJ3ZlIHJlZnJhaW5lZCBmcm9tIG1vZGVsbGluZyBydW50aW1lCmRlcGVuZGVuY2llcyB1c2lu ZyBLY29uZmlnIGJlY2F1c2UgaW4gc29tZSBjYXNlcyAoc3VjaCBhcyB0aGlzKSBpdCdsbAppbmNs dWRlIG1vcmUgdGhhbiBuZWNlc3NhcnkgKENoaXBJZGVhIHdpbGwgYXV0b21hdGljYWxseSBwdWxs IGluIHRoZQpVU0IgUEhZIGRyaXZlciBpcnJlc3BlY3RpdmUgb2Ygd2hldGhlciBvciBub3QgVGVn cmEgaXMgZW5hYmxlZCkuCgpUaGllcnJ5Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752340AbdLKKCy (ORCPT ); Mon, 11 Dec 2017 05:02:54 -0500 Received: from mail-qt0-f196.google.com ([209.85.216.196]:41557 "EHLO mail-qt0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750836AbdLKKCv (ORCPT ); Mon, 11 Dec 2017 05:02:51 -0500 X-Google-Smtp-Source: AGs4zMYwaEaHFnvNbASbg+R8K2CkdASwVBQGRRwI+9F3Kg4QYLlTi8jBxNlQnis0AAM3tibfLl616Q== Date: Mon, 11 Dec 2017 11:02:48 +0100 From: Thierry Reding To: Dmitry Osipenko Cc: Felipe Balbi , Alan Stern , Greg Kroah-Hartman , Jonathan Hunter , Peter Chen , linux-usb@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v1 1/2] usb: phy: Add Kconfig entry for Tegra's PHY driver Message-ID: <20171211100248.GD10671@ulmo> References: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="LKTjZJSUETSlgu2t" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --LKTjZJSUETSlgu2t Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Dec 11, 2017 at 02:09:59AM +0300, Dmitry Osipenko wrote: > Add Kconfig entry so that other drivers other than ehci-tegra > (like ChipIdea) could add Tegra's PHY to build dependencies. >=20 > Signed-off-by: Dmitry Osipenko > --- > drivers/usb/host/Kconfig | 2 +- > drivers/usb/phy/Kconfig | 8 ++++++++ > drivers/usb/phy/Makefile | 2 +- > 3 files changed, 10 insertions(+), 2 deletions(-) I don't think we actually build-depend on the PHY driver from the ChipIdea driver. In the past, we've refrained from modelling runtime dependencies using Kconfig because in some cases (such as this) it'll include more than necessary (ChipIdea will automatically pull in the USB PHY driver irrespective of whether or not Tegra is enabled). Thierry --LKTjZJSUETSlgu2t Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAlouV8gACgkQ3SOs138+ s6EX4Q/9G8MlT8qwMzotfpafj8jVbSIbGCe3+SfcCIRKmodJ1VWEYnRGhsAmfyVg GLfmNMFeGtMlte5J8t2oKYAgvv2Fd/GWztofNmlyhm142zDe6KEuVPXk5kiVCG0o 7xLFQ6PdzGFT9UysuiIL3U5YCN2Nfb4BUXgis0GoeZcqaPNaIZ72o9VIHGlbkC5u fDe1C0ODONd2UwBuwqCWcxxZ6Pf4SjjDnAELnKj9ayetWYpxNFQhX19t1lloGG3p 2nCYdyuDdQy7/HZ8ElI96W65akxcWjA7CFJa8wVCEmsSgiDsEnJNUCooKp9XEAVg QAAZm1OXA/njdLX3aAfaXTJsOKseSOkhMSk6wu77Y63o24PZpW0EwqyhjODgUpME OZAOPk6ncJieVx+VQKHOcGC/OWM8hLnP8tdfXLRIx7cyotExvLaspNmOw285A2uK Oraa/eSETKLzXc8yhZR1eD+YZvywpOoIJWv8LCVXyTgLs33rQmdw8a9McVjOFAqa nUOKzo3/bPOrIpLiMaRFyP+ahA1hXwUIwP6GxwOunXXB/tsAilFZ+C9utGaJ6uaN O6rGhNjB3a6moMi/lxtlN4SvbjpbqWpPMlw+JanezfED0poixEY2a0TrrI8e+vuv ogW0p/YlGLOQOT2G9FQ+AePSGTUAeQWGqeoQYDFtJQ7nMUlI6YU= =mUHF -----END PGP SIGNATURE----- --LKTjZJSUETSlgu2t--