On Sat, 2014-07-19 at 09:36 -0700, Greg KH wrote:

> >  /* /sys/devices/platform/visorchipset */
> >  static struct platform_device Visorchipset_platform_device = {
> >     .name = "visorchipset",
> >     .id = -1,
> > +   .dev.groups = visorchipset_dev_groups,
> 
> Only create this device when ControlVm_channel is present in the system,
> that should take out your check for it in the show/store function.
> 
> Same goes for the rest of these patches.
> 
> thanks,
> 
> greg k-h

I'm not sure I understand. Do you mean that we should only create sysfs
devices dynamically, and not use a static here?

I assume that any other sysfs devices in our driver set created this way
would have the same issue?

-- 
Ben Romer | Software Engineer |
Virtual Systems Development 

Unisys Corporation |  2476
Swedesford Rd |  Malvern, PA 19355
|  610-648-7140



_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to