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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6DF42C0015E for ; Wed, 19 Jul 2023 09:41:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230242AbjGSJlY (ORCPT ); Wed, 19 Jul 2023 05:41:24 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46616 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229639AbjGSJlW (ORCPT ); Wed, 19 Jul 2023 05:41:22 -0400 Received: from smtp.dudau.co.uk (dliviu.plus.com [80.229.23.120]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id B1814EC; Wed, 19 Jul 2023 02:41:20 -0700 (PDT) Received: from mail.dudau.co.uk (bart.dudau.co.uk [192.168.14.2]) by smtp.dudau.co.uk (Postfix) with SMTP id A2F6E41A7003; Wed, 19 Jul 2023 10:41:18 +0100 (BST) Received: by mail.dudau.co.uk (sSMTP sendmail emulation); Wed, 19 Jul 2023 10:41:18 +0100 Date: Wed, 19 Jul 2023 10:41:18 +0100 From: Liviu Dudau To: Sebastian Reichel Cc: linux-phy@lists.infradead.org, linux-rockchip@lists.infradead.org, Jingoo Han , Gustavo Pimentel , Bjorn Helgaas , Lorenzo Pieralisi , Vinod Koul , Kishon Vijay Abraham I , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Rob Herring , Serge Semin , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Shawn Lin , Simon Xue , John Clark , Qu Wenruo , devicetree@vger.kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, kernel@collabora.com Subject: Re: [PATCH v2 2/2] arm64: dts: rockchip: rk3588: add PCIe3 support Message-ID: References: <20230717173512.65169-1-sebastian.reichel@collabora.com> <20230717173512.65169-3-sebastian.reichel@collabora.com> <20230718160137.sfitnkl6gmyi75jx@mercury.elektranox.org> <20230718210601.6hrmnl5lbtl6vinp@mercury.elektranox.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20230718210601.6hrmnl5lbtl6vinp@mercury.elektranox.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Sebastian, On Tue, Jul 18, 2023 at 11:06:01PM +0200, Sebastian Reichel wrote: > Hi, > > On Tue, Jul 18, 2023 at 08:38:36PM +0100, Liviu Dudau wrote: > > On Tue, Jul 18, 2023 at 06:01:37PM +0200, Sebastian Reichel wrote: > > > On Tue, Jul 18, 2023 at 04:09:53PM +0100, Liviu Dudau wrote: > > > > On Mon, Jul 17, 2023 at 07:35:12PM +0200, Sebastian Reichel wrote: > > > > > pipe_phy1_grf: syscon@fd5c0000 { > > > > > compatible = "rockchip,rk3588-pipe-phy-grf", "syscon"; > > > > > reg = <0x0 0xfd5c0000 0x0 0x100>; > > > > > > > > What tree is based this on? Even after applying your PCIe2 series I don't have the above > > > > node so the patch doesn't apply to mainline. > > > > > > You are missing naneng-combphy support: > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git/commit/?h=v6.6-armsoc/dts64&id=6ebd55b3bba383e0523b0c014f17c97f3ce80708 > > > > Thanks! It looks like the PCIe2 commit that adds support to rk3588(s).dtsi > > files is also missing an #include for the PHY_TYPE_PCIE > > use, otherwise the DTS fail to compile. > > Yes, that's also already in linux-next: > > https://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git/commit/?h=v6.6-armsoc/dts64&id=34d6c15d8e86256ef2456c604b1c8d8242720871 I'm reading that as: "relevant patch that this series depends on has already been added to the tree that's going to pull this PCIe2 series so all will be good". Otherwise I think there should be some mention in the cover letter about dependencies, so that people like me don't report issues just because they are not using the linux-rockchip tree by default. Many thanks for the quick answers and the links to fix my tree. Best regards, Liviu > > Greetings, > > -- Sebastian -- Everyone who uses computers frequently has had, from time to time, a mad desire to attack the precocious abacus with an axe. -- John D. Clark, Ignition!