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;
Message-ID: <78072f5f-9bbd-971f-f57d-1cc86c0168c9@siemens.com>
Date:   Wed, 19 Apr 2023 14:28:20 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.9.1
Subject: Re: [PATCH 2/5] arm64: dts: ti: k3-am65*: Drop bootargs
Content-Language: en-US
To:     Nishanth Menon <nm@ti.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Vignesh Raghavendra <vigneshr@ti.com>
Cc:     linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        Tero Kristo <kristo@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
References: <20230418165212.1456415-1-nm@ti.com>
 <20230418165212.1456415-3-nm@ti.com>
From:   Jan Kiszka <jan.kiszka@siemens.com>
In-Reply-To: <20230418165212.1456415-3-nm@ti.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NkZobG9ZSnhSeE1jTDZ3WVFQaG5FV1dwUVh0N0RDYkZRKzhyYjJENEU0ZFA1?=
 =?utf-8?B?cjhHTC84ZEpkWEpGbDk1WjJkT0tGYloxazh1TnF3d2plbXJsOWthR25SQURR?=
 =?utf-8?B?UDJQNm9kMmxTS1BDQTdVN1poTjVyYm9MN1FEbG9sVXNVeGpJSkNRUXJackNw?=
 =?utf-8?B?aWFISnpHRHQ3cG9RazdWcHhlcDRBWmh0bEZ4SVJBcHZXMGF4OVhzSlR0Qmx2?=
 =?utf-8?B?TXdhYmhYdEsvNEJCbVBaUHVJQ0hnWlMvTWMzSjlTRzJzTG5MZkoyT3VtK29B?=
 =?utf-8?B?MlAyaUI1WU4xZjRzNDI2R0VxaXVEVk4zU1RrRnBpVFE2Qi8zcVFXM2taVFEy?=
 =?utf-8?B?YTU3SlBvT1VmVDZkVzE3N2dDdE5ObkVHWS9EUlc0OG0xekpzc0k4cTJxUmJa?=
 =?utf-8?B?MkU5MkZXUkZidG00R1hzQU9YaUhEZWdqeTdzalROb2hzNFpZVGxIZksyS01J?=
 =?utf-8?B?TlNtUkIvMEp0U2FreEtDR2d5R0dYbmZ4ZjN3Mm15a0lLejY1VFpqNEI2VmQx?=
 =?utf-8?B?dmk5ZXpPaDFVWEtiLzZlL2Y5OGVTSEVVd2FiVHd5a0NrNGRNRHNrT3FhQ2lY?=
 =?utf-8?B?cWlKM0RBTnRZL0laVUVyckZyZ1g5MUN2cTVVdXVIQTBvMU0rOTdrWFdqdkVo?=
 =?utf-8?B?TWZJK2JONU1nYVk1THBlZmdVQno5TGY0T09MczhESXpYRFptNzZ5eWNqSndS?=
 =?utf-8?B?TExLNGZ3VU0xTHdhbmhQTy9YQy84ZmVlNzZoWWZxdVVKZytnU1lPL3pXRy9v?=
 =?utf-8?B?STg1RGtydVo3RjBnNHoxcnZ3VzhzLzEvelpiVGh1R3FTWGdXZFlDTkxQTkw4?=
 =?utf-8?B?bkNrbEtmbDVOcUJ5bmxHNmhPREloc3dYTlNYc0QvTmZVdkZyKzduTC9RTFdj?=
 =?utf-8?B?ZWRmeHJ3Vko2OTVBYWVncUhiTC9abDNkcktUdVJyUjVyU2l6YlNrL1oyY08v?=
 =?utf-8?B?ZTEwWHFWZG1KY3JGQ08xTDVjaUkzYVBPU0daZ2tmMUNZWXVOZmhzcFp5YWV6?=
 =?utf-8?B?RmlrZXhVVTlPQmhTVzRPZEF6RjhTaTc2MUlVYmZ2M3N1YndtNkhVbThvYmE4?=
 =?utf-8?B?dXZCY0xNdWIxRFhSTVdoMkdVU2FRSWRrcWorcWo3ckNNN3RYMWJ0V3JnY2ky?=
 =?utf-8?B?V09RVXVSUnVCemFxMFhrdHBOUS85cDQ4RUdWTFYrSm9pVHc1SHlCWUo0V1hX?=
 =?utf-8?B?cG0xT3R0WGdKRDQvbEhOcnJybEx6VWF0cWNmOUpkcDNjS3Q0Tkg1akI3WDRP?=
 =?utf-8?B?Nk9DY0thS09hbHJENFRHZit6NEd4QlJLM1BkNG5ZUmxnOGVsR0x1R3FzdFBk?=
 =?utf-8?B?RzhyR1k5SE5vNDI5VkF2QkxRL1lYTlRLWldhOWlIWWZ1bFR3THpWLyt1NGp4?=
 =?utf-8?B?TFRmUDgyUmJHcEkvMzZOSkEyY3RrWE9VSUxqaXBoSWpFVEZLY0p0d3pHRjZU?=
 =?utf-8?B?UHNGSFl2eGg5dDN4N0JDdWVRYnFjTkZrWEF1bElBeDI2UHBQKzVxL2dNWkxH?=
 =?utf-8?B?NmxOcm82OEtUSmxGT1lwdk05MEhhbVhIZWVnWVJtNzJOMnlOTC96MG5hZk5p?=
 =?utf-8?B?Y216aE14K2VHM3pibU5uSDNTdEw3OHVMcUk3TkNIMUFmSTc4Y2lPUEtMRUs1?=
 =?utf-8?B?bTR2eU93YUpMMTdxTGFSVUhsY01lMVZwRHVNT2ZmWDBzL0l0eEJKNzVqdHNp?=
 =?utf-8?B?NFFvbFlsTzR4ZU5rUG1iclJhMFl1MVI5SHdSbGhTSmhaRlhEdDRsRURHR2lU?=
 =?utf-8?B?ZzRQcEd3YVJMb1NKZ3ptVVRQMWNPek5Rd0h6OGVvUXU5cXRmK0o2dzBQZks5?=
 =?utf-8?B?dE93c2Fvc2lUM1g1UG5GUTNUczl6bkNQcG13THd5Vk9SVnhEN0hQN1hLNDIz?=
 =?utf-8?B?QzNMbUhHMGZuNlh4NXg4bG9XZFRneFF2L0xzeXEyMGdlUm93YUJPRWtCV3ZH?=
 =?utf-8?B?cyt0Wi9yb3NRWGVCcXJSd2dPRTFHaENGVll4RHlvSzFndGxkNGhoVlZya2Vr?=
 =?utf-8?B?NGZnWjBFSkR6ZUxZejRLMEtLTEpHd04yZWNlZXlWOHMvTHpmZHgzbDhIVVp0?=
 =?utf-8?B?YnpNUHdqTldJQmZ5RjFVbTlrRmlqZy9oclBNSURla3hTWUFscnJIaFIxSUQz?=
 =?utf-8?Q?cRwfjW1ovweMDxXucVxFyaSM1?=
X-OriginatorOrg: siemens.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 733f2dba-9409-42ad-968f-08db40d19238
X-MS-Exchange-CrossTenant-AuthSource: AS4PR10MB6181.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Apr 2023 12:28:25.0433
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 38ae3bcd-9579-4fd4-adda-b42e1495d55a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: LsLKNSszQ9kukcucrEuqWgLN3i4cyx/gh0QPcuHt2tpr1gT2IBi/mmKHJCsKvEJEuzv0Xjhkm2Z/nxcDoYshvQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR10MB3682
X-Spam-Status: No, score=-3.7 required=5.0 tests=BAYES_00,DKIMWL_WL_MED,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED 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: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 18.04.23 18:52, Nishanth Menon wrote:
> Drop bootargs from the dts
> 

No reasoning provided, apparently also in the other patches of this
series. Given that these DTS here are not the last ones using bootargs
in upstream, that seems reasonable to state here.

Jan

> Cc: Jan Kiszka <jan.kiszka@siemens.com>
> Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
> Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 1 -
>  arch/arm64/boot/dts/ti/k3-am654-base-board.dts     | 1 -
>  2 files changed, 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> index 96ac2b476b11..7d256a1638ff 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> @@ -21,7 +21,6 @@ aliases {
>  
>  	chosen {
>  		stdout-path = "serial3:115200n8";
> -		bootargs = "earlycon=ns16550a,mmio32,0x02810000";
>  	};
>  
>  	reserved-memory {
> diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> index 592ab2b54cb3..0d6fc89eba7a 100644
> --- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> @@ -15,7 +15,6 @@ / {
>  
>  	chosen {
>  		stdout-path = "serial2:115200n8";
> -		bootargs = "earlycon=ns16550a,mmio32,0x02800000";
>  	};
>  
>  	memory@80000000 {

-- 
Siemens AG, Technology
Competence Center Embedded Linux