Re: [linux-usb-devel] [PATCH 6/9] isdn4linux: Siemens Gigaset drivers- procfs interface

From: Tilman Schmidt
Date: Wed Dec 14 2005 - 06:36:24 EST


Greg KH wrote:
> Are there existing userspace tools that rely on proc files for isdn
> control? If not, please do not add new proc files, these should be in
> sysfs instead.

Thanks for your comment. We'll move everything to sysfs. Most of it is
done already, but there is a problematic case I'd like your advice on:
the "info" and "hwinfo" entries, which emit several lines of status
and statistics information.

According to Documentation/filesystems/sysfs.txt:
"Mixing types, expressing multiple lines of data, and doing fancy
formatting of data is heavily frowned upon. Doing these things may get
you publically humiliated and your code rewritten without notice."
We certainly don't want to risk that. :-)

We have tentatively created an entry in /proc/tty/driver/ as proposed
in LDD3 chapter 18, section "proc and sysfs Handling of TTY Devices"
as a replacement. But that has the drawback of giving us only a
single file per driver as opposed to one or two files per device.
So the driver has to enumerate the devices and concatenate their data,
which appears cumbersome, even though normal users will probably
never connect more than one of these devices.

Is there a better alternative?

Thanks
Tilman

--
Tilman Schmidt E-Mail: tilman@xxxxxxx
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Ungeöffnet mindestens haltbar bis: (siehe Rückseite)
-
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/