From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from lelvem-ot02.ext.ti.com (lelvem-ot02.ext.ti.com [198.47.23.235]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2E13422F770; Wed, 4 Jun 2025 09:48:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.47.23.235 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749030540; cv=none; b=W3VJV8LNZ5cm80ddFzVcY79sGkFeJGszNyH2d4iCLzU5BbxK0Sap6OUKmx2tq3K2WbCLDeXr4CD/Zyz0IqgKa6cfKfOHkKYd4MA5FahF1mkOfxm1IPSYB50C74tISVa8Z8tgVIsyoA+tZ5wyX02vkLaVBabrWXWnP1hP6WASue8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749030540; c=relaxed/simple; bh=66KZLAI+Sn3EhX8f3kJupOtyRki6+CSXzYdA/EoACIo=; h=Message-ID:Date:MIME-Version:Subject:To:CC:References:From: In-Reply-To:Content-Type; b=Ah8F8yXUOr0H6Q7jKFP68HEyTdbwWUE9aPlypWsaJB/bQS8Nb896WSM6IAcwLPlIiiQSNL7pOFBIGs1EvTbiYpfWP3/GfYTzyaUfMleAXrBjSz5WHEclLqj9z9NQkF8kQeqqtRbAHVntTMDD7ILM5Ru2uLdajpnMcqYx7j4yYqQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com; spf=pass smtp.mailfrom=ti.com; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b=XOYNRiYG; arc=none smtp.client-ip=198.47.23.235 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ti.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b="XOYNRiYG" Received: from fllvem-sh03.itg.ti.com ([10.64.41.86]) by lelvem-ot02.ext.ti.com (8.15.2/8.15.2) with ESMTP id 5549mjQp3778763; Wed, 4 Jun 2025 04:48:45 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1749030525; bh=Bq4EZsLahywFgP1f+8VxyYxdSzxk7zu6x4ss6clsEj8=; h=Date:Subject:To:CC:References:From:In-Reply-To; b=XOYNRiYGr17hHOZRZ+Cx1giSNAXcFaNqGIK9Rvk/4iftOBLwP/AQ/sGCjIKoWl226 /OaE/4+l1padrWTmhELfMxVCgwP4xmswve3h7QevT8QiGViy39BXFwL+P3WplQjem0 +CxJsEdWv8JOrmS8A7d/x1JnuC2bScTubCz7bko0= Received: from DLEE114.ent.ti.com (dlee114.ent.ti.com [157.170.170.25]) by fllvem-sh03.itg.ti.com (8.18.1/8.18.1) with ESMTPS id 5549mj2f4179988 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA256 bits=128 verify=FAIL); Wed, 4 Jun 2025 04:48:45 -0500 Received: from DLEE113.ent.ti.com (157.170.170.24) by DLEE114.ent.ti.com (157.170.170.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Wed, 4 Jun 2025 04:48:45 -0500 Received: from lelvem-mr05.itg.ti.com (10.180.75.9) by DLEE113.ent.ti.com (157.170.170.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Wed, 4 Jun 2025 04:48:44 -0500 Received: from [172.24.227.14] (jayesh-hp-z2-tower-g5-workstation.dhcp.ti.com [172.24.227.14]) by lelvem-mr05.itg.ti.com (8.18.1/8.18.1) with ESMTP id 5549mfEG1853766; Wed, 4 Jun 2025 04:48:41 -0500 Message-ID: <0c69137d-93ad-4843-8512-9885b9cba452@ti.com> Date: Wed, 4 Jun 2025 15:18:40 +0530 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/4] arm64: dts: ti: k3-j721e-main: Make the "scm_conf" node a "simple-bus" To: Krzysztof Kozlowski , , , CC: , , , , , , , , References: <20250603095609.33569-1-j-choudhary@ti.com> <20250603095609.33569-2-j-choudhary@ti.com> <15da3051-c35e-4876-9185-9079493dc66c@kernel.org> Content-Language: en-US From: Jayesh Choudhary In-Reply-To: <15da3051-c35e-4876-9185-9079493dc66c@kernel.org> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit X-C2ProcessedOrg: 333ef613-75bf-4e12-a4b1-8e3623f5dcea Hello Krzysztof, On 03/06/25 16:32, Krzysztof Kozlowski wrote: > On 03/06/2025 11:56, Jayesh Choudhary wrote: >> The "scm_conf" bus need not be "syscon". Now that we have "pcie*_ctrl" >> child nodes for "ti,syscon-pcie-ctrl" property in "pcie" nodes, change >> "scm_conf" node to "simple-bus". >> Also remove "reg" property as it is not needed by "simple-bus". > > This (possibly) affects all other users of DTS which were expecting this > ABI. It's not only about forward-compatibility, but other projects. > > Maybe this doe snot matter for you, so explain that in commit msg and > provide rationale why you are affecting other users. > > This should not affect other users of DTS. J784S4 already has this change and it serves as a validation that DT nodes are compatible with other projects (for example uboot driver handles it). Other SoCs are being modified with taking J784S4 as a working reference. Considering this, do I still need to add something? Or is it okay since I am not exactly affecting other projects? Should I add something along the lines of "similar to j784s4...."? Warm Regards, Jayesh > > Best regards, > Krzysztof