From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 30F3BC7EE32 for ; Fri, 27 Jun 2025 00:48:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=psnWpSx1+WAOOcDnZt/LD/O55FZRNSoJnVqV6vDxllA=; b=ihFE4oTQMxpvbW C6M73FWhPgm873cJCZQ//RUy05L8owBaIGJK3EeM3CW0D5vopPXOjeRcb21yb68MN174oTN/WDQ/+ /dKT6Exjae8brRqh6W3MQxAPAdGODKZyz5ajVeXLAtvtvE2wUiGLDoDSQ+2Ar4MG87RIsUjK38Zv2 AYazIwpKLtwB05ozpKSIPQT8Nqe2DueWiWbOaJ5QmSMrqjFOjuG0M10HSdU/W7Y3klXLX7RbCLHLf rWsW/cD18lf6EEfXWk5c2CRcPhpGP8cmGW1n/dfo4Bw3f2NlT5+MQ6DruKnqcqLew9UD/LxbtIldx Z6BhlR5buVghhDV2HV3Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uUxGz-0000000DF1j-3wqk; Fri, 27 Jun 2025 00:48:57 +0000 Received: from nyc.source.kernel.org ([147.75.193.91]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uUwla-0000000DBVM-2A3k; Fri, 27 Jun 2025 00:16:31 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id A6106A4F04B; Fri, 27 Jun 2025 00:16:29 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4121FC4CEEB; Fri, 27 Jun 2025 00:16:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1750983389; bh=Drbs52CyKM+EsOpE81VcTWNKkqjmlBxF8J1HXfdBQek=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=fdd22rWuQTBh9UZwpcJ97lO61MCgnbk64wUJ0KWL/TfSmDcaBdS9qZjIUrQjytM6K KDr3+4K15j9mE/fT0wt2TOWeFArxvTtcixOOmKojFhgNs+FZVq66IFrP0NwW6WcqDs rvx3X5OcyWJR+WaPfS6RKKFebfGHosN7dVc87fCZ8clpT/4M8DLWKV/VHwqMUHjXWQ ezvuSAzQxaLKq7W000Lgglvpci/UVjIBYoxWC7L7xSZOyBYO86UbZnZxiz2Q4PLAxz Mqx9/5V/d3N/hye93jQpbShDpnxLaU9yMa4AQrVlqoQrQbg2kIVsTQBMHj67UE4xHr 8dgVn7h7iJQ9Q== Date: Thu, 26 Jun 2025 17:16:28 -0700 From: Vinod Koul To: Inochi Amaoto Cc: Kishon Vijay Abraham I , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Chen Wang , Paul Walmsley , Palmer Dabbelt , Albert Ou , Alexandre Ghiti , linux-phy@lists.infradead.org, devicetree@vger.kernel.org, sophgo@lists.linux.dev, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, Yixun Lan , Longbin Li Subject: Re: [PATCH v4 2/2] phy: sophgo: Add USB 2.0 PHY driver for Sophgo CV18XX/SG200X Message-ID: References: <20250611081804.1196397-1-inochiama@gmail.com> <20250611081804.1196397-3-inochiama@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20250611081804.1196397-3-inochiama@gmail.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250626_171630_702058_4B93B533 X-CRM114-Status: GOOD ( 30.45 ) X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org On 11-06-25, 16:18, Inochi Amaoto wrote: > Add USB 2.0 PHY driver for Sophgo CV18XX/SG200X. Currently > this driver does not support OTG mode as lack of document. > > Signed-off-by: Inochi Amaoto > --- > drivers/phy/Kconfig | 1 + > drivers/phy/Makefile | 1 + > drivers/phy/sophgo/Kconfig | 19 +++ > drivers/phy/sophgo/Makefile | 2 + > drivers/phy/sophgo/phy-cv1800-usb2.c | 222 +++++++++++++++++++++++++++ > 5 files changed, 245 insertions(+) > create mode 100644 drivers/phy/sophgo/Kconfig > create mode 100644 drivers/phy/sophgo/Makefile > create mode 100644 drivers/phy/sophgo/phy-cv1800-usb2.c > > diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig > index 58c911e1b2d2..678dd0452f0a 100644 > --- a/drivers/phy/Kconfig > +++ b/drivers/phy/Kconfig > @@ -122,6 +122,7 @@ source "drivers/phy/renesas/Kconfig" > source "drivers/phy/rockchip/Kconfig" > source "drivers/phy/samsung/Kconfig" > source "drivers/phy/socionext/Kconfig" > +source "drivers/phy/sophgo/Kconfig" > source "drivers/phy/st/Kconfig" > source "drivers/phy/starfive/Kconfig" > source "drivers/phy/sunplus/Kconfig" > diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile > index c670a8dac468..bfb27fb5a494 100644 > --- a/drivers/phy/Makefile > +++ b/drivers/phy/Makefile > @@ -35,6 +35,7 @@ obj-y += allwinner/ \ > rockchip/ \ > samsung/ \ > socionext/ \ > + sophgo/ \ > st/ \ > starfive/ \ > sunplus/ \ > diff --git a/drivers/phy/sophgo/Kconfig b/drivers/phy/sophgo/Kconfig > new file mode 100644 > index 000000000000..2c943bbe1f81 > --- /dev/null > +++ b/drivers/phy/sophgo/Kconfig > @@ -0,0 +1,19 @@ > +# SPDX-License-Identifier: GPL-2.0-only > +# > +# Phy drivers for Sophgo platforms > +# > + > +if ARCH_SOPHGO || COMPILE_TEST > + > +config PHY_SOPHGO_CV1800_USB2 > + tristate "Sophgo CV18XX/SG200X USB 2.0 PHY support" > + depends on MFD_SYSCON > + depends on USB_SUPPORT > + select GENERIC_PHY > + help > + Enable this to support the USB 2.0 PHY used with > + the DWC2 USB controller in Sophgo CV18XX/SG200X > + series SoC. > + If unsure, say N. > + > +endif # ARCH_SOPHGO || COMPILE_TEST > diff --git a/drivers/phy/sophgo/Makefile b/drivers/phy/sophgo/Makefile > new file mode 100644 > index 000000000000..318060661759 > --- /dev/null > +++ b/drivers/phy/sophgo/Makefile > @@ -0,0 +1,2 @@ > +# SPDX-License-Identifier: GPL-2.0 > +obj-$(CONFIG_PHY_SOPHGO_CV1800_USB2) += phy-cv1800-usb2.o > diff --git a/drivers/phy/sophgo/phy-cv1800-usb2.c b/drivers/phy/sophgo/phy-cv1800-usb2.c > new file mode 100644 > index 000000000000..1d21db7f875b > --- /dev/null > +++ b/drivers/phy/sophgo/phy-cv1800-usb2.c > @@ -0,0 +1,222 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright (C) 2025 Inochi Amaoto > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +#define REG_USB_PHY_CTRL 0x048 > + > +#define PHY_ID_OVERWRITE_EN BIT(6) > +#define PHY_ID_OVERWRITE_MODE BIT(7) > +#define PHY_ID_OVERWRITE_MODE_HOST FIELD_PREP(BIT(7), 0) > +#define PHY_ID_OVERWRITE_MODE_DEVICE FIELD_PREP(BIT(7), 1) > + > +#define PHY_APP_CLK_RATE 125000000 > +#define PHY_LPM_CLK_RATE 12000000 > +#define PHY_STB_CLK_RATE 333334 > + > +struct cv1800_usb_phy { > + struct phy *phy; > + struct regmap *syscon; > + spinlock_t lock; > + struct clk *usb_app_clk; > + struct clk *usb_lpm_clk; > + struct clk *usb_stb_clk; > + bool support_otg; > +}; > + > +static int cv1800_usb_phy_set_mode(struct phy *_phy, > + enum phy_mode mode, int submode) > +{ > + struct cv1800_usb_phy *phy = phy_get_drvdata(_phy); > + unsigned int regval = 0; > + int ret; > + > + switch (mode) { > + case PHY_MODE_USB_DEVICE: > + regval = PHY_ID_OVERWRITE_EN | PHY_ID_OVERWRITE_MODE_DEVICE; > + break; > + case PHY_MODE_USB_HOST: > + regval = PHY_ID_OVERWRITE_EN | PHY_ID_OVERWRITE_MODE_HOST; > + break; > + case PHY_MODE_USB_OTG: > + if (!phy->support_otg) > + return 0; > + > + ret = regmap_read(phy->syscon, REG_USB_PHY_CTRL, ®val); > + if (ret) > + return ret; > + > + regval = FIELD_GET(PHY_ID_OVERWRITE_MODE, regval); > + break; > + default: > + return -EINVAL; > + } > + > + return regmap_update_bits(phy->syscon, REG_USB_PHY_CTRL, > + PHY_ID_OVERWRITE_EN | PHY_ID_OVERWRITE_MODE, > + regval); > +} > + > +static ssize_t dr_mode_write(struct file *file, const char __user *_buf, > + size_t count, loff_t *ppos) > +{ > + struct seq_file *seq = file->private_data; > + struct cv1800_usb_phy *phy = seq->private; > + enum phy_mode mode; > + char buf[16]; > + > + if (copy_from_user(&buf, _buf, min_t(size_t, sizeof(buf) - 1, count))) > + return -EFAULT; > + > + if (sysfs_streq(buf, "host")) > + mode = PHY_MODE_USB_DEVICE; > + else if (sysfs_streq(buf, "peripheral")) > + mode = PHY_MODE_USB_DEVICE; > + else if (sysfs_streq(buf, "otg")) > + mode = PHY_MODE_USB_OTG; > + else > + return -EINVAL; > + > + return cv1800_usb_phy_set_mode(phy->phy, mode, 0); > +} > + > +static int dr_mode_show(struct seq_file *seq, void *v) > +{ > + struct cv1800_usb_phy *phy = seq->private; > + unsigned long flags; > + unsigned int regval; > + bool is_host = true; > + int ret; > + > + spin_lock_irqsave(&phy->lock, flags); > + ret = regmap_read(phy->syscon, REG_USB_PHY_CTRL, ®val); > + spin_unlock_irqrestore(&phy->lock, flags); > + > + if (ret) > + return ret; > + > + if (regval & PHY_ID_OVERWRITE_MODE) > + is_host = false; > + > + if (!(regval & PHY_ID_OVERWRITE_EN)) > + seq_puts(seq, "otg: "); > + > + seq_puts(seq, is_host ? "host\n" : "peripheral\n"); > + > + return 0; > +} This should be done by host controller and not phy and then use apis to set the mode for phy from controller, pls see other driver on how they do this > + > +DEFINE_SHOW_STORE_ATTRIBUTE(dr_mode); > + > +static int cv1800_usb_phy_set_clock(struct cv1800_usb_phy *phy) > +{ > + int ret; > + > + ret = clk_set_rate(phy->usb_app_clk, PHY_APP_CLK_RATE); > + if (ret) > + return ret; > + > + ret = clk_set_rate(phy->usb_lpm_clk, PHY_LPM_CLK_RATE); > + if (ret) > + return ret; > + > + ret = clk_set_rate(phy->usb_stb_clk, PHY_STB_CLK_RATE); > + if (ret) > + return ret; > + > + return 0; Should this not be return ret here? or just do return clk_set_rate() here > +} > + > +static const struct phy_ops cv1800_usb_phy_ops = { > + .set_mode = cv1800_usb_phy_set_mode, > + .owner = THIS_MODULE, > +}; > + > +static int cv1800_usb_phy_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct device *parent = dev->parent; > + struct cv1800_usb_phy *phy; > + struct phy_provider *phy_provider; > + int ret; > + > + if (!parent) > + return -ENODEV; > + > + phy = devm_kmalloc(dev, sizeof(*phy), GFP_KERNEL); > + if (!phy) > + return -ENOMEM; > + > + phy->syscon = syscon_node_to_regmap(parent->of_node); > + if (IS_ERR_OR_NULL(phy->syscon)) > + return -ENODEV; > + > + phy->support_otg = false; > + > + spin_lock_init(&phy->lock); > + > + phy->usb_app_clk = devm_clk_get_enabled(dev, "app"); > + if (IS_ERR(phy->usb_app_clk)) > + return dev_err_probe(dev, PTR_ERR(phy->usb_app_clk), > + "Failed to get app clock\n"); > + > + phy->usb_lpm_clk = devm_clk_get_enabled(dev, "lpm"); > + if (IS_ERR(phy->usb_lpm_clk)) > + return dev_err_probe(dev, PTR_ERR(phy->usb_lpm_clk), > + "Failed to get lpm clock\n"); > + > + phy->usb_stb_clk = devm_clk_get_enabled(dev, "stb"); > + if (IS_ERR(phy->usb_stb_clk)) > + return dev_err_probe(dev, PTR_ERR(phy->usb_stb_clk), > + "Failed to get stb clock\n"); > + > + phy->phy = devm_phy_create(dev, NULL, &cv1800_usb_phy_ops); > + if (IS_ERR(phy->phy)) > + return dev_err_probe(dev, PTR_ERR(phy->phy), > + "Failed to create phy\n"); > + > + ret = cv1800_usb_phy_set_clock(phy); > + if (ret) > + return ret; > + > + debugfs_create_file("dr_mode", 0644, phy->phy->debugfs, > + phy, &dr_mode_fops); > + > + phy_set_drvdata(phy->phy, phy); > + phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); > + > + return PTR_ERR_OR_ZERO(phy_provider); > +} > + > +static const struct of_device_id cv1800_usb_phy_ids[] = { > + { .compatible = "sophgo,cv1800b-usb2-phy" }, > + { }, > +}; > +MODULE_DEVICE_TABLE(of, cv1800_usb_phy_ids); > + > +static struct platform_driver cv1800_usb_phy_driver = { > + .probe = cv1800_usb_phy_probe, > + .driver = { > + .name = "cv1800-usb2-phy", > + .of_match_table = cv1800_usb_phy_ids, > + }, > +}; > +module_platform_driver(cv1800_usb_phy_driver); > + > +MODULE_AUTHOR("Inochi Amaoto "); > +MODULE_DESCRIPTION("CV1800/SG2000 SoC USB 2.0 PHY driver"); > +MODULE_LICENSE("GPL"); > -- > 2.49.0 -- ~Vinod -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy