[PATCH] ARM: dts: aspeed: Add dts for ASUS Kommando IPMI card

[Anirudh Srinivasan]

Hi Andrew,

On Wed, Mar 4, 2026 at 11:22 PM Andrew Jeffery
<andrew at codeconstruct.com.au> wrote:
>
> On Sat, 2026-02-28 at 22:00 -0600, Anirudh Srinivasan wrote:
> > Add dts for Asus Kommando IPMI card, an ast2600 based pcie bmc card
> >
> > Signed-off-by: Anirudh Srinivasan <anirudhsriniv at gmail.com>
> > ---
> >  arch/arm/dts/Makefile                            |  1 +
> >  arch/arm/dts/ast2600-asus-kommando-ipmi-card.dts | 98
> > ++++++++++++++++++++++++
> >  2 files changed, 99 insertions(+)

> > +
> > +&scu {
> > +     mac2-clk-delay = <0x08 0x04
> > +                       0x08 0x04
> > +                       0x08 0x04>;
>
> Before we go too far down this path, are you aware of the discussions
> on the kernel side?
>
> https://lore.kernel.org/all/20260302-rgmii_delay_2600-v6-0-68319a4c4110@aspeedtech.com/
>
> Essentially, can we defer configuring the delays via the SCU here so we
> can isolate that problem in the kernel?

I've been trying to follow that discussion over the last few days, and
the more I read it, the more I get confused.

I went back and looked at the u-boot DT from the Asus fw dump [1], and
there seems to be no mac2-clk-delay in it. That uses u-boot 2019 too.
Not sure how accurate/useful this info is though.

So (correct me if I am wrong)
1. If we don't need this clk delay at all, then I can skip this
clk-delay entry in u-boot and use "rgmii-id" in the linux and u-boot
DTs.
2. If we need this clk delay, u-boot is supposed to configure it
(u-boot DT uses "rgmii") and linux uses "rgmii-id". We don't need to
worry about u-boot patching the linux DT because both use separate
DTs.

This feels like something I should be able to test without any patches
to u-boot/linux. Just try booting u-boot with 2 different DTs (one
with rgmii and one with rgmii-id) and see which scenario networking
works.

Is anything I'm saying correct?

>
> I recognise that you might want networking in u-boot as well though :)

I then read this statement and got confused a bit more.

[1] https://gist.github.com/Genius1237/0b81fcbc8c25a8b516e63e466e8522b1

>
> > +};
> > +
> > +&sdrammc {
> > +     clock-frequency = <400000000>;
> > +};
> > +
> > +&uart5 {
> > +     status = "okay";
> > +     u-boot,dm-pre-reloc;
> > +};
> > +
> > +&wdt1 {
> > +     status = "okay";
> > +};
> > +
> > +&wdt2 {
> > +     status = "okay";
> > +};
> > +
> > +&wdt3 {
> > +     status = "okay";
> > +};
> >
> > ---
> > base-commit: d40130c7056b84dc5213f3ff6764d0e45bb84035
> > change-id: 20260228-asus-kommando-9de4d6b779a8
> >
> > Best regards,



-- 
Regards
Anirudh Srinivasan