[PATCH] board: add InnoComm i.MX8MM WB15EVK

Fabio Estevam festevam at gmail.com
Thu Apr 16 19:33:17 CEST 2020


Hi Matt,

On Thu, Apr 9, 2020 at 6:51 PM Matt Porter <mporter at konsulko.com> wrote:

Hopefully this dts will be sent upstream.

> +/ {
> +       model = "InnoComm i.MX8MM WB15EVK";
> +       compatible = "fsl,imx8mm-wb15evk", "fsl,imx8mm";

Should be "innocomm,imx8mm-wb15evk", "fsl,imx8mm" instead, since the
board manufacturer is InnoComm.

> +&gpio1 {
> +       phy_en {
> +               gpio-hog;
> +               gpios = <10 GPIO_ACTIVE_HIGH>;
> +               output-high;
> +               pinctrl-names = "default";
> +               pinctrl-0 = <&pinctrl_phy_en>;
> +       };

Shouldn't this be modelled as a phy-supply GPIO controlled regulator instead?

> +CONFIG_SPL_SYS_ICACHE_OFF=y
> +CONFIG_SPL_SYS_DCACHE_OFF=y

Can't we work with caches enabled by now?

> +/* USDHC */
> +#define CONFIG_FSL_USDHC

Better put it in the defconfig instead.

> +#define CONFIG_FEC_XCV_TYPE             RGMII
> +#define FEC_QUIRK_ENET_MAC

Shouldn't this be moved to a SoC header instead of each board file?

> +
> +#define IMX_FEC_BASE                    0x30BE0000

Not needed as you are using FEC DM.


More information about the U-Boot mailing list