Hi Kishon, Would you review this patch set? As I said on the cover letter, this patch set is based on some patches. http://thread.gmane.org/gmane.linux.drivers.devicetree/158563 Hi Chanwoo, Thank you very much for the review about extcon usage! Best regards, Yoshihiro Shimoda > From: Chanwoo Choi [mailto:cw00.choi@xxxxxxxxxxx] > Sent: Monday, March 14, 2016 12:36 PM > > Hi Yoshihiro, > > On 2016년 03월 03일 19:09, Yoshihiro Shimoda wrote: > > This patch adds extcon support for otg related channel. > > > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> > > --- > > drivers/phy/Kconfig | 1 + > > drivers/phy/phy-rcar-gen3-usb2.c | 26 ++++++++++++++++++++++++++ > > 2 files changed, 27 insertions(+) > > > > diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig > > index ba68f14..db57d6c 100644 > > --- a/drivers/phy/Kconfig > > +++ b/drivers/phy/Kconfig > > @@ -121,6 +121,7 @@ config PHY_RCAR_GEN2 > > config PHY_RCAR_GEN3_USB2 > > tristate "Renesas R-Car generation 3 USB 2.0 PHY driver" > > depends on OF && ARCH_RENESAS > > + depends on EXTCON > > select GENERIC_PHY > > help > > Support for USB 2.0 PHY found on Renesas R-Car generation 3 SoCs. > > diff --git a/drivers/phy/phy-rcar-gen3-usb2.c b/drivers/phy/phy-rcar-gen3-usb2.c > > index 7b14244..76bb88f 100644 > > --- a/drivers/phy/phy-rcar-gen3-usb2.c > > +++ b/drivers/phy/phy-rcar-gen3-usb2.c > > @@ -12,6 +12,7 @@ > > * published by the Free Software Foundation. > > */ > > > > +#include <linux/extcon.h> > > #include <linux/interrupt.h> > > #include <linux/io.h> > > #include <linux/module.h> > > @@ -77,6 +78,7 @@ > > > > struct rcar_gen3_chan { > > void __iomem *base; > > + struct extcon_dev *extcon; > > struct phy *phy; > > struct regulator *vbus; > > bool has_otg; > > @@ -127,6 +129,9 @@ static void rcar_gen3_init_for_host(struct rcar_gen3_chan *ch) > > rcar_gen3_set_linectrl(ch, 1, 1); > > rcar_gen3_set_host_mode(ch, 1); > > rcar_gen3_enable_vbus_ctrl(ch, 1); > > + > > + extcon_set_cable_state_(ch->extcon, EXTCON_USB_HOST, true); > > + extcon_set_cable_state_(ch->extcon, EXTCON_USB, false); > > } > > > > static void rcar_gen3_init_for_peri(struct rcar_gen3_chan *ch) > > @@ -134,6 +139,9 @@ static void rcar_gen3_init_for_peri(struct rcar_gen3_chan *ch) > > rcar_gen3_set_linectrl(ch, 0, 1); > > rcar_gen3_set_host_mode(ch, 0); > > rcar_gen3_enable_vbus_ctrl(ch, 0); > > + > > + extcon_set_cable_state_(ch->extcon, EXTCON_USB_HOST, false); > > + extcon_set_cable_state_(ch->extcon, EXTCON_USB, true); > > } > > > > static bool rcar_gen3_check_vbus(struct rcar_gen3_chan *ch) > > @@ -272,6 +280,12 @@ static const struct of_device_id rcar_gen3_phy_usb2_match_table[] = { > > }; > > MODULE_DEVICE_TABLE(of, rcar_gen3_phy_usb2_match_table); > > > > +static const unsigned int rcar_gen3_phy_cable[] = { > > + EXTCON_USB, > > + EXTCON_USB_HOST, > > + EXTCON_NONE, > > +}; > > + > > static int rcar_gen3_phy_usb2_probe(struct platform_device *pdev) > > { > > struct device *dev = &pdev->dev; > > @@ -297,11 +311,23 @@ static int rcar_gen3_phy_usb2_probe(struct platform_device *pdev) > > /* call request_irq for OTG */ > > irq = platform_get_irq(pdev, 0); > > if (irq >= 0) { > > + int ret; > > + > > irq = devm_request_irq(dev, irq, rcar_gen3_phy_usb2_irq, > > IRQF_SHARED, dev_name(dev), channel); > > if (irq < 0) > > dev_err(dev, "No irq handler (%d)\n", irq); > > channel->has_otg = true; > > + channel->extcon = devm_extcon_dev_allocate(dev, > > + rcar_gen3_phy_cable); > > + if (IS_ERR(channel->extcon)) > > + return PTR_ERR(channel->extcon); > > + > > + ret = devm_extcon_dev_register(dev, channel->extcon); > > + if (ret < 0) { > > + dev_err(dev, "Failed to register extcon\n"); > > + return ret; > > + } > > } > > > > /* devm_phy_create() will call pm_runtime_enable(dev); */ > > > > Looks good to me about extcon usgae. > > Reviewed-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx> > > Best Regards, > Chanwoo Choi ��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f
