On 12/29/25 11:00, Kane Chen wrote:
-----Original Message-----
From: Cédric Le Goater <[email protected]>
Sent: Sunday, December 28, 2025 1:51 AM
To: Kane Chen <[email protected]>; Peter Maydell
<[email protected]>; Steven Lee <[email protected]>; Troy
Lee <[email protected]>; Jamin Lin <[email protected]>; Andrew
Jeffery <[email protected]>; Joel Stanley <[email protected]>;
open list:ASPEED BMCs <[email protected]>; open list:All patches CC
here <[email protected]>
Cc: Troy Lee <[email protected]>; [email protected]
Subject: Re: [PATCH v4 06/19] hw/arm/aspeed: Integrate interrupt controller
for AST1700

Hello Kane,

Thank you for the suggestion. Since I need to submit a v5 patch to
split the I2C code changes anyway,

Can you please introduce the bus label property at the end of the patch series ?
Please consider adding a functional test and updating the documentation too.

I will handle the naming adjustments and other minor fixes myself in
that version.

Thanks,

C.

Hi Cédric,

If I move the bus label property to the end of this patch series, it will 
trigger
a test failure in the current patch series.

Which test ?

To avoid this, I plan to move the bus
label changes into a separate patch series and submit it before the AST1700
series. I believe this approach ensures both series pass the tests properly.
What are your thoughts on this?

I would like to understand the issue first.

Thanks,

C.

Reply via email to