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

[Marek Vasut]

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.