From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161130AbbEOMst (ORCPT ); Fri, 15 May 2015 08:48:49 -0400 Received: from mail-yh0-f45.google.com ([209.85.213.45]:32931 "EHLO mail-yh0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161018AbbEOMsp (ORCPT ); Fri, 15 May 2015 08:48:45 -0400 MIME-Version: 1.0 In-Reply-To: <5555C2A5.8010509@gmail.com> References: <1431557340-5421-1-git-send-email-robh@kernel.org> <5555C2A5.8010509@gmail.com> From: Rob Herring Date: Fri, 15 May 2015 07:48:24 -0500 X-Google-Sender-Auth: iqmZq0RRdJF6R-h7jp2PlC_h9n0 Message-ID: Subject: Re: [PATCH 0/5] Marvell PXA1928 USB support To: Sebastian Hesselbarth Cc: Greg Kroah-Hartman , Alan Stern , Kishon Vijay Abraham I , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Linux USB List , Jason Cooper , Andrew Lunn , Gregory Clement , Thomas Petazzoni Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 15, 2015 at 4:55 AM, Sebastian Hesselbarth wrote: > On 14.05.2015 00:48, Rob Herring wrote: >> >> This series adds USB PHYs and EHCI host drivers for the Marvell PXA1928 >> SOC. >> >> The OTG block is based on ChipIdea and works with "chipidea,usb2" >> compatible driver as is just by adding the PHY driver. Yay! >> >> Rob >> >> Rob Herring (5): >> dt-bindings: Add Marvell PXA1928 USB and HSIC PHY bindings >> dt-bindings: Add Marvell PXA1928 USB EHCI controller binding >> phy: Add Marvell USB 2.0 OTG 28nm PHY >> phy: add Marvell HSIC 28nm PHY >> usb: add pxa1928 ehci support >> >> .../devicetree/bindings/phy/pxa1928-usb-phy.txt | 18 ++ >> .../devicetree/bindings/usb/ehci-pxa1928.txt | 19 ++ >> drivers/phy/Kconfig | 20 ++ >> drivers/phy/Makefile | 2 + >> drivers/phy/phy-mv-hsic.c | 208 +++++++++++++ >> drivers/phy/phy-mv-usb2.c | 329 >> +++++++++++++++++++++ > > > [Adding some MVEBU guys] > > Rob, > > I had a look at the USB PHYs of some of the other Marvell SoCs a while > ago for the barebox bootloader [1]. Marvell seems to distinguish the > USB PHY type by technology node, e.g. 28nm like the one above. For the > most used Marvell SoCs, i.e. Kirkwood, Dove, and Armada 370/XP, they all > use a different technology node and we could either use the SoC name > or the technology node as compatible. It also seems to be by business unit. The registers in the 40nm version I have look nothing like what are in Armada. It could be the same IP, but PHY IP tends to just be signals and they leave the register interface up the to licensee's imagination. Any idea if there are 28nm Armada parts which have the same phy? It's a good idea to always include the SOC name in the compatible. Every new chip is an opportunity for changes or bugs. > Anyway, if you are introducing new PHY drivers with _that_ generic > names, it will either clash with every other Marvell USB PHYs - > or we'll have to add the PHY code into the drivers above. How about phy-pxa-28nm-{usb2,hsic}? Rob From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH 0/5] Marvell PXA1928 USB support Date: Fri, 15 May 2015 07:48:24 -0500 Message-ID: References: <1431557340-5421-1-git-send-email-robh@kernel.org> <5555C2A5.8010509@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <5555C2A5.8010509-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sebastian Hesselbarth Cc: Greg Kroah-Hartman , Alan Stern , Kishon Vijay Abraham I , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Linux USB List , Jason Cooper , Andrew Lunn , Gregory Clement , Thomas Petazzoni List-Id: devicetree@vger.kernel.org On Fri, May 15, 2015 at 4:55 AM, Sebastian Hesselbarth wrote: > On 14.05.2015 00:48, Rob Herring wrote: >> >> This series adds USB PHYs and EHCI host drivers for the Marvell PXA1928 >> SOC. >> >> The OTG block is based on ChipIdea and works with "chipidea,usb2" >> compatible driver as is just by adding the PHY driver. Yay! >> >> Rob >> >> Rob Herring (5): >> dt-bindings: Add Marvell PXA1928 USB and HSIC PHY bindings >> dt-bindings: Add Marvell PXA1928 USB EHCI controller binding >> phy: Add Marvell USB 2.0 OTG 28nm PHY >> phy: add Marvell HSIC 28nm PHY >> usb: add pxa1928 ehci support >> >> .../devicetree/bindings/phy/pxa1928-usb-phy.txt | 18 ++ >> .../devicetree/bindings/usb/ehci-pxa1928.txt | 19 ++ >> drivers/phy/Kconfig | 20 ++ >> drivers/phy/Makefile | 2 + >> drivers/phy/phy-mv-hsic.c | 208 +++++++++++++ >> drivers/phy/phy-mv-usb2.c | 329 >> +++++++++++++++++++++ > > > [Adding some MVEBU guys] > > Rob, > > I had a look at the USB PHYs of some of the other Marvell SoCs a while > ago for the barebox bootloader [1]. Marvell seems to distinguish the > USB PHY type by technology node, e.g. 28nm like the one above. For the > most used Marvell SoCs, i.e. Kirkwood, Dove, and Armada 370/XP, they all > use a different technology node and we could either use the SoC name > or the technology node as compatible. It also seems to be by business unit. The registers in the 40nm version I have look nothing like what are in Armada. It could be the same IP, but PHY IP tends to just be signals and they leave the register interface up the to licensee's imagination. Any idea if there are 28nm Armada parts which have the same phy? It's a good idea to always include the SOC name in the compatible. Every new chip is an opportunity for changes or bugs. > Anyway, if you are introducing new PHY drivers with _that_ generic > names, it will either clash with every other Marvell USB PHYs - > or we'll have to add the PHY code into the drivers above. How about phy-pxa-28nm-{usb2,hsic}? Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html