* [PATCH 0/3] Add support for PolarFire SoC's musb controller @ 2022-06-07 13:22 Conor Dooley 2022-06-07 13:22 ` [PATCH 1/3] usb: musb: " Conor Dooley ` (3 more replies) 0 siblings, 4 replies; 10+ messages in thread From: Conor Dooley @ 2022-06-07 13:22 UTC (permalink / raw) To: Bin Liu, Greg Kroah-Hartman Cc: Daire McNamara, Valentina Fernandez, linux-kernel, linux-usb, linux-riscv, Conor Dooley Hey Bin, Greg, Short series here adding support for USB on Microchip PolarFire SoC FPGAs. The kconfig dependancy for INVENTRA_DMA has become a bit of a mouthful, is there a better way of dealing with that? Thanks, Conor. Conor Dooley (3): usb: musb: Add support for PolarFire SoC's musb controller MAINTAINERS: add musb to PolarFire SoC entry riscv: dts: microchip: fix icicle kit's dr_mode MAINTAINERS | 1 + .../boot/dts/microchip/mpfs-icicle-kit.dts | 2 +- drivers/usb/musb/Kconfig | 13 +- drivers/usb/musb/Makefile | 1 + drivers/usb/musb/mpfs.c | 285 ++++++++++++++++++ 5 files changed, 300 insertions(+), 2 deletions(-) create mode 100644 drivers/usb/musb/mpfs.c -- 2.36.1 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/3] usb: musb: Add support for PolarFire SoC's musb controller 2022-06-07 13:22 [PATCH 0/3] Add support for PolarFire SoC's musb controller Conor Dooley @ 2022-06-07 13:22 ` Conor Dooley 2022-06-07 15:18 ` Ben Dooks 2022-06-07 13:22 ` [PATCH 2/3] MAINTAINERS: add musb to PolarFire SoC entry Conor Dooley ` (2 subsequent siblings) 3 siblings, 1 reply; 10+ messages in thread From: Conor Dooley @ 2022-06-07 13:22 UTC (permalink / raw) To: Bin Liu, Greg Kroah-Hartman Cc: Daire McNamara, Valentina Fernandez, linux-kernel, linux-usb, linux-riscv, Conor Dooley Add support for Microchips's PolarFire SoC's musb controller in host, peripheral and otg mode. Tested-by: Valentina Fernandez <valentina.fernandezalanis@microchip.com> Signed-off-by: Conor Dooley <conor.dooley@microchip.com> --- drivers/usb/musb/Kconfig | 13 +- drivers/usb/musb/Makefile | 1 + drivers/usb/musb/mpfs.c | 285 ++++++++++++++++++++++++++++++++++++++ 3 files changed, 298 insertions(+), 1 deletion(-) create mode 100644 drivers/usb/musb/mpfs.c diff --git a/drivers/usb/musb/Kconfig b/drivers/usb/musb/Kconfig index 4d61df6a9b5c..f906dfd360d3 100644 --- a/drivers/usb/musb/Kconfig +++ b/drivers/usb/musb/Kconfig @@ -123,6 +123,17 @@ config USB_MUSB_MEDIATEK select GENERIC_PHY select USB_ROLE_SWITCH +config USB_MUSB_POLARFIRE_SOC + tristate "Microchip PolarFire SoC platforms" + depends on SOC_MICROCHIP_POLARFIRE || COMPILE_TEST + depends on NOP_USB_XCEIV + select USB_MUSB_DUAL_ROLE + help + Say Y here to enable support for USB on Microchip's PolarFire SoC. + + This support is also available as a module. If so, the module + will be called mpfs. + comment "MUSB DMA mode" config MUSB_PIO_ONLY @@ -146,7 +157,7 @@ config USB_UX500_DMA config USB_INVENTRA_DMA bool 'Inventra' - depends on USB_MUSB_OMAP2PLUS || USB_MUSB_MEDIATEK || USB_MUSB_JZ4740 + depends on USB_MUSB_OMAP2PLUS || USB_MUSB_MEDIATEK || USB_MUSB_JZ4740 || USB_MUSB_POLARFIRE_SOC help Enable DMA transfers using Mentor's engine. diff --git a/drivers/usb/musb/Makefile b/drivers/usb/musb/Makefile index 932247360a9f..51dd54a8de49 100644 --- a/drivers/usb/musb/Makefile +++ b/drivers/usb/musb/Makefile @@ -25,6 +25,7 @@ obj-$(CONFIG_USB_MUSB_UX500) += ux500.o obj-$(CONFIG_USB_MUSB_JZ4740) += jz4740.o obj-$(CONFIG_USB_MUSB_SUNXI) += sunxi.o obj-$(CONFIG_USB_MUSB_MEDIATEK) += mediatek.o +obj-$(CONFIG_USB_MUSB_POLARFIRE_SOC) += mpfs.o # the kconfig must guarantee that only one of the # possible I/O schemes will be enabled at a time ... diff --git a/drivers/usb/musb/mpfs.c b/drivers/usb/musb/mpfs.c new file mode 100644 index 000000000000..2b3dda335783 --- /dev/null +++ b/drivers/usb/musb/mpfs.c @@ -0,0 +1,285 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * PolarFire SoC (MPFS) MUSB Glue Layer + * + * Copyright (c) 2020-2022 Microchip Corporation. All rights reserved. + * Based on {omap2430,tusb6010,ux500}.c + * + */ + +#include <linux/clk.h> +#include <linux/dma-mapping.h> +#include <linux/err.h> +#include <linux/io.h> +#include <linux/kernel.h> +#include <linux/module.h> +#include <linux/platform_device.h> +#include <linux/usb/usb_phy_generic.h> +#include "musb_core.h" +#include "musb_dma.h" + +#define MPFS_MUSB_MAX_EP_NUM 8 +#define MPFS_MUSB_RAM_BITS 12 + +struct mpfs_glue { + struct device *dev; + struct platform_device *musb; + struct platform_device *phy; + struct clk *clk; +}; + +static struct musb_fifo_cfg mpfs_musb_mode_cfg[] = { + { .hw_ep_num = 1, .style = FIFO_TX, .maxpacket = 512, }, + { .hw_ep_num = 1, .style = FIFO_RX, .maxpacket = 512, }, + { .hw_ep_num = 2, .style = FIFO_TX, .maxpacket = 512, }, + { .hw_ep_num = 2, .style = FIFO_RX, .maxpacket = 512, }, + { .hw_ep_num = 3, .style = FIFO_TX, .maxpacket = 512, }, + { .hw_ep_num = 3, .style = FIFO_RX, .maxpacket = 512, }, + { .hw_ep_num = 4, .style = FIFO_TX, .maxpacket = 1024, }, + { .hw_ep_num = 4, .style = FIFO_RX, .maxpacket = 4096, }, +}; + +static const struct musb_hdrc_config mpfs_musb_hdrc_config = { + .fifo_cfg = mpfs_musb_mode_cfg, + .fifo_cfg_size = ARRAY_SIZE(mpfs_musb_mode_cfg), + .multipoint = true, + .dyn_fifo = true, + .num_eps = MPFS_MUSB_MAX_EP_NUM, + .ram_bits = MPFS_MUSB_RAM_BITS, +}; + +static irqreturn_t mpfs_musb_interrupt(int irq, void *__hci) +{ + unsigned long flags; + irqreturn_t ret = IRQ_NONE; + struct musb *musb = __hci; + + spin_lock_irqsave(&musb->lock, flags); + + musb->int_usb = musb_readb(musb->mregs, MUSB_INTRUSB); + musb->int_tx = musb_readw(musb->mregs, MUSB_INTRTX); + musb->int_rx = musb_readw(musb->mregs, MUSB_INTRRX); + + if (musb->int_usb || musb->int_tx || musb->int_rx) { + musb_writeb(musb->mregs, MUSB_INTRUSB, musb->int_usb); + musb_writew(musb->mregs, MUSB_INTRTX, musb->int_tx); + musb_writew(musb->mregs, MUSB_INTRRX, musb->int_rx); + ret = musb_interrupt(musb); + } + + spin_unlock_irqrestore(&musb->lock, flags); + + return ret; +} + +static void mpfs_musb_set_vbus(struct musb *musb, int is_on) +{ + u8 devctl; + + /* + * HDRC controls CPEN, but beware current surges during device + * connect. They can trigger transient overcurrent conditions + * that must be ignored. + */ + devctl = musb_readb(musb->mregs, MUSB_DEVCTL); + + if (is_on) { + musb->is_active = 1; + musb->xceiv->otg->default_a = 1; + musb->xceiv->otg->state = OTG_STATE_A_WAIT_VRISE; + devctl |= MUSB_DEVCTL_SESSION; + MUSB_HST_MODE(musb); + } else { + musb->is_active = 0; + + /* + * NOTE: skipping A_WAIT_VFALL -> A_IDLE and + * jumping right to B_IDLE... + */ + musb->xceiv->otg->default_a = 0; + musb->xceiv->otg->state = OTG_STATE_B_IDLE; + devctl &= ~MUSB_DEVCTL_SESSION; + + MUSB_DEV_MODE(musb); + } + + musb_writeb(musb->mregs, MUSB_DEVCTL, devctl); + + dev_dbg(musb->controller, "VBUS %s, devctl %02x\n", + usb_otg_state_string(musb->xceiv->otg->state), + musb_readb(musb->mregs, MUSB_DEVCTL)); +} + +static int mpfs_musb_init(struct musb *musb) +{ + struct device *dev = musb->controller; + + musb->xceiv = devm_usb_get_phy(dev, USB_PHY_TYPE_USB2); + if (IS_ERR(musb->xceiv)) { + dev_err(dev, "HS UDC: no transceiver configured\n"); + return PTR_ERR(musb->xceiv); + } + + musb->dyn_fifo = true; + musb->isr = mpfs_musb_interrupt; + + musb_platform_set_vbus(musb, 1); + + return 0; +} + +static const struct musb_platform_ops mpfs_ops = { + .quirks = MUSB_DMA_INVENTRA, + .init = mpfs_musb_init, + .fifo_mode = 2, +#ifdef CONFIG_USB_INVENTRA_DMA + .dma_init = musbhs_dma_controller_create, + .dma_exit = musbhs_dma_controller_destroy, +#endif + .set_vbus = mpfs_musb_set_vbus +}; + +static int mpfs_probe(struct platform_device *pdev) +{ + struct musb_hdrc_platform_data *pdata = dev_get_platdata(&pdev->dev); + struct mpfs_glue *glue; + struct platform_device *musb_pdev; + struct device *dev = &pdev->dev; + struct resource musb_resources[3]; + struct clk *clk; + int ret; + + glue = devm_kzalloc(dev, sizeof(*glue), GFP_KERNEL); + if (!glue) + return -ENOMEM; + + musb_pdev = platform_device_alloc("musb-hdrc", PLATFORM_DEVID_AUTO); + if (!musb_pdev) { + dev_err(dev, "failed to allocate musb device\n"); + return -ENOMEM; + } + + clk = devm_clk_get(&pdev->dev, NULL); + if (IS_ERR(clk)) { + dev_err(&pdev->dev, "failed to get clock\n"); + ret = PTR_ERR(clk); + goto err_phy_release; + } + + ret = clk_prepare_enable(clk); + if (ret) { + dev_err(&pdev->dev, "failed to enable clock\n"); + goto err_phy_release; + } + + musb_pdev->dev.parent = dev; + musb_pdev->dev.coherent_dma_mask = DMA_BIT_MASK(39); + musb_pdev->dev.dma_mask = &musb_pdev->dev.coherent_dma_mask; + device_set_of_node_from_dev(&musb_pdev->dev, dev); + + glue->dev = dev; + glue->musb = musb_pdev; + glue->clk = clk; + + pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); + if (!pdata) + goto err_clk_disable; + + pdata->config = &mpfs_musb_hdrc_config; + pdata->platform_ops = &mpfs_ops; + + pdata->mode = usb_get_dr_mode(dev); + if (pdata->mode == USB_DR_MODE_UNKNOWN) { + dev_info(dev, "No dr_mode property found, defaulting to otg\n"); + pdata->mode = USB_DR_MODE_OTG; + } + + glue->phy = usb_phy_generic_register(); + if (IS_ERR(glue->phy)) { + dev_err(dev, "failed to register usb-phy %ld\n", + PTR_ERR(glue->phy)); + goto err_clk_disable; + } + + platform_set_drvdata(pdev, glue); + + memset(musb_resources, 0x00, + sizeof(*musb_resources) * ARRAY_SIZE(musb_resources)); + + musb_resources[0].name = pdev->resource[0].name; + musb_resources[0].start = pdev->resource[0].start; + musb_resources[0].end = pdev->resource[0].end; + musb_resources[0].flags = pdev->resource[0].flags; + + musb_resources[1].name = pdev->resource[1].name; + musb_resources[1].start = pdev->resource[1].start; + musb_resources[1].end = pdev->resource[1].end; + musb_resources[1].flags = pdev->resource[1].flags; + + musb_resources[2].name = pdev->resource[2].name; + musb_resources[2].start = pdev->resource[2].start; + musb_resources[2].end = pdev->resource[2].end; + musb_resources[2].flags = pdev->resource[2].flags; + + ret = platform_device_add_resources(musb_pdev, musb_resources, + ARRAY_SIZE(musb_resources)); + if (ret) { + dev_err(dev, "failed to add resources\n"); + goto err_clk_disable; + } + + ret = platform_device_add_data(musb_pdev, pdata, sizeof(*pdata)); + if (ret) { + dev_err(dev, "failed to add platform_data\n"); + goto err_clk_disable; + } + + ret = platform_device_add(musb_pdev); + if (ret) { + dev_err(dev, "failed to register musb device\n"); + goto err_clk_disable; + } + + dev_info(&pdev->dev, "Registered MPFS MUSB driver\n"); + return 0; + +err_clk_disable: + clk_disable_unprepare(clk); + +err_phy_release: + usb_phy_generic_unregister(glue->phy); + platform_device_put(musb_pdev); + return ret; +} + +static int mpfs_remove(struct platform_device *pdev) +{ + struct mpfs_glue *glue = platform_get_drvdata(pdev); + + platform_device_unregister(glue->musb); + usb_phy_generic_unregister(pdev); + + return 0; +} + +#ifdef CONFIG_OF +static const struct of_device_id mpfs_id_table[] = { + { .compatible = "microchip,mpfs-musb" }, + { } +}; +MODULE_DEVICE_TABLE(of, mpfs_id_table); +#endif + +static struct platform_driver mpfs_musb_driver = { + .probe = mpfs_probe, + .remove = mpfs_remove, + .driver = { + .name = "mpfs-musb", + .of_match_table = of_match_ptr(mpfs_id_table) + }, +}; + +module_platform_driver(mpfs_musb_driver); + +MODULE_DESCRIPTION("PolarFire SoC MUSB Glue Layer"); +MODULE_LICENSE("GPL"); -- 2.36.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] usb: musb: Add support for PolarFire SoC's musb controller 2022-06-07 13:22 ` [PATCH 1/3] usb: musb: " Conor Dooley @ 2022-06-07 15:18 ` Ben Dooks 2022-06-12 22:14 ` Conor.Dooley 0 siblings, 1 reply; 10+ messages in thread From: Ben Dooks @ 2022-06-07 15:18 UTC (permalink / raw) To: Conor Dooley, Bin Liu, Greg Kroah-Hartman Cc: Daire McNamara, Valentina Fernandez, linux-kernel, linux-usb, linux-riscv On 07/06/2022 14:22, Conor Dooley wrote: > Add support for Microchips's PolarFire SoC's musb controller in host, > peripheral and otg mode. > > Tested-by: Valentina Fernandez <valentina.fernandezalanis@microchip.com> > Signed-off-by: Conor Dooley <conor.dooley@microchip.com> > --- > drivers/usb/musb/Kconfig | 13 +- > drivers/usb/musb/Makefile | 1 + > drivers/usb/musb/mpfs.c | 285 ++++++++++++++++++++++++++++++++++++++ > 3 files changed, 298 insertions(+), 1 deletion(-) > create mode 100644 drivers/usb/musb/mpfs.c > > diff --git a/drivers/usb/musb/Kconfig b/drivers/usb/musb/Kconfig > index 4d61df6a9b5c..f906dfd360d3 100644 > --- a/drivers/usb/musb/Kconfig > +++ b/drivers/usb/musb/Kconfig > @@ -123,6 +123,17 @@ config USB_MUSB_MEDIATEK > select GENERIC_PHY > select USB_ROLE_SWITCH > > +config USB_MUSB_POLARFIRE_SOC > + tristate "Microchip PolarFire SoC platforms" > + depends on SOC_MICROCHIP_POLARFIRE || COMPILE_TEST > + depends on NOP_USB_XCEIV > + select USB_MUSB_DUAL_ROLE > + help > + Say Y here to enable support for USB on Microchip's PolarFire SoC. > + > + This support is also available as a module. If so, the module > + will be called mpfs. > + > comment "MUSB DMA mode" > > config MUSB_PIO_ONLY > @@ -146,7 +157,7 @@ config USB_UX500_DMA > > config USB_INVENTRA_DMA > bool 'Inventra' > - depends on USB_MUSB_OMAP2PLUS || USB_MUSB_MEDIATEK || USB_MUSB_JZ4740 > + depends on USB_MUSB_OMAP2PLUS || USB_MUSB_MEDIATEK || USB_MUSB_JZ4740 || USB_MUSB_POLARFIRE_SOC > help > Enable DMA transfers using Mentor's engine. > > diff --git a/drivers/usb/musb/Makefile b/drivers/usb/musb/Makefile > index 932247360a9f..51dd54a8de49 100644 > --- a/drivers/usb/musb/Makefile > +++ b/drivers/usb/musb/Makefile > @@ -25,6 +25,7 @@ obj-$(CONFIG_USB_MUSB_UX500) += ux500.o > obj-$(CONFIG_USB_MUSB_JZ4740) += jz4740.o > obj-$(CONFIG_USB_MUSB_SUNXI) += sunxi.o > obj-$(CONFIG_USB_MUSB_MEDIATEK) += mediatek.o > +obj-$(CONFIG_USB_MUSB_POLARFIRE_SOC) += mpfs.o > > # the kconfig must guarantee that only one of the > # possible I/O schemes will be enabled at a time ... > diff --git a/drivers/usb/musb/mpfs.c b/drivers/usb/musb/mpfs.c > new file mode 100644 > index 000000000000..2b3dda335783 > --- /dev/null > +++ b/drivers/usb/musb/mpfs.c > @@ -0,0 +1,285 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * PolarFire SoC (MPFS) MUSB Glue Layer > + * > + * Copyright (c) 2020-2022 Microchip Corporation. All rights reserved. > + * Based on {omap2430,tusb6010,ux500}.c > + * > + */ > + > +#include <linux/clk.h> > +#include <linux/dma-mapping.h> > +#include <linux/err.h> > +#include <linux/io.h> > +#include <linux/kernel.h> > +#include <linux/module.h> > +#include <linux/platform_device.h> > +#include <linux/usb/usb_phy_generic.h> > +#include "musb_core.h" > +#include "musb_dma.h" > + > +#define MPFS_MUSB_MAX_EP_NUM 8 > +#define MPFS_MUSB_RAM_BITS 12 > + > +struct mpfs_glue { > + struct device *dev; > + struct platform_device *musb; > + struct platform_device *phy; > + struct clk *clk; > +}; > + > +static struct musb_fifo_cfg mpfs_musb_mode_cfg[] = { > + { .hw_ep_num = 1, .style = FIFO_TX, .maxpacket = 512, }, > + { .hw_ep_num = 1, .style = FIFO_RX, .maxpacket = 512, }, > + { .hw_ep_num = 2, .style = FIFO_TX, .maxpacket = 512, }, > + { .hw_ep_num = 2, .style = FIFO_RX, .maxpacket = 512, }, > + { .hw_ep_num = 3, .style = FIFO_TX, .maxpacket = 512, }, > + { .hw_ep_num = 3, .style = FIFO_RX, .maxpacket = 512, }, > + { .hw_ep_num = 4, .style = FIFO_TX, .maxpacket = 1024, }, > + { .hw_ep_num = 4, .style = FIFO_RX, .maxpacket = 4096, }, > +}; > + > +static const struct musb_hdrc_config mpfs_musb_hdrc_config = { > + .fifo_cfg = mpfs_musb_mode_cfg, > + .fifo_cfg_size = ARRAY_SIZE(mpfs_musb_mode_cfg), > + .multipoint = true, > + .dyn_fifo = true, > + .num_eps = MPFS_MUSB_MAX_EP_NUM, > + .ram_bits = MPFS_MUSB_RAM_BITS, > +}; > + > +static irqreturn_t mpfs_musb_interrupt(int irq, void *__hci) > +{ > + unsigned long flags; > + irqreturn_t ret = IRQ_NONE; > + struct musb *musb = __hci; > + > + spin_lock_irqsave(&musb->lock, flags); > + > + musb->int_usb = musb_readb(musb->mregs, MUSB_INTRUSB); > + musb->int_tx = musb_readw(musb->mregs, MUSB_INTRTX); > + musb->int_rx = musb_readw(musb->mregs, MUSB_INTRRX); > + > + if (musb->int_usb || musb->int_tx || musb->int_rx) { > + musb_writeb(musb->mregs, MUSB_INTRUSB, musb->int_usb); > + musb_writew(musb->mregs, MUSB_INTRTX, musb->int_tx); > + musb_writew(musb->mregs, MUSB_INTRRX, musb->int_rx); > + ret = musb_interrupt(musb); > + } > + > + spin_unlock_irqrestore(&musb->lock, flags); > + > + return ret; > +} > + > +static void mpfs_musb_set_vbus(struct musb *musb, int is_on) > +{ > + u8 devctl; > + > + /* > + * HDRC controls CPEN, but beware current surges during device > + * connect. They can trigger transient overcurrent conditions > + * that must be ignored. > + */ > + devctl = musb_readb(musb->mregs, MUSB_DEVCTL); > + > + if (is_on) { > + musb->is_active = 1; > + musb->xceiv->otg->default_a = 1; > + musb->xceiv->otg->state = OTG_STATE_A_WAIT_VRISE; > + devctl |= MUSB_DEVCTL_SESSION; > + MUSB_HST_MODE(musb); > + } else { > + musb->is_active = 0; > + > + /* > + * NOTE: skipping A_WAIT_VFALL -> A_IDLE and > + * jumping right to B_IDLE... > + */ > + musb->xceiv->otg->default_a = 0; > + musb->xceiv->otg->state = OTG_STATE_B_IDLE; > + devctl &= ~MUSB_DEVCTL_SESSION; > + > + MUSB_DEV_MODE(musb); > + } > + > + musb_writeb(musb->mregs, MUSB_DEVCTL, devctl); > + > + dev_dbg(musb->controller, "VBUS %s, devctl %02x\n", > + usb_otg_state_string(musb->xceiv->otg->state), > + musb_readb(musb->mregs, MUSB_DEVCTL)); > +} > + > +static int mpfs_musb_init(struct musb *musb) > +{ > + struct device *dev = musb->controller; > + > + musb->xceiv = devm_usb_get_phy(dev, USB_PHY_TYPE_USB2); > + if (IS_ERR(musb->xceiv)) { > + dev_err(dev, "HS UDC: no transceiver configured\n"); > + return PTR_ERR(musb->xceiv); > + } > + > + musb->dyn_fifo = true; > + musb->isr = mpfs_musb_interrupt; > + > + musb_platform_set_vbus(musb, 1); > + > + return 0; > +} > + > +static const struct musb_platform_ops mpfs_ops = { > + .quirks = MUSB_DMA_INVENTRA, > + .init = mpfs_musb_init, > + .fifo_mode = 2, > +#ifdef CONFIG_USB_INVENTRA_DMA > + .dma_init = musbhs_dma_controller_create, > + .dma_exit = musbhs_dma_controller_destroy, > +#endif > + .set_vbus = mpfs_musb_set_vbus > +}; > + > +static int mpfs_probe(struct platform_device *pdev) > +{ > + struct musb_hdrc_platform_data *pdata = dev_get_platdata(&pdev->dev); > + struct mpfs_glue *glue; > + struct platform_device *musb_pdev; > + struct device *dev = &pdev->dev; > + struct resource musb_resources[3]; > + struct clk *clk; > + int ret; > + > + glue = devm_kzalloc(dev, sizeof(*glue), GFP_KERNEL); > + if (!glue) > + return -ENOMEM; > + > + musb_pdev = platform_device_alloc("musb-hdrc", PLATFORM_DEVID_AUTO); > + if (!musb_pdev) { > + dev_err(dev, "failed to allocate musb device\n"); > + return -ENOMEM; > + } > + > + clk = devm_clk_get(&pdev->dev, NULL); > + if (IS_ERR(clk)) { > + dev_err(&pdev->dev, "failed to get clock\n"); > + ret = PTR_ERR(clk); > + goto err_phy_release; > + } > + > + ret = clk_prepare_enable(clk); > + if (ret) { > + dev_err(&pdev->dev, "failed to enable clock\n"); > + goto err_phy_release; > + } > + > + musb_pdev->dev.parent = dev; > + musb_pdev->dev.coherent_dma_mask = DMA_BIT_MASK(39); > + musb_pdev->dev.dma_mask = &musb_pdev->dev.coherent_dma_mask; > + device_set_of_node_from_dev(&musb_pdev->dev, dev); > + > + glue->dev = dev; > + glue->musb = musb_pdev; > + glue->clk = clk; > + > + pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); > + if (!pdata) > + goto err_clk_disable; > + > + pdata->config = &mpfs_musb_hdrc_config; > + pdata->platform_ops = &mpfs_ops; > + > + pdata->mode = usb_get_dr_mode(dev); > + if (pdata->mode == USB_DR_MODE_UNKNOWN) { > + dev_info(dev, "No dr_mode property found, defaulting to otg\n"); > + pdata->mode = USB_DR_MODE_OTG; > + } > + > + glue->phy = usb_phy_generic_register(); > + if (IS_ERR(glue->phy)) { > + dev_err(dev, "failed to register usb-phy %ld\n", > + PTR_ERR(glue->phy)); > + goto err_clk_disable; > + } > + > + platform_set_drvdata(pdev, glue); > + > + memset(musb_resources, 0x00, > + sizeof(*musb_resources) * ARRAY_SIZE(musb_resources)); > + > + musb_resources[0].name = pdev->resource[0].name; > + musb_resources[0].start = pdev->resource[0].start; > + musb_resources[0].end = pdev->resource[0].end; > + musb_resources[0].flags = pdev->resource[0].flags; > + > + musb_resources[1].name = pdev->resource[1].name; > + musb_resources[1].start = pdev->resource[1].start; > + musb_resources[1].end = pdev->resource[1].end; > + musb_resources[1].flags = pdev->resource[1].flags; > + > + musb_resources[2].name = pdev->resource[2].name; > + musb_resources[2].start = pdev->resource[2].start; > + musb_resources[2].end = pdev->resource[2].end; > + musb_resources[2].flags = pdev->resource[2].flags; I wonder if you could memcpy or just use compiler assignment for these? > + > + ret = platform_device_add_resources(musb_pdev, musb_resources, > + ARRAY_SIZE(musb_resources)); > + if (ret) { > + dev_err(dev, "failed to add resources\n"); > + goto err_clk_disable; > + } > + > + ret = platform_device_add_data(musb_pdev, pdata, sizeof(*pdata)); > + if (ret) { > + dev_err(dev, "failed to add platform_data\n"); > + goto err_clk_disable; > + } > + > + ret = platform_device_add(musb_pdev); > + if (ret) { > + dev_err(dev, "failed to register musb device\n"); > + goto err_clk_disable; > + } > + > + dev_info(&pdev->dev, "Registered MPFS MUSB driver\n"); > + return 0; > + > +err_clk_disable: > + clk_disable_unprepare(clk); > + > +err_phy_release: > + usb_phy_generic_unregister(glue->phy); > + platform_device_put(musb_pdev); > + return ret; > +} > + > +static int mpfs_remove(struct platform_device *pdev) > +{ > + struct mpfs_glue *glue = platform_get_drvdata(pdev); > + > + platform_device_unregister(glue->musb); > + usb_phy_generic_unregister(pdev); > + > + return 0; > +} > + > +#ifdef CONFIG_OF > +static const struct of_device_id mpfs_id_table[] = { > + { .compatible = "microchip,mpfs-musb" }, > + { } > +}; > +MODULE_DEVICE_TABLE(of, mpfs_id_table); > +#endif > + > +static struct platform_driver mpfs_musb_driver = { > + .probe = mpfs_probe, > + .remove = mpfs_remove, > + .driver = { > + .name = "mpfs-musb", > + .of_match_table = of_match_ptr(mpfs_id_table) > + }, > +}; > + > +module_platform_driver(mpfs_musb_driver); > + > +MODULE_DESCRIPTION("PolarFire SoC MUSB Glue Layer"); > +MODULE_LICENSE("GPL"); -- Ben Dooks http://www.codethink.co.uk/ Senior Engineer Codethink - Providing Genius https://www.codethink.co.uk/privacy.html ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] usb: musb: Add support for PolarFire SoC's musb controller 2022-06-07 15:18 ` Ben Dooks @ 2022-06-12 22:14 ` Conor.Dooley 0 siblings, 0 replies; 10+ messages in thread From: Conor.Dooley @ 2022-06-12 22:14 UTC (permalink / raw) To: ben.dooks, b-liu, gregkh Cc: Daire.McNamara, Valentina.FernandezAlanis, linux-kernel, linux-usb, linux-riscv On 07/06/2022 16:18, Ben Dooks wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On 07/06/2022 14:22, Conor Dooley wrote: >> Add support for Microchips's PolarFire SoC's musb controller in host, >> peripheral and otg mode. >> >> Tested-by: Valentina Fernandez <valentina.fernandezalanis@microchip.com> >> Signed-off-by: Conor Dooley <conor.dooley@microchip.com> >> --- >> drivers/usb/musb/Kconfig | 13 +- >> drivers/usb/musb/Makefile | 1 + >> drivers/usb/musb/mpfs.c | 285 ++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 298 insertions(+), 1 deletion(-) >> create mode 100644 drivers/usb/musb/mpfs.c >> ---8<--- >> + >> + platform_set_drvdata(pdev, glue); >> + >> + memset(musb_resources, 0x00, >> + sizeof(*musb_resources) * ARRAY_SIZE(musb_resources)); >> + >> + musb_resources[0].name = pdev->resource[0].name; >> + musb_resources[0].start = pdev->resource[0].start; >> + musb_resources[0].end = pdev->resource[0].end; >> + musb_resources[0].flags = pdev->resource[0].flags; >> + >> + musb_resources[1].name = pdev->resource[1].name; >> + musb_resources[1].start = pdev->resource[1].start; >> + musb_resources[1].end = pdev->resource[1].end; >> + musb_resources[1].flags = pdev->resource[1].flags; >> + >> + musb_resources[2].name = pdev->resource[2].name; >> + musb_resources[2].start = pdev->resource[2].start; >> + musb_resources[2].end = pdev->resource[2].end; >> + musb_resources[2].flags = pdev->resource[2].flags; > > I wonder if you could memcpy or just use compiler assignment for > these? > Looks like since we wrote the driver, Rob's changed all the others to do this more nicely: https://git.kernel.org/torvalds/c/9879c81b6807 >> + >> + ret = platform_device_add_resources(musb_pdev, musb_resources, >> + ARRAY_SIZE(musb_resources)); >> + if (ret) { >> + dev_err(dev, "failed to add resources\n"); >> + goto err_clk_disable; >> + } >> + >> + ret = platform_device_add_data(musb_pdev, pdata, sizeof(*pdata)); >> + if (ret) { >> + dev_err(dev, "failed to add platform_data\n"); >> + goto err_clk_disable; >> + } >> + >> + ret = platform_device_add(musb_pdev); >> + if (ret) { >> + dev_err(dev, "failed to register musb device\n"); >> + goto err_clk_disable; >> + } >> + >> + dev_info(&pdev->dev, "Registered MPFS MUSB driver\n"); >> + return 0; >> + >> +err_clk_disable: >> + clk_disable_unprepare(clk); >> + >> +err_phy_release: >> + usb_phy_generic_unregister(glue->phy); >> + platform_device_put(musb_pdev); >> + return ret; >> +} >> + >> +static int mpfs_remove(struct platform_device *pdev) >> +{ >> + struct mpfs_glue *glue = platform_get_drvdata(pdev); >> + >> + platform_device_unregister(glue->musb); >> + usb_phy_generic_unregister(pdev); >> + >> + return 0; >> +} >> + >> +#ifdef CONFIG_OF >> +static const struct of_device_id mpfs_id_table[] = { >> + { .compatible = "microchip,mpfs-musb" }, >> + { } >> +}; >> +MODULE_DEVICE_TABLE(of, mpfs_id_table); >> +#endif >> + >> +static struct platform_driver mpfs_musb_driver = { >> + .probe = mpfs_probe, >> + .remove = mpfs_remove, >> + .driver = { >> + .name = "mpfs-musb", >> + .of_match_table = of_match_ptr(mpfs_id_table) >> + }, >> +}; >> + >> +module_platform_driver(mpfs_musb_driver); >> + >> +MODULE_DESCRIPTION("PolarFire SoC MUSB Glue Layer"); >> +MODULE_LICENSE("GPL"); > > > -- > Ben Dooks http://www.codethink.co.uk/ > Senior Engineer Codethink - Providing Genius > > https://www.codethink.co.uk/privacy.html ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 2/3] MAINTAINERS: add musb to PolarFire SoC entry 2022-06-07 13:22 [PATCH 0/3] Add support for PolarFire SoC's musb controller Conor Dooley 2022-06-07 13:22 ` [PATCH 1/3] usb: musb: " Conor Dooley @ 2022-06-07 13:22 ` Conor Dooley 2022-06-07 13:22 ` [PATCH 3/3] riscv: dts: microchip: fix icicle kit's dr_mode Conor Dooley 2022-06-07 17:53 ` [PATCH 0/3] Add support for PolarFire SoC's musb controller Conor.Dooley 3 siblings, 0 replies; 10+ messages in thread From: Conor Dooley @ 2022-06-07 13:22 UTC (permalink / raw) To: Bin Liu, Greg Kroah-Hartman Cc: Daire McNamara, Valentina Fernandez, linux-kernel, linux-usb, linux-riscv, Conor Dooley Add the newly introduced musb driver to the existing PolarFire SoC entry. Signed-off-by: Conor Dooley <conor.dooley@microchip.com> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index a6d3bd9d2a8d..b07b2cfa3567 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -17143,6 +17143,7 @@ S: Supported F: arch/riscv/boot/dts/microchip/ F: drivers/mailbox/mailbox-mpfs.c F: drivers/soc/microchip/ +F: drivers/usb/musb/mpfs.c F: include/soc/microchip/mpfs.h RNBD BLOCK DRIVERS -- 2.36.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/3] riscv: dts: microchip: fix icicle kit's dr_mode 2022-06-07 13:22 [PATCH 0/3] Add support for PolarFire SoC's musb controller Conor Dooley 2022-06-07 13:22 ` [PATCH 1/3] usb: musb: " Conor Dooley 2022-06-07 13:22 ` [PATCH 2/3] MAINTAINERS: add musb to PolarFire SoC entry Conor Dooley @ 2022-06-07 13:22 ` Conor Dooley 2022-06-08 12:21 ` Heinrich Schuchardt 2022-06-07 17:53 ` [PATCH 0/3] Add support for PolarFire SoC's musb controller Conor.Dooley 3 siblings, 1 reply; 10+ messages in thread From: Conor Dooley @ 2022-06-07 13:22 UTC (permalink / raw) To: Bin Liu, Greg Kroah-Hartman Cc: Daire McNamara, Valentina Fernandez, linux-kernel, linux-usb, linux-riscv, Conor Dooley The usb on icicle kit is actually configured for otg, not host mode. Swap it over. Fixes: 528a5b1f2556 ("riscv: dts: microchip: add new peripherals to icicle kit device tree") Signed-off-by: Conor Dooley <conor.dooley@microchip.com> --- arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts b/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts index 044982a11df5..121b01c8784f 100644 --- a/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts +++ b/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts @@ -158,5 +158,5 @@ &syscontroller { &usb { status = "okay"; - dr_mode = "host"; + dr_mode = "otg"; }; -- 2.36.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 3/3] riscv: dts: microchip: fix icicle kit's dr_mode 2022-06-07 13:22 ` [PATCH 3/3] riscv: dts: microchip: fix icicle kit's dr_mode Conor Dooley @ 2022-06-08 12:21 ` Heinrich Schuchardt 2022-06-08 12:49 ` Conor.Dooley 0 siblings, 1 reply; 10+ messages in thread From: Heinrich Schuchardt @ 2022-06-08 12:21 UTC (permalink / raw) To: Conor Dooley Cc: Daire McNamara, Valentina Fernandez, linux-kernel, linux-usb, linux-riscv, Greg Kroah-Hartman, Bin Liu On 6/7/22 15:22, Conor Dooley wrote: > The usb on icicle kit is actually configured for otg, not host mode. > Swap it over. To my understanding both host mode and OTG mode could be supported by the hardware. Jumper J17 on https://www.microsemi.com/images/10/MicrosoftTeams-image%201.png. USB device mode selection - Open: USB client. Closed: USB host Is OTG really the most important scenario for running the board? I personally would prefer to run it in host mode to add more peripherals. What is missing on the driver side to enable host mode? Can we detect the 5 Volt jumper setting via GPIO? Best regards Heinrich > > Fixes: 528a5b1f2556 ("riscv: dts: microchip: add new peripherals to icicle kit device tree") > Signed-off-by: Conor Dooley<conor.dooley@microchip.com> > --- > arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts b/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts > index 044982a11df5..121b01c8784f 100644 > --- a/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts > +++ b/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts > @@ -158,5 +158,5 @@ &syscontroller { > > &usb { > status = "okay"; > - dr_mode = "host"; > + dr_mode = "otg"; > }; > -- 2.36.1 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 3/3] riscv: dts: microchip: fix icicle kit's dr_mode 2022-06-08 12:21 ` Heinrich Schuchardt @ 2022-06-08 12:49 ` Conor.Dooley 2022-06-09 8:23 ` Conor.Dooley 0 siblings, 1 reply; 10+ messages in thread From: Conor.Dooley @ 2022-06-08 12:49 UTC (permalink / raw) To: heinrich.schuchardt Cc: Daire.McNamara, Valentina.FernandezAlanis, linux-kernel, linux-usb, linux-riscv, gregkh, b-liu On 08/06/2022 13:21, Heinrich Schuchardt wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On 6/7/22 15:22, Conor Dooley wrote: >> The usb on icicle kit is actually configured for otg, not host mode. >> Swap it over. > > To my understanding both host mode and OTG mode could be supported by > the hardware. > > Jumper J17 on > https://www.microsemi.com/images/10/MicrosoftTeams-image%201.png. > USB device mode selection - Open: USB client. Closed: USB host > > Is OTG really the most important scenario for running the board? I > personally would prefer to run it in host mode to add more peripherals. > What is missing on the driver side to enable host mode? Nothing. I'll drop the the patch for v2 :) > Can we detect the 5 Volt jumper setting via GPIO? I have to check if a GPIO can be routed that way & I'll get back to you. Thanks, Conor. > > Best regards > > Heinrich > >> >> Fixes: 528a5b1f2556 ("riscv: dts: microchip: add new peripherals to icicle kit device tree") >> Signed-off-by: Conor Dooley<conor.dooley@microchip.com> >> --- >> arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts b/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts >> index 044982a11df5..121b01c8784f 100644 >> --- a/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts >> +++ b/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts >> @@ -158,5 +158,5 @@ &syscontroller { >> >> &usb { >> status = "okay"; >> - dr_mode = "host"; >> + dr_mode = "otg"; >> }; >> -- 2.36.1 > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 3/3] riscv: dts: microchip: fix icicle kit's dr_mode 2022-06-08 12:49 ` Conor.Dooley @ 2022-06-09 8:23 ` Conor.Dooley 0 siblings, 0 replies; 10+ messages in thread From: Conor.Dooley @ 2022-06-09 8:23 UTC (permalink / raw) To: heinrich.schuchardt Cc: Daire.McNamara, Valentina.FernandezAlanis, linux-kernel, linux-usb, linux-riscv, gregkh, b-liu On 08/06/2022 13:47, Conor Dooley wrote: > On 08/06/2022 13:21, Heinrich Schuchardt wrote: >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe >> >> On 6/7/22 15:22, Conor Dooley wrote: >>> The usb on icicle kit is actually configured for otg, not host mode. >>> Swap it over. >> >> To my understanding both host mode and OTG mode could be supported by >> the hardware. >> >> Jumper J17 on >> https://www.microsemi.com/images/10/MicrosoftTeams-image%201.png. >> USB device mode selection - Open: USB client. Closed: USB host >> >> Is OTG really the most important scenario for running the board? I >> personally would prefer to run it in host mode to add more peripherals. >> What is missing on the driver side to enable host mode? > > Nothing. I'll drop the the patch for v2 :) > >> Can we detect the 5 Volt jumper setting via GPIO? > > I have to check if a GPIO can be routed that way & I'll get back to you. Unfortunately not, that jumper can't be connected to a GPIO. > Thanks, > Conor. > >> >> Best regards >> >> Heinrich >> >>> >>> Fixes: 528a5b1f2556 ("riscv: dts: microchip: add new peripherals to icicle kit device tree") >>> Signed-off-by: Conor Dooley<conor.dooley@microchip.com> >>> --- >>> arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts b/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts >>> index 044982a11df5..121b01c8784f 100644 >>> --- a/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts >>> +++ b/arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dts >>> @@ -158,5 +158,5 @@ &syscontroller { >>> >>> &usb { >>> status = "okay"; >>> - dr_mode = "host"; >>> + dr_mode = "otg"; >>> }; >>> -- 2.36.1 >> > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 0/3] Add support for PolarFire SoC's musb controller 2022-06-07 13:22 [PATCH 0/3] Add support for PolarFire SoC's musb controller Conor Dooley ` (2 preceding siblings ...) 2022-06-07 13:22 ` [PATCH 3/3] riscv: dts: microchip: fix icicle kit's dr_mode Conor Dooley @ 2022-06-07 17:53 ` Conor.Dooley 3 siblings, 0 replies; 10+ messages in thread From: Conor.Dooley @ 2022-06-07 17:53 UTC (permalink / raw) To: b-liu, gregkh Cc: Daire.McNamara, Valentina.FernandezAlanis, linux-kernel, linux-usb, linux-riscv On 07/06/2022 14:22, Conor Dooley wrote: > Hey Bin, Greg, > Short series here adding support for USB on Microchip PolarFire SoC FPGAs. > The kconfig dependancy for INVENTRA_DMA has become a bit of a mouthful, > is there a better way of dealing with that? > Thanks, > Conor. Hmm, I now realise that I forgot to mention this is one of three patchsets I have sent this cycle that touches this MAINTAINERS entry. The others are to the PWM and SPI trees. The SPI one is already applied. I will include this in the covers for future revsions Thanks, Conor > > Conor Dooley (3): > usb: musb: Add support for PolarFire SoC's musb controller > MAINTAINERS: add musb to PolarFire SoC entry > riscv: dts: microchip: fix icicle kit's dr_mode > > MAINTAINERS | 1 + > .../boot/dts/microchip/mpfs-icicle-kit.dts | 2 +- > drivers/usb/musb/Kconfig | 13 +- > drivers/usb/musb/Makefile | 1 + > drivers/usb/musb/mpfs.c | 285 ++++++++++++++++++ > 5 files changed, 300 insertions(+), 2 deletions(-) > create mode 100644 drivers/usb/musb/mpfs.c > ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-06-12 22:19 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-06-07 13:22 [PATCH 0/3] Add support for PolarFire SoC's musb controller Conor Dooley 2022-06-07 13:22 ` [PATCH 1/3] usb: musb: " Conor Dooley 2022-06-07 15:18 ` Ben Dooks 2022-06-12 22:14 ` Conor.Dooley 2022-06-07 13:22 ` [PATCH 2/3] MAINTAINERS: add musb to PolarFire SoC entry Conor Dooley 2022-06-07 13:22 ` [PATCH 3/3] riscv: dts: microchip: fix icicle kit's dr_mode Conor Dooley 2022-06-08 12:21 ` Heinrich Schuchardt 2022-06-08 12:49 ` Conor.Dooley 2022-06-09 8:23 ` Conor.Dooley 2022-06-07 17:53 ` [PATCH 0/3] Add support for PolarFire SoC's musb controller Conor.Dooley
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).