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 B206AC0032E for ; Wed, 25 Oct 2023 13:16:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233757AbjJYNQ5 (ORCPT ); Wed, 25 Oct 2023 09:16:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37946 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233875AbjJYNQ4 (ORCPT ); Wed, 25 Oct 2023 09:16:56 -0400 Received: from mail-qt1-x832.google.com (mail-qt1-x832.google.com [IPv6:2607:f8b0:4864:20::832]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C3849133; Wed, 25 Oct 2023 06:16:53 -0700 (PDT) Received: by mail-qt1-x832.google.com with SMTP id d75a77b69052e-41cc537ed54so36850601cf.2; Wed, 25 Oct 2023 06:16:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1698239813; x=1698844613; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=g1tU5kbwVb+Wg1uvhO3+D2ibDL+pMHtlvIijrkuaPh8=; b=JztT+RB5tT9OlVCWDTDHrk8Yb+mSxQGALK/XRzNVQ6Uai4dbyc4lOxSADoG1tjvg3/ 1Oz9hBZSd1LbPvqKLtANYd9hsSSEhbRvLRBmG/VEMua51TfZi/Lq/9JNgfsfVm4mLK2z uHybcIwIPPdQRwtsee0u0prDHLQwcQHlButJulpCLb8QJWxrdqb6NkfKvFDrIfxKa97F 3LpLMTuhSjtpu1dGiXEybaSidDKCWVFMOQqJV+hwFPvgVsYc6zNyUNRYenUJOLhFX50E 5jXXsVhRS6r7o4T4eRvCxRhskMd97iPJMBj9NH5vqX8awtOYbST1unPib0bHwC6QPY4u xs5w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1698239813; x=1698844613; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=g1tU5kbwVb+Wg1uvhO3+D2ibDL+pMHtlvIijrkuaPh8=; b=MJjZZFkTAzcKG5RK8lbTrZz7ZAfUAwXYz8P9ged/IxhpB3exi4zfglrl4KoJsTI0xf YI7CThxWVdT4Ay7w73pfdabl57wtzszasenSe1Cz8kYrA6avUNOxoX2uIjubwj0JFUdG Z1eq5LQzAhhd/8tg6J16F5aTE8ZF1qIERVk8YFQCllfxq/qaiU3r3TK/9NeNqLq/uOt4 h70N8BG6WQqJ4HZvngea9nlGeozBvNXrQYHD8Pvpvvswx9jLyLrQLr/mkzIM/0TJbjpH HiDBZp6ph3bWjyPNxP5/Tcrrj+3cNxdnRG5l6YjMFsQKnRFvPHRLO/O0JbvOkK+QT+Ba QZGg== X-Gm-Message-State: AOJu0Yyj9DnhhIfJfYEgRFZywojeJETWmvXRbnHR8Z24SHHk2dPw7WSA Giy7YJs7OSVHzkU88wYsb+g= X-Google-Smtp-Source: AGHT+IGDdTkY4rbUQVfZKtiVec1opfPoUHwbLgxu9buNeZQ8XoSBU+I/OT31DE12H3tKf3Lub14xEg== X-Received: by 2002:ac8:5715:0:b0:418:11ee:6315 with SMTP id 21-20020ac85715000000b0041811ee6315mr16789523qtw.43.1698239812806; Wed, 25 Oct 2023 06:16:52 -0700 (PDT) Received: from [192.168.20.212] (cpe-107-15-246-199.nc.res.rr.com. [107.15.246.199]) by smtp.gmail.com with ESMTPSA id g3-20020ac842c3000000b0041977932fc6sm4188578qtm.18.2023.10.25.06.16.51 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 25 Oct 2023 06:16:52 -0700 (PDT) Message-ID: Date: Wed, 25 Oct 2023 09:16:50 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Subject: Re: [PATCH v9 01/20] dt-bindings: PCI: Add PLDA XpressRICH PCIe host common properties To: Krzysztof Kozlowski , Conor Dooley Cc: Minda Chen , =?UTF-8?Q?Krzysztof_Wilczy=c5=84ski?= , Rob Herring , Bjorn Helgaas , Lorenzo Pieralisi , Daire McNamara , Emil Renner Berthing , Krzysztof Kozlowski , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, linux-pci@vger.kernel.org, Paul Walmsley , Palmer Dabbelt , Albert Ou , Philipp Zabel , Mason Huo , Leyfoon Tan , Kevin Xie References: <20231020104341.63157-1-minda.chen@starfivetech.com> <20231020104341.63157-2-minda.chen@starfivetech.com> <8ced1915-7b94-4abc-bd8b-cb4bf027fa01@linaro.org> <97e2decd-f6a3-91cb-6ca7-539f53b686f3@gmail.com> <20231025-dotted-almighty-ae489e9eb764@spud> <3d7a2dba-53f6-4625-8981-fbeb469418c1@linaro.org> Content-Language: en-US From: John Clark In-Reply-To: <3d7a2dba-53f6-4625-8981-fbeb469418c1@linaro.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org That is correct, I tested the entire series against Linux 6.6-rc7 and confirmed that pcie/nvme/usb work on the VisionFive2 device. I was unable to test the Microchip side of the equation. I will be more clear as to what I verified if I comment in the future. John Clark On 10/25/23 6:41 AM, Krzysztof Kozlowski wrote: > On 25/10/2023 12:28, Conor Dooley wrote: >> Hi John, >> >> On Mon, Oct 23, 2023 at 11:09:50PM -0400, John Clark wrote: >>>> On 2023/10/20 19:04, Krzysztof Kozlowski wrote: >>>>> On 20/10/2023 12:43, Minda Chen wrote: >>>>>> Add PLDA XpressRICH PCIe host common properties dt-binding doc. >>>>>> Microchip PolarFire PCIe host using PLDA IP. >>>>>> Move common properties from Microchip PolarFire PCIe host >>>>>> to PLDA files. >>>>>> >>>>>> Signed-off-by: Minda Chen >>>>>> Reviewed-by: Hal Feng >>>>>> Reviewed-by: Conor Dooley >>>>>> Reviewed-by: Rob Herring >>>>>> --- >>>>>> .../bindings/pci/microchip,pcie-host.yaml | 55 +------------- >>>>>> .../pci/plda,xpressrich3-axi-common.yaml | 75 +++++++++++++++++++ >>>>> Where was this patch reviewed? >>>>> >>>>> Best regards, >>>>> Krzysztof >>>>> >>>> This Conor's review tag. v2 : https://patchwork.kernel.org/project/linux-pci/patch/20230727103949.26149-2-minda.chen@starfivetech.com/ >>>> This is Rob's review tag v3: https://patchwork.kernel.org/project/linux-pci/patch/20230814082016.104181-2-minda.chen@starfivetech.com/ >>> Tested-by: John Clark : >>> https://github.com/inindev/visionfive2/tree/main/kernel/patches >> I suspect you don't mean that you tested this individual dt-binding, but >> rather that you tested the whole series. If so, you should probably >> provide this tested-by against the cover-letter instead of this bindings >> patch. > Yeah, otherwise I would like to hear how do you test bindings (other > than dt_binding_check which is something similar to testing as compiling > code). > > Best regards, > Krzysztof > 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 017F6C0032E for ; Wed, 25 Oct 2023 13:17:13 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=vxwklsG6qSoTYbxaWBRrn5qdZEGZeQ+AyG3iAuys2cQ=; b=tGQRsmumTI6vgA lAqZYz5wUkms17lY6SXqxs/90rZpDU2o+FgmE3gzmcWoMG3YIE4tJN60TiOo5jjp8tbgD1FGnuaET kAd/X7u7wQ6MxnTUpzyWz7hgspMcfQKzHNlOI7GkRQIs2hkE8QrwQgHvq1fAAtaPmyzEfdmeZPNZy Yj2D16Z+i0M2BztNKuW2tqU3lBzAfm9LNa+sQZYMKgl/BFeGRo5hqz5Kc+e+2QRsz17DBmsnBS1ik HODhj5kW1ZurK60q0LGS3iZttlMAUWyKWw82Eztr6bpcM9rFYEWjimimooiJACG4d/6o3xKc/tWvX 9GQE9+KDJo/Uh82tcLZg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qvdkw-00CODe-0r; Wed, 25 Oct 2023 13:17:06 +0000 Received: from mail-qt1-x832.google.com ([2607:f8b0:4864:20::832]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qvdkq-00COCI-21 for linux-riscv@lists.infradead.org; Wed, 25 Oct 2023 13:17:04 +0000 Received: by mail-qt1-x832.google.com with SMTP id d75a77b69052e-41cc0e9d92aso36845401cf.3 for ; Wed, 25 Oct 2023 06:16:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1698239813; x=1698844613; darn=lists.infradead.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=g1tU5kbwVb+Wg1uvhO3+D2ibDL+pMHtlvIijrkuaPh8=; b=XkZCFHI63X7SAGkp61t8v249RyEQTNyE17iHjgOpQ7qaiOwrDBdfjBGzFx6ixS55fR v/obztq9uzc+zyDQvKUk9EpX805rBiiwsrOxnE3GxYglbjwb+5uJo2o9tDBChjJly3EX XnXN5gzHxNixWCpLoSehkrVZRuCsr7jvSMT4ERbENockm40EFh2Vj6BfwIZmxR/lb+fp 0J7nX5P4H4nCHoU7EWITClanSzHRTkKHj47n/21NxSnbWljuKtStrL/CMT0ZkBQ9AOoI e4NjCcUZLLr8o6m1HapWwbSMwW1mmkl4+Wydo8+L4dEbXOTL2RVoyI2EeUw6/Rnsdf+7 o/4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1698239813; x=1698844613; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=g1tU5kbwVb+Wg1uvhO3+D2ibDL+pMHtlvIijrkuaPh8=; b=MAZglECzF5CQGOBYKrAbVMM8xplKy7AQpcW9CliLOQjev3r5vyxOqy0OmS5J4skVKc BdGJ9KiB0b9QSxVx+uLJuk4sKaAEgJh6IBjK5gxTQc9/vQVu38KncoXbtel5Y2PkQ5ey wHegLcg0t2AY+Me4VpQfUkVYaheooGSxWCh8QyBTcUoGubdNKf3l7a2SZzk8NumUXIph aXcou4xttrjDyBAtxpIugVZ6IswKKBMja0w19T+x5aQYK7sQCequLTORbkJKfpCm0VHk V9Zu1K/28WNU77f4gJcvYFSLy7TlzvxAhrf43o0/skCK4nd5XpWEXnPEv1T0pmXAnVxi PmXg== X-Gm-Message-State: AOJu0YxuopIEc8ZPXrlLQozRceVtjRGCp4p7wZOnEJovXSuySYJiaI0c GMudAN8eY3dOtZImfBANyOw= X-Google-Smtp-Source: AGHT+IGDdTkY4rbUQVfZKtiVec1opfPoUHwbLgxu9buNeZQ8XoSBU+I/OT31DE12H3tKf3Lub14xEg== X-Received: by 2002:ac8:5715:0:b0:418:11ee:6315 with SMTP id 21-20020ac85715000000b0041811ee6315mr16789523qtw.43.1698239812806; Wed, 25 Oct 2023 06:16:52 -0700 (PDT) Received: from [192.168.20.212] (cpe-107-15-246-199.nc.res.rr.com. [107.15.246.199]) by smtp.gmail.com with ESMTPSA id g3-20020ac842c3000000b0041977932fc6sm4188578qtm.18.2023.10.25.06.16.51 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 25 Oct 2023 06:16:52 -0700 (PDT) Message-ID: Date: Wed, 25 Oct 2023 09:16:50 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Subject: Re: [PATCH v9 01/20] dt-bindings: PCI: Add PLDA XpressRICH PCIe host common properties To: Krzysztof Kozlowski , Conor Dooley Cc: Minda Chen , =?UTF-8?Q?Krzysztof_Wilczy=c5=84ski?= , Rob Herring , Bjorn Helgaas , Lorenzo Pieralisi , Daire McNamara , Emil Renner Berthing , Krzysztof Kozlowski , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, linux-pci@vger.kernel.org, Paul Walmsley , Palmer Dabbelt , Albert Ou , Philipp Zabel , Mason Huo , Leyfoon Tan , Kevin Xie References: <20231020104341.63157-1-minda.chen@starfivetech.com> <20231020104341.63157-2-minda.chen@starfivetech.com> <8ced1915-7b94-4abc-bd8b-cb4bf027fa01@linaro.org> <97e2decd-f6a3-91cb-6ca7-539f53b686f3@gmail.com> <20231025-dotted-almighty-ae489e9eb764@spud> <3d7a2dba-53f6-4625-8981-fbeb469418c1@linaro.org> Content-Language: en-US From: John Clark In-Reply-To: <3d7a2dba-53f6-4625-8981-fbeb469418c1@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231025_061700_663666_989E7236 X-CRM114-Status: GOOD ( 14.55 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org That is correct, I tested the entire series against Linux 6.6-rc7 and confirmed that pcie/nvme/usb work on the VisionFive2 device. I was unable to test the Microchip side of the equation. I will be more clear as to what I verified if I comment in the future. John Clark On 10/25/23 6:41 AM, Krzysztof Kozlowski wrote: > On 25/10/2023 12:28, Conor Dooley wrote: >> Hi John, >> >> On Mon, Oct 23, 2023 at 11:09:50PM -0400, John Clark wrote: >>>> On 2023/10/20 19:04, Krzysztof Kozlowski wrote: >>>>> On 20/10/2023 12:43, Minda Chen wrote: >>>>>> Add PLDA XpressRICH PCIe host common properties dt-binding doc. >>>>>> Microchip PolarFire PCIe host using PLDA IP. >>>>>> Move common properties from Microchip PolarFire PCIe host >>>>>> to PLDA files. >>>>>> >>>>>> Signed-off-by: Minda Chen >>>>>> Reviewed-by: Hal Feng >>>>>> Reviewed-by: Conor Dooley >>>>>> Reviewed-by: Rob Herring >>>>>> --- >>>>>> .../bindings/pci/microchip,pcie-host.yaml | 55 +------------- >>>>>> .../pci/plda,xpressrich3-axi-common.yaml | 75 +++++++++++++++++++ >>>>> Where was this patch reviewed? >>>>> >>>>> Best regards, >>>>> Krzysztof >>>>> >>>> This Conor's review tag. v2 : https://patchwork.kernel.org/project/linux-pci/patch/20230727103949.26149-2-minda.chen@starfivetech.com/ >>>> This is Rob's review tag v3: https://patchwork.kernel.org/project/linux-pci/patch/20230814082016.104181-2-minda.chen@starfivetech.com/ >>> Tested-by: John Clark : >>> https://github.com/inindev/visionfive2/tree/main/kernel/patches >> I suspect you don't mean that you tested this individual dt-binding, but >> rather that you tested the whole series. If so, you should probably >> provide this tested-by against the cover-letter instead of this bindings >> patch. > Yeah, otherwise I would like to hear how do you test bindings (other > than dt_binding_check which is something similar to testing as compiling > code). > > Best regards, > Krzysztof > _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv