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 67981C7619A for ; Wed, 12 Apr 2023 10:11:32 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230090AbjDLKL3 (ORCPT ); Wed, 12 Apr 2023 06:11:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58088 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230368AbjDLKL2 (ORCPT ); Wed, 12 Apr 2023 06:11:28 -0400 Received: from mail-pj1-x102e.google.com (mail-pj1-x102e.google.com [IPv6:2607:f8b0:4864:20::102e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4F33C2D73 for ; Wed, 12 Apr 2023 03:11:22 -0700 (PDT) Received: by mail-pj1-x102e.google.com with SMTP id d22-20020a17090a111600b0023d1b009f52so13968375pja.2 for ; Wed, 12 Apr 2023 03:11:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1681294282; x=1683886282; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=E8Dyl4+NdOephBJ8a5CGLrM2Bu7+ex2R8fnMkaTpqs4=; b=mz5OpN3ySFLWfaIDtFk+QVy7GrbLBKNedf9Vu+hPa+TK0mu13m7MrIQ8OEe6pbJAWQ fbNT7LpAG2fd4KTvtHfzUu6jof2ugDDGynUWoavd+e/Jt3PY6P5L+vYcwrabo+RSYWri ILmzz2EOWH/mZlpfbeU2MtvwJUMnUsDeK0FDzNJtDxRnR6SbeaMGJil5T9RGDlncPbib uAC5ZIexDvQORv2cI+I38V/ubGCDciwBgIFDqlFLf5Ia4SOWcFXnIYWEmKKQBgcIi/vy c86DUgPPBy+nlDR9ZeEfbfqxyM6Y9R7QN+6mFnclLLmy/UjO/QlzcGXgQ3uMC2tgB3Ib 86NQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1681294282; x=1683886282; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=E8Dyl4+NdOephBJ8a5CGLrM2Bu7+ex2R8fnMkaTpqs4=; b=x+bPB9bJ2tdVTBqO+7yDwtJxxVhfQNDj/xp0/iSE9Jx4anCLfyIK/7RNHZ3w5WWsdG o+i5QPTlLFAxk6X2n+OpQc08xNKYcAzUppVHDSPQUlnW0qTpan9dwVcKtIdycIRp/UJn fGrd0DNN8Wyddwe0DLwwaRhC4y9fg5VqqJFKJVTHVdOzJs3OKZ6/nGS5GAWepYaEYr0V Rj0Z5r/ZpHG7xlieV4wQeZXCEmw+OATKHm5NgSEBC3+J2FdDWEcZAbw/2ALJV3YQUqda GfCWqJSi68TO1ikl3FGqCcZDd/eqXMv8zZ+IYnQk1b4gmTOMY5Ib2Tv3aC5VD5jV67Mk vFFA== X-Gm-Message-State: AAQBX9doTo7DMH98YR0pkrkMoIVfJjHIyyycHsXmnMHHq7B0IFF36oAz nwO2UZYOJeB1BmQWL783fHpe0MPdYBATOm5HAg== X-Google-Smtp-Source: AKy350ag0nT+T/UdRSRwoJJZyisBeaEREEvNiKolME4G238ayYlNBBq1ZXotsF5BB8/0cI9XV4UrbA== X-Received: by 2002:a17:902:ce86:b0:1a6:4532:1159 with SMTP id f6-20020a170902ce8600b001a645321159mr7580106plg.63.1681294281569; Wed, 12 Apr 2023 03:11:21 -0700 (PDT) Received: from thinkpad ([117.216.120.128]) by smtp.gmail.com with ESMTPSA id iz15-20020a170902ef8f00b001a1dc2be791sm6121654plb.259.2023.04.12.03.11.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 12 Apr 2023 03:11:21 -0700 (PDT) Date: Wed, 12 Apr 2023 15:41:12 +0530 From: Manivannan Sadhasivam To: Lorenzo Pieralisi Cc: Rob Herring , andersson@kernel.org, krzysztof.kozlowski+dt@linaro.org, bhelgaas@google.com, konrad.dybcio@linaro.org, linux-arm-msm@vger.kernel.org, linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Revert "dt-bindings: PCI: qcom: Add iommu-map properties" Message-ID: <20230412101112.GA9463@thinkpad> References: <20230411121533.22454-1-manivannan.sadhasivam@linaro.org> <20230411174742.GA3428751-robh@kernel.org> <20230411184231.GA59982@thinkpad> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Wed, Apr 12, 2023 at 10:15:53AM +0200, Lorenzo Pieralisi wrote: > On Wed, Apr 12, 2023 at 12:12:31AM +0530, Manivannan Sadhasivam wrote: > > On Tue, Apr 11, 2023 at 12:47:42PM -0500, Rob Herring wrote: > > > On Tue, Apr 11, 2023 at 05:45:33PM +0530, Manivannan Sadhasivam wrote: > > > > This reverts commit 6ebfa40b63ae65eac20834ef4f45355fc5ef6899. > > > > > > > > "iommu-map" property is already documented in commit > > > > > > Need the commit hash here. > > > > > > > ("dt-bindings: PCI: qcom: Add SM8550 compatible") along with the "iommus" > > > > property. > > > > > > Shouldn't there be a patch removing "iommus" as discussed? > > > > > > > Yeah, that was my intention after the dts patches were merged. And since the > > dts patches are in linux-next now, I could finally send the patch. > > I don't understand what's the plan here. By the way, instead of merging > this revert I just dropped the commit that this patch is reverting from > the controller/qcom branch, please have a look to check if everything is > what you expect it to be there. > This is fine. The plan is to remove the "iommus" property from Qcom PCI binding since we have removed the usage of that property from devicetree [1]. Initially the iommu properties were not documented at all in the binding. But commit, "dt-bindings: PCI: qcom: Add SM8550 compatible" added them to the binding to satisfy dtbs check. But in parallel, the patch removing "iommus" property from dts got merged to qcom tree. So now we have 2 options here: 1. Amend the commit "dt-bindings: PCI: qcom: Add SM8550 compatible" to remove the "iommus" property. 2. I will submit a separate patch removing that property. Lorenzo, let me know what works for you. Sorry for the mess! Confusion happened due to patches getting applied without sync. - Mani [1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=6340b391e15094575911ab0d96bfff09deadafba > Lorenzo > > > - Mani > > > > > > > > > > So let's revert the commit that just added "iommu-map" to avoid > > > > duplication. > > > > > > > > Signed-off-by: Manivannan Sadhasivam > > > > --- > > > > Documentation/devicetree/bindings/pci/qcom,pcie.yaml | 2 -- > > > > 1 file changed, 2 deletions(-) > > > > > > > > diff --git a/Documentation/devicetree/bindings/pci/qcom,pcie.yaml b/Documentation/devicetree/bindings/pci/qcom,pcie.yaml > > > > index 5d236bac99b6..a1318a4ecadf 100644 > > > > --- a/Documentation/devicetree/bindings/pci/qcom,pcie.yaml > > > > +++ b/Documentation/devicetree/bindings/pci/qcom,pcie.yaml > > > > @@ -78,8 +78,6 @@ properties: > > > > > > > > dma-coherent: true > > > > > > > > - iommu-map: true > > > > - > > > > interconnects: > > > > maxItems: 2 > > > > > > > > -- > > > > 2.25.1 > > > > > > > > -- > > மணிவண்ணன் சதாசிவம் -- மணிவண்ணன் சதாசிவம்