From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH v3] phy: rcar-gen2 usb: Add Host/Function switching for USB0 Date: Fri, 10 Jul 2015 19:36:10 +0300 Message-ID: <559FF47A.5000001@cogentembedded.com> References: <1436270121-25924-1-git-send-email-phil.edworthy@renesas.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1436270121-25924-1-git-send-email-phil.edworthy-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Phil Edworthy , Kishon Vijay Abraham I Cc: Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Yoshihiro Shimoda , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org Hello. On 07/07/2015 02:55 PM, Phil Edworthy wrote: > Instead of statically selecting the PHY connection to either the > USBHS (Function) or PCI0 (Host) IP blocks, this change allows the > dts to specifiy gpio pins for the vbus and id signals. Additional These GPIOs don't have anything to do with the PHY, they're interfacing Maxim MAX3355 OTG chip for which I have submitted the extcon driver. > gpio pins are used to control power to an external OTG device and > an override to turn vbus on/off. > Note: the R-Car USB PHY only allows this Host/Function switching > on channel 0. > This has been tested on a r8a7791 based Koelsch board, which uses > a MAX3355 device to supply vbus power when needed. > Signed-off-by: Phil Edworthy I'm inclined to NAK this. You're modifying the wrong driver, I think. WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html