From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrice CHOTARD Date: Wed, 21 Oct 2020 12:34:28 +0000 Subject: [PATCH 26/33] mtd: stm32_fmc2: migrate trace to dev and log macro In-Reply-To: <20201014091646.4233-27-patrick.delaunay@st.com> References: <20201014091646.4233-1-patrick.delaunay@st.com> <20201014091646.4233-27-patrick.delaunay@st.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Patrick On 10/14/20 11:16 AM, Patrick Delaunay wrote: > Change pr_* to dev_ or log_ macro and define LOG_CATEGORY. > > Signed-off-by: Patrick Delaunay > --- > > drivers/mtd/nand/raw/stm32_fmc2_nand.c | 40 ++++++++++++-------------- > 1 file changed, 19 insertions(+), 21 deletions(-) > > diff --git a/drivers/mtd/nand/raw/stm32_fmc2_nand.c b/drivers/mtd/nand/raw/stm32_fmc2_nand.c > index 47fe61090d..9624583b8c 100644 > --- a/drivers/mtd/nand/raw/stm32_fmc2_nand.c > +++ b/drivers/mtd/nand/raw/stm32_fmc2_nand.c > @@ -4,12 +4,15 @@ > * Author: Christophe Kerello > */ > > +#define LOG_CATEGORY UCLASS_MTD > + > #include > #include > #include > #include > #include > #include > +#include > #include > #include > #include > @@ -324,7 +327,7 @@ static int stm32_fmc2_nfc_ham_calculate(struct mtd_info *mtd, const u8 *data, > ret = readl_poll_timeout(nfc->io_base + FMC2_SR, sr, > sr & FMC2_SR_NWRF, FMC2_TIMEOUT_5S); > if (ret < 0) { > - pr_err("Ham timeout\n"); > + log_err("Ham timeout\n"); > return ret; > } > > @@ -409,7 +412,7 @@ static int stm32_fmc2_nfc_bch_calculate(struct mtd_info *mtd, const u8 *data, > ret = readl_poll_timeout(nfc->io_base + FMC2_BCHISR, bchisr, > bchisr & FMC2_BCHISR_EPBRF, FMC2_TIMEOUT_5S); > if (ret < 0) { > - pr_err("Bch timeout\n"); > + log_err("Bch timeout\n"); > return ret; > } > > @@ -457,7 +460,7 @@ static int stm32_fmc2_nfc_bch_correct(struct mtd_info *mtd, u8 *dat, > ret = readl_poll_timeout(nfc->io_base + FMC2_BCHISR, bchisr, > bchisr & FMC2_BCHISR_DERF, FMC2_TIMEOUT_5S); > if (ret < 0) { > - pr_err("Bch timeout\n"); > + log_err("Bch timeout\n"); > return ret; > } > > @@ -795,26 +798,24 @@ static int stm32_fmc2_nfc_parse_child(struct stm32_fmc2_nfc *nfc, ofnode node) > > nand->ncs /= sizeof(u32); > if (!nand->ncs) { > - pr_err("Invalid reg property size\n"); > + log_err("Invalid reg property size\n"); > return -EINVAL; > } > > ret = ofnode_read_u32_array(node, "reg", cs, nand->ncs); > if (ret < 0) { > - pr_err("Could not retrieve reg property\n"); > + log_err("Could not retrieve reg property\n"); > return -EINVAL; > } > > for (i = 0; i < nand->ncs; i++) { > if (cs[i] >= FMC2_MAX_CE) { > - pr_err("Invalid reg value: %d\n", > - nand->cs_used[i]); > + log_err("Invalid reg value: %d\n", nand->cs_used[i]); > return -EINVAL; > } > > if (nfc->cs_assigned & BIT(cs[i])) { > - pr_err("Cs already assigned: %d\n", > - nand->cs_used[i]); > + log_err("Cs already assigned: %d\n", nand->cs_used[i]); > return -EINVAL; > } > > @@ -837,12 +838,12 @@ static int stm32_fmc2_nfc_parse_dt(struct udevice *dev, > nchips++; > > if (!nchips) { > - pr_err("NAND chip not defined\n"); > + log_err("NAND chip not defined\n"); > return -EINVAL; > } > > if (nchips > 1) { > - pr_err("Too many NAND chips defined\n"); > + log_err("Too many NAND chips defined\n"); > return -EINVAL; > } > > @@ -918,24 +919,21 @@ static int stm32_fmc2_nfc_probe(struct udevice *dev) > > addr = dev_read_addr_index(dev, mem_region); > if (addr == FDT_ADDR_T_NONE) { > - pr_err("Resource data_base not found for cs%d", > - chip_cs); > + dev_err(dev, "Resource data_base not found for cs%d", chip_cs); > return ret; > } > nfc->data_base[chip_cs] = addr; > > addr = dev_read_addr_index(dev, mem_region + 1); > if (addr == FDT_ADDR_T_NONE) { > - pr_err("Resource cmd_base not found for cs%d", > - chip_cs); > + dev_err(dev, "Resource cmd_base not found for cs%d", chip_cs); > return ret; > } > nfc->cmd_base[chip_cs] = addr; > > addr = dev_read_addr_index(dev, mem_region + 2); > if (addr == FDT_ADDR_T_NONE) { > - pr_err("Resource addr_base not found for cs%d", > - chip_cs); > + dev_err(dev, "Resource addr_base not found for cs%d", chip_cs); > return ret; > } > nfc->addr_base[chip_cs] = addr; > @@ -985,14 +983,14 @@ static int stm32_fmc2_nfc_probe(struct udevice *dev) > * ECC sector size = 512 > */ > if (chip->ecc.mode != NAND_ECC_HW) { > - pr_err("Nand_ecc_mode is not well defined in the DT\n"); > + dev_err(dev, "Nand_ecc_mode is not well defined in the DT\n"); > return -EINVAL; > } > > ret = nand_check_ecc_caps(chip, &stm32_fmc2_nfc_ecc_caps, > mtd->oobsize - FMC2_BBM_LEN); > if (ret) { > - pr_err("No valid ECC settings set\n"); > + dev_err(dev, "No valid ECC settings set\n"); > return ret; > } > > @@ -1045,6 +1043,6 @@ void board_nand_init(void) > DM_GET_DRIVER(stm32_fmc2_nfc), > &dev); > if (ret && ret != -ENODEV) > - pr_err("Failed to initialize STM32 FMC2 NFC controller. (error %d)\n", > - ret); > + log_err("Failed to initialize STM32 FMC2 NFC controller. (error %d)\n", > + ret); > } Reviewed-by: Patrice Chotard Thanks