Hallo Stefan,

* Stefan Monnier <[email protected]> [26-03-09 10:02]:
> > added pwron attr wich is the same as power_on
> 
> Not sure if I understand the code correctly, but does this add a new
> "file" /sys/.../pwron ?  If so, shouldn't it rather be a symlink to
> "power_on"?
Yes, would be better i think ... but i don't know how to do so!

I looked around and found a way to link kobjects but not the entries of 
an kobject.

But i would be a good idea to remove the other created link.. so, here a patch!

Tim Niemeyer
From f3971bf59f909d56291692c6cc096fc2e7e40822 Mon Sep 17 00:00:00 2001
From: Tim Niemeyer <[email protected]>
Date: Fri, 27 Mar 2009 17:26:18 +0100
Subject: [PATCH] remove sysfs-links on remove

Signed-off-by: Tim Niemeyer <[email protected]>
---
 arch/arm/plat-s3c24xx/neo1973_pm_gps.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/plat-s3c24xx/neo1973_pm_gps.c b/arch/arm/plat-s3c24xx/neo1973_pm_gps.c
index b5322f2..93b4911 100644
--- a/arch/arm/plat-s3c24xx/neo1973_pm_gps.c
+++ b/arch/arm/plat-s3c24xx/neo1973_pm_gps.c
@@ -764,6 +764,7 @@ static int gta01_pm_gps_remove(struct platform_device *pdev)
 			regulator_put(neo1973_gps.regulator[i]);
 		}
 #endif
+		bus_remove_device_link(&platform_bus_type, "gta01-pm-gps.0");
 		sysfs_remove_group(&pdev->dev.kobj, &gta01_gps_attr_group);
 	}
 
-- 
1.5.6.5

Attachment: signature.asc
Description: Digital signature

Reply via email to