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: <396a3c24-2a86-0d1c-07f9-386a0de1d473@os.amperecomputing.com>
Date:   Mon, 5 Sep 2022 16:45:45 +0700
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0)
 Gecko/20100101 Thunderbird/102.1.2
Subject: Re: [PATCH v3 2/2] ARM: dts: aspeed: Add device tree for Ampere's Mt.
 Mitchell BMC
Content-Language: en-CA
To:     Arnd Bergmann <arnd@arndb.de>, Joel Stanley <joel@jms.id.au>
Cc:     openbmc@lists.ozlabs.org, Olof Johansson <olof@lixom.net>,
        soc@kernel.org, Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Andrew Jeffery <andrew@aj.id.au>,
        linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-aspeed@lists.ozlabs.org,
        Open Source Submission <patches@amperecomputing.com>,
        Phong Vo <phong@os.amperecomputing.com>,
        thang@os.amperecomputing.com
References: <20220817071539.176110-1-quan@os.amperecomputing.com>
 <20220817071539.176110-3-quan@os.amperecomputing.com>
 <CACPK8XdDpG3ONM1=-E6qvHL1FgMNWSMPoL_sVGJK6BmmnT3w_w@mail.gmail.com>
 <CAK8P3a2LZKfZpdTQ-R4o9mJ6dk52VRF+Bxj=PJEx-1MA4yH8+w@mail.gmail.com>
 <673e200f-f458-7866-f956-3d5bd7160a49@os.amperecomputing.com>
 <CAK8P3a3MU7shuBpcpRNC5L6xxQmSy8FXCX1jvYXhv-NT3PMYOw@mail.gmail.com>
From:   Quan Nguyen <quan@os.amperecomputing.com>
In-Reply-To: <CAK8P3a3MU7shuBpcpRNC5L6xxQmSy8FXCX1jvYXhv-NT3PMYOw@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MFlneHN2WVRxcHJKVzBQZkZFQ0NqSTZmek8yd2ZOTTVxYlVodmZrLys3K0VL?=
 =?utf-8?B?K1Jtb3dpWVFjRHJXU1pLS3hLR2NIOGNLb3RueWdLSHhTMURHclNOQmVrKysv?=
 =?utf-8?B?ZEEvTVBiRkNuNkkxM0Q5dnN3TGpCZU5aVDJVc1BPSG4wUTRhbjlwYi9ja1ht?=
 =?utf-8?B?V1FFcDNPOTdWRGpJaE1YaDI2MkVHRnRydjI3YUlZWWE4VHlBaVhxN1lmc0Vr?=
 =?utf-8?B?Z1FkTUZ5cEdXbXY0ZU1FVkFVNHNWeEhCS1dqYmN1cTdIZnE0aUt6ZVNzUWJs?=
 =?utf-8?B?U0c4SEoxY2EwS1ZocXorZUZxN0M3NUlRYloyZmZZd1krRGt4ZkFzallLbnlp?=
 =?utf-8?B?dnFZeG5UL2hiV0J3em13ZUo2VW9FM2hGL0trYTJ4RWtDcDVxcm42ZUhuZEtH?=
 =?utf-8?B?S000QXhZeFFmc1JxbUFuYkVDb1JVRklxTVk5OVN0eEs5VEpsL3hXbVVWTndL?=
 =?utf-8?B?VkxwOXA1bVluZ3Q1ZStHT21SM1ZiVUhia0pJOC9KS0lPV25tQWltNW45R0NV?=
 =?utf-8?B?aWhhczJZaVYwdElRYTBFWFZpOXdzRm50NDArV0R5RXdxRTZVZDlXYzNldzZY?=
 =?utf-8?B?cVdnL2VobnlmRjBiVFYxVDlQU09RZnRwcGo5NXR5ZXUyeVYrNFNkMDhXY1Nm?=
 =?utf-8?B?TjFtZzNFQzhCRm5kUnVSeEVWMzhSYllHSWZtbmxDTjFmUkVRZnZGQlB0U3dj?=
 =?utf-8?B?M0RPZGZIZDhId3Y1NDl0VFZOYWd3Y3o3Y0VldzVFMmpsNzdvWi9VT2RqeUhK?=
 =?utf-8?B?MEpjVmxxakRCdWV0blFJUzRPMFFOTC9mb0lIenRUdVU3WUcyWmw4SVVkVkNh?=
 =?utf-8?B?WFA5c1N5a0I1MHRnamtsU3ZWeS9pZ24rWW5PdjFoV3NvTTdybGxKbzZYd2ZK?=
 =?utf-8?B?UDlUYTlCa2lhVUVaUm1QbHp3ZS9QZy8wYnRkOXhCdVQ2UEFwOTRJdnpKSnhW?=
 =?utf-8?B?blhnMDZORzVsek4vQW5OWUNhNWV1RGJIOHhaeklsb1dtMWxGdWY4MjAvVXBs?=
 =?utf-8?B?R1RQRmNISlRsbndsQWlwMXZYZWpROUdXVjhNemt0Q2M4YUM1NFVrNW43VkU1?=
 =?utf-8?B?OHBYN1BicW1RMGhVMUdOc1VXTzZndEsrYWxNdFBBbS90TlU0NkZZMFVPeVVX?=
 =?utf-8?B?QnFIeFlBeEJqamRMeDMzYTZVWHRxMG9Na1JQbTdoeDVsbWFpNjZ5cnpaQzV0?=
 =?utf-8?B?UjhzUlFUeUJXWmNhQ1FzZHRFMVg4Yi9TVTBOZlpsMmRzVGYvOXhOSlNsWDkx?=
 =?utf-8?B?RkQ2MDZZRk9vMTNTVnU1emlLazdnWk5nUjYxOEVsTTRSMXpKS2x0R2hFY1dO?=
 =?utf-8?B?RXZaM1Q3MWNoYTY4MUZ6U2NjZlNIeGlLUTl4SCtCNTBQUmk3eVlDdlVueGFx?=
 =?utf-8?B?R0xyT01JbWsrY3ZKNWk2ZS9PcnBWbE1Jd0U5ZjJtSVF3Y3dBeE9iQkM2RWZO?=
 =?utf-8?B?bHVZU3ZEMUF0RS9KbFFOaFVCSWFkdVZOaVBSSHlRKytUMnRMbzRIbVY0anZB?=
 =?utf-8?B?TXgyVHdNeVl0aE5rZ2MxV2I4QlZLZUFBQ0FldjdKMXJxYzJSd3VON1kzN1dk?=
 =?utf-8?B?d0ZyUk9GTkNlMlpyZy9uUTQwdElkRnQzVTBlVUVyd0krSDVNSHUyMVgrZDBi?=
 =?utf-8?B?THdjSnpWbU9rcE1EYi8yUmdFSDU1Z21kVW4wd0ozQ0VTRTNjdWphVktVd0NH?=
 =?utf-8?B?bFFDVGVtRWFVallzOVpWUEtRd2RybXNZOWFxU1RRMlphYmp0MHJTbEcvb0Rv?=
 =?utf-8?B?L1NhSnUwUTlnWlRQbVF0NWdGa2d4Y3ZEZG9iYm9vcHd3b0xnZ1BDbmQ5WXFB?=
 =?utf-8?B?SmF2aGQrcmM3USs0RFpSbTE4a2t6UmpEQTA2NWR4WExQOVhqWURkNFI5Nzhl?=
 =?utf-8?B?TWZ3dnh1M0JiQWFKZjh5ZVl1ejFQSm9WTTZoU1JvU0Z5T0ZBT3M0L3pPZGpk?=
 =?utf-8?B?RFFhK3krR1FMY2c5L1NBODllNERJZGhrd1pWOVlTYlNLczB4eXBVc1hGc01W?=
 =?utf-8?B?bkdDNVJ6Nk1uYWRmRnVYc2o1MnRLRTA2U09tWTBnaW9Pcm5oa2RtZ3RYc1M0?=
 =?utf-8?B?Tk1ZOE5xT0R4clBkVjJuQTJITGRVaTdMSXJFY2pBd2l0ZjgzOE1FQ0JKamFW?=
 =?utf-8?B?dW5WS1ZuT1hWeFpLMGkvNytMWlpScm1lS1lFcGJrU2d1elBESDZwV0MyU0lL?=
 =?utf-8?Q?c+su9Fi3h/yw6+akkcHMfMs=3D?=
X-OriginatorOrg: os.amperecomputing.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1a0364ea-7f3b-4715-5af7-08da8f236ce7
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR01MB7282.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Sep 2022 09:45:54.3668
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: E9nTPhYltf4mNp8qgXg6vObL7hk+uhSHTEpVzbOgKdZKZc6JKjXtEat1iguPNX+44WolLVcdCAwxdiG1NxAvxDOlR4/Q7PV0vPZmkkt9D8Q=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL3PR01MB6868
X-Spam-Status: No, score=-3.7 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        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 25/08/2022 19:35, Arnd Bergmann wrote:
> On Tue, Aug 23, 2022 at 10:27 AM Quan Nguyen
> <quan@os.amperecomputing.com> wrote:
>> On 18/08/2022 19:06, Arnd Bergmann wrote:
>>
>> Thanks Arnd for the comment.
>>
>> I think adding -append could solve the issue.
>>
>> But as the bootargs still exist in all other
>> arch/arm/boot/dts/aspeed-bmc-*.dts should we still keep bootargs for
>> this dts?
> 
> I think it should still be removed. Only 238 of 2547 set the console using
> bootargs, so that would make it more consistent with the other files.
> 
> Changing the files that have the same issue is a separate matter.
> 

Hi Joel,

In the meantime, I'm just wonder if you could help to test the patch 
with -append and get it merged.

Thank you and best regards,
- Quan