From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: misc: fastrpc: Expand context ID mask for DSP polling mode support Date: Mon, 16 Feb 2026 08:26:14 +1000 Message-ID: In-Reply-To: <20260215182136.3995111-4-ekansh.gupta@oss.qualcomm.com> References: <20260215182136.3995111-1-ekansh.gupta@oss.qualcomm.com> <20260215182136.3995111-4-ekansh.gupta@oss.qualcomm.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Assessment: Acceptable, good commit message** The commit message provides a clear explanation of the bit layout change: ``` [15:8] =3D CCCCCCCC (context ID) [7:5] =3D xxx (reserved) [4] =3D A (async mode) [3:0] =3D PPPP (PD type) ``` Changing `GENMASK(11, 4)` to `GENMASK(15, 8)` shifts the context ID into the = upper byte, leaving the lower byte for DSP flags. Because patch 2 already con= verted everything to use `FIELD_PREP`/`FIELD_GET` with the symbolic mask, thi= s is a one-line change. Clean and correct. Already has Reviewed-by from Dmitry Baryshkov. --- Generated by Claude Code Patch Reviewer