Claude review: drm/bridge: ite-it66121: Disable HDMI audio capture stream

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

Patch Review

**Correctness:** The patch correctly sets both `no_i2s_capture` and `no_spdif_capture` to `1` in the `hdmi_codec_pdata` initializer at `ite-it66121.c:1475-1476`. Both fields are valid members of `struct hdmi_codec_pdata` (defined in `include/sound/hdmi-codec.h:126,129`). Since the driver already sets `.spdif = 0` (SPDIF disabled entirely) and `.i2s = 1`, adding `.no_i2s_capture = 1` is the important one here, while `.no_spdif_capture = 1` is a belt-and-suspenders addition that doesn't hurt.

**Placement:** The new fields are placed logically between `.max_i2s_channels` and `.no_capture_mute`, matching the struct field order in the header, which is good style.

**Precedent:** The `tda998x` bridge driver (`drivers/gpu/drm/bridge/tda998x_drv.c:1177`) already uses `.no_i2s_capture = 1` in the same manner, confirming this is the established pattern.

**Commit message:** Clear and well-written — explains the hardware limitation, the symptom (PulseAudio errors), and the fix.

**Minor observation:** Since `.spdif = 0` already means SPDIF is disabled, setting `.no_spdif_capture = 1` is technically redundant but harmless and defensive. This is fine.

**No issues found.** 

Reviewed-by quality: this patch is ready to merge as-is.

---
Generated by Claude Code Patch Reviewer