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 X-Spam-Level: X-Spam-Status: No, score=-15.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7F493C433DB for ; Mon, 4 Jan 2021 13:23:58 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 45B49207B1 for ; Mon, 4 Jan 2021 13:23:58 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 45B49207B1 Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=ti.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=W38Pb/HK9xw9aNFg6rhBegP9AqTMNwBZskRwXq/4aSc=; b=ShlWN8qXSRfXlb/D6BtzV/Jda Ew4BwCMqKmcm9MHRLcExCgj5m/GnBYnQx0PSyR6lIObc6GUnPCDvO0dCWVwWvvYoW7tFjtR9mf0tf jH51M/t71ncp3raUkqRSBgKeTGpvcjW5hcoD+Yvn3lAO71nIebocEvqk1PBg2ZlzBIfawKHQBiqec 7XRilY80iuaxmKbdo1QipWp7GsOlkEF3gN22xUPEbmqmHJC7KF5WAk7b8Ax3uYZcG4tKbYtuGmzqt SlbyaPk+bdLjKl3iY+VE4KnAt79naCiNm9donDmHavr9+LFLZ5U9DInfypJwhfnDfPXflul6sY71c gF0XQwx9A==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kwPoK-0004rP-1P; Mon, 04 Jan 2021 13:22:12 +0000 Received: from fllv0016.ext.ti.com ([198.47.19.142]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kwPoG-0004qC-Kn for linux-arm-kernel@lists.infradead.org; Mon, 04 Jan 2021 13:22:09 +0000 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 104DM6s7111648; Mon, 4 Jan 2021 07:22:06 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1609766526; bh=KMUX6lV3Dn1JFy6a5yFEO2PfMa2RRfs32yPF/fHx0W4=; h=Subject:To:CC:References:From:Date:In-Reply-To; b=uk8DDwoWQY8Lx2OGRWESttTbmRkaKW/QLwB9fICwWsMakitcgvzQpo4NMdx8E8SLx uQdV9B35Tb5NGL0pjnI1ZnnjgeskKnWOwbIWilh08yatHO5uCwUYeNiS8iAWhnpl5g j+LOywVCbJoEvMtxqNGfleOijVxe1Od/um6lpjBU= Received: from DLEE105.ent.ti.com (dlee105.ent.ti.com [157.170.170.35]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 104DM6UA023884 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Mon, 4 Jan 2021 07:22:06 -0600 Received: from DLEE105.ent.ti.com (157.170.170.35) by DLEE105.ent.ti.com (157.170.170.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3; Mon, 4 Jan 2021 07:22:06 -0600 Received: from fllv0040.itg.ti.com (10.64.41.20) by DLEE105.ent.ti.com (157.170.170.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3 via Frontend Transport; Mon, 4 Jan 2021 07:22:06 -0600 Received: from [10.250.235.36] (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 104DM3K6003991; Mon, 4 Jan 2021 07:22:04 -0600 Subject: Re: [PATCH v3 0/6] PCI: J7200/J721E PCIe bindings To: Nishanth Menon References: <20210104122232.24071-1-kishon@ti.com> <20210104125910.qaf7vi3dx6vsne6t@backfield> <465097c1-2d38-ad45-cc32-d92c385114c5@ti.com> <20210104131642.m5rgpkhnrffq5nrx@circular> From: Kishon Vijay Abraham I Message-ID: Date: Mon, 4 Jan 2021 18:52:03 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20210104131642.m5rgpkhnrffq5nrx@circular> Content-Language: en-US X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210104_082208_834619_60E2827C X-CRM114-Status: GOOD ( 23.71 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Rob Herring , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Nishanth, On 04/01/21 6:46 pm, Nishanth Menon wrote: > On 18:40-20210104, Kishon Vijay Abraham I wrote: >> Nishanth, >> >> On 04/01/21 6:29 pm, Nishanth Menon wrote: >>> On 17:52-20210104, Kishon Vijay Abraham I wrote: >>>> Patch series adds DT nodes in order to get PCIe working in J7200. >>>> Also includes couple of fixes for J721e. >>>> >>>> v1 of the patch series can be found @ [1] >>>> v2 of the patch series can be found @ [2] >>>> >>>> Changes from v2: >>>> 1) Moved serdes_refclk node out of interconnect node and also replaced >>>> "_" with "-" >>>> >>>> Changes from v1: >>>> 1) Include only the device tree patches here (the binding patch is sent >>>> separately) >>>> 2) Include couple of patches that fixes J721E DTS. >>>> >>>> [1] -> http://lore.kernel.org/r/20201102101154.13598-1-kishon@ti.com >>>> [2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com >>>> >>>> Kishon Vijay Abraham I (6): >>>> arm64: dts: ti: k3-j721e-main: Fix supported max outbound regions >>>> arm64: dts: ti: k3-j721e-main: Remove "syscon" nodes added for >>>> pcieX_ctrl >>>> arm64: dts: ti: k3-j7200-main: Add SERDES and WIZ device tree node >>>> arm64: dts: ti: k3-j7200-main: Add PCIe device tree node >>>> arm64: dts: ti: k3-j7200-common-proc-board: Enable SERDES0 >>>> arm64: dts: ti: k3-j7200-common-proc-board: Enable PCIe >>>> >>>> .../dts/ti/k3-j7200-common-proc-board.dts | 38 ++++++ >>>> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 111 ++++++++++++++++++ >>>> arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 52 ++------ >>>> 3 files changed, 157 insertions(+), 44 deletions(-) >>> >>> >>> A bit confused on the dependency here. is there something merged into >>> next-20210104 that makes this series ready for pickup? is there a way >>> I can get a immutable tag for driver fixups to pull so that my dts >>> next is not broken for PCIe (I am assuming looking at the series that >>> this is probably not a backward compatible series?)? >> >> There are no driver changes for the basic J7200 PCIe support and the DT >> bindings are already merged [1]. >> >> There are few errata fixes applicable for J721E which has to be removed >> for J7200 but that depends on other patches to be merged [1] but that >> doesn't impact j7200 functionality. >> >> [1] -> >> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml#n19 >> [2] -> http://lore.kernel.org/r/20201230120515.2348-1-nadeem@cadence.com > > So, Dropping stuff like "cdns,max-outbound-regions" (change from 16 to > 32) will work on older kernels? Could you do a quick sanity check on the > couple of "fixes" patches in this thread is not breaking functionality > introduced in the older stable kernels? Okay, the driver changes are done such that it works with both old DT and new DT however the newer DT will not work with old kernel. So I think I should drop the "Fixes" tag in the DT patches. Thanks Kishon _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel