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: <b34ab4d0-c14f-caeb-afde-01655b35a236@os.amperecomputing.com>
Date:   Tue, 13 Sep 2022 14:42:14 +0700
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0)
 Gecko/20100101 Thunderbird/102.2.2
Subject: Re: [PATCH v3 2/2] ARM: dts: aspeed: Add device tree for Ampere's Mt.
 Mitchell BMC
Content-Language: en-CA
From:   Quan Nguyen <quan@os.amperecomputing.com>
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>
 <396a3c24-2a86-0d1c-07f9-386a0de1d473@os.amperecomputing.com>
In-Reply-To: <396a3c24-2a86-0d1c-07f9-386a0de1d473@os.amperecomputing.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?aktxajV3Lzd3VUpldkNzbTJ5a0RBdk8wVW91T2R6QVhqZCtCbjZPeWRwSnpy?=
 =?utf-8?B?VVFabmRqdTVNS3k3QXo2UDJrVDNEV2JQWEFEZnlCN3FPQ2gyNVF6clRiZXIr?=
 =?utf-8?B?QjUrYlVTanE2QkU3b3pGdkJRblFnbld2RE12a0pZaGRKbjNvWGE3VFpaR0Fi?=
 =?utf-8?B?a2Z3eXBmdEQ1M2VLR2ltVzRpaklMUzk2VDFiMFAwU2dLUmd3Vkl4U0FkMGZK?=
 =?utf-8?B?dWgrSktaNHJiZVZkNGh2WDJxRlhKY0pka3lrdEM1RGV4MDhLTmI3bkRzbGZI?=
 =?utf-8?B?ajVJSGloWi9nTCtDNXBUMWVlSHByTzU3MFgwRTNzVmJaV24zOEdwMVNnS0ZY?=
 =?utf-8?B?K2l3d3VjREd3Y0QzYW5JUWJ3MTFYbUo5eGg1ZGw3K0JBbnJJR05tR1BQNVNu?=
 =?utf-8?B?NlZQVEtvVGRGRGIwb1lHdm52M0kzTFptaGFGamJscTg3TVVHQjRYT2d6c0do?=
 =?utf-8?B?WS95K2wzTlJBY2dnTmcxNVlidHJ6QXlvN3lFOTYxcnkreXkzQ0Q4c2ZkWkth?=
 =?utf-8?B?UHZBZERIVm5kcUErcTF1eVJXR0F6M3FtWUsrMU1xZGplSDB0VUJVNVRKUXBE?=
 =?utf-8?B?Z3NOckVJcUswd3VwS2pqcXFwd3hyVEVQdTRIMnJkcFRheHpyWDI3TmU1bnJo?=
 =?utf-8?B?MFU2ZGkzTWhYa25ibGVWTGt0OXZkaCtPaHZXQk9DR1hCOVpsV3cxUWdTdWsz?=
 =?utf-8?B?K1JWMlBQeHJZdWNlTm0wR0FCQUFYOUZXalpFNlMrWkpEd3ZiblhKUUlNbjQ0?=
 =?utf-8?B?YjJ6Mks4VDIreDAzd25KbE9XeTBpbEtrTUpMWW1xVnF1NFRualZYZTZvMFE5?=
 =?utf-8?B?RVNYNTAxa2Nkcm9lbXNBazRwbWRhcTVPQ3FoZytFa0JYVnc4T3pWUE1lelFK?=
 =?utf-8?B?eTlSb0ZvOUdPTmRZcHBqQXMzcHpFY1hxbzlZQU9iZUlrbXFuRXF6L2xpUkZu?=
 =?utf-8?B?UGQ2alBaY1dYcGNFdXdibGFFK3JWS3p5bHNSMitYeXY5aWNNSHN5TmoxQVFQ?=
 =?utf-8?B?OC9TQWRhcVRYS01HbUpGZGc0ejNWWUNQcWFvMnRRK3BiQlNxMmY1MVBreUFO?=
 =?utf-8?B?QVVHQ3ZuUEN6aW9LSlYxRi9ycDdIY3RsQm83a2MvZ1o2Z2daV1VEV0hPQkpS?=
 =?utf-8?B?SDZxMHU4bXhxQkZQVko1dDM2d0NXOEc5Q1ZCNzM1TWJNSHJ5cWUyTFpFdE13?=
 =?utf-8?B?QUIwTkJOYm9JOG5HRnExOTBlMGZTbDgvUm1CUFFHbkk2bXhUc1VrZUJZZmpL?=
 =?utf-8?B?czF1c0hVaHdDZ1J1a3lBWW15UGdrbXYzdlBVR1RmSVBjVlNXSkpUN0d3ZHlT?=
 =?utf-8?B?R3dqZzZ5cUlkYnFZREZxelgxczVNem50Wk1hUzhDNmRRck85dmhzZlVOdDBm?=
 =?utf-8?B?M0haUGQ1Zk1UNEE1UUFMOW5JaHowSFZ4THcrU1RIelFGaENSc2JkU3FCRUVT?=
 =?utf-8?B?VVltWDk5YnVnaERST29PWjUxS3ZweVFpZEJaSzdvY2NLTDVYRkpVaDdUdUtJ?=
 =?utf-8?B?eUFqZWV0ZGJuK2ErSVVUcWZFYUFUNmlIUWpOYnU3N1pxTGZOZmtWVFVzNEtC?=
 =?utf-8?B?Y2k5YWJaRWJUc0dYY2czZmRyR1YrL3JocXVoNlVnREsyZ3JxQi9kckNod2dy?=
 =?utf-8?B?L1JEcmxmWG1pQzVnZC9PVkNUOTRqVmh2QUtZZkVieFEzS0hTUDljNEhRRjlK?=
 =?utf-8?B?aFU0d0YrbVRHQU1EWFhTNzNVVW96ZGVIekVnRElJdWowWlBOM3RJL3VaVHJ6?=
 =?utf-8?B?c04vbHZ4Y1hYRlBYQ1pjZmNSOGJkVnB4elU5WlZkZ3B6M3Jubnp4d1hQdHB4?=
 =?utf-8?B?SG5vczdQU3BMVFNwMlFaWk9IcjQyaTcvaFE2aG00QWJnQXQzTjdXdmFURDhQ?=
 =?utf-8?B?bUtESk9JUHlQQTB5NkVFd3BGWGJuaHVycTh3WFYyTldJaW9Qd00vWmJMQTds?=
 =?utf-8?B?WnpqMzhkbEFGcVVFRUdReWJ6RnhYKzZIU1EwbjlYMkVrQXVCWlAvQWdjNGZx?=
 =?utf-8?B?YmxiMHZKdWtRR2trK1phQ21wUWh0dE1MQ2h1bWtpWUR6bGZuVW9ITURWSXY1?=
 =?utf-8?B?N2J5TmE1NTYvUUdYVENIZGQ5YWNBei9xS2V2TVV2SEp2eVVrYTdZaStGSGsv?=
 =?utf-8?B?MlJ4Mm1JRTlJMExwc3ZBOStyMXNrRWhNMm9XdU4yRCtudGpFdVNZMzJDUzlJ?=
 =?utf-8?Q?x7lGFpTwh676YNNWaeX6r2M=3D?=
X-OriginatorOrg: os.amperecomputing.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a308a95a-2f5f-4fd7-e043-08da955b8057
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR01MB7282.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Sep 2022 07:42:25.7603
 (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: yGNrL2TE7NHbQCqmpP5mzdUvkrEjiO/nDf4gimb5LSa7wUKnYHf6+c6zIeSofQBdWKHXXYtKmSd9nRPiESHSkJpdJRmDhcEA5kJ6ZZ3N+m4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR01MB4205
X-Spam-Status: No, score=-4.2 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

Hi,

Just a gentle ping on the patch.

Thanks and best regards,
- Quan

On 05/09/2022 16:45, Quan Nguyen wrote:
> 
> 
> 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