On Sat, Oct 12, 2024 at 11:12:09AM +0800, Zhangfei Gao wrote: > > + case CMDQ_OP_ATC_INV: > > + case CMDQ_OP_CFGI_CD: > > + case CMDQ_OP_CFGI_CD_ALL: > > + u32 sid, vsid = FIELD_GET(CMDQ_CFGI_0_SID, cmd->cmd[0]); > > Here got build error > > drivers/iommu/arm/arm-smmu-v3/arm-smmu-v3-iommufd.c:302:3: error: a > label can only be part of a statement and a declaration is not a > statement > 302 | u32 sid, vsid = FIELD_GET(CMDQ_CFGI_0_SID, cmd->cmd[0]); > | ^~~ > > Need {} to include. > case CMDQ_OP_CFGI_CD_ALL: { > ... > } Oh, yea. Will fix. Thanks! Nicolin