From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754911Ab3LEJSq (ORCPT ); Thu, 5 Dec 2013 04:18:46 -0500 Received: from ch1ehsobe002.messaging.microsoft.com ([216.32.181.182]:11204 "EHLO ch1outboundpool.messaging.microsoft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752154Ab3LEJSk (ORCPT ); Thu, 5 Dec 2013 04:18:40 -0500 X-Forefront-Antispam-Report: CIP:70.37.183.190;KIP:(null);UIP:(null);IPV:NLI;H:mail.freescale.net;RD:none;EFVD:NLI X-SpamScore: -3 X-BigFish: VS-3(z579ehzbb2dI98dI9371I1432Izz1f42h2148h208ch1ee6h1de0h1fdah2073h2146h1202h1e76h2189h1d1ah1d2ah1fc6hzz1de098h177df4h8275eh17326ah8275bh8275dh1de097h186068ha1495iz2dh2a8h839h944hd25hf0ah1220h1288h12a5h12a9h12bdh137ah13b6h1441h1504h1537h153bh162dh1631h1758h18e1h1946h19b5h1ad9h1b0ah1b2fh2222h224fh1fb3h1d0ch1d2eh1d3fh1dfeh1dffh1fe8h1ff5h209eh2216h22d0h2336h1155h) Date: Thu, 5 Dec 2013 16:49:09 +0800 From: Peter Chen To: Daniel Tang CC: , , , Subject: Re: [PATCH 1/2] usb: chipidea: fix mistake in device tree binding of nspire-usb to use vendor name 'lsi' instead of SoC name 'zevio' Message-ID: <20131205084909.GA25943@shlinux1.ap.freescale.net> References: <1386148808-15065-1-git-send-email-dt.tangr@gmail.com> <20131204134433.GD1573@shlinux1.ap.freescale.net> <38AC4BA3-DE10-47A6-B3EF-26872A88AABD@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <38AC4BA3-DE10-47A6-B3EF-26872A88AABD@gmail.com> User-Agent: Mutt/1.5.20 (2009-06-14) X-OriginatorOrg: freescale.com X-FOPE-CONNECTOR: Id%0$Dn%*$RO%0$TLS%0$FQDN%$TlsDn% Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Dec 05, 2013 at 04:44:13PM +1100, Daniel Tang wrote: > Hi, > > On 05/12/2013, at 12:44 AM, Peter Chen wrote: > > > > > lsi is vendor name, what are zevio and nspire? > > Usually, the compatible string should be "vendor_name,soc_name-module_name" > > > > Because this port uses documentation from reverse engineering, it's difficult to work out what is SoC specific and what is device specific. The SoC is Zevio but the driver is written for the TI-Nspire. > Please wrap the line to 80 characters. The driver is written for the TI-Nspire, and you port this driver for SoC Zevio platform? Since you use chipidea ip, we don't care the usb module name at your platform, we only care the soc name which you are running, it can detect the SoC platform at runtime. At your dts patch, it still uses nspire-usb. http://marc.info/?l=linux-usb&m=138614886720024&w=2 Peter > If it's usually "vendor_name,soc_name-module_name", I'll fix up this patch with zevio instead of nspire (and it'll be more consistent with the other drivers). > > >> - reg: Should contain registers location and length > >> - interrupts: Should contain controller interrupt > >> > >> @@ -11,7 +11,7 @@ Recommended properies: > >> Examples: > >> usb0: usb@B0000000 { > >> reg = <0xB0000000 0x1000>; > >> - compatible = "zevio,nspire-usb"; > >> + compatible = "lsi,nspire-usb"; > >> interrupts = <8>; > >> vbus-supply = <&vbus_reg>; > >> }; > >> diff --git a/drivers/usb/chipidea/ci_hdrc_nspire.c b/drivers/usb/chipidea/ci_hdrc_nspire.c > >> index 517ce41..c5c2dde 100644 > >> --- a/drivers/usb/chipidea/ci_hdrc_nspire.c > >> +++ b/drivers/usb/chipidea/ci_hdrc_nspire.c > >> @@ -52,7 +52,7 @@ static int ci_hdrc_nspire_remove(struct platform_device *pdev) > >> } > >> > >> static const struct of_device_id ci_hdrc_nspire_dt_ids[] = { > >> - { .compatible = "zevio,nspire-usb", }, > >> + { .compatible = "lsi,nspire-usb", }, > >> { /* sentinel */ } > >> }; > >> > >> -- > >> 1.7.10.4 > >> > >> -- > >> To unsubscribe from this list: send the line "unsubscribe linux-usb" in > >> the body of a message to majordomo@vger.kernel.org > >> More majordomo info at http://vger.kernel.org/majordomo-info.html > >> > > > > -- > > > > Best Regards, > > Peter Chen > > > > Cheers, > Daniel Tang -- Best Regards, Peter Chen