From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: platform/x86/intel/vsec: Make driver_data info const Date: Fri, 27 Feb 2026 14:12:36 +1000 Message-ID: In-Reply-To: <20260224-upstream-pmt-acpi-v5-v5-2-8dd73bcf049c@linux.intel.com> References: <20260224-upstream-pmt-acpi-v5-v5-0-8dd73bcf049c@linux.intel.com> <20260224-upstream-pmt-acpi-v5-v5-2-8dd73bcf049c@linux.intel.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Overall: Clean const-correctness improvement.** Makes `vsec_priv->info` a `const` pointer and updates all function signatures that accept `intel_vsec_platform_info *` to `const`. The cast in `intel_vsec_pci_probe()`: ```c info = (const struct intel_vsec_platform_info *)id->driver_data; ``` is correct since `driver_data` is `unsigned long` and this is a standard PCI ID table pattern. Minor nit: the alignment of `const` in the `intel_vsec_register` declaration differs slightly between the two hunks but this is cosmetic. No issues found. --- --- Generated by Claude Code Patch Reviewer