From nobody Fri Apr 3 03:20:27 2026 Received: from mailout2.w1.samsung.com (mailout2.w1.samsung.com [210.118.77.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C38D931716F for ; Wed, 25 Mar 2026 09:00:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=210.118.77.12 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774429235; cv=none; b=hwaxDTbweMq+eSySZytwUMyFnAV4BfF8Ns3Hvs/wsePIa7RKyy7vbrQ8022hyP8NQR2O14cnk1f0UIRTqS6k4DTzcb/3q4dV66HW3dix4ePmy6kxtz6an/lc4FGhNgA0sCPi5Mq74HPMeJsElMXCn17tH+enaE8TNp9HitIeD6I= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774429235; c=relaxed/simple; bh=rYdxiFd8V1p9D8axzTRdYHSmxWdNu1JUKInL/cS/0xI=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:MIME-Version: Content-Type:References; b=Es745eEcxrdq5/bt6hM1y29V6Lq774/wdlNcqU2/BRwEA/UP9K9P49dRl+hDgQTVKa1OQmuEG39aXUa99GsE8843xRJistR6dzX9a4XwA72CfWk/Ih87vsPJzS3HkChAAKT5uIdSOom3HKnNinB3xnwp8U3MHhPDOLie60DGEnI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=samsung.com; spf=pass smtp.mailfrom=samsung.com; dkim=pass (1024-bit key) header.d=samsung.com header.i=@samsung.com header.b=gxSREEvd; arc=none smtp.client-ip=210.118.77.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=samsung.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=samsung.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=samsung.com header.i=@samsung.com header.b="gxSREEvd" Received: from eucas1p1.samsung.com (unknown [182.198.249.206]) by mailout2.w1.samsung.com (KnoxPortal) with ESMTP id 20260325090032euoutp02143b10275c7bc5a0a3b31679a56f8141~gCfxSdKnQ0087600876euoutp02b for ; Wed, 25 Mar 2026 09:00:32 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout2.w1.samsung.com 20260325090032euoutp02143b10275c7bc5a0a3b31679a56f8141~gCfxSdKnQ0087600876euoutp02b DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1774429232; bh=wwurihy1qMKafazYkjQUEhOehZg0+KQHoNwHol2qBYc=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=gxSREEvdokyuM4SJvoYNxRBCw5lJqnKxsVDEvWM8vPv+T0ROzMuuoUJpSVYpaS58c Y8AwYTshgbVkhx1d2g5lOXTl51tvwt0eyQioLsQNKunLt3jhQVTQkqE+Z672ZdpERL yC0g5fL8R1nNpgjHlQzpfEg3CwvoujmFLw42FFo8= Received: from eusmtip1.samsung.com (unknown [203.254.199.221]) by eucas1p1.samsung.com (KnoxPortal) with ESMTPA id 20260325090032eucas1p1bf0dfc6d73868e576d829d9d2208672b~gCfw1aKZu1802018020eucas1p1t; Wed, 25 Mar 2026 09:00:32 +0000 (GMT) Received: from AMDC4653.digital.local (unknown [106.120.51.32]) by eusmtip1.samsung.com (KnoxPortal) with ESMTPA id 20260325090031eusmtip187bfb06cefb9c4211a0093e10bb3ac8a~gCfwSg-9U3134631346eusmtip1N; Wed, 25 Mar 2026 09:00:31 +0000 (GMT) From: Marek Szyprowski To: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-mm@kvack.org, iommu@lists.linux.dev Cc: Marek Szyprowski , Rob Herring , Krzysztof Kozlowski , Oreoluwa Babatunde , Saravana Kannan , Andrew Morton , Robin Murphy Subject: [PATCH v3 3/7] of: reserved_mem: switch to ops based OF_DECLARE() Date: Wed, 25 Mar 2026 10:00:19 +0100 Message-Id: <20260325090023.3175348-4-m.szyprowski@samsung.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260325090023.3175348-1-m.szyprowski@samsung.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-CMS-MailID: 20260325090032eucas1p1bf0dfc6d73868e576d829d9d2208672b X-Msg-Generator: CA Content-Type: text/plain; charset="utf-8" X-RootMTR: 20260325090032eucas1p1bf0dfc6d73868e576d829d9d2208672b X-EPHeader: CA X-CMS-RootMailID: 20260325090032eucas1p1bf0dfc6d73868e576d829d9d2208672b References: <20260325090023.3175348-1-m.szyprowski@samsung.com> Move init function from OF_DECLARE() argument to the given reserved memory region ops structure and then pass that structure to the OF_DECLARE() initializer. This node_init callback is mandatory for the reserved mem driver. Such change makes it possible in the future to add more functions called by the generic code before given memory region is initialized and rmem object is created. Signed-off-by: Marek Szyprowski --- drivers/memory/tegra/tegra210-emc-table.c | 16 ++++++++-------- drivers/of/of_reserved_mem.c | 17 +++++++++++++---- include/linux/of_reserved_mem.h | 13 ++++++------- kernel/dma/coherent.c | 13 +++++++------ kernel/dma/contiguous.c | 15 ++++++++------- kernel/dma/swiotlb.c | 14 +++++++------- 6 files changed, 49 insertions(+), 39 deletions(-) diff --git a/drivers/memory/tegra/tegra210-emc-table.c b/drivers/memory/teg= ra/tegra210-emc-table.c index ac1d1e13482a..4b3c478b2743 100644 --- a/drivers/memory/tegra/tegra210-emc-table.c +++ b/drivers/memory/tegra/tegra210-emc-table.c @@ -70,20 +70,20 @@ static void tegra210_emc_table_device_release(struct re= served_mem *rmem, memunmap(timings); } =20 -static const struct reserved_mem_ops tegra210_emc_table_ops =3D { - .device_init =3D tegra210_emc_table_device_init, - .device_release =3D tegra210_emc_table_device_release, -}; - static int tegra210_emc_table_init(unsigned long node, struct reserved_mem *rmem) { pr_debug("Tegra210 EMC table at %pa, size %lu bytes\n", &rmem->base, (unsigned long)rmem->size); =20 - rmem->ops =3D &tegra210_emc_table_ops; - return 0; } + +static const struct reserved_mem_ops tegra210_emc_table_ops =3D { + .node_init =3D tegra210_emc_table_init, + .device_init =3D tegra210_emc_table_device_init, + .device_release =3D tegra210_emc_table_device_release, +}; + RESERVEDMEM_OF_DECLARE(tegra210_emc_table, "nvidia,tegra210-emc-table", - tegra210_emc_table_init); + &tegra210_emc_table_ops); diff --git a/drivers/of/of_reserved_mem.c b/drivers/of/of_reserved_mem.c index 9aff460a0420..4dd0d6f6a4b0 100644 --- a/drivers/of/of_reserved_mem.c +++ b/drivers/of/of_reserved_mem.c @@ -480,8 +480,16 @@ static int __init __reserved_mem_alloc_size(unsigned l= ong node, const char *unam static const struct of_device_id __rmem_of_table_sentinel __used __section("__reservedmem_of_table_end"); =20 -/* - * __reserved_mem_init_node() - call region specific reserved memory init = code +/** + * __reserved_mem_init_node() - initialize a reserved memory region + * @rmem: reserved_mem structure to initialize + * @node: FDT node describing the reserved memory region + * + * This function iterates through the reserved memory drivers and calls the + * node_init callback for the compatible entry matching the node. On succe= ss, + * the operations pointer is stored in the reserved_mem structure. + * + * Return: 0 on success, -ENODEV if no compatible match found */ static int __init __reserved_mem_init_node(struct reserved_mem *rmem, unsigned long node) @@ -492,14 +500,15 @@ static int __init __reserved_mem_init_node(struct res= erved_mem *rmem, =20 for (i =3D __reservedmem_of_table; ret =3D=3D -ENODEV && i < &__rmem_of_table_sentinel; i++) { - reservedmem_of_init_fn initfn =3D i->data; + const struct reserved_mem_ops *ops =3D i->data; const char *compat =3D i->compatible; =20 if (!of_flat_dt_is_compatible(node, compat)) continue; =20 - ret =3D initfn(node, rmem); + ret =3D ops->node_init(node, rmem); if (ret =3D=3D 0) { + rmem->ops =3D ops; pr_info("initialized node %s, compatible id %s\n", rmem->name, compat); break; diff --git a/include/linux/of_reserved_mem.h b/include/linux/of_reserved_me= m.h index 5159938bfe03..747a1e73d5dd 100644 --- a/include/linux/of_reserved_mem.h +++ b/include/linux/of_reserved_mem.h @@ -18,19 +18,17 @@ struct reserved_mem { }; =20 struct reserved_mem_ops { + int (*node_init)(unsigned long fdt_node, struct reserved_mem *rmem); int (*device_init)(struct reserved_mem *rmem, struct device *dev); void (*device_release)(struct reserved_mem *rmem, struct device *dev); }; =20 -typedef int (*reservedmem_of_init_fn)(unsigned long node, - struct reserved_mem *rmem); - #ifdef CONFIG_OF_RESERVED_MEM =20 -#define RESERVEDMEM_OF_DECLARE(name, compat, init) \ - _OF_DECLARE(reservedmem, name, compat, init, reservedmem_of_init_fn) +#define RESERVEDMEM_OF_DECLARE(name, compat, ops) \ + _OF_DECLARE(reservedmem, name, compat, ops, struct reserved_mem_ops *) =20 int of_reserved_mem_device_init_by_idx(struct device *dev, struct device_node *np, int idx); @@ -48,8 +46,9 @@ int of_reserved_mem_region_count(const struct device_node= *np); =20 #else =20 -#define RESERVEDMEM_OF_DECLARE(name, compat, init) \ - _OF_DECLARE_STUB(reservedmem, name, compat, init, reservedmem_of_init_fn) +#define RESERVEDMEM_OF_DECLARE(name, compat, ops) \ + _OF_DECLARE_STUB(reservedmem, name, compat, ops, \ + struct reserved_mem_ops *) =20 static inline int of_reserved_mem_device_init_by_idx(struct device *dev, struct device_node *np, int idx) diff --git a/kernel/dma/coherent.c b/kernel/dma/coherent.c index 64f9ba618e19..bcdc0f76d2e8 100644 --- a/kernel/dma/coherent.c +++ b/kernel/dma/coherent.c @@ -362,10 +362,6 @@ static void rmem_dma_device_release(struct reserved_me= m *rmem, dev->dma_mem =3D NULL; } =20 -static const struct reserved_mem_ops rmem_dma_ops =3D { - .device_init =3D rmem_dma_device_init, - .device_release =3D rmem_dma_device_release, -}; =20 static int __init rmem_dma_setup(unsigned long node, struct reserved_mem *= rmem) { @@ -388,7 +384,6 @@ static int __init rmem_dma_setup(unsigned long node, st= ruct reserved_mem *rmem) } #endif =20 - rmem->ops =3D &rmem_dma_ops; pr_info("Reserved memory: created DMA memory pool at %pa, size %ld MiB\n", &rmem->base, (unsigned long)rmem->size / SZ_1M); return 0; @@ -405,5 +400,11 @@ static int __init dma_init_reserved_memory(void) core_initcall(dma_init_reserved_memory); #endif /* CONFIG_DMA_GLOBAL_POOL */ =20 -RESERVEDMEM_OF_DECLARE(dma, "shared-dma-pool", rmem_dma_setup); +static const struct reserved_mem_ops rmem_dma_ops =3D { + .node_init =3D rmem_dma_setup, + .device_init =3D rmem_dma_device_init, + .device_release =3D rmem_dma_device_release, +}; + +RESERVEDMEM_OF_DECLARE(dma, "shared-dma-pool", &rmem_dma_ops); #endif diff --git a/kernel/dma/contiguous.c b/kernel/dma/contiguous.c index e6fc6906b5c0..efeebda92537 100644 --- a/kernel/dma/contiguous.c +++ b/kernel/dma/contiguous.c @@ -470,11 +470,6 @@ static void rmem_cma_device_release(struct reserved_me= m *rmem, dev->cma_area =3D NULL; } =20 -static const struct reserved_mem_ops rmem_cma_ops =3D { - .device_init =3D rmem_cma_device_init, - .device_release =3D rmem_cma_device_release, -}; - static int __init rmem_cma_setup(unsigned long node, struct reserved_mem *= rmem) { bool default_cma =3D of_get_flat_dt_prop(node, "linux,cma-default", NULL); @@ -499,7 +494,6 @@ static int __init rmem_cma_setup(unsigned long node, st= ruct reserved_mem *rmem) if (default_cma) dma_contiguous_default_area =3D cma; =20 - rmem->ops =3D &rmem_cma_ops; rmem->priv =3D cma; =20 pr_info("Reserved memory: created CMA memory pool at %pa, size %ld MiB\n", @@ -511,5 +505,12 @@ static int __init rmem_cma_setup(unsigned long node, s= truct reserved_mem *rmem) =20 return 0; } -RESERVEDMEM_OF_DECLARE(cma, "shared-dma-pool", rmem_cma_setup); + +static const struct reserved_mem_ops rmem_cma_ops =3D { + .node_init =3D rmem_cma_setup, + .device_init =3D rmem_cma_device_init, + .device_release =3D rmem_cma_device_release, +}; + +RESERVEDMEM_OF_DECLARE(cma, "shared-dma-pool", &rmem_cma_ops); #endif diff --git a/kernel/dma/swiotlb.c b/kernel/dma/swiotlb.c index 2f53d8c599d1..9a15e7231e39 100644 --- a/kernel/dma/swiotlb.c +++ b/kernel/dma/swiotlb.c @@ -1877,11 +1877,6 @@ static void rmem_swiotlb_device_release(struct reser= ved_mem *rmem, dev->dma_io_tlb_mem =3D &io_tlb_default_mem; } =20 -static const struct reserved_mem_ops rmem_swiotlb_ops =3D { - .device_init =3D rmem_swiotlb_device_init, - .device_release =3D rmem_swiotlb_device_release, -}; - static int __init rmem_swiotlb_setup(unsigned long node, struct reserved_mem *rmem) { @@ -1891,11 +1886,16 @@ static int __init rmem_swiotlb_setup(unsigned long = node, of_get_flat_dt_prop(node, "no-map", NULL)) return -EINVAL; =20 - rmem->ops =3D &rmem_swiotlb_ops; pr_info("Reserved memory: created restricted DMA pool at %pa, size %ld Mi= B\n", &rmem->base, (unsigned long)rmem->size / SZ_1M); return 0; } =20 -RESERVEDMEM_OF_DECLARE(dma, "restricted-dma-pool", rmem_swiotlb_setup); +static const struct reserved_mem_ops rmem_swiotlb_ops =3D { + .node_init =3D rmem_swiotlb_setup, + .device_init =3D rmem_swiotlb_device_init, + .device_release =3D rmem_swiotlb_device_release, +}; + +RESERVEDMEM_OF_DECLARE(dma, "restricted-dma-pool", &rmem_swiotlb_ops); #endif /* CONFIG_DMA_RESTRICTED_POOL */ --=20 2.34.1