[andrew.patterson@hp.com: [PATCH v6] cciss: add cciss driver sysfs entries]

From: Mike Miller (OS Dev)
Date: Fri May 29 2009 - 10:55:25 EST

I fat fingered Andrew & Jens' addresses. Sorry for nay duplicates.
----- Forwarded message from Andrew Patterson <andrew.patterson@xxxxxx> -----

Date: Mon, 13 Apr 2009 10:45:25 -0600
From: Andrew Patterson <andrew.patterson@xxxxxx>
To: mikem@xxxxxxxxxxxxxxxxxxxxxxx
Cc: gerry.morong@xxxxxx, andrew.patterson@xxxxxx,
scameron@xxxxxxxxxxxxxxxxxxxxxxx, scott.benesh@xxxxxx
Subject: [PATCH v6] cciss: add cciss driver sysfs entries

This is version six of my patch to add sysfs entries to the cciss driver.


Version 3
Got rid of /sys/class/cciss_* entries. Now only show vendor, model,
rev, and unique_id at /sys/bus/pci/devices/ccissX/cXdY. Where X is the
controller nubmer and Y is the logical drive number.
Fixed a double memcpy of the serial number.
Documented new sysfs entries in
Fixed tabs in Documentation/ABI/testing/sysfs-bus-pci-devices-cciss
Removed unneeded device_attribute release functions
Fixed pahole-reported holes.

Version 4
Moved cciss_sysfs.c into cciss.c
Added cciss_bus_type to logical drive so we get /sys/bus/cciss/device/ccissX.

Version 5
Added cciss_bus_type to hba so we get /sys/bus/cciss/device/cYdZ.

Version 6

Use "sizeof() + 1" instead of "*LEN + 2" for *_show() routines.
Added back in ABI docs.
The following patch adds sysfs entries for the cciss driver. It adds
vendor, model, rev, and unique_id under /sys/block/cciss!c#d#/device
needed for RedHat BZ:


This patch is based off os linus's 2.6 tree (2.6.30-rc1).

.../ABI/testing/sysfs-bus-pci-devices-cciss | 33 ++
drivers/block/cciss.c | 267 ++++++++++++++++++++
drivers/block/cciss.h | 24 +-
3 files changed, 314 insertions(+), 10 deletions(-)
create mode 100644 Documentation/ABI/testing/sysfs-bus-pci-devices-cciss

Andrew Patterson

----- End forwarded message -----
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/