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 8273AC7618D for ; Thu, 6 Apr 2023 08:58:17 +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:MIME-Version:References:In-Reply-To: Date:Cc:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=+06K8RL1X86o2m4Q0+I/WNKcCOlqUieCmtKzeBGfg5s=; b=diBj0yhflEwoes SIZEKrQbWHP/oCd/IRaaph8NRkGqASL/p3A5PdnF54BRkSPIeJuGEaUSmBOs1nbi8kCKkEJT2fNhi aEdKgqIqzyw3kGrd1/0jd444C83tytKMouS/yzJtzoszhWGoXwk26LC0Qfct5rMxbaOmgj/YyK3Ev cbSD6wktXEvnpu/yIs7YhtUpUu18za7xDHa/fwXCr5KIsIKeZEWpIiLzZpDm5YirxekjsYulhakpu J3+Akco7c204ZWajcmk7nOZc1GUcuC6Eye2G0M+YHATTTSzjesrZDBKD0TfZWwkRggtKX9ozC0S8+ Wau7qimQYz2/aalNxCtw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pkLQq-006oUs-0E; Thu, 06 Apr 2023 08:57:24 +0000 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pkLQn-006oTZ-0A for linux-arm-kernel@lists.infradead.org; Thu, 06 Apr 2023 08:57:22 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1680771439; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=LZNl+uqbccaR6P7ROzn8PC1aJ1TDsT5g+QeX5B6yqoA=; b=bac5UiIHCsk1/ZBKvSm85M0e9fQMArl9QMWH2uPoCzddwl8lZb3jDtZ/1agyQ3UP8NMVHC fZGB1MVdmH/dO7XDZYN1xwPBju8+1OHaRZaebQy4EW4RgrElETXQZNVrTubsaoNe8d6rR8 K8XSCbQ3bSjYNQZZpAPua0GvbCByko0= Received: from mail-qt1-f199.google.com (mail-qt1-f199.google.com [209.85.160.199]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-396-UsP03_IBN96xvS3SXV0y1g-1; Thu, 06 Apr 2023 04:57:16 -0400 X-MC-Unique: UsP03_IBN96xvS3SXV0y1g-1 Received: by mail-qt1-f199.google.com with SMTP id d75a77b69052e-3e2daffa0d4so13711691cf.0 for ; Thu, 06 Apr 2023 01:57:16 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1680771435; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=srqW/Zc6uyMlDdpAQsQE29ZbVva96BUjEhTIphyrjK4=; b=mMqM/gdbrdbCl+K1OjFpeF+xm+L3avrltx7QHOJSMJjXDxEjgKE/Nvdcp4ruyI1LU7 LMWMEGkeZ8+hGNYOy2hw/lvgi/lhOqoIaSWomowv5PbM/mH1g4bswH2ZkUyfWctveceK y1u88HE1YW8FfJd8JJwXhZgisZm1oHD1ZJUfnuQj2wnY5RE5gqnfyS6rzIMBOFJI8Jsv z8LwNogDHrZd7mrb3IC3NPst7o33czQEnw5sTXGGkbXvJhYcd15AoB+0Sp4ZwcHmz0FL zEFL0ACsjTyUz/pj/u5NlvmFka7m5/mSgDOUqTnyHSnCyrwhY515cyY8BTTaYEZLBYtz SIBQ== X-Gm-Message-State: AAQBX9dUd5ap8LcI7+M3h4F3TPNXPVGNO/G3iHTtuNpSX6fUTi4dwSVn F4eb7YB06V+KpXWT1zqPe7cD/wOhoFQutmfXrH8TUN6x4jSrqSWrNnrweldbSEzRfZkHkideFsI xtAZWbtk19eFVAkrQYgWl7JLL0cclUHhAtfw= X-Received: by 2002:a05:622a:1815:b0:3e4:eb39:eb8b with SMTP id t21-20020a05622a181500b003e4eb39eb8bmr11113064qtc.5.1680771435670; Thu, 06 Apr 2023 01:57:15 -0700 (PDT) X-Google-Smtp-Source: AKy350bzXKpcq5d/id+6ZQprS3naXkWcpRtP62YORjWExdv5Im2GPPasqjiU0Vii9Sx8y+GhdWYImw== X-Received: by 2002:a05:622a:1815:b0:3e4:eb39:eb8b with SMTP id t21-20020a05622a181500b003e4eb39eb8bmr11113044qtc.5.1680771435377; Thu, 06 Apr 2023 01:57:15 -0700 (PDT) Received: from gerbillo.redhat.com (146-241-227-151.dyn.eolo.it. [146.241.227.151]) by smtp.gmail.com with ESMTPSA id 15-20020a05620a040f00b0073b8512d2dbsm318849qkp.72.2023.04.06.01.57.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 06 Apr 2023 01:57:15 -0700 (PDT) Message-ID: Subject: Re: [PATCH net-next v3 12/12] net: stmmac: dwmac-qcom-ethqos: Add EMAC3 support From: Paolo Abeni To: Andrew Halaney , linux-kernel@vger.kernel.org Cc: agross@kernel.org, andersson@kernel.org, konrad.dybcio@linaro.org, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, vkoul@kernel.org, bhupesh.sharma@linaro.org, wens@csie.org, jernej.skrabec@gmail.com, samuel@sholland.org, mturquette@baylibre.com, peppe.cavallaro@st.com, alexandre.torgue@foss.st.com, joabreu@synopsys.com, mcoquelin.stm32@gmail.com, richardcochran@gmail.com, linux@armlinux.org.uk, veekhee@apple.com, tee.min.tan@linux.intel.com, mohammad.athari.ismail@intel.com, jonathanh@nvidia.com, ruppala@nvidia.com, bmasney@redhat.com, andrey.konovalov@linaro.org, linux-arm-msm@vger.kernel.org, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, ncai@quicinc.com, jsuraj@qti.qualcomm.com, hisunil@quicinc.com, echanude@redhat.com Date: Thu, 06 Apr 2023 10:57:08 +0200 In-Reply-To: <20230403165229.4jhpo2xs7tuclekw@halaney-x13s> References: <20230331214549.756660-1-ahalaney@redhat.com> <20230403165229.4jhpo2xs7tuclekw@halaney-x13s> User-Agent: Evolution 3.46.4 (3.46.4-1.fc37) MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230406_015721_187541_AA74D663 X-CRM114-Status: GOOD ( 13.67 ) 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, On Mon, 2023-04-03 at 11:52 -0500, Andrew Halaney wrote: > @@ -327,9 +370,17 @@ static int ethqos_rgmii_macro_init(struct qcom_ethqos *ethqos) > RGMII_CONFIG2_RX_PROG_SWAP, > RGMII_IO_MACRO_CONFIG2); > > - /* Set PRG_RCLK_DLY to 57 for 1.8 ns delay */ > - rgmii_updatel(ethqos, SDCC_DDR_CONFIG_PRG_RCLK_DLY, > - 57, SDCC_HC_REG_DDR_CONFIG); > + /* PRG_RCLK_DLY = TCXO period * TCXO_CYCLES_CNT / 2 * RX delay ns, > + * in practice this becomes PRG_RCLK_DLY = 52 * 4 / 2 * RX delay ns > + */ > + if (ethqos->has_emac3) > + /* 0.9 ns */ > + rgmii_updatel(ethqos, SDCC_DDR_CONFIG_PRG_RCLK_DLY, > + 115, SDCC_HC_REG_DDR_CONFIG); > + else > + /* 1.8 ns */ > + rgmii_updatel(ethqos, SDCC_DDR_CONFIG_PRG_RCLK_DLY, > + 57, SDCC_HC_REG_DDR_CONFIG); The only (very minor) comment I have is that AFAIK the preferred style for the above block is: if (ethqos->has_emac3) { /* 0.9 ns */ rgmii_updatel(ethqos, SDCC_DDR_CONFIG_PRG_RCLK_DLY, 115, SDCC_HC_REG_DDR_CONFIG); } else { ... due to the comment presence this should be threaded as a multi-line statement. (even if checkpatch does not complain). Cheers, Paolo > SDCC_HC_REG_DDR_CONFIG); > @@ -355,8 +406,15 @@ static int ethqos_rgmii_macro_init(struct qcom_ethqos *ethqos) > BIT(6), RGMII_IO_MACRO_CONFIG); > rgmii_updatel(ethqos, RGMII_CONFIG2_RSVD_CONFIG15, > 0, RGMII_IO_MACRO_CONFIG2); > - rgmii_updatel(ethqos, RGMII_CONFIG2_RX_PROG_SWAP, > - 0, RGMII_IO_MACRO_CONFIG2); > + > + if (ethqos->has_emac3) > + rgmii_updatel(ethqos, RGMII_CONFIG2_RX_PROG_SWAP, > + RGMII_CONFIG2_RX_PROG_SWAP, > + RGMII_IO_MACRO_CONFIG2); > + else > + rgmii_updatel(ethqos, RGMII_CONFIG2_RX_PROG_SWAP, > + 0, RGMII_IO_MACRO_CONFIG2); > + > /* Write 0x5 to PRG_RCLK_DLY_CODE */ > rgmii_updatel(ethqos, SDCC_DDR_CONFIG_EXT_PRG_RCLK_DLY_CODE, > (BIT(29) | BIT(27)), SDCC_HC_REG_DDR_CONFIG); > @@ -389,8 +447,13 @@ static int ethqos_rgmii_macro_init(struct qcom_ethqos *ethqos) > RGMII_IO_MACRO_CONFIG); > rgmii_updatel(ethqos, RGMII_CONFIG2_RSVD_CONFIG15, > 0, RGMII_IO_MACRO_CONFIG2); > - rgmii_updatel(ethqos, RGMII_CONFIG2_RX_PROG_SWAP, > - 0, RGMII_IO_MACRO_CONFIG2); > + if (ethqos->has_emac3) > + rgmii_updatel(ethqos, RGMII_CONFIG2_RX_PROG_SWAP, > + RGMII_CONFIG2_RX_PROG_SWAP, > + RGMII_IO_MACRO_CONFIG2); > + else > + rgmii_updatel(ethqos, RGMII_CONFIG2_RX_PROG_SWAP, > + 0, RGMII_IO_MACRO_CONFIG2); > /* Write 0x5 to PRG_RCLK_DLY_CODE */ > rgmii_updatel(ethqos, SDCC_DDR_CONFIG_EXT_PRG_RCLK_DLY_CODE, > (BIT(29) | BIT(27)), SDCC_HC_REG_DDR_CONFIG); > @@ -433,6 +496,17 @@ static int ethqos_configure(struct qcom_ethqos *ethqos) > rgmii_updatel(ethqos, SDCC_DLL_CONFIG_PDN, > SDCC_DLL_CONFIG_PDN, SDCC_HC_REG_DLL_CONFIG); > > + if (ethqos->has_emac3) { > + if (ethqos->speed == SPEED_1000) { > + rgmii_writel(ethqos, 0x1800000, SDCC_TEST_CTL); > + rgmii_writel(ethqos, 0x2C010800, SDCC_USR_CTL); > + rgmii_writel(ethqos, 0xA001, SDCC_HC_REG_DLL_CONFIG2); > + } else { > + rgmii_writel(ethqos, 0x40010800, SDCC_USR_CTL); > + rgmii_writel(ethqos, 0xA001, SDCC_HC_REG_DLL_CONFIG2); > + } > + } > + > /* Clear DLL_RST */ > rgmii_updatel(ethqos, SDCC_DLL_CONFIG_DLL_RST, 0, > SDCC_HC_REG_DLL_CONFIG); > @@ -452,7 +526,9 @@ static int ethqos_configure(struct qcom_ethqos *ethqos) > SDCC_HC_REG_DLL_CONFIG); > > /* Set USR_CTL bit 26 with mask of 3 bits */ > - rgmii_updatel(ethqos, GENMASK(26, 24), BIT(26), SDCC_USR_CTL); > + if (!ethqos->has_emac3) > + rgmii_updatel(ethqos, GENMASK(26, 24), BIT(26), > + SDCC_USR_CTL); > > /* wait for DLL LOCK */ > do { > @@ -547,6 +623,7 @@ static int qcom_ethqos_probe(struct platform_device *pdev) > ethqos->por = data->por; > ethqos->num_por = data->num_por; > ethqos->rgmii_config_loopback_en = data->rgmii_config_loopback_en; > + ethqos->has_emac3 = data->has_emac3; > > ethqos->rgmii_clk = devm_clk_get(&pdev->dev, "rgmii"); > if (IS_ERR(ethqos->rgmii_clk)) { > @@ -566,6 +643,7 @@ static int qcom_ethqos_probe(struct platform_device *pdev) > plat_dat->fix_mac_speed = ethqos_fix_mac_speed; > plat_dat->dump_debug_regs = rgmii_dump; > plat_dat->has_gmac4 = 1; > + plat_dat->dwmac4_addrs = &data->dwmac4_addrs; > plat_dat->pmt = 1; > plat_dat->tso_en = of_property_read_bool(np, "snps,tso"); > if (of_device_is_compatible(np, "qcom,qcs404-ethqos")) > @@ -603,6 +681,7 @@ static int qcom_ethqos_remove(struct platform_device *pdev) > > static const struct of_device_id qcom_ethqos_match[] = { > { .compatible = "qcom,qcs404-ethqos", .data = &emac_v2_3_0_data}, > + { .compatible = "qcom,sc8280xp-ethqos", .data = &emac_v3_0_0_data}, > { .compatible = "qcom,sm8150-ethqos", .data = &emac_v2_1_0_data}, > { } > }; _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel