On Thu, Jul 26, 2012 at 2:54 AM, Colin Cross <ccr...@android.com> wrote: > On Tue, Jul 24, 2012 at 11:11 PM, Bryan Wu <bryan...@canonical.com> wrote: >> On Wed, Jul 25, 2012 at 8:32 AM, Colin Cross <ccr...@android.com> wrote: >>> Some triggers create sysfs files when they are enabled. Send a uevent >>> "change" notification whenever the trigger is changed to allow userspace >>> processes such as udev to modify permissions on the new files. >>> >> >> This looks like an workaround only for led trigger, can we fix this in >> sysfs level? > > See the previous discussion here: https://lkml.org/lkml/2012/7/20/458
Thanks, I went through this thread here. Actually it was archived in my email account, so I missed that during a trip. Basically, I think this issue is a kind of general issue related to sysfs, not just only for led trigger system. And adding this uevent notification to a upper level LED driver is not good to me, if we got similar issue in other subsystem, we should add similar fix there. Why not we add this in sysfs when we call device_create_file(). And this will be benefit for other drivers. Please point out me why we can't do that in sysfs level. Thanks. -- Bryan Wu <bryan...@canonical.com> Kernel Developer +86.186-168-78255 Mobile Canonical Ltd. www.canonical.com Ubuntu - Linux for human beings | www.ubuntu.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/