From: Tony Lindgren <tony@atomide.com> To: Kishon Vijay Abraham I <kishon@ti.com> Cc: Roger Quadros <rogerq@ti.com>, t-kristo@ti.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, balbi@ti.com, bcousson@baylibre.com, linux@arm.linux.org.uk, nsekhar@ti.com Subject: Re: [PATCH 10/10] ARM: dts: omap4: Use "syscon-otghs" instead of "ctrl-module" in USB node Date: Thu, 6 Aug 2015 01:48:13 -0700 [thread overview] Message-ID: <20150806084812.GC4215@atomide.com> (raw) In-Reply-To: <55C21CCD.6050101@ti.com> * Kishon Vijay Abraham I <kishon@ti.com> [150805 07:28]: > Hi Roger, > > On Wednesday 05 August 2015 01:38 PM, Roger Quadros wrote: > > On 05/08/15 11:02, Roger Quadros wrote: > >> Kishon, > >> > >> On 04/08/15 18:30, Kishon Vijay Abraham I wrote: > >>> Add "syscon-otghs" property and remove the deprecated "ctrl-module" > >>> property from MUSB devicetree node. > >>> > >>> Since "omap_control_usbotg" devicetree node is no longer used, remove > >>> it. > >>> > >>> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com> > >>> --- > >>> arch/arm/boot/dts/omap4.dtsi | 8 +------- > >>> 1 file changed, 1 insertion(+), 7 deletions(-) > >>> > >>> diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi > >>> index b27634d..2fb49a5 100644 > >>> --- a/arch/arm/boot/dts/omap4.dtsi > >>> +++ b/arch/arm/boot/dts/omap4.dtsi > >>> @@ -854,12 +854,6 @@ > >>> }; > >>> }; > >>> > >>> - omap_control_usbotg: control-phy@4a00233c { > >>> - compatible = "ti,control-phy-otghs"; > >>> - reg = <0x4a00233c 0x4>; > >>> - reg-names = "otghs_control"; > >>> - }; > >>> - > >>> usb_otg_hs: usb_otg_hs@4a0ab000 { > >>> compatible = "ti,omap4-musb"; > >>> reg = <0x4a0ab000 0x7ff>; > >>> @@ -872,7 +866,7 @@ > >>> multipoint = <1>; > >>> num-eps = <16>; > >>> ram-bits = <12>; > >>> - ctrl-module = <&omap_control_usbotg>; > >>> + syscon-otghs = <&scm_conf 0x33c>; > >> > >> All other properties were of the format "syscon-phy-foo". > >> Why use a different format here? > >> > > > > Ah, looks like we don't have a dedicated phy driver for this? > > We will need to add a PHY driver I guess then and handle this register > > in that driver rather than in the musb driver. > > I was explaining in the other thread to Tony on why this shouldn't be done in a > PHY driver [1]. And I just suggested thhere that it should be in the drivers/phy/phy-omap-usb2.c PHY driver instead of the MUSB driver. Regards, Tony > [1] -> https://lkml.org/lkml/2015/8/5/455
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 10/10] ARM: dts: omap4: Use "syscon-otghs" instead of "ctrl-module" in USB node Date: Thu, 6 Aug 2015 01:48:13 -0700 [thread overview] Message-ID: <20150806084812.GC4215@atomide.com> (raw) In-Reply-To: <55C21CCD.6050101@ti.com> * Kishon Vijay Abraham I <kishon@ti.com> [150805 07:28]: > Hi Roger, > > On Wednesday 05 August 2015 01:38 PM, Roger Quadros wrote: > > On 05/08/15 11:02, Roger Quadros wrote: > >> Kishon, > >> > >> On 04/08/15 18:30, Kishon Vijay Abraham I wrote: > >>> Add "syscon-otghs" property and remove the deprecated "ctrl-module" > >>> property from MUSB devicetree node. > >>> > >>> Since "omap_control_usbotg" devicetree node is no longer used, remove > >>> it. > >>> > >>> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com> > >>> --- > >>> arch/arm/boot/dts/omap4.dtsi | 8 +------- > >>> 1 file changed, 1 insertion(+), 7 deletions(-) > >>> > >>> diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi > >>> index b27634d..2fb49a5 100644 > >>> --- a/arch/arm/boot/dts/omap4.dtsi > >>> +++ b/arch/arm/boot/dts/omap4.dtsi > >>> @@ -854,12 +854,6 @@ > >>> }; > >>> }; > >>> > >>> - omap_control_usbotg: control-phy at 4a00233c { > >>> - compatible = "ti,control-phy-otghs"; > >>> - reg = <0x4a00233c 0x4>; > >>> - reg-names = "otghs_control"; > >>> - }; > >>> - > >>> usb_otg_hs: usb_otg_hs at 4a0ab000 { > >>> compatible = "ti,omap4-musb"; > >>> reg = <0x4a0ab000 0x7ff>; > >>> @@ -872,7 +866,7 @@ > >>> multipoint = <1>; > >>> num-eps = <16>; > >>> ram-bits = <12>; > >>> - ctrl-module = <&omap_control_usbotg>; > >>> + syscon-otghs = <&scm_conf 0x33c>; > >> > >> All other properties were of the format "syscon-phy-foo". > >> Why use a different format here? > >> > > > > Ah, looks like we don't have a dedicated phy driver for this? > > We will need to add a PHY driver I guess then and handle this register > > in that driver rather than in the musb driver. > > I was explaining in the other thread to Tony on why this shouldn't be done in a > PHY driver [1]. And I just suggested thhere that it should be in the drivers/phy/phy-omap-usb2.c PHY driver instead of the MUSB driver. Regards, Tony > [1] -> https://lkml.org/lkml/2015/8/5/455
next prev parent reply other threads:[~2015-08-06 8:48 UTC|newest] Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-08-04 15:30 [PATCH 00/10] ARM: dts: use syscon property instead of ctrl-module Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` [PATCH 01/10] ARM: dts: dra7: Add dt node for PCIe registers in sysctrl space Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-05 7:55 ` Roger Quadros 2015-08-05 7:55 ` Roger Quadros 2015-08-05 7:55 ` Roger Quadros 2015-08-04 15:30 ` [PATCH 02/10] ARM: dts: dra7: Use "syscon-phy-power" instead of "ctrl-module" in SATA PHY node Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-05 7:55 ` Roger Quadros 2015-08-05 7:55 ` Roger Quadros 2015-08-05 7:55 ` Roger Quadros 2015-08-04 15:30 ` [PATCH 03/10] ARM: dts: dra7: Use "syscon-phy-power" and "syscon-pcs" in PCIe " Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` [PATCH 04/10] ARM: dts: dra7: Use "ti,dra7x-usb2-phy2" compatible string for USB2 PHY2 Kishon Vijay Abraham I 2015-08-04 15:30 ` [PATCH 04/10] ARM: dts: dra7: Use "ti, dra7x-usb2-phy2" " Kishon Vijay Abraham I 2015-08-04 15:30 ` [PATCH 04/10] ARM: dts: dra7: Use "ti,dra7x-usb2-phy2" " Kishon Vijay Abraham I 2015-08-04 15:30 ` [PATCH 05/10] ARM: dts: dra7: Use "syscon-phy-power" instead of "ctrl-module" in USB PHY node Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-05 7:56 ` Roger Quadros 2015-08-05 7:56 ` Roger Quadros 2015-08-05 7:56 ` Roger Quadros 2015-08-04 15:30 ` [PATCH 06/10] ARM: dts: am4372: " Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-05 7:58 ` Roger Quadros 2015-08-05 7:58 ` Roger Quadros 2015-08-05 7:58 ` Roger Quadros 2015-08-04 15:30 ` [PATCH 07/10] ARM: dts: OMAP5: " Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-05 7:58 ` Roger Quadros 2015-08-05 7:58 ` Roger Quadros 2015-08-05 7:58 ` Roger Quadros 2015-08-04 15:30 ` [PATCH 08/10] ARM: dts: OMAP5: Use "syscon-phy-power" instead of "ctrl-module" in SATA " Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-05 7:59 ` Roger Quadros 2015-08-05 7:59 ` Roger Quadros 2015-08-05 7:59 ` Roger Quadros 2015-08-04 15:30 ` [PATCH 09/10] ARM: dts: omap4: Use "syscon-phy-power" instead of "ctrl-module" in USB " Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-05 8:01 ` Roger Quadros 2015-08-05 8:01 ` Roger Quadros 2015-08-05 8:01 ` Roger Quadros 2015-08-04 15:30 ` [PATCH 10/10] ARM: dts: omap4: Use "syscon-otghs" instead of "ctrl-module" in USB node Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-04 15:30 ` Kishon Vijay Abraham I 2015-08-05 8:02 ` Roger Quadros 2015-08-05 8:02 ` Roger Quadros 2015-08-05 8:02 ` Roger Quadros 2015-08-05 8:08 ` Roger Quadros 2015-08-05 8:08 ` Roger Quadros 2015-08-05 8:08 ` Roger Quadros 2015-08-05 14:25 ` Kishon Vijay Abraham I 2015-08-05 14:25 ` Kishon Vijay Abraham I 2015-08-05 14:25 ` Kishon Vijay Abraham I 2015-08-06 8:48 ` Tony Lindgren [this message] 2015-08-06 8:48 ` Tony Lindgren
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20150806084812.GC4215@atomide.com \ --to=tony@atomide.com \ --cc=balbi@ti.com \ --cc=bcousson@baylibre.com \ --cc=devicetree@vger.kernel.org \ --cc=kishon@ti.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=nsekhar@ti.com \ --cc=rogerq@ti.com \ --cc=t-kristo@ti.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.