Re: [PATCH v4 3/4] cxl/hdm: Make switch decoder target parsing endian-safe

On Wed, Jul 22, 2026 at 02:07:49AM +0100, Jonathan Cameron wrote:
> On Mon, 20 Jul 2026 18:59:39 -0700
> Alison Schofield <[email protected]> wrote:
> 
> > Switch decoder target IDs are stored one per byte, starting with
> > target 0 in the least significant byte. The current code accesses
> > those bytes in memory order, which only matches register order on
> > little-endian hosts.
> > 
> > On big-endian hosts, the target IDs are reversed and the decoder is
> > associated with the wrong downstream ports.
> > 
> > Extract each target ID from the register value with an explicit shift
> > and mask so the target mapping is independent of host endianness.
> > 
> > Fixes: d17d0540a0db ("cxl/core/hdm: Add CXL standard decoder enumeration to the core")
> > Reviewed-by: Richard Cheng <[email protected]>
> > Reviewed-by: Li Ming <[email protected]>
> > Signed-off-by: Alison Schofield <[email protected]>
> 
> Suggestion for explicitly handling the endian issue inline.

Done in v5.


> I'm also fine if you stick with what you have here.
> 
> Reviewed-by: Jonathan Cameron <[email protected]>
>
Dropped all the Reviewed-by: tags for that change though. :(