From patchwork Tue Apr 3 14:24:13 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Oliver O'Halloran X-Patchwork-Id: 10321291 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 3847E60390 for ; Tue, 3 Apr 2018 14:25:18 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 247B1283B0 for ; Tue, 3 Apr 2018 14:25:18 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 1841B28426; Tue, 3 Apr 2018 14:25:18 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.8 required=2.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE, T_DKIM_INVALID autolearn=no version=3.3.1 Received: from ml01.01.org (ml01.01.org [198.145.21.10]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 8FADF283B0 for ; Tue, 3 Apr 2018 14:25:17 +0000 (UTC) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 86A09226085D1; Tue, 3 Apr 2018 07:25:17 -0700 (PDT) X-Original-To: linux-nvdimm@lists.01.org Delivered-To: linux-nvdimm@lists.01.org Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:400e:c01::242; helo=mail-pl0-x242.google.com; envelope-from=oohall@gmail.com; receiver=linux-nvdimm@lists.01.org Received: from mail-pl0-x242.google.com (mail-pl0-x242.google.com [IPv6:2607:f8b0:400e:c01::242]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 5A464226085BC for ; Tue, 3 Apr 2018 07:25:16 -0700 (PDT) Received: by mail-pl0-x242.google.com with SMTP id g20-v6so9001459plo.9 for ; Tue, 03 Apr 2018 07:25:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=OxQ+iVoYs4yXEEp6zgtQjPcaouSGZezyzr/K5cOtOzE=; b=UjDqkgXkh/WZuTCs8kQrwQ3RH32kq6ZHxogrP1Z+BGJukb9y4Pxf/8niccGNhHuCMk eApYdWhOAKJfJL7Cy7W0p4Osg/gvq3Xg0LpjoYse8b1elObZirrzYz1FvdV9rHBlukQo 5SInsACHP9Zy04JipS043F5etn88TpKvFw4pic9X+YCJOPvhWI90I6zr99rdRZzf6I84 k1hA6w+Tvh9dolt22nZ2b9deJBt89XMNsLEX++hWcBqZV7fmNUJ6KhToIXljjMbDvb+i S86Cnyc/Gv3YCiy1A8O8vlrMhtd4OLsDY1LCqTXoSTDN7tRH+WJhKWsQsyKh8Dyh014r Vf5Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=OxQ+iVoYs4yXEEp6zgtQjPcaouSGZezyzr/K5cOtOzE=; b=orqXWLR1+oWQ47CkCbYguNIEg5wOnsKIwAD67zkpngXF+Fchzow5vKDleEVcThf4G4 M4/E8ly5XhJNidY4FthQNFZC4wOz+rBli5mxifd2VAZ6/QrEFvOUY59JDU++NmYDUhwC 8cTSPfBel+5KHEa+za8ZQzFvKvf0sh2biURFdsjTA0bG0P7nOtPylcQu+EyvZ1geSCwZ SDztHUOuk+GW0Jv+8vJJvfS4ePRuMfZX0vWOsVQ5tHUpnbIFIdUXuhkBEko3H3noVVRQ XrzIfHRDFTNv+CPpm0wDSgPUQ3lWpFmPsCNtu/3VpbD5Jqd81Vd7t8hqUV5L1w7pjIor 7Trw== X-Gm-Message-State: AElRT7FgtMK84jrJp3AqXrs9ZMiivbAgJy7Viv+8+EBDvg3p6YTHeqAX taSA78utU0TLFfBlpCeat9Ie5g== X-Google-Smtp-Source: AIpwx49UX0zlvqT9kz1J2tREfok3kfxVyRCQ3lxxt5JpqyFUI92bm2KLaRmpZab32+9u9JFL74LU1g== X-Received: by 2002:a17:902:6849:: with SMTP id f9-v6mr14717212pln.139.1522765515922; Tue, 03 Apr 2018 07:25:15 -0700 (PDT) Received: from localhost.localdomain ([61.68.80.171]) by smtp.gmail.com with ESMTPSA id j83sm6987587pfe.178.2018.04.03.07.25.13 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 03 Apr 2018 07:25:15 -0700 (PDT) From: Oliver O'Halloran To: linuxppc-dev@lists.ozlabs.org Subject: [RESEND v2 2/4] libnvdimm: Add device-tree based driver Date: Wed, 4 Apr 2018 00:24:13 +1000 Message-Id: <20180403142415.30083-2-oohall@gmail.com> X-Mailer: git-send-email 2.9.5 In-Reply-To: <20180403142415.30083-1-oohall@gmail.com> References: <20180403142415.30083-1-oohall@gmail.com> X-BeenThere: linux-nvdimm@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: "Linux-nvdimm developer list." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-nvdimm@lists.01.org MIME-Version: 1.0 Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" X-Virus-Scanned: ClamAV using ClamSMTP This patch adds peliminary device-tree bindings for persistent memory regions. The driver registers a libnvdimm bus for each pmem-region node and each address range under the node is converted to a region within that bus. Signed-off-by: Oliver O'Halloran --- v2: Made each bus have a separate node rather having a shared bus. Renamed to of_pmem rather than of_nvdimm. Changed log level of happy-path messages to debug. --- MAINTAINERS | 7 +++ drivers/nvdimm/Kconfig | 10 ++++ drivers/nvdimm/Makefile | 1 + drivers/nvdimm/of_pmem.c | 119 +++++++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 137 insertions(+) create mode 100644 drivers/nvdimm/of_pmem.c diff --git a/MAINTAINERS b/MAINTAINERS index 4e62756936fa..6ef38be700e8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -8035,6 +8035,13 @@ Q: https://patchwork.kernel.org/project/linux-nvdimm/list/ S: Supported F: drivers/nvdimm/pmem* +LIBNVDIMM: DEVICETREE BINDINGS +M: Oliver O'Halloran +L: linux-nvdimm@lists.01.org +Q: https://patchwork.kernel.org/project/linux-nvdimm/list/ +S: Supported +F: drivers/nvdimm/of_pmem.c + LIBNVDIMM: NON-VOLATILE MEMORY DEVICE SUBSYSTEM M: Dan Williams L: linux-nvdimm@lists.01.org diff --git a/drivers/nvdimm/Kconfig b/drivers/nvdimm/Kconfig index a65f2e1d9f53..2d6862bf7436 100644 --- a/drivers/nvdimm/Kconfig +++ b/drivers/nvdimm/Kconfig @@ -102,4 +102,14 @@ config NVDIMM_DAX Select Y if unsure +config OF_PMEM + tristate "Device-tree support for persistent memory regions" + depends on OF + default LIBNVDIMM + help + Allows regions of persistent memory to be described in the + device-tree. + + Select Y if unsure. + endif diff --git a/drivers/nvdimm/Makefile b/drivers/nvdimm/Makefile index 70d5f3ad9909..e8847045dac0 100644 --- a/drivers/nvdimm/Makefile +++ b/drivers/nvdimm/Makefile @@ -4,6 +4,7 @@ obj-$(CONFIG_BLK_DEV_PMEM) += nd_pmem.o obj-$(CONFIG_ND_BTT) += nd_btt.o obj-$(CONFIG_ND_BLK) += nd_blk.o obj-$(CONFIG_X86_PMEM_LEGACY) += nd_e820.o +obj-$(CONFIG_OF_PMEM) += of_pmem.o nd_pmem-y := pmem.o diff --git a/drivers/nvdimm/of_pmem.c b/drivers/nvdimm/of_pmem.c new file mode 100644 index 000000000000..374c796ea1de --- /dev/null +++ b/drivers/nvdimm/of_pmem.c @@ -0,0 +1,119 @@ +// SPDX-License-Identifier: GPL-2.0+ + +#define pr_fmt(fmt) "of_pmem: " fmt + +#include +#include +#include +#include +#include +#include + +static const struct attribute_group *region_attr_groups[] = { + &nd_region_attribute_group, + &nd_device_attribute_group, + NULL, +}; + +static const struct attribute_group *bus_attr_groups[] = { + &nvdimm_bus_attribute_group, + NULL, +}; + +struct of_nd_private { + struct nvdimm_bus_descriptor bus_desc; + struct nvdimm_bus *bus; +}; + +static int of_nd_region_probe(struct platform_device *pdev) +{ + struct of_nd_private *priv; + struct device_node *np; + struct nvdimm_bus *bus; + bool is_volatile; + int i; + + np = dev_of_node(&pdev->dev); + if (!np) + return -ENXIO; + + priv = kzalloc(sizeof(*priv), GFP_KERNEL); + if (!priv) + return -ENOMEM; + + priv->bus_desc.attr_groups = bus_attr_groups; + priv->bus_desc.provider_name = "of_pmem"; + priv->bus_desc.module = THIS_MODULE; + priv->bus_desc.of_node = np; + + priv->bus = bus = nvdimm_bus_register(&pdev->dev, &priv->bus_desc); + if (!bus) { + kfree(priv); + return -ENODEV; + } + platform_set_drvdata(pdev, priv); + + is_volatile = !!of_find_property(np, "volatile", NULL); + dev_dbg(&pdev->dev, "Registering %s regions from %pOF\n", + is_volatile ? "volatile" : "non-volatile", np); + + for (i = 0; i < pdev->num_resources; i++) { + struct nd_region_desc ndr_desc; + struct nd_region *region; + + /* + * NB: libnvdimm copies the data from ndr_desc into it's own + * structures so passing a stack pointer is fine. + */ + memset(&ndr_desc, 0, sizeof(ndr_desc)); + ndr_desc.attr_groups = region_attr_groups; + ndr_desc.numa_node = of_node_to_nid(np); + ndr_desc.res = &pdev->resource[i]; + ndr_desc.of_node = np; + set_bit(ND_REGION_PAGEMAP, &ndr_desc.flags); + + if (is_volatile) + region = nvdimm_volatile_region_create(bus, &ndr_desc); + else + region = nvdimm_pmem_region_create(bus, &ndr_desc); + + if (!region) + dev_warn(&pdev->dev, "Unable to register region %pR from %pOF\n", + ndr_desc.res, np); + else + dev_dbg(&pdev->dev, "Registered region %pR from %pOF\n", + ndr_desc.res, np); + } + + return 0; +} + +static int of_nd_region_remove(struct platform_device *pdev) +{ + struct of_nd_private *priv = platform_get_drvdata(pdev); + + nvdimm_bus_unregister(priv->bus); + kfree(priv); + + return 0; +} + +static const struct of_device_id of_nd_region_match[] = { + { .compatible = "pmem-region" }, + { }, +}; + +static struct platform_driver of_nd_region_driver = { + .probe = of_nd_region_probe, + .remove = of_nd_region_remove, + .driver = { + .name = "of_pmem", + .owner = THIS_MODULE, + .of_match_table = of_nd_region_match, + }, +}; + +module_platform_driver(of_nd_region_driver); +MODULE_DEVICE_TABLE(of, of_nd_region_match); +MODULE_LICENSE("GPL"); +MODULE_AUTHOR("IBM Corporation");