From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: ARM: dts: imx6ul-var-som: fix warning for boolean property with a value
Date: Tue, 03 Mar 2026 12:58:05 +1000
Message-ID: <review-patch2-20260302190953.669325-3-hugo@hugovil.com>
In-Reply-To: <20260302190953.669325-3-hugo@hugovil.com>
References: <20260302190953.669325-1-hugo@hugovil.com> <20260302190953.669325-3-hugo@hugovil.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

Correct fix: `micrel,rmii-reference-clock-select-25-mhz` is a boolean property and should not have `= <1>`. However, this patch is missing a Fixes tag and `Cc: stable@kernel.org`, unlike patch 1 which is a similar DT warning fix. Consider adding:
```
Fixes: 9d6a67d9c7a9 ("ARM: dts: imx6ul: Add Variscite VAR-SOM-MX6UL SoM support")
Cc: stable@kernel.org
```

---
Generated by Claude Code Patch Reviewer