From patchwork Wed Aug 16 07:29:05 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kefeng Wang X-Patchwork-Id: 110206 Delivered-To: patch@linaro.org Received: by 10.140.95.78 with SMTP id h72csp429623qge; Wed, 16 Aug 2017 00:27:27 -0700 (PDT) X-Received: by 10.84.232.14 with SMTP id h14mr870543plk.359.1502868447024; Wed, 16 Aug 2017 00:27:27 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1502868447; cv=none; d=google.com; s=arc-20160816; b=Tt7Wt10MmbUuHfSeh2GBa0CSmB51WaD/e8zsU0GQya6RB59CEGbeUwbjGJjommivI3 kzjPuHW8Ey/CsjfF4QMOQJIYuXys56RMf9Sg5vs+OVYARN9/ILug5hosO6dwHk/UAw8G 6GWmIR/2wFs94SfBDbL6cVVLKgQeZssYNd76L65KJVCTlDet0JWkIUl1r/UZtq6rcw2f L7W8mJUjzESEgt9TxPhdHLjilt0fFGOUs9nuNEj2DBgPvWSF8EzzbB9eVqaQml3s1hfb fusBcv4Alits75ISnCnABOhemVv0ndw8TT1YXiYK/nm9cN6zTkyhLYS/Ngl6/6Po1EvR kRSQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:message-id:date:subject:cc :to:from:arc-authentication-results; bh=7HjCzFYb8Lk9Di7p6mcNIkrcpLmYmrmpznh8+jooS4w=; b=Huma7rih1pzE0M6QjuNFGStMkPtLgGtSgxpHziaCZx2LSmPzBn5caB7ohHWUPt147r G8xsmEy3YTQWrFMto46lChBe9htkSCnkchWsL0oc0cE1jnAxTp0kvO8gKoiMbeEExCRu 6tRT4PjAhGDYVpXX1eLDLsajQBRFgmAtwIk3o0oo2BrYua+ZWWNqaXwg/Ct4fUow/vo7 8JZQNM5oMGhM9bpAMzaNmRf0hLQEJkzpKT1DWDJee7GvnGxHKHbGp+XLMbDkKVAyHWnQ y9RYqfU7fnK2BCCWxjm9DVsgU6OquczyA1V+vR7hq8lQ/89Rac3KarPWaly9IGjPcZvD mdhA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id n22si130307pgd.517.2017.08.16.00.27.26; Wed, 16 Aug 2017 00:27:27 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751542AbdHPH1W (ORCPT + 26 others); Wed, 16 Aug 2017 03:27:22 -0400 Received: from szxga05-in.huawei.com ([45.249.212.191]:3552 "EHLO szxga05-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751068AbdHPH1V (ORCPT ); Wed, 16 Aug 2017 03:27:21 -0400 Received: from 172.30.72.60 (EHLO DGGEMS403-HUB.china.huawei.com) ([172.30.72.60]) by dggrg05-dlp.huawei.com (MOS 4.4.6-GA FastPath queued) with ESMTP id DFI29358; Wed, 16 Aug 2017 15:27:16 +0800 (CST) Received: from localhost.localdomain.localdomain (10.175.113.25) by DGGEMS403-HUB.china.huawei.com (10.3.19.203) with Microsoft SMTP Server id 14.3.301.0; Wed, 16 Aug 2017 15:27:05 +0800 From: Kefeng Wang To: Boris Brezillon CC: Richard Weinberger , , , , Kefeng Wang Subject: [PATCH] mtd: nand: convert to unified device property interface Date: Wed, 16 Aug 2017 15:29:05 +0800 Message-ID: <1502868545-124616-1-git-send-email-wangkefeng.wang@huawei.com> X-Mailer: git-send-email 1.8.3.1 MIME-Version: 1.0 X-Originating-IP: [10.175.113.25] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020201.5993F3D5.0037, ss=1, re=0.000, recu=0.000, reip=0.000, cl=1, cld=1, fgs=0, ip=0.0.0.0, so=2014-11-16 11:51:01, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: b8a779f89d8373ac4310e2eec7b6fc83 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Changing from of_* to device_* interface, then we can also extract the properties from ACPI tables as well as from DT. Signed-off-by: Kefeng Wang --- - APCI will be supported in hisi504_nand.c, and it will use nand_scan_ident(). drivers/mtd/nand/nand_base.c | 54 ++++++++++++++++++++++---------------------- 1 file changed, 27 insertions(+), 27 deletions(-) -- 1.8.3.1 diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c index c6c18b8..27a0947 100644 --- a/drivers/mtd/nand/nand_base.c +++ b/drivers/mtd/nand/nand_base.c @@ -46,7 +46,7 @@ #include #include #include -#include +#include static int nand_get_device(struct mtd_info *mtd, int new_state); @@ -4209,12 +4209,12 @@ static int nand_detect(struct nand_chip *chip, struct nand_flash_dev *type) [NAND_ECC_ON_DIE] = "on-die", }; -static int of_get_nand_ecc_mode(struct device_node *np) +static int device_get_nand_ecc_mode(struct device *dev) { const char *pm; int err, i; - err = of_property_read_string(np, "nand-ecc-mode", &pm); + err = device_property_read_string(dev, "nand-ecc-mode", &pm); if (err < 0) return err; @@ -4238,12 +4238,12 @@ static int of_get_nand_ecc_mode(struct device_node *np) [NAND_ECC_BCH] = "bch", }; -static int of_get_nand_ecc_algo(struct device_node *np) +static int device_get_nand_ecc_algo(struct device *dev) { const char *pm; int err, i; - err = of_property_read_string(np, "nand-ecc-algo", &pm); + err = device_property_read_string(dev, "nand-ecc-algo", &pm); if (!err) { for (i = NAND_ECC_HAMMING; i < ARRAY_SIZE(nand_ecc_algos); i++) if (!strcasecmp(pm, nand_ecc_algos[i])) @@ -4255,7 +4255,7 @@ static int of_get_nand_ecc_algo(struct device_node *np) * For backward compatibility we also read "nand-ecc-mode" checking * for some obsoleted values that were specifying ECC algorithm. */ - err = of_property_read_string(np, "nand-ecc-mode", &pm); + err = device_property_read_string(dev, "nand-ecc-mode", &pm); if (err < 0) return err; @@ -4267,29 +4267,29 @@ static int of_get_nand_ecc_algo(struct device_node *np) return -ENODEV; } -static int of_get_nand_ecc_step_size(struct device_node *np) +static int device_get_nand_ecc_step_size(struct device *dev) { int ret; u32 val; - ret = of_property_read_u32(np, "nand-ecc-step-size", &val); + ret = device_property_read_u32(dev, "nand-ecc-step-size", &val); return ret ? ret : val; } -static int of_get_nand_ecc_strength(struct device_node *np) +static int device_get_nand_ecc_strength(struct device *dev) { int ret; u32 val; - ret = of_property_read_u32(np, "nand-ecc-strength", &val); + ret = device_property_read_u32(dev, "nand-ecc-strength", &val); return ret ? ret : val; } -static int of_get_nand_bus_width(struct device_node *np) +static int device_get_nand_bus_width(struct device *dev) { u32 val; - if (of_property_read_u32(np, "nand-bus-width", &val)) + if (device_property_read_u32(dev, "nand-bus-width", &val)) return 8; switch (val) { @@ -4301,29 +4301,28 @@ static int of_get_nand_bus_width(struct device_node *np) } } -static bool of_get_nand_on_flash_bbt(struct device_node *np) +static bool device_get_nand_on_flash_bbt(struct device *dev) { - return of_property_read_bool(np, "nand-on-flash-bbt"); + return device_property_read_bool(dev, "nand-on-flash-bbt"); } -static int nand_dt_init(struct nand_chip *chip) +static int nand_chip_init(struct nand_chip *chip, struct device *dev) { - struct device_node *dn = nand_get_flash_node(chip); int ecc_mode, ecc_algo, ecc_strength, ecc_step; - if (!dn) + if (!dev) return 0; - if (of_get_nand_bus_width(dn) == 16) + if (device_get_nand_bus_width(dev) == 16) chip->options |= NAND_BUSWIDTH_16; - if (of_get_nand_on_flash_bbt(dn)) + if (device_get_nand_on_flash_bbt(dev)) chip->bbt_options |= NAND_BBT_USE_FLASH; - ecc_mode = of_get_nand_ecc_mode(dn); - ecc_algo = of_get_nand_ecc_algo(dn); - ecc_strength = of_get_nand_ecc_strength(dn); - ecc_step = of_get_nand_ecc_step_size(dn); + ecc_mode = device_get_nand_ecc_mode(dev); + ecc_algo = device_get_nand_ecc_algo(dev); + ecc_strength = device_get_nand_ecc_strength(dev); + ecc_step = device_get_nand_ecc_step_size(dev); if (ecc_mode >= 0) chip->ecc.mode = ecc_mode; @@ -4337,7 +4336,7 @@ static int nand_dt_init(struct nand_chip *chip) if (ecc_step > 0) chip->ecc.size = ecc_step; - if (of_property_read_bool(dn, "nand-ecc-maximize")) + if (device_property_read_bool(dev, "nand-ecc-maximize")) chip->ecc.options |= NAND_ECC_MAXIMIZE; return 0; @@ -4358,14 +4357,15 @@ int nand_scan_ident(struct mtd_info *mtd, int maxchips, { int i, nand_maf_id, nand_dev_id; struct nand_chip *chip = mtd_to_nand(mtd); + struct device *dev = mtd->dev.parent; int ret; - ret = nand_dt_init(chip); + ret = nand_chip_init(chip, dev); if (ret) return ret; - if (!mtd->name && mtd->dev.parent) - mtd->name = dev_name(mtd->dev.parent); + if (!mtd->name && dev) + mtd->name = dev_name(dev); if ((!chip->cmdfunc || !chip->select_chip) && !chip->cmd_ctrl) { /*