From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wm1-x342.google.com ([2a00:1450:4864:20::342]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1ihZmR-0005OJ-D6 for barebox@lists.infradead.org; Wed, 18 Dec 2019 13:54:25 +0000 Received: by mail-wm1-x342.google.com with SMTP id f129so1994524wmf.2 for ; Wed, 18 Dec 2019 05:54:22 -0800 (PST) MIME-Version: 1.0 References: <20191217111854.3164-1-l.stach@pengutronix.de> <20191217111854.3164-2-l.stach@pengutronix.de> In-Reply-To: <20191217111854.3164-2-l.stach@pengutronix.de> From: Andrey Smirnov Date: Wed, 18 Dec 2019 05:54:10 -0800 Message-ID: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 2/3] ARM: zii-imx8mq-dev: add DT fixups To: Lucas Stach Cc: Barebox List On Tue, Dec 17, 2019 at 3:19 AM Lucas Stach wrote: > > There are only two fixups we need to apply at the moment: > - The 27" RMB3 based unit has a eGalax Touchscreen instead of Synaptics. > - The 10.1" SCU/CCU unit has no DEB and thus no switch, but instead the > i210 ethernet is routed to the external connector directly. > > Signed-off-by: Lucas Stach > --- > arch/arm/boards/zii-imx8mq-dev/board.c | 138 +++++++++++++++++++++++++ > arch/arm/dts/imx8mq-zii-ultra.dtsi | 10 ++ > 2 files changed, 148 insertions(+) > > diff --git a/arch/arm/boards/zii-imx8mq-dev/board.c b/arch/arm/boards/zii-imx8mq-dev/board.c > index dcf945db495a..0be68423d9bf 100644 > --- a/arch/arm/boards/zii-imx8mq-dev/board.c > +++ b/arch/arm/boards/zii-imx8mq-dev/board.c > @@ -11,6 +11,15 @@ > #include > #include > #include > +#include "../zii-common/pn-fixup.h" > + > +#define LRU_FLAG_EGALAX BIT(0) > +#define LRU_FLAG_NO_DEB BIT(1) > + > +struct zii_imx8mq_dev_lru_fixup { > + struct zii_pn_fixup fixup; > + unsigned int flags; > +}; > > static int zii_imx8mq_dev_init(void) > { > @@ -32,3 +41,132 @@ static int zii_imx8mq_dev_init(void) > return 0; > } > device_initcall(zii_imx8mq_dev_init); > + > +static int zii_imx8mq_dev_fixup_egalax_ts(struct device_node *root, void *ctx) > +{ > + struct device_node *np; > + > + /* > + * The 27" unit has a EETI eGalax touchscreen instead of the > + * Synaptics RMI4 found on other units. > + */ > + pr_info("Enabling eGalax touchscreen instead of RMI4\n"); > + > + np = of_find_compatible_node(root, NULL, "syna,rmi4-i2c"); > + if (!np) > + return -ENODEV; > + > + of_device_disable(np); > + > + np = of_find_compatible_node(root, NULL, "eeti,exc3000"); > + if (!np) > + return -ENODEV; > + > + of_device_enable(np); > + > + return 0; > +} > + > +static int zii_imx8mq_dev_fixup_deb_internal(void) > +{ > + struct device_node *np, *aliases; > + struct device_d *dev; > + > + /* > + * In the internal DT remove the complete FEC hierarchy and move the > + * i210 to be the eth0 interface to allow network boot to work without > + * rewriting all the boot scripts. > + */ > + aliases = of_find_node_by_path("/aliases"); > + if (!aliases) > + return -ENODEV; > + > + np = of_find_compatible_node(NULL, NULL, "fsl,imx8mq-fec"); > + if (!np) > + return -ENODEV; > + > + of_device_disable(np); > + > + of_property_write_string(aliases, "ethernet1", np->full_name); > + > + dev = get_device_by_device_node(np); The patch adding this function might be missing from the series. Or did I miss it going in earlier? > + if (!dev) > + return -ENODEV; > + > + unregister_device(dev); > + > + np = of_find_node_by_name(NULL, "i210@0"); > + if (!np) > + return -ENODEV; > + > + of_property_write_string(aliases, "ethernet0", np->full_name); > + > + /* Refresh the internal aliases list from the patched DT */ > + of_alias_scan(); > + > + return 0; > +} > + > +static int zii_imx8mq_dev_fixup_deb(struct device_node *root, void *ctx) > +{ > + struct device_node *np; > + > + /* > + * In the kernel DT remove all devices from the DEB, which isn't > + * present on this system. > + */ > + np = of_find_compatible_node(root, NULL, "marvell,mv88e6085"); > + if (!np) > + return -ENODEV; > + > + of_device_disable(np); > + > + np = of_find_compatible_node(root, NULL, "zii,rave-wdt"); > + if (!np) > + return -ENODEV; > + > + of_device_disable(np); > + > + return 0; > +} > + > +static void zii_imx8mq_dev_lru_fixup(const struct zii_pn_fixup *context) > +{ > + const struct zii_imx8mq_dev_lru_fixup *fixup = > + container_of(context, struct zii_imx8mq_dev_lru_fixup, fixup); > + > + if (fixup->flags & LRU_FLAG_EGALAX) > + of_register_fixup(zii_imx8mq_dev_fixup_egalax_ts, NULL); > + > + if (fixup->flags & LRU_FLAG_NO_DEB) { > + zii_imx8mq_dev_fixup_deb_internal(); > + of_register_fixup(zii_imx8mq_dev_fixup_deb, NULL); > + } > +} > + > +#define ZII_IMX8MQ_DEV_LRU_FIXUP(__pn, __flags) \ > + { \ > + { __pn, zii_imx8mq_dev_lru_fixup }, \ > + __flags \ > + } > + > +static const struct zii_imx8mq_dev_lru_fixup zii_imx8mq_dev_lru_fixups[] = { > + ZII_IMX8MQ_DEV_LRU_FIXUP("00-5131-02", LRU_FLAG_EGALAX), > + ZII_IMX8MQ_DEV_LRU_FIXUP("00-5131-03", LRU_FLAG_EGALAX), > + ZII_IMX8MQ_DEV_LRU_FIXUP("00-5170-01", LRU_FLAG_NO_DEB), > +}; > + > +/* > + * This initcall needs to be executed before coredevices, so we have a chance > + * to fix up the devices with the correct information. > + */ > +static int zii_imx8mq_dev_process_fixups(void) > +{ > + if (!of_machine_is_compatible("zii,imx8mq-ultra")) > + return 0; > + > + zii_process_lru_fixups(zii_imx8mq_dev_lru_fixups); > + > + return 0; > +} > +postmmu_initcall(zii_imx8mq_dev_process_fixups); > diff --git a/arch/arm/dts/imx8mq-zii-ultra.dtsi b/arch/arm/dts/imx8mq-zii-ultra.dtsi > index 6180f21ab0a4..50bad9b1a27e 100644 > --- a/arch/arm/dts/imx8mq-zii-ultra.dtsi > +++ b/arch/arm/dts/imx8mq-zii-ultra.dtsi > @@ -22,6 +22,11 @@ > }; > }; > > + device-info { > + nvmem-cells = <&lru_part_number>; > + nvmem-cell-names = "lru-part-number"; > + }; > + > aliases { > ethernet0 = &fec1; > ethernet1 = &i210; > @@ -64,6 +69,11 @@ > &uart2 { > rave-sp { > eeprom@a4 { > + lru_part_number: lru-part-number@21 { > + reg = <0x21 15>; > + read-only; > + }; > + > mac_address_0: mac-address@180 { > reg = <0x180 6>; > }; > -- > 2.20.1 > > > _______________________________________________ > barebox mailing list > barebox@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/barebox _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox