Re: [PATCH v2 2/2] arm64: zynqmp: Add ifm ZynqMP VHIP6 EvalBoard

[Marek Vasut <marex@nabladev.com>]

On 9/11/26 9:32 AM, Krzysztof Kozlowski wrote:
> On Wed, Sep 09, 2026 at 08:26:05PM +0200, Marek Vasut wrote:
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +#include <dt-bindings/interrupt-controller/irq.h>
>> +#include <dt-bindings/phy/phy.h>
>> +
>> +#include "zynqmp.dtsi"
>> +#include "zynqmp-clk-ccf.dtsi"
>> +
>> +/ {
>> +	model = "ifm ZynqMP VHIP6 Evaluation Board";
>> +	compatible = "ifm,zynqmp-vhip6-evalboard", "ifm,zynqmp-vhip6", "xlnx,zynqmp";
>> +
>> +	aliases {
>> +		ethernet0 = &gem0;
>> +		ethernet1 = &gem1;
>> +		ethernet2 = &gem2;
>> +		ethernet3 = &gem3;
>> +		gpio0 = &gpio;
>> +		i2c0 = &i2c0;
>> +		i2c1 = &i2c1;
>> +		rtc0 = &i2crtc;
>> +		serial0 = &uart0;
>> +		spi0 = &qspi;
>> +		usb0 = &usb0;
>> +		usb1 = &usb1;
>> +	};
>> +
>> +	chosen {
>> +		bootargs = "clk_ignore_unused";
> 
> You should not have such bootargs in mainline DTS. It ties DTS to
> currecnt Linux driver implementation, not mentioning that it is
> subjective choice. And once such bootargs appear, then they are tricky
> to remove because users of DTS might rely on them.
This can be dropped, thanks.