[PATCH v3 7/8] ARM: dts: omap4: update omap-control-usb nodes
Benoit Cousson
bcousson at baylibre.com
Fri Aug 16 10:30:51 EDT 2013
Hi Roger,
Sorry I missed something in the previous revision :-(
On 16/08/2013 15:09, Roger Quadros wrote:
> Split otghs_ctrl and USB2 PHY power down into separate
> omap-control-usb nodes. Get rid of "ti,type" property.
You should add that you update the usb_otg_hs node accordingly as well.
> CC: Benoit Cousson <bcousson at baylibre.com>
> Signed-off-by: Roger Quadros <rogerq at ti.com>
> ---
> arch/arm/boot/dts/omap4.dtsi | 20 ++++++++++++--------
> 1 files changed, 12 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
> index 22d9f2b..a77dd0a 100644
> --- a/arch/arm/boot/dts/omap4.dtsi
> +++ b/arch/arm/boot/dts/omap4.dtsi
> @@ -519,7 +519,7 @@
> usb2_phy: usb2phy at 4a0ad080 {
> compatible = "ti,omap-usb2";
> reg = <0x4a0ad080 0x58>;
> - ctrl-module = <&omap_control_usb>;
> + ctrl-module = <&omap_control_usb2phy>;
> };
> };
>
> @@ -643,12 +643,16 @@
> };
> };
>
> - omap_control_usb: omap-control-usb at 4a002300 {
> - compatible = "ti,omap-control-usb";
> - reg = <0x4a002300 0x4>,
> - <0x4a00233c 0x4>;
> - reg-names = "control_dev_conf", "otghs_control";
> - ti,type = <1>;
> + omap_control_usb2phy: omap-control-usb at 4a002300 {
> + compatible = "ti,usb2-control-usb";
> + reg = <0x4a002300 0x4>;
> + reg-names = "power";
> + };
> +
> + omap_control_usbotg: omap-control-usb at 4a00233c {
> + compatible = "ti,omap4-control-usb";
> + reg = <0x4a00233c 0x4>;
> + reg-names = "otghs_control";
> };
>
> usb_otg_hs: usb_otg_hs at 4a0ab000 {
> @@ -661,7 +665,7 @@
> multipoint = <1>;
> num-eps = <16>;
> ram-bits = <12>;
> - ti,has-mailbox;
> + ctrl-module = <&omap_control_usbotg>;
In omap-usb.txt, ti,has-mailbox is still marked as mandatory whereas the
ctrl-module is optional.
You should update the usb-otg-hs bindings as well.
BTW, why is that property not prefixed with "ti,"? Is ctrl-module really
meaningful for other arch?
Regards,
Benoit
More information about the linux-arm-kernel
mailing list