linux-stable.git/drivers/base/core.c, branch linux-2.6.27.y

Driver core: Clarify device cleanup.

2008-10-22T21:21:03+00:00

commit 5739411acbaa63a6c22c91e340fdcdbcc7d82a51 upstream

Make the comments on how to use device_initialize(), device_add()
and device_register() a bit clearer - in particular, explicitly
note that put_device() must be used once we tried to add the device
to the hierarchy.

Signed-off-by: Cornelia Huck 
Signed-off-by: Greg Kroah-Hartman

Driver core: Fix cleanup in device_create_vargs().

2008-10-22T21:21:01+00:00

commit 286661b3777897220ecfcd774bccc68a34667f39 upstream

If device_register() in device_create_vargs() fails, the device
must be cleaned up with put_device() (which is also fine on NULL)
instead of kfree().

Signed-off-by: Cornelia Huck 
Signed-off-by: Greg Kroah-Hartman

driver core: add init_name to struct device

2008-08-21T17:15:37+00:00

This gives us a way to handle both the bus_id and init_name values being
used for a while during the transition period.

Cc: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman

PM: don't skip device PM init when CONFIG_PM_SLEEP isn't set and CONFIG_PM is set

2008-08-21T17:15:36+00:00

This patch (as1124) fixes a couple of bugs in the PM core.  The new
dev->power.status field should be initialized regardless of whether
CONFIG_PM_SLEEP is enabled, and similarly dpm_sysfs_add() should be
called whenever CONFIG_PM is enabled.

The patch separates out the call to dpm_sysfs_add() from the call to
device_pm_add().  As a result device_pm_add() can no longer return an
error, so its return type is changed to void.

Signed-off-by: Alan Stern 
Tested-by: Romit Dasgupta 
Acked-by: Rafael J. Wysocki 
Signed-off-by: Greg Kroah-Hartman

dev_printk(): constify the `dev' argument

2008-08-21T17:15:36+00:00

Add const markings to dev_name and dev_driver_string to make it clear that
dev_printk doesn't modify dev.  This is a prerequisite to adding more
const markings to other functions make it clearer, which functions can
modify dev and which can't.

Signed-off-by: Jean Delvare 
Cc: Kay Sievers 
Signed-off-by: Andrew Morton 
Signed-off-by: Greg Kroah-Hartman

Use WARN() in drivers/base/

2008-07-26T19:00:07+00:00

Use WARN() instead of a printk+WARN_ON() pair; this way the message
becomes part of the warning section for better reporting/collection.

Signed-off-by: Arjan van de Ven 
Cc: Greg KH 
Cc: Kay Sievers 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

driver core: Suppress sysfs warnings for device_rename().

2008-07-22T04:55:01+00:00

driver core: Suppress sysfs warnings for device_rename().

Renaming network devices to an already existing name is not
something we want sysfs to print a scary warning for, since the
callers can deal with this correctly. So let's introduce
sysfs_create_link_nowarn() which gets rid of the common warning.

Signed-off-by: Cornelia Huck 
Signed-off-by: Greg Kroah-Hartman

class: change internal semaphore to a mutex

2008-07-22T04:54:52+00:00

Now that the lockdep infrastructure in the class core is in place, we
should be able to properly change the internal class semaphore to be a
mutex.

David wrote the original patch, and Greg fixed it up to apply properly
due to all of the recent changes in this area.

From: Dave Young 
Cc: Matthew Wilcox 
Cc: Kay Sievers 
Cc: Andrew Morton 
Cc: James Bottomley 
Signed-off-by: Greg Kroah-Hartman

class: rename "sem" to "class_sem" in internal class structure

2008-07-22T04:54:52+00:00

This renames the struct class "sem" field to be "class_sem" to make
things easier when struct bus_type and struct class merge in the future.
It also makes grepping for fields easier as well.

Based on an idea from Kay.

Cc: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman

class: rename "subsys" to "class_subsys" in internal class structure

2008-07-22T04:54:52+00:00

This renames the struct class "subsys" field to be "class_subsys" to
make things easier when struct bus_type and struct class merge in the
future.  It also makes grepping for fields easier as well.

Based on an idea from Kay.

Cc: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman