On Tue, Dec 02, 2025 at 01:49:24AM +0000, Peng Fan wrote: > > Subject: Re: [PATCH v1 1/2] imx8ulp_evk: Move environment variables > > to .env file > > > > On Mon, Dec 01, 2025 at 06:56:56PM +0800, [email protected] > > wrote: > > > > > From: Alice Guo <[email protected]> > > > > > > Add board-specific environment variables to imx8ulp_evk.env for > > better > > > maintainability. > > > > > > Signed-off-by: Alice Guo <[email protected]> > > [snip] > > > +fdt_high=0xffffffffffffffff > > > > NAK. I just the other week sent out 40+ patches removing that setting > > from other platforms and you weren't even setting it before. > > How about add a check in checkpatch.pl to give a warning on this?
There's already one which this triggered and is an ERROR level as well. It also tells you not to disable initrd relocation, which is just a bad idea in many cases, rather than often fatal to the OS like fdt_high is. -- Tom
signature.asc
Description: PGP signature
