examples: Refactor device model with version-based split

EricccTaiwan · EricccTaiwan · commit 38a991d303eb · 2025-05-18T17:20:48.000+08:00
The previous implementation of devicemodel_remove used conditional compilation to switch between void and int return types based on the kernel version. While functionally correct, the interleaved `#if/#endif` blocks made the logic harder to follow. This refactor cleanly separates the two function definitions using a full `#if/#else` block around the entire function. No functional behavior is changed. Close: #313 Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
diff --git a/examples/devicemodel.c b/examples/devicemodel.c
@@ -23,19 +23,21 @@ static int devicemodel_probe(struct platform_device *dev)
 
     return 0;
 }
+
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(6, 11, 0)
 static void devicemodel_remove(struct platform_device *dev)
+{
+    pr_info("devicemodel example removed\n");
+    /* Your device removal code */
+}
 #else
 static int devicemodel_remove(struct platform_device *dev)
-#endif
 {
     pr_info("devicemodel example removed\n");
-
     /* Your device removal code */
-#if LINUX_VERSION_CODE < KERNEL_VERSION(6, 11, 0)
     return 0;
-#endif
 }
+#endif
 
 static int devicemodel_suspend(struct device *dev)
 {

Original file line number	Diff line number	Diff line change
`@@ -23,19 +23,21 @@ static int devicemodel_probe(struct platform_device *dev)`
`23`	`23`
`24`	`24`	`return 0;`
`25`	`25`	`}`
	`26`	`+`
`26`	`27`	`#if LINUX_VERSION_CODE >= KERNEL_VERSION(6, 11, 0)`
`27`	`28`	`static void devicemodel_remove(struct platform_device *dev)`
	`29`	`+{`
	`30`	`+ pr_info("devicemodel example removed\n");`
	`31`	`+ /* Your device removal code */`
	`32`	`+}`
`28`	`33`	`#else`
`29`	`34`	`static int devicemodel_remove(struct platform_device *dev)`
`30`		`-#endif`
`31`	`35`	`{`
`32`	`36`	`pr_info("devicemodel example removed\n");`
`33`		`-`
`34`	`37`	`/* Your device removal code */`
`35`		`-#if LINUX_VERSION_CODE < KERNEL_VERSION(6, 11, 0)`
`36`	`38`	`return 0;`
`37`		`-#endif`
`38`	`39`	`}`
	`40`	`+#endif`
`39`	`41`
`40`	`42`	`static int devicemodel_suspend(struct device *dev)`
`41`	`43`	`{`