22 Jul, 2008
40 commits
-
debugfs_remove_recursive() will remove a dentry and all its children.
Drivers can use this to zap their whole debugfs tree so that they don't
need to keep track of every single debugfs dentry they created.It may fail to remove the whole tree in certain cases:
sh-3.2# rmmod atmel-mci < /sys/kernel/debug/mmc0/ios/clock
mmc0: card b368 removed
atmel_mci atmel_mci.0: Lost dma0chan1, falling back to PIO
sh-3.2# ls /sys/kernel/debug/mmc0/
iosBut I'm not sure if that case can be handled in any sane manner.
Signed-off-by: Haavard Skinnemoen
Cc: Pierre Ossman
Signed-off-by: Greg Kroah-Hartman -
Signed-off-by: Tsugikazu Shibata
Signed-off-by: Greg Kroah-Hartman -
James Bottomley recently discovered that we have
{request,release}_firmware() dummies for the case of the actual
functions not being available and has a fix for the bug that was
actually causing build errors for built-in users with
CONFIG_FW_LOADER=m.But now missing selects on FW_LOADER are no longer visible at
compile-time at all and can become runtime problems.FW_LOADER is infrastructure with relatively small codesize we can safely
enable for everyone, and only for people who really need small kernels
(and can be expected to know what they are doing) it matters being able
to disable it.This patch therefore always sets FW_LOADER=y and allows users only to
disable it with EMBEDDED=y.As a bonus, we can then get rid of all "select FW_LOADER" plus the due
to it required "depends on HOTPLUG" which removes some complexity from
our Kconfig files.Signed-off-by: Adrian Bunk
Signed-off-by: Greg Kroah-Hartman -
The Userspace I/O HOWTO template sets two different sections with the same
html output name (about.html). This clearly won't work, so change the
first one to a unique "aboutthis.html" to prevent clobbering.Signed-off-by: Mike Frysinger
Acked-by: Hans J. Koch
Cc: Randy Dunlap
Signed-off-by: Andrew Morton
Signed-off-by: Greg Kroah-Hartman -
The Userspace I/O HOWTO document uses straight tags and plain text
to describe copyright/legal information. It should instead use the
and tags like all other documents in the kernel.Signed-off-by: Mike Frysinger
Acked-by: Hans J. Koch
Cc: Randy Dunlap
Signed-off-by: Andrew Morton
Signed-off-by: Greg Kroah-Hartman -
sysfs_chmod_file() calls notify_change() to change the permission bits
on a sysfs file. Replace with explicit call to sysfs_setattr() and
fsnotify_change().This is equivalent, except that security_inode_setattr() is not
called. This function is called by drivers, so the security checks do
not make any sense.Signed-off-by: Miklos Szeredi
Signed-off-by: Greg Kroah-Hartman -
The systdev_class_register() and sysdev_register() functions have
pr_debug() statements which are enabled when the user selects the
driver core debug. Both of these routines do not produce the
correct output, as they make assumptions about data which has not
been initialised.In sysdev_class_register() the code uses the kobject_name(&cls->kset.kobj)
at the start of the function, but this is not setup until later in the
same call. Change this to use cls->name which is passed in from the caller.The sysdev_register() function tries to get the name of the sysdev by
kobject_name(&sysdev->kobj), but that isn't setup until later in the same
function so change this message to use the name of the sysdev's class and
add another message once the name is initialised.Signed-off-by: Ben Dooks
Signed-off-by: Greg Kroah-Hartman -
We should call kobject_put() instead of kfree() if kobject_init_and_add()
returns an error, shouldn't we? Don't set up a bad example ;)Signed-off-by: Li Zefan
Signed-off-by: Greg Kroah-Hartman -
reorder kobject to save space on 64 bit builds.
shrinks from 72 to 64 bytes & moves allocated kobject to a smaller
slab.Signed-off-by: Richard Kennedy
Signed-off-by: Greg Kroah-Hartman -
Signed-off-by: Uwe Kleine-König
Signed-off-by: Hans J. Koch -
This patch adds a generic UIO platform driver. It eliminates the need for a
dedicated kernel module for simple platform devices. Users only need to
implement their irq handler in platform code and fill a struct uio_info
there. This helps avoiding code duplication as UIO platform drivers often
share a lot of common code.Signed-off-by: Uwe Kleine-König
Signed-off-by: Hans J. Koch
Signed-off-by: Greg Kroah-Hartman -
Sometimes it is necessary to enable/disable the interrupt of a UIO device
from the userspace part of the driver. With this patch, the UIO kernel driver
can implement an "irqcontrol()" function that does this. Userspace can write
an s32 value to /dev/uioX (usually 0 or 1 to turn the irq off or on). The
UIO core will then call the driver's irqcontrol function.Signed-off-by: Hans J. Koch
Acked-by: Uwe Kleine-König
Acked-by: Magnus Damm
Signed-off-by: Greg Kroah-Hartman -
ae210f188614bb3d1ee3f19c64e28e3cdd44877c introduced a big "if UIO"/"endif"
where all uio drivers are defined. So know there is no need for them to
depend explicitly on UIO.Signed-off-by: Uwe Kleine-König
Signed-off-by: Hans J. Koch
Signed-off-by: Greg Kroah-Hartman -
Along with the non-modalias conformant "pnp:*" aliases, we add "acpi:*"
entries to PNP drivers, to allow module autoloading by ACPI PNP device
entries, which export proper modalias information, without any specific
userspace modprobe mangling.Cc: Adam Belay
Cc: Thomas Renninger
Signed-off-by: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
We have the dev_printk() variants for this kind of thing, use them
instead of directly trying to access the bus_id field of struct device.This is done in order to remove bus_id entirely.
Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
There is no such thing as a "device id size" in the driver core, so
remove the define and fix up any users of this odd define in the rest of
the kernel.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
There is no such thing as a "device name size" in the driver core, so
remove the define and fix up any users of this odd define in the rest of
the kernel.Signed-off-by: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
Kobjects do not have a limit in name size since a while, so stop
pretending that they do.Signed-off-by: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
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 -
This adds the infrastructure to properly handle lockdep issues when the
internal class semaphore is changed to a mutex.Matthew wrote the original patch, and Greg fixed it up to work properly
with the class_create() function.From: Matthew Wilcox
Cc: Kay Sievers
Cc: Dave Young
Cc: Andrew Morton
Cc: James Bottomley
Cc: Peter Zijlstra
Cc: Ingo Molnar
Signed-off-by: Greg Kroah-Hartman -
Removes a field that has been deleted, and adds a description fo the
class_dirs field which was previously undocumented.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
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 -
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 -
This renames the struct class "interfaces" field to be
"class_interfaces" 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 -
This renames the struct class "devices" field to be "class_devices" 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 -
This moves the portions of struct class that are dynamic (kobject and
lock and lists) out of the main structure and into a dynamic, private,
structure.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
Don't create symlinks in a class to a device that is not owned by the
class. If the bluetooth subsystem really wants to point to all of the
devices it controls, it needs to create real devices, not fake symlinks.Cc: Maxim Krasnyansky
Cc: Kay Sievers
Acked-by: Marcel Holtmann
Signed-off-by: Greg Kroah-Hartman -
This object really should be a struct device, or at least contain a
pointer to a struct device, as it is trying to create a separate device
tree outside of the main device tree. This patch fixes this problem.It is needed for the class core rework that is being done in the driver
core.Cc: Kay Sievers
Cc: Roland Dreier
Cc: Sean Hefty
Cc: Hal Rosenstock
Signed-off-by: Greg Kroah-Hartman -
This pointer really is a struct ib_device, not a struct device, so name
it properly to help prevent confusion.This makes the followon patch in this series much smaller and easier to
understand as well.Cc: Kay Sievers
Cc: Roland Dreier
Cc: Hal Rosenstock
Acked-by: Sean Hefty
Signed-off-by: Greg Kroah-Hartman -
Use the proper class iterator function instead of mucking around in the
internals of the class structures.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
The seq_start call is the better place for the header for the file, that
way we don't have to be mucking in the class structure to try to figure
out if this is the first partition or not.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
Use the proper class iterator function instead of mucking around in the
internals of the class structures.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
These functions are only needed if CONFIG_PROC_FS is enabled, so save
the space when it is not.This also makes it easier for a patch later in this series to work
properly if CONFIG_PROC_FS is not enabled.Signed-off-by: Randy Dunlap
Signed-off-by: Greg Kroah-Hartman -
Use the proper class iterator function instead of mucking around in the
internals of the class structures.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
Use the proper class iterator function instead of mucking around in the
internals of the class structures.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
Warn if something really bad happens if we can't create this link.
Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
This mirrors the functionality that driver_find_device has as well.
We add a start variable, and all callers of the function are fixed up at
the same time.The block layer will be using this new functionality in a follow-on
patch.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
This mirrors the functionality that driver_for_each_device has as well.
We add a start variable, and all callers of the function are fixed up at
the same time.The block layer will be using this new functionality in a follow-on
patch.Cc: Kay Sievers
Signed-off-by: Greg Kroah-Hartman -
Now that device_create() has been audited, rename things back to the
original call to be sane.Keep the device_create_drvdata macro around to make merges easier.
Signed-off-by: Greg Kroah-Hartman
-
There are no more users of this, and it is racy. Use
device_create_drvdata() or device_create_vargs() instead.Signed-off-by: Greg Kroah-Hartman
