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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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: [email protected]
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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux