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 00DD9C3DA49 for ; Thu, 18 Jul 2024 09:29:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type: Content-Transfer-Encoding:MIME-Version:References:In-Reply-To:Message-ID:Date :Subject:Cc:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=R/FaCrVz5BoNpLahWC4Bh5/7f60+JG/HUoQTFCwAgZQ=; b=zVLBwuzhu5EL0W/a1HOCxY10w9 GZt+Ul7fLsewWkPy0yotZyNAS+EMzBGkpOkHoh5Yj6qWYYfJTtC5c22dGn4FIQjCTT7GFdzOtJnb3 f8bj20wRzFu7qbfTf8hLh97+6ZBBYKVethMaXJ0aKEnRPbof/iEah7iY9j90n1CVHDHQolmzmYgGg /6S31hseIqxTWGNnEIlsRa8vFVgjMICYzVW9NN7K4PouCLz/XUo/kdqDi7P1YWqPdUOUJfaLEqftc 7SeWgljA5BbNW7yFKwTnPHqXKoiim5jmEIuGDvluY9UOKAG5pkU0zVddYloUilWPNrGs2tNa06MyH J0NNbDTg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sUNSA-0000000GZ2l-1256; Thu, 18 Jul 2024 09:29:34 +0000 Received: from gloria.sntech.de ([185.11.138.130]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sUNRo-0000000GYxu-2jNQ; Thu, 18 Jul 2024 09:29:13 +0000 Received: from i5e860d09.versanet.de ([94.134.13.9] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1sUNRf-0003M8-Iy; Thu, 18 Jul 2024 11:29:03 +0200 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: Dragan Simic , Anand Moon Cc: mturquette@baylibre.com, sboyd@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org Subject: Re: [PATCH v2 3/3] arm64: dts: rockchip: fix the pcie refclock oscillator on Rock 5 ITX Date: Thu, 18 Jul 2024 11:29:02 +0200 Message-ID: <1829575.Zkmt1EvEu4@diego> In-Reply-To: References: <20240715110251.261844-1-heiko@sntech.de> <72b84761ef93c1d4d6d61b16ef1bb1ce@manjaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240718_022912_721495_339EC047 X-CRM114-Status: GOOD ( 30.28 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Donnerstag, 18. Juli 2024, 10:00:51 CEST schrieb Anand Moon: > Hi Dragan > > On Thu, 18 Jul 2024 at 13:28, Dragan Simic wrote: > > > > On 2024-07-18 09:52, Anand Moon wrote: > > > On Thu, 18 Jul 2024 at 13:02, Dragan Simic wrote: > > >> On 2024-07-18 09:26, Anand Moon wrote: > > >> > On Mon, 15 Jul 2024 at 16:35, Heiko Stuebner wrote: > > >> >> > > >> >> The Rock 5 ITX uses two PCIe controllers to drive both a M.2 slot and > > >> >> its > > >> >> SATA controller with 2 lanes each. The supply for the refclk > > >> >> oscillator is > > >> >> the same that supplies the M.2 slot, but the SATA controller port is > > >> >> supplied by a different rail. > > >> >> > > >> >> This leads to the effect that if the PCIe30x4 controller for the M.2 > > >> >> probes first, everything works normally. But if the PCIe30x2 > > >> >> controller > > >> >> that is connected to the SATA controller probes first, it will hang on > > >> >> the first DBI read as nothing will have enabled the refclock before. > > >> > > > >> > I just checked the rk3588-rock-5-itx.dts in the linux-next. > > >> > You have not enabled sata0 and sata2, which might be the problem > > >> > for the SATA controller not getting initialized. > > >> > > >> Rock 5 ITX doesn't use RK5588's built-in SATA interfaces, so that's > > >> fine. > > >> Please have a look at the board schematic, it uses a separate PCI > > >> Express > > >> SATA controller for its four SATA ports. > > >> > > > yes, But I am referring to sata node not enabled which enable > > > the PHY_TYPE_SATA. > > > > > > see rk3588-coolpi-cm5-evb.dts and rk3588-edgeble-neu6a-io.dtsi > > > rk3588-quartzpro64.dts > > > which have sata port on board. > > > > > > &sata0 { > > > status = "okay"; > > > }; > > > > QuartzPro64, as an example, uses RK3588's built-in SATA interfaces, > > so it enables sata0 in its board dts. Rock 5 ITX doesn't do that, > > as I already described. > > Ok no problem, For the Rock 5 ITX it really only routes 2 PCIe lanes to one M.2 port and the other 2 lanes to the separate ASMedia SATA controller. So from the Rock5 PoV, it's really just 2 PCIe 2-lane slots and the SATA controller simply gets probed as PCIe device. I even have a sample of the Rock 5+ here, that actually drops the separate SATA controller and instead provides a 2nd M.2 slot ;-) Heiko