Re: [PATCH v3 2/2] ARM: dts: aspeed: Add device tree for Ampere's Mt. Mitchell BMC

From: Quan Nguyen
Date: Wed Sep 21 2022 - 03:57:47 EST


Hi Arnd,

I'm really sorry but I have to ping you again as I'm confused and still wonder if anything else could be updated to this patchset. Could you share some suggestion on what to do next ? or should I resend it?

The only thing left is Joel's complaint about there is no output to test with bootargs removed. I have tried and see it could be fixed by using the -append option with qemu.

Besides, Joel also says it "looks ready to merge" and give a Reviewed-by already.

Thank you and best regards,
- Quan


On 13/09/2022 14:42, Quan Nguyen wrote:
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@xxxxxxxxxxxxxxxxxxxxxx> 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