![Zhang Rui](/assets/img/avatar_default.png)
Well, first of all, I don't want to change so many files either. What I do: Adding a new parameter "struct bin_attribute *" in the .read/.write methods for the sysfs binary attributes. In fact, only the four lines change in fs/sysfs/bin.c and include/linux/sysfs.h do the real work. But I have to update all the files that use binary attributes to make them compatible with the new .read and .write methods. I'm not sure if I missed any. :( Why I do this: For a sysfs attribute, we can get a pointer pointing to the struct attribute in the .show/.store method, while we can't do this for the binary attributes. I don't know why this is different, but this does make it not so handy to use the binary attributes as the regular ones. So I think this patch is reasonable. :) Who benefits from it: The patch that exposes ACPI tables in sysfs requires such an improvement. All the table binary attributes share the same .read method. Parameter "struct bin_attribute *" is used to get the table signature and instance number which are used to distinguish different ACPI table binary attributes. Without this parameter, we need to offer different .read methods for different ACPI table binary attributes. This is impossible as there are various ACPI tables on different platforms, and we don't know what they are until they are loaded. Signed-off-by: Zhang Rui <rui.zhang@intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
101 lines
2.7 KiB
C
101 lines
2.7 KiB
C
/*
|
|
* File Attributes for Zorro Devices
|
|
*
|
|
* Copyright (C) 2003 Geert Uytterhoeven
|
|
*
|
|
* Loosely based on drivers/pci/pci-sysfs.c
|
|
*
|
|
* This file is subject to the terms and conditions of the GNU General Public
|
|
* License. See the file COPYING in the main directory of this archive
|
|
* for more details.
|
|
*/
|
|
|
|
|
|
#include <linux/kernel.h>
|
|
#include <linux/zorro.h>
|
|
#include <linux/stat.h>
|
|
#include <linux/string.h>
|
|
|
|
#include "zorro.h"
|
|
|
|
|
|
/* show configuration fields */
|
|
#define zorro_config_attr(name, field, format_string) \
|
|
static ssize_t \
|
|
show_##name(struct device *dev, struct device_attribute *attr, char *buf) \
|
|
{ \
|
|
struct zorro_dev *z; \
|
|
\
|
|
z = to_zorro_dev(dev); \
|
|
return sprintf(buf, format_string, z->field); \
|
|
} \
|
|
static DEVICE_ATTR(name, S_IRUGO, show_##name, NULL);
|
|
|
|
zorro_config_attr(id, id, "0x%08x\n");
|
|
zorro_config_attr(type, rom.er_Type, "0x%02x\n");
|
|
zorro_config_attr(serial, rom.er_SerialNumber, "0x%08x\n");
|
|
zorro_config_attr(slotaddr, slotaddr, "0x%04x\n");
|
|
zorro_config_attr(slotsize, slotsize, "0x%04x\n");
|
|
|
|
static ssize_t zorro_show_resource(struct device *dev, struct device_attribute *attr, char *buf)
|
|
{
|
|
struct zorro_dev *z = to_zorro_dev(dev);
|
|
|
|
return sprintf(buf, "0x%08lx 0x%08lx 0x%08lx\n",
|
|
(unsigned long)zorro_resource_start(z),
|
|
(unsigned long)zorro_resource_end(z),
|
|
zorro_resource_flags(z));
|
|
}
|
|
|
|
static DEVICE_ATTR(resource, S_IRUGO, zorro_show_resource, NULL);
|
|
|
|
static ssize_t zorro_read_config(struct kobject *kobj,
|
|
struct bin_attribute *bin_attr,
|
|
char *buf, loff_t off, size_t count)
|
|
{
|
|
struct zorro_dev *z = to_zorro_dev(container_of(kobj, struct device,
|
|
kobj));
|
|
struct ConfigDev cd;
|
|
unsigned int size = sizeof(cd);
|
|
|
|
if (off > size)
|
|
return 0;
|
|
if (off+count > size)
|
|
count = size-off;
|
|
|
|
/* Construct a ConfigDev */
|
|
memset(&cd, 0, sizeof(cd));
|
|
cd.cd_Rom = z->rom;
|
|
cd.cd_SlotAddr = z->slotaddr;
|
|
cd.cd_SlotSize = z->slotsize;
|
|
cd.cd_BoardAddr = (void *)zorro_resource_start(z);
|
|
cd.cd_BoardSize = zorro_resource_len(z);
|
|
|
|
memcpy(buf, (void *)&cd+off, count);
|
|
return count;
|
|
}
|
|
|
|
static struct bin_attribute zorro_config_attr = {
|
|
.attr = {
|
|
.name = "config",
|
|
.mode = S_IRUGO | S_IWUSR,
|
|
},
|
|
.size = sizeof(struct ConfigDev),
|
|
.read = zorro_read_config,
|
|
};
|
|
|
|
void zorro_create_sysfs_dev_files(struct zorro_dev *z)
|
|
{
|
|
struct device *dev = &z->dev;
|
|
|
|
/* current configuration's attributes */
|
|
device_create_file(dev, &dev_attr_id);
|
|
device_create_file(dev, &dev_attr_type);
|
|
device_create_file(dev, &dev_attr_serial);
|
|
device_create_file(dev, &dev_attr_slotaddr);
|
|
device_create_file(dev, &dev_attr_slotsize);
|
|
device_create_file(dev, &dev_attr_resource);
|
|
sysfs_create_bin_file(&dev->kobj, &zorro_config_attr);
|
|
}
|
|
|