Skip to content

Commit

Permalink
Driver core: Fix up build when CONFIG_BLOCK=N
Browse files Browse the repository at this point in the history
This fixes up the driver core build errors when CONFIG_BLOCK=N

Thanks to Alexander van Heukelum <heukelum@mailshack.com> for the basis
of this patch, and to Jeremy Fitzhardinge <jeremy@goop.org> for
reporting the problem.


Cc: Alexander van Heukelum <heukelum@mailshack.com>
Cc: Jeremy Fitzhardinge <jeremy@goop.org>
Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Greg Kroah-Hartman committed Jan 27, 2008
1 parent 9152530 commit 4e886c2
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 12 deletions.
2 changes: 1 addition & 1 deletion drivers/base/class.c
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ int class_register(struct class *cls)
if (error)
return error;

#ifdef CONFIG_SYSFS_DEPRECATED
#if defined(CONFIG_SYSFS_DEPRECATED) && defined(CONFIG_BLOCK)
/* let the block class directory show up in the root of sysfs */
if (cls != &block_class)
cls->subsys.kobj.kset = class_kset;
Expand Down
30 changes: 19 additions & 11 deletions drivers/base/core.c
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,17 @@
int (*platform_notify)(struct device *dev) = NULL;
int (*platform_notify_remove)(struct device *dev) = NULL;

/*
* sysfs bindings for devices.
*/
#ifdef CONFIG_BLOCK
static inline int device_is_not_partition(struct device *dev)
{
return !(dev->type == &part_type);
}
#else
static inline int device_is_not_partition(struct device *dev)
{
return 1;
}
#endif

/**
* dev_driver_string - Return a device's driver name, if at all possible
Expand Down Expand Up @@ -652,14 +660,14 @@ static int device_add_class_symlinks(struct device *dev)
#ifdef CONFIG_SYSFS_DEPRECATED
/* stacked class devices need a symlink in the class directory */
if (dev->kobj.parent != &dev->class->subsys.kobj &&
dev->type != &part_type) {
device_is_not_partition(dev)) {
error = sysfs_create_link(&dev->class->subsys.kobj, &dev->kobj,
dev->bus_id);
if (error)
goto out_subsys;
}

if (dev->parent && dev->type != &part_type) {
if (dev->parent && device_is_not_partition(dev)) {
struct device *parent = dev->parent;
char *class_name;

Expand Down Expand Up @@ -688,11 +696,11 @@ static int device_add_class_symlinks(struct device *dev)
return 0;

out_device:
if (dev->parent && dev->type != &part_type)
if (dev->parent && device_is_not_partition(dev))
sysfs_remove_link(&dev->kobj, "device");
out_busid:
if (dev->kobj.parent != &dev->class->subsys.kobj &&
dev->type != &part_type)
device_is_not_partition(dev))
sysfs_remove_link(&dev->class->subsys.kobj, dev->bus_id);
#else
/* link in the class directory pointing to the device */
Expand All @@ -701,7 +709,7 @@ static int device_add_class_symlinks(struct device *dev)
if (error)
goto out_subsys;

if (dev->parent && dev->type != &part_type) {
if (dev->parent && device_is_not_partition(dev)) {
error = sysfs_create_link(&dev->kobj, &dev->parent->kobj,
"device");
if (error)
Expand All @@ -725,7 +733,7 @@ static void device_remove_class_symlinks(struct device *dev)
return;

#ifdef CONFIG_SYSFS_DEPRECATED
if (dev->parent && dev->type != &part_type) {
if (dev->parent && device_is_not_partition(dev)) {
char *class_name;

class_name = make_class_name(dev->class->name, &dev->kobj);
Expand All @@ -737,10 +745,10 @@ static void device_remove_class_symlinks(struct device *dev)
}

if (dev->kobj.parent != &dev->class->subsys.kobj &&
dev->type != &part_type)
device_is_not_partition(dev))
sysfs_remove_link(&dev->class->subsys.kobj, dev->bus_id);
#else
if (dev->parent && dev->type != &part_type)
if (dev->parent && device_is_not_partition(dev))
sysfs_remove_link(&dev->kobj, "device");

sysfs_remove_link(&dev->class->subsys.kobj, dev->bus_id);
Expand Down

0 comments on commit 4e886c2

Please sign in to comment.