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 4BBCACD4851 for ; Tue, 19 May 2026 08:25:33 +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=jomEvBus/VbrEmVoh83Zt3hBmxs/4BnxMqkO2SAtT64=; b=kzZ6O1P2WibcOo 5o/noMyqvQe5tbWTwjjDksDyDv9c8OdN3pOORfXlmU/fShayWm+aS5lc8TbUJUV/rFwD3Mz+zV8YJ 5WkY8GzJQFDPf60kBL4yLa2B3dYHDI1S98ieH+YNHu2agH7Fv4Hq17YPAtUpHvM4vY+uajEoj8zFC Zmmz3To0q+rZTJgvlxwUbC2kqjTuShsYj3UFW9smsG8dmFEOwumlviVHnWiZBHitxkTKginafS9GY Rj2KzyOMe8TIDSehQS0+FMK7BTPKDaDf1RbrQNfGBZIYYHUoe2WBlaJ577sPrXlluGPdKj89SA1Af UYcJdyVeCihk2g1iyQJA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wPFlc-00000000fN5-45mM; Tue, 19 May 2026 08:25:32 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wPFlZ-00000000fKh-2hxO; Tue, 19 May 2026 08:25:30 +0000 Received: from killaraus.ideasonboard.com (unknown [IPv6:2a01:cb1d:8f2:800:42d6:38fa:3bdf:70df]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id F3B6797F; Tue, 19 May 2026 10:25:14 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1779179115; bh=yKScYuuUJS7xZJRRgYdhSorlTkDWtLSZgrLiv6hxvFw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Ew1EM6jePLOZoXy6YhFq5m77ZX/aS9JzSp8f8++IIR18M4ChpvY692tVYuHAzCqEd ljBbt8YnaPo2zrvvABSe7S5+Gaz/slCnRTSp7Eu4Ajla19YLjrzqOjqvnKTTvRsnNN jnlqRkBSdm8d2JNzZH4QesDZXW3KzMBnngFeT6OE= Date: Tue, 19 May 2026 10:25:26 +0200 From: Laurent Pinchart To: Radhey Shyam Pandey Cc: vkoul@kernel.org, neil.armstrong@linaro.org, michal.simek@amd.com, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, linux-arm-kernel@lists.infradead.org, git@amd.com, Tomi Valkeinen Subject: Re: [PATCH 0/3] phy: zynqmp: fix SERDES scrambler register handling and enable for USB Message-ID: <20260519082526.GB16205@killaraus.ideasonboard.com> References: <20260511163135.2924642-1-radhey.shyam.pandey@amd.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20260511163135.2924642-1-radhey.shyam.pandey@amd.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260519_012529_825764_1C9B392E X-CRM114-Status: GOOD ( 14.26 ) 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 Hi Radhey, I haven't really been involved with the phy-zynqmp driver for a while, despite still being listed as a maintainer. I have just sent a patch (you're on CC) to hand maintainership duties over to Tomi Valkeinen, who took over maintainership of the ZynqMP DPSUB driver. As Tomi isn't really involved with the PHYs, in particular with the non-DP PHYs supported by the driver, it could also make more sense for someone from AMD to take over maintainer duties for phy-zynqmp. On Mon, May 11, 2026 at 10:01:32PM +0530, Radhey Shyam Pandey wrote: > This series fixes three related issues in the ZynqMP SERDES PHY > scrambler/encoder bypass path: > > 1. The L0_TM_DISABLE_SCRAMBLE_ENCODER mask incorrectly included bit 2 > of L0_TX_DIG_61, which is a reserved read-only field. Correct the > mask to (BIT(3) | GENMASK(1, 0)). > > 2. xpsgtr_bypass_scrambler_8b10b() used xpsgtr_write_phy() which > performs a full register write, clobbering unrelated bits. Switch > to xpsgtr_clr_set_phy() with clr=mask, set=mask to preserve other > register fields. > > 3. USB Gen1 requires PHY-side scrambling and 8b/10b encoding as > mandated by the USB 3.x specification. The driver was incorrectly > bypassing these for USB, the same as SATA and SGMII where encoding > is handled in the controller. > > Nava kishore Manne (3): > phy: zynqmp: fix L0_TM_DISABLE_SCRAMBLE_ENCODER mask > phy: zynqmp: use read-modify-write for SERDES scrambler bypass > phy: zynqmp: keep SERDES scrambler and 8b/10b enabled for USB > > drivers/phy/xilinx/phy-zynqmp.c | 37 ++++++++++++++++++++++++++------- > 1 file changed, 30 insertions(+), 7 deletions(-) > > > base-commit: 5d6919055dec134de3c40167a490f33c74c12581 -- Regards, Laurent Pinchart -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy