From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761498AbbA3KbH (ORCPT ); Fri, 30 Jan 2015 05:31:07 -0500 Received: from mga11.intel.com ([192.55.52.93]:56010 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755803AbbA3KbC (ORCPT ); Fri, 30 Jan 2015 05:31:02 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,862,1389772800"; d="scan'208";a="447472743" Date: Fri, 30 Jan 2015 12:30:39 +0200 From: Heikki Krogerus To: David Cohen Cc: Felipe Balbi , Greg Kroah-Hartman , Baolu Lu , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Kishon Vijay Abraham I , david.e.box@linux.intel.com Subject: Re: [PATCH 8/8] phy: add driver for TI TUSB1210 ULPI PHY Message-ID: <20150130103039.GF2570@kuha.fi.intel.com> References: <1422025978-178336-1-git-send-email-heikki.krogerus@linux.intel.com> <1422025978-178336-9-git-send-email-heikki.krogerus@linux.intel.com> <20150129050951.GC9098@saruman.tx.rr.com> <20150129143053.GC2570@kuha.fi.intel.com> <20150129162049.GG21217@saruman.tx.rr.com> <20150129180416.GB2784@psi-dev26.jf.intel.com> <20150129182538.GC2784@psi-dev26.jf.intel.com> <20150129184719.GD2784@psi-dev26.jf.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150129184719.GD2784@psi-dev26.jf.intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > > > > > > Also, I was chatting in private with David and, apparently, there's a > > > > > > way to request for eye diagram data from BIOS straight. That's more > > > > > > in-line with what we would do for DT-based boots, passing that > > > > > > eye-diagram data as a DT attribute. > > > > > > > > > > > > Care to comment ? If that's the case, I'd rather use that BIOS hook and > > > > > > remove ulpi_read() from probe(). > > > > > > > > > > I'm not familiar with such method. But if there is one, I'm not > > > > > against it. I need to check it. > > > > > > > > David ? Care to comment ? > > > > > > I agree with Heikki's proposal to use BIOS hook and remove ulpi_read() > > > from probe(). That address one of my chicken/egg concerns. > > > > Oops. I misinterpreted the thread. Heikki didn't propose that :) > > Let me get things mature and reply to here. > > I talked to David Box (CC'ed him to this reply). > We can add an integer value to ACPI and request it during phy's probe > using ACPI's Device Specific Data (_DSD). That would end the need to > have phy functional during probe (being compatible to BYT-CR and to > module support) and would be more compatible to DT as well. You can still make modifications to the DSDT?! I am familiar with _DSD, and it does not just allow us to add integer values, but complete named key to value properties compatible with dt, including strings. And in case you guys were not familiar with the unified device property interface, it's a wrapper for apci and dt, and should btw. be used instead of acpi_* or of_* to get properties in drivers from now on. So if we add these properties for the dwc3 device object: "ulpi,vendor", "ulpi,product", we can check them in ULPI bus driver as an alternative for reading the vendor and product registers, and indeed be able to bind the PHY to a driver before it's powered on! P.S. We should not mix BIOS to ACPI. Thanks! -- heikki