Received: by 2002:a05:6358:53a8:b0:117:f937:c515 with SMTP id z40csp4691062rwe; Mon, 17 Apr 2023 17:23:21 -0700 (PDT) X-Google-Smtp-Source: AKy350ZM35xDBqSrDAa/cTzQm0YO4KQ1KW5rj6dQMbylKvydLZTSZVhwmM3cbEQKVhuFYbv7ekp7 X-Received: by 2002:a05:6a20:8e0c:b0:f0:1ae9:f221 with SMTP id y12-20020a056a208e0c00b000f01ae9f221mr4340192pzj.12.1681777401136; Mon, 17 Apr 2023 17:23:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1681777401; cv=none; d=google.com; s=arc-20160816; b=H2NHP2dfCkw2bVW5CRAsd5EWM0NTtpTibhWlZRiN6Rv4chNQHPM4N08pnvvKimGAYT vn2mfTYwCwV3Tu1NtBt+FqX5pI7Gl4Z2b1xedAqnAbPSdScT6N+B8N4lpzMFCmV6o8aV DlGlOJ4CKvGxonDlbQYlIQeUwxq2ElwmClmbE0z+AB5IWpnbswFIgrJBtGygwA4aQtQO KzrSl5tk8BFlml2zzRNEr7Ohm4O3GsrofbBxCa7Pgzsn0xJK1zZsEU/nUVpN10a1x2P+ ywIJA+olRMynlIhnpyqS4Vq4NxGYUyKfb7tiMoyiJeSGzkI9o8uwB19lqzZtaKb3B9/W oYQA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to :organization:from:references:cc:to:content-language:subject :user-agent:mime-version:date:message-id:dkim-signature; bh=ECeuGlXLjzEciLP6NA4AGXBvtb7yudMi8OtlQBWXTIk=; b=oxCWQYpK/B9kDHrOD5BSUOTFNnwZFo493XAY2MhIfdPB9QVOxGSbNS7u3+ls6NAddx bbi5NGylCNuZlK+0cxmRR9SQ+0e88Tf/soSAcqd3c/fy9SSpO4hvMVqhI2SHruNx8xdV XzX0ovwcSpv7ag7DIW1WAIXDy/9/brdCLyrhCdv9wNaCkbS8tY/L3v7l7xNOqfAQSfwN d4qtCXIjgBucbt1AmzZxDZjTJyj4m4lJdsEX05hrzxhFDfvNEEMJIjnCxIPRGRQAD4Hj IKK2qgQ0NHkjILfLMeTp/9DNKxm+gKo7y8zofidci9t0E49Dlsvln/e6yaKR8Q+8fzbR Oz3Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=mWOwpl9x; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id w22-20020a63c116000000b005077dda0477si12908500pgf.802.2023.04.17.17.23.07; Mon, 17 Apr 2023 17:23:21 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=mWOwpl9x; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230368AbjDRAJe (ORCPT + 99 others); Mon, 17 Apr 2023 20:09:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44296 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229951AbjDRAJc (ORCPT ); Mon, 17 Apr 2023 20:09:32 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 764744C3A; Mon, 17 Apr 2023 17:09:18 -0700 (PDT) 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 dfw.source.kernel.org (Postfix) with ESMTPS id 0D2BE624E0; Tue, 18 Apr 2023 00:09:18 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 44E44C433D2; Tue, 18 Apr 2023 00:09:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1681776557; bh=b71MwB1GPHPGUGJNYm/USXjO8vdrSbhJfyNLXt/caJk=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=mWOwpl9xJtkdrH9tLNqKB6OsMG+6PTWKD7b5eyaeARTd0g8XoMdsxLi9++Jyl34kj w9teMIc8GVbmeBD7NbCBX8K+c+HUVQNlPv/SfSdus5l9qkKqUFP8xjXRchJYIUeBKH LP8t71q9iSLOM0q5g9vGJtighGEcOt7UdUpk1FyUMwtQ0Yfryh+G8auiEhgGJqZzrJ rm8QQoSWnKoUdtL3RqkuhLca/s77+4/ahVRhHuSj0wWt18cCdMEb4Kt0pvJ+n05luv 11Ka5G1Uud0dhING8D2xJGP2vKC5/3LSAU1LjPiT91j9k74SmVLdD+C/vtxJjidMJS PXHhTnQY5fXkA== Message-ID: Date: Tue, 18 Apr 2023 09:09:12 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [PATCH v4 00/11] PCI: rockchip: Fix RK3399 PCIe endpoint controller driver Content-Language: en-US To: Rick Wertenbroek , alberto.dassatti@heig-vd.ch Cc: xxm@rock-chips.com, Shawn Lin , Bjorn Helgaas , Lorenzo Pieralisi , =?UTF-8?Q?Krzysztof_Wilczy=c5=84ski?= , Rob Herring , Krzysztof Kozlowski , Heiko Stuebner , Brian Norris , Johan Jonker , Caleb Connolly , Corentin Labbe , Judy Hsiao , Arnaud Ferraris , Hugh Cole-Baker , linux-pci@vger.kernel.org, linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20230417092631.347976-1-rick.wertenbroek@gmail.com> From: Damien Le Moal Organization: Western Digital Research In-Reply-To: <20230417092631.347976-1-rick.wertenbroek@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-9.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A, RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 4/17/23 18:26, Rick Wertenbroek wrote: > This is a series of patches that fixes the PCIe endpoint controller driver > for the Rockchip RK3399 SoC. The driver was introduced in commit > cf590b078391 ("PCI: rockchip: Add EP driver for Rockchip PCIe controller") > The original driver had issues and would not allow for the RK3399 to > operate in PCIe endpoint mode correctly. This patch series fixes that so > that the PCIe core controller of the RK3399 SoC can now act as a PCIe > endpoint. This is v4 of the patch series and addresses the comments received > during the review of the v3 [1]. The changes to the v3 are minor and none of > them change the logic of the driver. > > Thank you in advance for reviewing this patch series and hopefully > getting this merged. Having a functional PCIe endpoint controller > driver for the RK3399 would allow to develop further PCIe endpoint > functions through the Linux PCIe endpoint framework using this SoC. > > [1] https://lore.kernel.org/linux-pci/20230404082426.3880812-1-rick.wertenbroek@gmail.com/ > > Summary of changes to V3 : > > * Set the fields in the standard order in the dtsi and removed unnecessary > change in associated documentation, thanks to Krzysztof for pointing this out. > * Added comment to explain how MSI-X capabilities advertisement was removed. > * Changed Damien Le Moal address to his > kernel.org address Damien Le Moal in tags. > * [minor] Simplified code where a power of 2 was applied followed by a ilog2 > operation. > * [minor] Small code changes, replaced constant variable by macro, split > remaining long lines. I retested this series, all good. So my Tested-by tag stands. Lorenzo, Krzysztof, Bjorn, Let's please get this series queued for 6.4 ASAP as otherwise the rockchip EP controller is unusable. Also please note that working with Rick off-list, we found out that more fixes are in fact needed for the PCI address mapping (ATU programming) on top of this series. But the problems detected are subtle and harder to trigger. The fixes needed involve changes to the EPC core API so we are still working on the best way to do that. We'll work on another series on top of this one for addressing these issues.