From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf0-x22f.google.com ([2607:f8b0:400e:c00::22f]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1b5MqJ-0000Zl-B0 for linux-mtd@lists.infradead.org; Wed, 25 May 2016 00:38:36 +0000 Received: by mail-pf0-x22f.google.com with SMTP id b124so12186933pfb.0 for ; Tue, 24 May 2016 17:38:14 -0700 (PDT) From: Moritz Fischer To: dwmw2@infradead.org Cc: computersforpeace@gmail.com, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, moritz.fischer.private@gmail.com, Moritz Fischer Subject: [PATCH] mtd: ofpart: Assign device node to partitions parsed with ofpart Date: Tue, 24 May 2016 17:37:40 -0700 Message-Id: <1464136660-13222-1-git-send-email-moritz.fischer@ettus.com> List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , This patch adds a struct device_node pointer to struct mtd_partition. Partitions parsed via ofpart can now be referenced via devicetree. Cc: David Woodhouse Cc: Brian Norris Signed-off-by: Moritz Fischer --- Hi all, during my work on another patch series for supporting otp regions as nvmem providers via dt (follow up series), I realized we don't actually assign node pointers. While not used for ofpart atm, I can see cases where having the pointer is useful to reference raw mtd partitions. I can also (re)submit this as part of my upcoming series for nvmem / otp support Cheers, Moritz --- drivers/mtd/mtdpart.c | 1 + drivers/mtd/ofpart.c | 1 + include/linux/mtd/partitions.h | 4 ++++ 3 files changed, 6 insertions(+) diff --git a/drivers/mtd/mtdpart.c b/drivers/mtd/mtdpart.c index 6778ba2..3af9cf5 100644 --- a/drivers/mtd/mtdpart.c +++ b/drivers/mtd/mtdpart.c @@ -439,6 +439,7 @@ static struct mtd_part *allocate_partition(struct mtd_info *master, slave->mtd.name = name; slave->mtd.owner = master->owner; + slave->mtd.dev.of_node = part->node; /* NOTE: Historically, we didn't arrange MTDs as a tree out of * concern for showing the same data in multiple partitions. diff --git a/drivers/mtd/ofpart.c b/drivers/mtd/ofpart.c index 6127ee6..caeaef0 100644 --- a/drivers/mtd/ofpart.c +++ b/drivers/mtd/ofpart.c @@ -108,6 +108,7 @@ static int parse_ofpart_partitions(struct mtd_info *master, parts[i].offset = of_read_number(reg, a_cells); parts[i].size = of_read_number(reg + a_cells, s_cells); + parts[i].node = pp; partname = of_get_property(pp, "label", &len); if (!partname) diff --git a/include/linux/mtd/partitions.h b/include/linux/mtd/partitions.h index 70736e1..51d36ac 100644 --- a/include/linux/mtd/partitions.h +++ b/include/linux/mtd/partitions.h @@ -31,6 +31,9 @@ * master MTD flag set for the corresponding MTD partition. * For example, to force a read-only partition, simply adding * MTD_WRITEABLE to the mask_flags will do the trick. + * node: contains device node pointer, necessary to reference device using + * devicetree. + * * Note: writeable partitions require their size and offset be * erasesize aligned (e.g. use MTDPART_OFS_NEXTBLK). @@ -41,6 +44,7 @@ struct mtd_partition { uint64_t size; /* partition size */ uint64_t offset; /* offset within the master MTD space */ uint32_t mask_flags; /* master MTD flags to mask out for this partition */ + struct device_node *node; /* pointer to the device node */ }; #define MTDPART_OFS_RETAIN (-3) -- 2.5.5