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 5B31AECAAA1 for ; Fri, 28 Oct 2022 10:24:37 +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:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=2F2rhSVUsd5xGFB8nXTVLVlKlaCqAxceqkDdhhiC+BE=; b=0VL7DHdR1UmfrW TB0LhF6WvDt/3PZkkAIbsfbZjRVnNkHG98gTq7GQ7XS/2u7FgQJz0Afbyytqv1C2/iHYJ3/O/94DD /nEiMa3bY2UwD6r/Y5r2q3fRSdHvcnyn5GhqjHh3ZwQcQffoggRcbmL32uy3KIut17b/KYZDodEOt YwsCROgffu1gvc+/KdAvPHt/6IfEW6SpVZlsqCzUUGdErPbDuese/EucX8LdFVnFY/3xHrr6uD2hi 4KNEk+VktoGeiZ4zn/AIRXMklAGRvBwxJQEGJ++HRkgX832eljHz+/463uM3o1KNFA98HGuNNn+Z6 PLsLMomtLT2cFrQH/xcQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ooMWW-00Gdkx-BG; Fri, 28 Oct 2022 10:23:36 +0000 Received: from ams.source.kernel.org ([145.40.68.75]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ooMWQ-00GdiC-NM; Fri, 28 Oct 2022 10:23:34 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id E82C0B80189; Fri, 28 Oct 2022 10:23:26 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id ED3E5C433D7; Fri, 28 Oct 2022 10:23:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1666952605; bh=pBCdGb929chY5ppQKuDgVYnGog6hqh/6E2N9jwb1/Zc=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=CK14ehMMVSXCsyikJLRvsO703TSBA8hqBDTbHmB56WimZGqIliGCQ1TWofoZSDoCr t6/KsOtbaegq8xYk4ebOZgF9CcDB3s7586rzQHivwFpOGMmlUjgqvoun8vpJr+4jFn YhfILNBOUKzylZp6ud4Yzuo1sVZgd1eciJniazjy12vcMOZ2N1VwvW7yXpUvUkFf7O F53a48BF4p9aOPto6Ft5NUoKonXeRbRrCO+gqK+xMmQv0dnXcJb/7QtCXaofZntwSP 4ZmlXl4yibOboMPcPGehGwKvgyVGmXsVSP42/BGXJMyKdY4TxacyBfPiEhsgX97rh3 BJDnM2eOXBnMw== Message-ID: Date: Fri, 28 Oct 2022 13:23:20 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.1 Subject: Re: [PATCH v3 2/3] phy: ti: gmii-sel: Update methods for fetching and using qsgmii main port Content-Language: en-US To: Siddharth Vadapalli , robh+dt@kernel.org, lee@kernel.org, krzysztof.kozlowski@linaro.org, krzysztof.kozlowski+dt@linaro.org, kishon@kernel.org, vkoul@kernel.org, dan.carpenter@oracle.com Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, linux-arm-kernel@lists.infradead.org References: <20221026074532.109220-1-s-vadapalli@ti.com> <20221026074532.109220-3-s-vadapalli@ti.com> From: Roger Quadros In-Reply-To: <20221026074532.109220-3-s-vadapalli@ti.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221028_032331_107155_396D45A5 X-CRM114-Status: GOOD ( 28.90 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Siddharth, On 26/10/2022 10:45, Siddharth Vadapalli wrote: > The number of QSGMII main ports are specific to the device. TI's J7200 for > which the QSGMII main port property is fetched from the device-tree has > only one QSGMII main port. However, devices like TI's J721e support up to > two QSGMII main ports. Thus, the existing methods for fetching and using > the QSGMII main port are not scalable. > > Update the existing methods for handling the QSGMII main ports and its > associated requirements to make it scalable for future devices. > > Signed-off-by: Siddharth Vadapalli > --- > drivers/phy/ti/phy-gmii-sel.c | 29 ++++++++++++++++++++++------- > 1 file changed, 22 insertions(+), 7 deletions(-) > > diff --git a/drivers/phy/ti/phy-gmii-sel.c b/drivers/phy/ti/phy-gmii-sel.c > index 0bcfd6d96b4d..c8f30d2e1f46 100644 > --- a/drivers/phy/ti/phy-gmii-sel.c > +++ b/drivers/phy/ti/phy-gmii-sel.c > @@ -50,6 +50,7 @@ struct phy_gmii_sel_soc_data { > const struct reg_field (*regfields)[PHY_GMII_SEL_LAST]; > bool use_of_data; > u64 extra_modes; > + u32 num_qsgmii_main_ports; > }; > > struct phy_gmii_sel_priv { > @@ -213,6 +214,8 @@ struct phy_gmii_sel_soc_data phy_gmii_sel_cpsw5g_soc_j7200 = { > .use_of_data = true, > .regfields = phy_gmii_sel_fields_am654, > .extra_modes = BIT(PHY_INTERFACE_MODE_QSGMII), > + .num_ports = 4, > + .num_qsgmii_main_ports = 1, > }; > > static const struct of_device_id phy_gmii_sel_id_table[] = { > @@ -378,11 +381,13 @@ static int phy_gmii_sel_init_ports(struct phy_gmii_sel_priv *priv) > static int phy_gmii_sel_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > + const struct phy_gmii_sel_soc_data *soc_data; > struct device_node *node = dev->of_node; > const struct of_device_id *of_id; > struct phy_gmii_sel_priv *priv; > u32 main_ports = 1; > int ret; > + u32 i; > > of_id = of_match_node(phy_gmii_sel_id_table, pdev->dev.of_node); > if (!of_id) > @@ -394,16 +399,26 @@ static int phy_gmii_sel_probe(struct platform_device *pdev) > > priv->dev = &pdev->dev; > priv->soc_data = of_id->data; > + soc_data = priv->soc_data; > priv->num_ports = priv->soc_data->num_ports; > - of_property_read_u32(node, "ti,qsgmii-main-ports", &main_ports); > + priv->qsgmii_main_ports = 0; > + > /* > - * Ensure that main_ports is within bounds. If the property > - * ti,qsgmii-main-ports is not mentioned, or the value mentioned > - * is out of bounds, default to 1. > + * Based on the compatible, try to read the appropriate number of > + * QSGMII main ports from the "ti,qsgmii-main-ports" property from > + * the device-tree node. > */ > - if (main_ports < 1 || main_ports > 4) > - main_ports = 1; > - priv->qsgmii_main_ports = PHY_GMII_PORT(main_ports); > + for (i = 0; i < soc_data->num_qsgmii_main_ports; i++) { > + of_property_read_u32_index(node, "ti,qsgmii-main-ports", i, &main_ports); > + /* > + * Ensure that main_ports is within bounds. > + */ > + if (main_ports < 1 || main_ports > soc_data->num_ports) { > + dev_err(dev, "Invalid qsgmii main port provided\n"); nit: This message is a bit misleading if the property does not exist in DT. How about just "Invalid ti,qsgmii-main-ports" > + return -EINVAL; > + } > + priv->qsgmii_main_ports |= PHY_GMII_PORT(main_ports); > + } > > priv->regmap = syscon_node_to_regmap(node->parent); > if (IS_ERR(priv->regmap)) { Reviewed-by: Roger Quadros cheers, -roger _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel