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: <63446da5.050a0220.92e81.d3fb@mx.google.com>
Date:   Mon, 10 Oct 2022 14:44:46 +0200
From:   Christian Marangi <ansuelsmth@gmail.com>
To:     Andrew Lunn <andrew@lunn.ch>
Cc:     Vivien Didelot <vivien.didelot@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Vladimir Oltean <olteanv@gmail.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        "Russell King (Oracle)" <rmk+kernel@armlinux.org.uk>,
        netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        Pawel Dembicki <paweldembicki@gmail.com>,
        Lech Perczak <lech.perczak@gmail.com>
Subject: Re: [net PATCH 1/2] net: dsa: qca8k: fix inband mgmt for big-endian
 systems
References: <20221010111459.18958-1-ansuelsmth@gmail.com>
 <Y0RqDd/P3XkrSzc3@lunn.ch>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Y0RqDd/P3XkrSzc3@lunn.ch>
Precedence: bulk

On Mon, Oct 10, 2022 at 08:53:01PM +0200, Andrew Lunn wrote:
> >  /* Special struct emulating a Ethernet header */
> >  struct qca_mgmt_ethhdr {
> > -	u32 command;		/* command bit 31:0 */
> > -	u32 seq;		/* seq 63:32 */
> > -	u32 mdio_data;		/* first 4byte mdio */
> > +	__le32 command;		/* command bit 31:0 */
> > +	__le32 seq;		/* seq 63:32 */
> > +	__le32 mdio_data;		/* first 4byte mdio */
> >  	__be16 hdr;		/* qca hdr */
> >  } __packed;
> 
> It looks odd that hdr is BE while the rest are LE. Did you check this?
> 
>    Andrew

Yes we did many test to analyze this and I just checked with some
tcpdump that the hdr is BE everytime. If you want I can provide you some
tcpdump from 2 different systems.

Anyway it looks like this family switch treats the hdr in a standard way
with the network byte order and for anything else stick to LE.

Also as a side note the tagger worked correctly before the mgmt feature
on BE systems and also works correctly now... just any command is slow
as the mgmt system has to timeout and fallback to legacy mdio.

-- 
	Ansuel