Hi Dave, It looks like the review bot does not see any extra contexts than those included in the patch file?
I am trying to address the issues (if valid) raised by the bot of my patchset `[PATCH drm-misc-next 0/3] drm: verisilicon: convert drm_format to vs_format in atomic_check' . In the bot review of 3/3, I got the following review note: ``` After this patch, the format is converted and stored in atomic_check. However, we should verify the existing `drm_atomic_helper_check_plane_state()` call (if any) happens after our format validation. Looking at the patch, I don't see where plane state validation happens. ``` However, the function already returns with `drm_atomic_helper_check_plane_state()` call at the tail. It's not part of the patch (because `git diff` just truncates at the code acquiring the new CRTC state, which is prequisite for calling `drm_atomic_helper_check_plane_state()` ), but always part of the codebase. This shows that the bot cannot see the existing codebase, but only the patches themselves. This sounds like some problem. Hope this could be fixed, although it sounds like this will make the bot more token-burning. Thanks Icenowy 在 2026-02-12四的 05:44 +1000，Dave Airlie写道： > Hi all, > > This came up at kernel maintainers summit, so I've been trying to see > what I can piece together, and have a small demonstration that may be > useful to some people. > > I didn't want to pollute the mailing list with AI patch reviews, so I > decided to set up a public-inbox that the reviews are pushed into. > This isn't currently automated, I'm just asking claude to pull the > last 2-3 days of patches and review what is new every so often. > > The workflow use lei to pull mails to local PC, use review-prompts + > my own prompt to try and review a patch series, both as a complete > work, and per-patch reviews, then create the reply emails and put > them > into a public inbox git tree for publishing. > > I've no idea if it's using review-prompts properly or at all, this is > all very vibe coded so far. > > https://lore.gitlab.freedesktop.org/drm-ai-reviews/ > > This is a public inbox, you can also git clone > > https://gitlab.freedesktop.org/drm/ai-reviews-public-inbox > > I'm currently just using my Red Hat provided claude with opus 4.6, > until I get told I've burned enough money. > > The list below are the patches with reviews, if someone wants to look > and give feedback on whether the reviews for their series are useful, > find any bugs or regressions, that would be cool. > > I've bcc'd anyone who has a patch on the list. > > This is also just an experiment to see what might stick, it might > disappear at any time, and it probably needs a lot of tuning. > > Thanks, > Dave. > > [PATCH v2 0/2] drm/buddy: Documentation and internal helper cleanup > [PATCH] drm/amd/display: Remove duplicate include > [PATCH -next v9 0/3] rust: Add CList and GPU buddy allocator bindings > [PATCH V1] accel/amdxdna: Fix suspend failure after enabling turbo > mode > [PATCH V1] accel/amdxdna: Fix dead lock for suspend and resume > [PATCH v1] drm/tyr: gpu: fix GpuInfo::log model/version decoding > [PATCH v2 0/2] drm/vkms: Fix bad matrix offset component > multiplication > [PATCH 1/2] accel/amdxdna: Fix NULL pointer dereference in mailbox > channel cleanup > [PATCH] drm/msm: always recover the gpu > [PATCH drm-misc-next] drm: verisilicon: assign git tree to drm/misc > in > MAINTAINERS > [PATCH drm-misc-next 0/3] drm: verisilicon: convert drm_format to > vs_format in atomic_check > [PATCH v3 3/3] drm/panel: add LXD M9189A panel driver > [PATCH v1 0/2] ARM: tegra: document Tegra20 HDMI port > [PATCH] fbcon: Declare struct fb_info.fbcon_par as of type struct > fbcon_par > [PATCH v1] drm/amdgpu: fix sync handling in > amdgpu_dma_buf_move_notify > [PATCH v9 0/7] User readable error codes on atomic_ioctl failure > [PATCH] accel/qaic: Fix dma_free_attrs() buffer size > [PATCH] drm/radeon: Add HAINAN clock adjustment > [PATCH] drm/amdgpu: Add HAINAN clock adjustment > [PATCH v9 01/15] drm/bridge: analogix_dp: Add > &analogix_dp_plat_data.next_bridge > [PATCH v2 0/5] drm/ci: add new jobs, uprev IGT and mesa > [PATCH] drm/bridge: lt9611: Remove DRM_BRIDGE_OP_MODES flag > [PATCH 0/6] Support for the Pixel 3a XL with the Tianma panel > [PATCH -next v8 0/3] rust: Add CList and GPU buddy allocator bindings > [PATCH] drm/bridge: samsung-dsim: Fix memory leak in error path > [PATCH] drm/rockchip: vop2: Use drm_err_ratelimited() for wait > timeouts > [PATCH] fbcon: Remove struct fbcon_display.inverse > [PATCH 1/5] dma-mapping: avoid random addr value print out on error > path > [PATCH v3 2/6] drm/gem-shmem: Test for existence of page in mmap > fault handler > [PATCH] gpu: host1x: Fix passing zero to ERR_PTR in > host1x_iommu_attach() > [PATCH AUTOSEL 6.18-5.10] drm/tegra: hdmi: sor: Fix error: variable > ‘j’ set but not used > [PATCH] drm/mediatek: dsi: Store driver data before invoking > mipi_dsi_host_register > [PATCH] drm/i915/guc: fix corrupted copyright symbols in selftest > files > [PATCH v7 4/5] ARM: dts: microchip: sam9x7: Add GFX2D GPU > [PATCH] drm/panel: ilitek-ili9882t: Fine-tune HFP for tianma, > tl121bvms07-00 > [PATCH v4 1/8] drm/amdkfd: Add userptr batch allocation UAPI > structures
