Thread (20 messages) 20 messages, 5 authors, 2012-05-30

[PATCH v2 6/6] spi: s3c64xx: add device tree support

From: Grant Likely <hidden>
Date: 2012-05-20 05:10:26
Also in: linux-devicetree, linux-samsung-soc, linux-spi

On Fri, 18 May 2012 15:03:33 +0530, Thomas Abraham [off-list ref] wrote:
Add support for device based discovery.

Signed-off-by: Thomas Abraham <redacted>
Acked-by: Jaswinder Singh <redacted>
Acked-by: Grant Likely <redacted>

... but one nitpick below...
+#ifdef CONFIG_OF
+static const struct of_device_id s3c64xx_spi_dt_match[] = {
+	{ .compatible = "samsung,exynos4210-spi",
+			.data = (void *)EXYNOS4_SPI_PORT_CONFIG,
This looks completely backwards.  EXYNOS4_SPI_PORT_CONFIG casts a
pointer to a kernel_ulong_t, and this casts it right back to a
pointer.  The cast should be removed entirely from the macro, and the
platform_device_id table should have the cast from pointer to ulong.

g.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help