From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Mon, 07 Aug 2023 09:01:23 +0200 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1qSuF2-00AiYv-5d for lore@lore.pengutronix.de; Mon, 07 Aug 2023 09:01:23 +0200 Received: from bombadil.infradead.org ([2607:7c80:54:3::133]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qSuEy-0006k1-Ie for lore@pengutronix.de; Mon, 07 Aug 2023 09:01:22 +0200 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:From:In-Reply-To: Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=3AAM1I77kl4pC30WpX3tMbVbSRrxSoFefncdv9rK1bM=; b=OkZpmX5LPYi2BtFlAMGisXw0gx tYZ6grZH3p1RORacIhsp8vHN/Kdh1rPD0jZABI7+hgn0Et2lPX/WMzdtBDKygrHmv3DKcd+GSQDu0 kjQGIKHt57o2jFae55jfZpVG1wbUVVeoXrfFYGd/3WDljvbt3fBE+lX6/ZL3vldZ+fj+NEtJ4M5w3 yqbc5z2RsIcE8zjsSCeUrm6q2hTinfrAOigGJ9rTmHRv/vZgHP7DPanBIfkudegfARnYEYpzmw2uB 5nL7B2Wbz7mCvaYG8t3rgekA21qSfMU4DwfM5ISrZjwCuBJG+k9Ivrx+yPQ3aEhMLLkBja+1F26LC 4FIzo7sQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qSuDZ-00GJGB-0f; Mon, 07 Aug 2023 06:59:53 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qSuDS-00GJFk-26 for barebox@lists.infradead.org; Mon, 07 Aug 2023 06:59:50 +0000 Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qSuDR-0006Kn-8u; Mon, 07 Aug 2023 08:59:45 +0200 Received: from sha by ptx.hi.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1qSuDR-0004af-2K; Mon, 07 Aug 2023 08:59:45 +0200 Date: Mon, 7 Aug 2023 08:59:45 +0200 To: Alexander Shiyan Cc: barebox@lists.infradead.org Message-ID: <20230807065945.GJ26314@pengutronix.de> References: <20230803103238.51706-1-eagle.alexander923@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230803103238.51706-1-eagle.alexander923@gmail.com> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-Accept-Language: de,en X-Accept-Content-Type: text/plain User-Agent: Mutt/1.10.1 (2018-07-13) From: Sascha Hauer X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230806_235947_071496_594A4F44 X-CRM114-Status: GOOD ( 37.57 ) X-BeenThere: barebox@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "barebox" X-SA-Exim-Connect-IP: 2607:7c80:54:3::133 X-SA-Exim-Mail-From: barebox-bounces+lore=pengutronix.de@lists.infradead.org X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.ext.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-4.8 required=4.0 tests=AWL,BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_NONE autolearn=unavailable autolearn_force=no version=3.4.2 Subject: Re: [PATCH 1/2] memory: Add driver for FMC2 External Bus Interface on STM32MP SoCs X-SA-Exim-Version: 4.2.1 (built Wed, 08 May 2019 21:11:16 +0000) X-SA-Exim-Scanned: Yes (on metis.ext.pengutronix.de) On Thu, Aug 03, 2023 at 01:32:37PM +0300, Alexander Shiyan wrote: > This adds support for FMC2 External Bus Interface on STM32MP SoCs. > The original source is taken from the STMicroelectronics/u-boot repository [1]. > > [1] https://github.com/STMicroelectronics/u-boot/blob/v2022.10-stm32mp/drivers/memory/stm32-fmc2-ebi.c > > Signed-off-by: Alexander Shiyan > --- > drivers/memory/Kconfig | 9 + > drivers/memory/Makefile | 1 + > drivers/memory/stm32-fmc2-ebi.c | 1063 +++++++++++++++++++++++++++++++ > 3 files changed, 1073 insertions(+) > create mode 100644 drivers/memory/stm32-fmc2-ebi.c Applied, thanks Sascha > > diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig > index e18b452009..b0d99c3a54 100644 > --- a/drivers/memory/Kconfig > +++ b/drivers/memory/Kconfig > @@ -23,4 +23,13 @@ config MC_TEGRA124 > the Tegra124 SoC. This driver performs the necessary initialization > to provide a function GPU when the OS is running. > > +config STM32_FMC2_EBI > + bool "Support for FMC2 External Bus Interface on STM32MP SoCs" > + depends on ARCH_STM32MP || COMPILE_TEST > + help > + Select this option to enable the STM32 FMC2 External Bus Interface > + controller. This driver configures the transactions with external > + devices (like SRAM, ethernet adapters, FPGAs, LCD displays, ...) on > + SOCs containing the FMC2 External Bus Interface. > + > endmenu > diff --git a/drivers/memory/Makefile b/drivers/memory/Makefile > index bdf8db66e8..67d3c47621 100644 > --- a/drivers/memory/Makefile > +++ b/drivers/memory/Makefile > @@ -1,3 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0-only > obj-$(CONFIG_ATMEL_EBI) += atmel-ebi.o > obj-$(CONFIG_MC_TEGRA124) += mc-tegra124.o > +obj-$(CONFIG_STM32_FMC2_EBI) += stm32-fmc2-ebi.o > diff --git a/drivers/memory/stm32-fmc2-ebi.c b/drivers/memory/stm32-fmc2-ebi.c > new file mode 100644 > index 0000000000..ac2ea1b0e0 > --- /dev/null > +++ b/drivers/memory/stm32-fmc2-ebi.c > @@ -0,0 +1,1063 @@ > +// SPDX-License-Identifier: GPL-2.0+ OR BSD-3-Clause > +/* > + * Copyright (C) STMicroelectronics 2020 > + */ > + > +#define pr_fmt(fmt) "stm32-fmc2-ebi: " fmt > + > +#include > +#include > +#include > +#include > +#include > +#include > + > +/* FMC2 Controller Registers */ > +#define FMC2_BCR1 0x0 > +#define FMC2_BTR1 0x4 > +#define FMC2_BCR(x) ((x) * 0x8 + FMC2_BCR1) > +#define FMC2_BTR(x) ((x) * 0x8 + FMC2_BTR1) > +#define FMC2_PCSCNTR 0x20 > +#define FMC2_BWTR1 0x104 > +#define FMC2_BWTR(x) ((x) * 0x8 + FMC2_BWTR1) > + > +/* Register: FMC2_BCR1 */ > +#define FMC2_BCR1_CCLKEN BIT(20) > +#define FMC2_BCR1_FMC2EN BIT(31) > + > +/* Register: FMC2_BCRx */ > +#define FMC2_BCR_MBKEN BIT(0) > +#define FMC2_BCR_MUXEN BIT(1) > +#define FMC2_BCR_MTYP GENMASK(3, 2) > +#define FMC2_BCR_MWID GENMASK(5, 4) > +#define FMC2_BCR_FACCEN BIT(6) > +#define FMC2_BCR_BURSTEN BIT(8) > +#define FMC2_BCR_WAITPOL BIT(9) > +#define FMC2_BCR_WAITCFG BIT(11) > +#define FMC2_BCR_WREN BIT(12) > +#define FMC2_BCR_WAITEN BIT(13) > +#define FMC2_BCR_EXTMOD BIT(14) > +#define FMC2_BCR_ASYNCWAIT BIT(15) > +#define FMC2_BCR_CPSIZE GENMASK(18, 16) > +#define FMC2_BCR_CBURSTRW BIT(19) > +#define FMC2_BCR_NBLSET GENMASK(23, 22) > + > +/* Register: FMC2_BTRx/FMC2_BWTRx */ > +#define FMC2_BXTR_ADDSET GENMASK(3, 0) > +#define FMC2_BXTR_ADDHLD GENMASK(7, 4) > +#define FMC2_BXTR_DATAST GENMASK(15, 8) > +#define FMC2_BXTR_BUSTURN GENMASK(19, 16) > +#define FMC2_BTR_CLKDIV GENMASK(23, 20) > +#define FMC2_BTR_DATLAT GENMASK(27, 24) > +#define FMC2_BXTR_ACCMOD GENMASK(29, 28) > +#define FMC2_BXTR_DATAHLD GENMASK(31, 30) > + > +/* Register: FMC2_PCSCNTR */ > +#define FMC2_PCSCNTR_CSCOUNT GENMASK(15, 0) > +#define FMC2_PCSCNTR_CNTBEN(x) BIT((x) + 16) > + > +#define FMC2_MAX_EBI_CE 4 > +#define FMC2_MAX_BANKS 5 > + > +#define FMC2_BCR_CPSIZE_0 0x0 > +#define FMC2_BCR_CPSIZE_128 0x1 > +#define FMC2_BCR_CPSIZE_256 0x2 > +#define FMC2_BCR_CPSIZE_512 0x3 > +#define FMC2_BCR_CPSIZE_1024 0x4 > + > +#define FMC2_BCR_MWID_8 0x0 > +#define FMC2_BCR_MWID_16 0x1 > + > +#define FMC2_BCR_MTYP_SRAM 0x0 > +#define FMC2_BCR_MTYP_PSRAM 0x1 > +#define FMC2_BCR_MTYP_NOR 0x2 > + > +#define FMC2_BXTR_EXTMOD_A 0x0 > +#define FMC2_BXTR_EXTMOD_B 0x1 > +#define FMC2_BXTR_EXTMOD_C 0x2 > +#define FMC2_BXTR_EXTMOD_D 0x3 > + > +#define FMC2_BCR_NBLSET_MAX 0x3 > +#define FMC2_BXTR_ADDSET_MAX 0xf > +#define FMC2_BXTR_ADDHLD_MAX 0xf > +#define FMC2_BXTR_DATAST_MAX 0xff > +#define FMC2_BXTR_BUSTURN_MAX 0xf > +#define FMC2_BXTR_DATAHLD_MAX 0x3 > +#define FMC2_BTR_CLKDIV_MAX 0xf > +#define FMC2_BTR_DATLAT_MAX 0xf > +#define FMC2_PCSCNTR_CSCOUNT_MAX 0xff > + > +enum stm32_fmc2_ebi_bank { > + FMC2_EBI1 = 0, > + FMC2_EBI2, > + FMC2_EBI3, > + FMC2_EBI4, > + FMC2_NAND > +}; > + > +enum stm32_fmc2_ebi_register_type { > + FMC2_REG_BCR = 1, > + FMC2_REG_BTR, > + FMC2_REG_BWTR, > + FMC2_REG_PCSCNTR > +}; > + > +enum stm32_fmc2_ebi_transaction_type { > + FMC2_ASYNC_MODE_1_SRAM = 0, > + FMC2_ASYNC_MODE_1_PSRAM, > + FMC2_ASYNC_MODE_A_SRAM, > + FMC2_ASYNC_MODE_A_PSRAM, > + FMC2_ASYNC_MODE_2_NOR, > + FMC2_ASYNC_MODE_B_NOR, > + FMC2_ASYNC_MODE_C_NOR, > + FMC2_ASYNC_MODE_D_NOR, > + FMC2_SYNC_READ_SYNC_WRITE_PSRAM, > + FMC2_SYNC_READ_ASYNC_WRITE_PSRAM, > + FMC2_SYNC_READ_SYNC_WRITE_NOR, > + FMC2_SYNC_READ_ASYNC_WRITE_NOR > +}; > + > +enum stm32_fmc2_ebi_buswidth { > + FMC2_BUSWIDTH_8 = 8, > + FMC2_BUSWIDTH_16 = 16 > +}; > + > +enum stm32_fmc2_ebi_cpsize { > + FMC2_CPSIZE_0 = 0, > + FMC2_CPSIZE_128 = 128, > + FMC2_CPSIZE_256 = 256, > + FMC2_CPSIZE_512 = 512, > + FMC2_CPSIZE_1024 = 1024 > +}; > + > +struct stm32_fmc2_ebi { > + struct clk *clk; > + void __iomem *io_base; > + u8 bank_assigned; > +}; > + > +/* > + * struct stm32_fmc2_prop - STM32 FMC2 EBI property > + * @name: the device tree binding name of the property > + * @bprop: indicate that it is a boolean property > + * @mprop: indicate that it is a mandatory property > + * @reg_type: the register that have to be modified > + * @reg_mask: the bit that have to be modified in the selected register > + * in case of it is a boolean property > + * @reset_val: the default value that have to be set in case the property > + * has not been defined in the device tree > + * @check: this callback ckecks that the property is compliant with the > + * transaction type selected > + * @calculate: this callback is called to calculate for exemple a timing > + * set in nanoseconds in the device tree in clock cycles or in > + * clock period > + * @set: this callback applies the values in the registers > + */ > +struct stm32_fmc2_prop { > + const char *name; > + bool bprop; > + bool mprop; > + int reg_type; > + u32 reg_mask; > + u32 reset_val; > + int (*check)(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, int cs); > + u32 (*calculate)(struct stm32_fmc2_ebi *ebi, int cs, u32 setup); > + int (*set)(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup); > +}; > + > +static int stm32_fmc2_ebi_check_mux(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs) > +{ > + u32 bcr = readl(ebi->io_base + FMC2_BCR(cs)); > + > + if (bcr & FMC2_BCR_MTYP) > + return 0; > + > + return -EINVAL; > +} > + > +static int stm32_fmc2_ebi_check_waitcfg(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs) > +{ > + u32 bcr = readl(ebi->io_base + FMC2_BCR(cs)); > + u32 val = FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_NOR); > + > + if ((bcr & FMC2_BCR_MTYP) == val && bcr & FMC2_BCR_BURSTEN) > + return 0; > + > + return -EINVAL; > +} > + > +static int stm32_fmc2_ebi_check_sync_trans(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs) > +{ > + u32 bcr = readl(ebi->io_base + FMC2_BCR(cs)); > + > + if (bcr & FMC2_BCR_BURSTEN) > + return 0; > + > + return -EINVAL; > +} > + > +static int stm32_fmc2_ebi_check_async_trans(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs) > +{ > + u32 bcr = readl(ebi->io_base + FMC2_BCR(cs)); > + > + if (!(bcr & FMC2_BCR_BURSTEN) || !(bcr & FMC2_BCR_CBURSTRW)) > + return 0; > + > + return -EINVAL; > +} > + > +static int stm32_fmc2_ebi_check_cpsize(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs) > +{ > + u32 bcr = readl(ebi->io_base + FMC2_BCR(cs)); > + u32 val = FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_PSRAM); > + > + if ((bcr & FMC2_BCR_MTYP) == val && bcr & FMC2_BCR_BURSTEN) > + return 0; > + > + return -EINVAL; > +} > + > +static int stm32_fmc2_ebi_check_address_hold(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs) > +{ > + u32 bcr = readl(ebi->io_base + FMC2_BCR(cs)); > + u32 bxtr = prop->reg_type == FMC2_REG_BWTR ? > + readl(ebi->io_base + FMC2_BWTR(cs)) : > + readl(ebi->io_base + FMC2_BTR(cs)); > + u32 val = FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_D); > + > + if ((!(bcr & FMC2_BCR_BURSTEN) || !(bcr & FMC2_BCR_CBURSTRW)) && > + ((bxtr & FMC2_BXTR_ACCMOD) == val || bcr & FMC2_BCR_MUXEN)) > + return 0; > + > + return -EINVAL; > +} > + > +static int stm32_fmc2_ebi_check_clk_period(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs) > +{ > + u32 bcr = readl(ebi->io_base + FMC2_BCR(cs)); > + u32 bcr1 = cs ? readl(ebi->io_base + FMC2_BCR1) : bcr; > + > + if (bcr & FMC2_BCR_BURSTEN && (!cs || !(bcr1 & FMC2_BCR1_CCLKEN))) > + return 0; > + > + return -EINVAL; > +} > + > +static int stm32_fmc2_ebi_check_cclk(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs) > +{ > + if (cs) > + return -EINVAL; > + > + return stm32_fmc2_ebi_check_sync_trans(ebi, prop, cs); > +} > + > +static u32 stm32_fmc2_ebi_ns_to_clock_cycles(struct stm32_fmc2_ebi *ebi, > + int cs, u32 setup) > +{ > + unsigned long hclk = clk_get_rate(ebi->clk); > + unsigned long hclkp = NSEC_PER_SEC / (hclk / 1000); > + > + return DIV_ROUND_UP(setup * 1000, hclkp); > +} > + > +static u32 stm32_fmc2_ebi_ns_to_clk_period(struct stm32_fmc2_ebi *ebi, > + int cs, u32 setup) > +{ > + u32 nb_clk_cycles = stm32_fmc2_ebi_ns_to_clock_cycles(ebi, cs, setup); > + u32 bcr = readl(ebi->io_base + FMC2_BCR1); > + u32 btr = bcr & FMC2_BCR1_CCLKEN || !cs ? > + readl(ebi->io_base + FMC2_BTR1) : > + readl(ebi->io_base + FMC2_BTR(cs)); > + u32 clk_period = FIELD_GET(FMC2_BTR_CLKDIV, btr) + 1; > + > + return DIV_ROUND_UP(nb_clk_cycles, clk_period); > +} > + > +static int stm32_fmc2_ebi_get_reg(int reg_type, int cs, u32 *reg) > +{ > + switch (reg_type) { > + case FMC2_REG_BCR: > + *reg = FMC2_BCR(cs); > + break; > + case FMC2_REG_BTR: > + *reg = FMC2_BTR(cs); > + break; > + case FMC2_REG_BWTR: > + *reg = FMC2_BWTR(cs); > + break; > + case FMC2_REG_PCSCNTR: > + *reg = FMC2_PCSCNTR; > + break; > + default: > + return -EINVAL; > + } > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_bit_field(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 reg; > + int ret; > + > + ret = stm32_fmc2_ebi_get_reg(prop->reg_type, cs, ®); > + if (ret) > + return ret; > + > + clrsetbits_le32(ebi->io_base + reg, prop->reg_mask, > + setup ? prop->reg_mask : 0); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_trans_type(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 bcr_mask, bcr = FMC2_BCR_WREN; > + u32 btr_mask, btr = 0; > + u32 bwtr_mask, bwtr = 0; > + > + bwtr_mask = FMC2_BXTR_ACCMOD; > + btr_mask = FMC2_BXTR_ACCMOD; > + bcr_mask = FMC2_BCR_MUXEN | FMC2_BCR_MTYP | FMC2_BCR_FACCEN | > + FMC2_BCR_WREN | FMC2_BCR_WAITEN | FMC2_BCR_BURSTEN | > + FMC2_BCR_EXTMOD | FMC2_BCR_CBURSTRW; > + > + switch (setup) { > + case FMC2_ASYNC_MODE_1_SRAM: > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_SRAM); > + /* > + * MUXEN = 0, MTYP = 0, FACCEN = 0, BURSTEN = 0, WAITEN = 0, > + * WREN = 1, EXTMOD = 0, CBURSTRW = 0, ACCMOD = 0 > + */ > + break; > + case FMC2_ASYNC_MODE_1_PSRAM: > + /* > + * MUXEN = 0, MTYP = 1, FACCEN = 0, BURSTEN = 0, WAITEN = 0, > + * WREN = 1, EXTMOD = 0, CBURSTRW = 0, ACCMOD = 0 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_PSRAM); > + break; > + case FMC2_ASYNC_MODE_A_SRAM: > + /* > + * MUXEN = 0, MTYP = 0, FACCEN = 0, BURSTEN = 0, WAITEN = 0, > + * WREN = 1, EXTMOD = 1, CBURSTRW = 0, ACCMOD = 0 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_SRAM); > + bcr |= FMC2_BCR_EXTMOD; > + btr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_A); > + bwtr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_A); > + break; > + case FMC2_ASYNC_MODE_A_PSRAM: > + /* > + * MUXEN = 0, MTYP = 1, FACCEN = 0, BURSTEN = 0, WAITEN = 0, > + * WREN = 1, EXTMOD = 1, CBURSTRW = 0, ACCMOD = 0 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_PSRAM); > + bcr |= FMC2_BCR_EXTMOD; > + btr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_A); > + bwtr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_A); > + break; > + case FMC2_ASYNC_MODE_2_NOR: > + /* > + * MUXEN = 0, MTYP = 2, FACCEN = 1, BURSTEN = 0, WAITEN = 0, > + * WREN = 1, EXTMOD = 0, CBURSTRW = 0, ACCMOD = 0 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_NOR); > + bcr |= FMC2_BCR_FACCEN; > + break; > + case FMC2_ASYNC_MODE_B_NOR: > + /* > + * MUXEN = 0, MTYP = 2, FACCEN = 1, BURSTEN = 0, WAITEN = 0, > + * WREN = 1, EXTMOD = 1, CBURSTRW = 0, ACCMOD = 1 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_NOR); > + bcr |= FMC2_BCR_FACCEN | FMC2_BCR_EXTMOD; > + btr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_B); > + bwtr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_B); > + break; > + case FMC2_ASYNC_MODE_C_NOR: > + /* > + * MUXEN = 0, MTYP = 2, FACCEN = 1, BURSTEN = 0, WAITEN = 0, > + * WREN = 1, EXTMOD = 1, CBURSTRW = 0, ACCMOD = 2 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_NOR); > + bcr |= FMC2_BCR_FACCEN | FMC2_BCR_EXTMOD; > + btr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_C); > + bwtr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_C); > + break; > + case FMC2_ASYNC_MODE_D_NOR: > + /* > + * MUXEN = 0, MTYP = 2, FACCEN = 1, BURSTEN = 0, WAITEN = 0, > + * WREN = 1, EXTMOD = 1, CBURSTRW = 0, ACCMOD = 3 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_NOR); > + bcr |= FMC2_BCR_FACCEN | FMC2_BCR_EXTMOD; > + btr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_D); > + bwtr |= FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_D); > + break; > + case FMC2_SYNC_READ_SYNC_WRITE_PSRAM: > + /* > + * MUXEN = 0, MTYP = 1, FACCEN = 0, BURSTEN = 1, WAITEN = 0, > + * WREN = 1, EXTMOD = 0, CBURSTRW = 1, ACCMOD = 0 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_PSRAM); > + bcr |= FMC2_BCR_BURSTEN | FMC2_BCR_CBURSTRW; > + break; > + case FMC2_SYNC_READ_ASYNC_WRITE_PSRAM: > + /* > + * MUXEN = 0, MTYP = 1, FACCEN = 0, BURSTEN = 1, WAITEN = 0, > + * WREN = 1, EXTMOD = 0, CBURSTRW = 0, ACCMOD = 0 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_PSRAM); > + bcr |= FMC2_BCR_BURSTEN; > + break; > + case FMC2_SYNC_READ_SYNC_WRITE_NOR: > + /* > + * MUXEN = 0, MTYP = 2, FACCEN = 1, BURSTEN = 1, WAITEN = 0, > + * WREN = 1, EXTMOD = 0, CBURSTRW = 1, ACCMOD = 0 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_NOR); > + bcr |= FMC2_BCR_FACCEN | FMC2_BCR_BURSTEN | FMC2_BCR_CBURSTRW; > + break; > + case FMC2_SYNC_READ_ASYNC_WRITE_NOR: > + /* > + * MUXEN = 0, MTYP = 2, FACCEN = 1, BURSTEN = 1, WAITEN = 0, > + * WREN = 1, EXTMOD = 0, CBURSTRW = 0, ACCMOD = 0 > + */ > + bcr |= FIELD_PREP(FMC2_BCR_MTYP, FMC2_BCR_MTYP_NOR); > + bcr |= FMC2_BCR_FACCEN | FMC2_BCR_BURSTEN; > + break; > + default: > + /* Type of transaction not supported */ > + return -EINVAL; > + } > + > + if (bcr & FMC2_BCR_EXTMOD) > + clrsetbits_le32(ebi->io_base + FMC2_BWTR(cs), > + bwtr_mask, bwtr); > + clrsetbits_le32(ebi->io_base + FMC2_BTR(cs), btr_mask, btr); > + clrsetbits_le32(ebi->io_base + FMC2_BCR(cs), bcr_mask, bcr); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_buswidth(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 val; > + > + switch (setup) { > + case FMC2_BUSWIDTH_8: > + val = FIELD_PREP(FMC2_BCR_MWID, FMC2_BCR_MWID_8); > + break; > + case FMC2_BUSWIDTH_16: > + val = FIELD_PREP(FMC2_BCR_MWID, FMC2_BCR_MWID_16); > + break; > + default: > + /* Buswidth not supported */ > + return -EINVAL; > + } > + > + clrsetbits_le32(ebi->io_base + FMC2_BCR(cs), FMC2_BCR_MWID, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_cpsize(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 val; > + > + switch (setup) { > + case FMC2_CPSIZE_0: > + val = FIELD_PREP(FMC2_BCR_CPSIZE, FMC2_BCR_CPSIZE_0); > + break; > + case FMC2_CPSIZE_128: > + val = FIELD_PREP(FMC2_BCR_CPSIZE, FMC2_BCR_CPSIZE_128); > + break; > + case FMC2_CPSIZE_256: > + val = FIELD_PREP(FMC2_BCR_CPSIZE, FMC2_BCR_CPSIZE_256); > + break; > + case FMC2_CPSIZE_512: > + val = FIELD_PREP(FMC2_BCR_CPSIZE, FMC2_BCR_CPSIZE_512); > + break; > + case FMC2_CPSIZE_1024: > + val = FIELD_PREP(FMC2_BCR_CPSIZE, FMC2_BCR_CPSIZE_1024); > + break; > + default: > + /* Cpsize not supported */ > + return -EINVAL; > + } > + > + clrsetbits_le32(ebi->io_base + FMC2_BCR(cs), FMC2_BCR_CPSIZE, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_bl_setup(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 val; > + > + val = min_t(u32, setup, FMC2_BCR_NBLSET_MAX); > + val = FIELD_PREP(FMC2_BCR_NBLSET, val); > + clrsetbits_le32(ebi->io_base + FMC2_BCR(cs), FMC2_BCR_NBLSET, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_address_setup(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 bcr = readl(ebi->io_base + FMC2_BCR(cs)); > + u32 bxtr = prop->reg_type == FMC2_REG_BWTR ? > + readl(ebi->io_base + FMC2_BWTR(cs)) : > + readl(ebi->io_base + FMC2_BTR(cs)); > + u32 reg, val = FIELD_PREP(FMC2_BXTR_ACCMOD, FMC2_BXTR_EXTMOD_D); > + int ret; > + > + ret = stm32_fmc2_ebi_get_reg(prop->reg_type, cs, ®); > + if (ret) > + return ret; > + > + if ((bxtr & FMC2_BXTR_ACCMOD) == val || bcr & FMC2_BCR_MUXEN) > + val = clamp_val(setup, 1, FMC2_BXTR_ADDSET_MAX); > + else > + val = min_t(u32, setup, FMC2_BXTR_ADDSET_MAX); > + val = FIELD_PREP(FMC2_BXTR_ADDSET, val); > + clrsetbits_le32(ebi->io_base + reg, FMC2_BXTR_ADDSET, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_address_hold(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 val, reg; > + int ret; > + > + ret = stm32_fmc2_ebi_get_reg(prop->reg_type, cs, ®); > + if (ret) > + return ret; > + > + val = clamp_val(setup, 1, FMC2_BXTR_ADDHLD_MAX); > + val = FIELD_PREP(FMC2_BXTR_ADDHLD, val); > + clrsetbits_le32(ebi->io_base + reg, FMC2_BXTR_ADDHLD, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_data_setup(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 val, reg; > + int ret; > + > + ret = stm32_fmc2_ebi_get_reg(prop->reg_type, cs, ®); > + if (ret) > + return ret; > + > + val = clamp_val(setup, 1, FMC2_BXTR_DATAST_MAX); > + val = FIELD_PREP(FMC2_BXTR_DATAST, val); > + clrsetbits_le32(ebi->io_base + reg, FMC2_BXTR_DATAST, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_bus_turnaround(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 val, reg; > + int ret; > + > + ret = stm32_fmc2_ebi_get_reg(prop->reg_type, cs, ®); > + if (ret) > + return ret; > + > + val = setup ? min_t(u32, setup - 1, FMC2_BXTR_BUSTURN_MAX) : 0; > + val = FIELD_PREP(FMC2_BXTR_BUSTURN, val); > + clrsetbits_le32(ebi->io_base + reg, FMC2_BXTR_BUSTURN, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_data_hold(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 val, reg; > + int ret; > + > + ret = stm32_fmc2_ebi_get_reg(prop->reg_type, cs, ®); > + if (ret) > + return ret; > + > + if (prop->reg_type == FMC2_REG_BWTR) > + val = setup ? min_t(u32, setup - 1, FMC2_BXTR_DATAHLD_MAX) : 0; > + else > + val = min_t(u32, setup, FMC2_BXTR_DATAHLD_MAX); > + val = FIELD_PREP(FMC2_BXTR_DATAHLD, val); > + clrsetbits_le32(ebi->io_base + reg, FMC2_BXTR_DATAHLD, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_clk_period(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 val; > + > + val = setup ? clamp_val(setup - 1, 1, FMC2_BTR_CLKDIV_MAX) : 1; > + val = FIELD_PREP(FMC2_BTR_CLKDIV, val); > + clrsetbits_le32(ebi->io_base + FMC2_BTR(cs), FMC2_BTR_CLKDIV, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_data_latency(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 val; > + > + val = setup > 1 ? min_t(u32, setup - 2, FMC2_BTR_DATLAT_MAX) : 0; > + val = FIELD_PREP(FMC2_BTR_DATLAT, val); > + clrsetbits_le32(ebi->io_base + FMC2_BTR(cs), FMC2_BTR_DATLAT, val); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_set_max_low_pulse(struct stm32_fmc2_ebi *ebi, > + const struct stm32_fmc2_prop *prop, > + int cs, u32 setup) > +{ > + u32 old_val, new_val, pcscntr; > + > + if (setup < 1) > + return 0; > + > + pcscntr = readl(ebi->io_base + FMC2_PCSCNTR); > + > + /* Enable counter for the bank */ > + setbits_le32(ebi->io_base + FMC2_PCSCNTR, FMC2_PCSCNTR_CNTBEN(cs)); > + > + new_val = min_t(u32, setup - 1, FMC2_PCSCNTR_CSCOUNT_MAX); > + old_val = FIELD_GET(FMC2_PCSCNTR_CSCOUNT, pcscntr); > + if (old_val && new_val > old_val) > + /* Keep current counter value */ > + return 0; > + > + new_val = FIELD_PREP(FMC2_PCSCNTR_CSCOUNT, new_val); > + clrsetbits_le32(ebi->io_base + FMC2_PCSCNTR, > + FMC2_PCSCNTR_CSCOUNT, new_val); > + > + return 0; > +} > + > +static const struct stm32_fmc2_prop stm32_fmc2_child_props[] = { > + /* st,fmc2-ebi-cs-trans-type must be the first property */ > + { > + .name = "st,fmc2-ebi-cs-transaction-type", > + .mprop = true, > + .set = stm32_fmc2_ebi_set_trans_type, > + }, > + { > + .name = "st,fmc2-ebi-cs-cclk-enable", > + .bprop = true, > + .reg_type = FMC2_REG_BCR, > + .reg_mask = FMC2_BCR1_CCLKEN, > + .check = stm32_fmc2_ebi_check_cclk, > + .set = stm32_fmc2_ebi_set_bit_field, > + }, > + { > + .name = "st,fmc2-ebi-cs-mux-enable", > + .bprop = true, > + .reg_type = FMC2_REG_BCR, > + .reg_mask = FMC2_BCR_MUXEN, > + .check = stm32_fmc2_ebi_check_mux, > + .set = stm32_fmc2_ebi_set_bit_field, > + }, > + { > + .name = "st,fmc2-ebi-cs-buswidth", > + .reset_val = FMC2_BUSWIDTH_16, > + .set = stm32_fmc2_ebi_set_buswidth, > + }, > + { > + .name = "st,fmc2-ebi-cs-waitpol-high", > + .bprop = true, > + .reg_type = FMC2_REG_BCR, > + .reg_mask = FMC2_BCR_WAITPOL, > + .set = stm32_fmc2_ebi_set_bit_field, > + }, > + { > + .name = "st,fmc2-ebi-cs-waitcfg-enable", > + .bprop = true, > + .reg_type = FMC2_REG_BCR, > + .reg_mask = FMC2_BCR_WAITCFG, > + .check = stm32_fmc2_ebi_check_waitcfg, > + .set = stm32_fmc2_ebi_set_bit_field, > + }, > + { > + .name = "st,fmc2-ebi-cs-wait-enable", > + .bprop = true, > + .reg_type = FMC2_REG_BCR, > + .reg_mask = FMC2_BCR_WAITEN, > + .check = stm32_fmc2_ebi_check_sync_trans, > + .set = stm32_fmc2_ebi_set_bit_field, > + }, > + { > + .name = "st,fmc2-ebi-cs-asyncwait-enable", > + .bprop = true, > + .reg_type = FMC2_REG_BCR, > + .reg_mask = FMC2_BCR_ASYNCWAIT, > + .check = stm32_fmc2_ebi_check_async_trans, > + .set = stm32_fmc2_ebi_set_bit_field, > + }, > + { > + .name = "st,fmc2-ebi-cs-cpsize", > + .check = stm32_fmc2_ebi_check_cpsize, > + .set = stm32_fmc2_ebi_set_cpsize, > + }, > + { > + .name = "st,fmc2-ebi-cs-byte-lane-setup-ns", > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_bl_setup, > + }, > + { > + .name = "st,fmc2-ebi-cs-address-setup-ns", > + .reg_type = FMC2_REG_BTR, > + .reset_val = FMC2_BXTR_ADDSET_MAX, > + .check = stm32_fmc2_ebi_check_async_trans, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_address_setup, > + }, > + { > + .name = "st,fmc2-ebi-cs-address-hold-ns", > + .reg_type = FMC2_REG_BTR, > + .reset_val = FMC2_BXTR_ADDHLD_MAX, > + .check = stm32_fmc2_ebi_check_address_hold, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_address_hold, > + }, > + { > + .name = "st,fmc2-ebi-cs-data-setup-ns", > + .reg_type = FMC2_REG_BTR, > + .reset_val = FMC2_BXTR_DATAST_MAX, > + .check = stm32_fmc2_ebi_check_async_trans, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_data_setup, > + }, > + { > + .name = "st,fmc2-ebi-cs-bus-turnaround-ns", > + .reg_type = FMC2_REG_BTR, > + .reset_val = FMC2_BXTR_BUSTURN_MAX + 1, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_bus_turnaround, > + }, > + { > + .name = "st,fmc2-ebi-cs-data-hold-ns", > + .reg_type = FMC2_REG_BTR, > + .check = stm32_fmc2_ebi_check_async_trans, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_data_hold, > + }, > + { > + .name = "st,fmc2-ebi-cs-clk-period-ns", > + .reset_val = FMC2_BTR_CLKDIV_MAX + 1, > + .check = stm32_fmc2_ebi_check_clk_period, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_clk_period, > + }, > + { > + .name = "st,fmc2-ebi-cs-data-latency-ns", > + .check = stm32_fmc2_ebi_check_sync_trans, > + .calculate = stm32_fmc2_ebi_ns_to_clk_period, > + .set = stm32_fmc2_ebi_set_data_latency, > + }, > + { > + .name = "st,fmc2-ebi-cs-write-address-setup-ns", > + .reg_type = FMC2_REG_BWTR, > + .reset_val = FMC2_BXTR_ADDSET_MAX, > + .check = stm32_fmc2_ebi_check_async_trans, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_address_setup, > + }, > + { > + .name = "st,fmc2-ebi-cs-write-address-hold-ns", > + .reg_type = FMC2_REG_BWTR, > + .reset_val = FMC2_BXTR_ADDHLD_MAX, > + .check = stm32_fmc2_ebi_check_address_hold, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_address_hold, > + }, > + { > + .name = "st,fmc2-ebi-cs-write-data-setup-ns", > + .reg_type = FMC2_REG_BWTR, > + .reset_val = FMC2_BXTR_DATAST_MAX, > + .check = stm32_fmc2_ebi_check_async_trans, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_data_setup, > + }, > + { > + .name = "st,fmc2-ebi-cs-write-bus-turnaround-ns", > + .reg_type = FMC2_REG_BWTR, > + .reset_val = FMC2_BXTR_BUSTURN_MAX + 1, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_bus_turnaround, > + }, > + { > + .name = "st,fmc2-ebi-cs-write-data-hold-ns", > + .reg_type = FMC2_REG_BWTR, > + .check = stm32_fmc2_ebi_check_async_trans, > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_data_hold, > + }, > + { > + .name = "st,fmc2-ebi-cs-max-low-pulse-ns", > + .calculate = stm32_fmc2_ebi_ns_to_clock_cycles, > + .set = stm32_fmc2_ebi_set_max_low_pulse, > + }, > +}; > + > +static int stm32_fmc2_ebi_parse_prop(struct stm32_fmc2_ebi *ebi, > + struct device_node *node, > + const struct stm32_fmc2_prop *prop, > + int cs) > +{ > + u32 setup = 0; > + > + if (!prop->set) { > + pr_err("property %s is not well defined\n", prop->name); > + return -EINVAL; > + } > + > + if (prop->check && prop->check(ebi, prop, cs)) > + /* Skip this property */ > + return 0; > + > + if (prop->bprop) { > + bool bprop; > + > + bprop = of_property_read_bool(node, prop->name); > + if (prop->mprop && !bprop) { > + pr_err("mandatory property %s not defined in the device tree\n", > + prop->name); > + return -EINVAL; > + } > + > + if (bprop) > + setup = 1; > + } else { > + u32 val; > + int ret; > + > + ret = of_property_read_u32(node, prop->name, &val); > + if (prop->mprop && ret) { > + pr_err("mandatory property %s not defined in the device tree\n", > + prop->name); > + return ret; > + } > + > + if (ret) > + setup = prop->reset_val; > + else if (prop->calculate) > + setup = prop->calculate(ebi, cs, val); > + else > + setup = val; > + } > + > + return prop->set(ebi, prop, cs, setup); > +} > + > +static void stm32_fmc2_ebi_enable_bank(struct stm32_fmc2_ebi *ebi, int cs) > +{ > + setbits_le32(ebi->io_base + FMC2_BCR(cs), FMC2_BCR_MBKEN); > +} > + > +static void stm32_fmc2_ebi_disable_bank(struct stm32_fmc2_ebi *ebi, int cs) > +{ > + clrbits_le32(ebi->io_base + FMC2_BCR(cs), FMC2_BCR_MBKEN); > +} > + > +/* NWAIT signal can not be connected to EBI controller and NAND controller */ > +static bool stm32_fmc2_ebi_nwait_used_by_ctrls(struct stm32_fmc2_ebi *ebi) > +{ > + unsigned int cs; > + u32 bcr; > + > + for (cs = 0; cs < FMC2_MAX_EBI_CE; cs++) { > + if (!(ebi->bank_assigned & BIT(cs))) > + continue; > + > + bcr = readl(ebi->io_base + FMC2_BCR(cs)); > + if ((bcr & FMC2_BCR_WAITEN || bcr & FMC2_BCR_ASYNCWAIT) && > + ebi->bank_assigned & BIT(FMC2_NAND)) > + return true; > + } > + > + return false; > +} > + > +static void stm32_fmc2_ebi_enable(struct stm32_fmc2_ebi *ebi) > +{ > + setbits_le32(ebi->io_base + FMC2_BCR1, FMC2_BCR1_FMC2EN); > +} > + > +static int stm32_fmc2_ebi_setup_cs(struct stm32_fmc2_ebi *ebi, > + struct device_node *node, u32 cs) > +{ > + unsigned int i; > + int ret; > + > + stm32_fmc2_ebi_disable_bank(ebi, cs); > + > + for (i = 0; i < ARRAY_SIZE(stm32_fmc2_child_props); i++) { > + const struct stm32_fmc2_prop *p = &stm32_fmc2_child_props[i]; > + > + ret = stm32_fmc2_ebi_parse_prop(ebi, node, p, cs); > + if (ret) { > + pr_err("property %s could not be set: %d\n", p->name, ret); > + return ret; > + } > + } > + > + stm32_fmc2_ebi_enable_bank(ebi, cs); > + > + return 0; > +} > + > +static int stm32_fmc2_ebi_parse_dt(struct device *dev, > + struct stm32_fmc2_ebi *ebi) > +{ > + struct device_node *child; > + bool child_found = false; > + u32 bank; > + int ret; > + > + for_each_available_child_of_node(dev->of_node, child) { > + ret = of_property_read_u32(child, "reg", &bank); > + if (ret) { > + dev_err(dev, "could not retrieve reg property: %d\n", ret); > + return ret; > + } > + > + if (bank >= FMC2_MAX_BANKS) { > + dev_err(dev, "invalid reg value: %d\n", bank); > + return -EINVAL; > + } > + > + if (ebi->bank_assigned & BIT(bank)) { > + dev_err(dev, "bank already assigned: %d\n", bank); > + return -EINVAL; > + } > + > + if (bank < FMC2_MAX_EBI_CE) { > + ret = stm32_fmc2_ebi_setup_cs(ebi, child, bank); > + if (ret) { > + dev_err(dev, "setup chip select %d failed: %d\n", bank, ret); > + return ret; > + } > + } > + > + ebi->bank_assigned |= BIT(bank); > + child_found = true; > + } > + > + if (!child_found) { > + dev_warn(dev, "no subnodes found.\n"); > + return -ENODEV; > + } > + > + if (stm32_fmc2_ebi_nwait_used_by_ctrls(ebi)) { > + dev_err(dev, "NWAIT signal connected to EBI and NAND controllers\n"); > + return -EINVAL; > + } > + > + stm32_fmc2_ebi_enable(ebi); > + > + return 0; > +} > + > +static int __init stm32_fmc2_ebi_probe(struct device *dev) > +{ > + struct stm32_fmc2_ebi *ebi; > + int ret; > + > + ebi = xzalloc(sizeof(*ebi)); > + > + ebi->clk = clk_get(dev, NULL); > + if (IS_ERR(ebi->clk)) { > + ret = PTR_ERR(ebi->clk); > + goto out_kfree; > + } > + > + clk_enable(ebi->clk); > + > + ebi->io_base = of_iomap(dev->of_node, 0); > + if (!ebi->io_base) { > + ret = -ENOMEM; > + goto out_clk; > + } > + > + ret = device_reset_us(dev, 2); > + if (ret) > + goto out_clk; > + > + ret = stm32_fmc2_ebi_parse_dt(dev, ebi); > + if (ret) > + goto out_clk; > + > + return of_platform_populate(dev->of_node, NULL, dev); > + > +out_clk: > + clk_disable(ebi->clk); > + > +out_kfree: > + kfree(ebi); > + > + return ret; > +} > + > +static __maybe_unused struct of_device_id stm32_fmc2_ebi_dt_ids[] = { > + { .compatible = "st,stm32mp1-fmc2-ebi", }, > + { /* sentinel */ } > +}; > +MODULE_DEVICE_TABLE(of, stm32_fmc2_ebi_dt_ids); > + > +static struct driver stm32_fmc2_ebi_driver = { > + .name = "stm32_fmc2_ebi", > + .probe = stm32_fmc2_ebi_probe, > + .of_compatible = DRV_OF_COMPAT(stm32_fmc2_ebi_dt_ids), > +}; > +coredevice_platform_driver(stm32_fmc2_ebi_driver); > -- > 2.39.1 > > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |