Claude review: accel/qaic Fix HELLO_RESP mode sourced from wrong rx field

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Verdict: Correct fix, minor subject-line nit.**

The bug: In `sahara_hello()`, when building the `HELLO_RESP` packet to send back, the code was reading the mode from `context->rx->hello_resp.mode` instead of `context->rx->hello.mode`. Since this function handles an incoming `HELLO_CMD` (not a `HELLO_RESP`), the received packet's payload should be accessed via the `.hello` union member.

Looking at the struct definition at `sahara.c:63-98`:

```c
union {
    struct {
        __le32 version;
        __le32 version_compat;
        __le32 max_length;
        __le32 mode;
    } hello;
    struct {
        __le32 version;
        __le32 version_compat;
        __le32 status;
        __le32 mode;
    } hello_resp;
    ...
};
```

Both `.hello` and `.hello_resp` have `.mode` at the same offset (the 4th `__le32` field in the union), so **in practice the old code would read the correct value**. This makes it a correctness/readability fix rather than a functional bug fix. The commit message says "Fix" which implies a functional bug; it might be more accurate to describe this as "Use correct union member" to avoid overstating the impact.

**Nits:**

1. **Subject line missing colon**: The subject reads `accel/qaic Fix HELLO_RESP...` but kernel convention is `accel/qaic: Fix HELLO_RESP...` (colon after subsystem prefix).

2. **Commit message could note no runtime impact**: Since the union layout means both members alias to the same offset, the commit message could mention that the old code happened to produce the correct value but accessed the wrong semantic field. This helps reviewers understand the actual risk level.

Otherwise, the one-line change is obviously correct — the entire rest of `sahara_hello()` consistently uses `context->rx->hello.*` to access the received packet fields (lines 299–302, 306, 309–311, 315–319), and only the mode assignment on the old line 322 was the odd one out.

**Reviewed-by worthy**: Yes, with the subject-line colon fix.

---
Generated by Claude Code Patch Reviewer