Em Fri, 28 Oct 2016 17:57:29 -0400 Greg Kroah-Hartman <gre...@linuxfoundation.org> escreveu:
> On Fri, Oct 28, 2016 at 11:33:51AM -0200, Mauro Carvalho Chehab wrote: > > Em Fri, 28 Oct 2016 08:49:12 -0400 > > Greg Kroah-Hartman <gre...@linuxfoundation.org> escreveu: > > > > > On Fri, Oct 28, 2016 at 08:31:57AM -0400, Greg Kroah-Hartman wrote: > > > > On Fri, Oct 28, 2016 at 10:19:47AM -0200, Mauro Carvalho Chehab wrote: > > > > > > > > > Use a script to parse the Documenation/ABI directory and output > > > > > it at the admin-guide. > > > > > > > > > > Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> > > > > > --- > > > > > Documentation/admin-guide/abi.rst | 5 + > > > > > Documentation/admin-guide/index.rst | 1 + > > > > > Documentation/sphinx/abi_book.pl | 208 > > > > > ++++++++++++++++++++++++++++++++++++ > > > > > 3 files changed, 214 insertions(+) > > > > > create mode 100644 Documentation/admin-guide/abi.rst > > > > > create mode 100755 Documentation/sphinx/abi_book.pl > > > > > > > > Any hint as to what the output of this actually looks like? > > > > > > Doh, you already showed it in patch 0/3, that's what I get for never > > > reading cover letters... > > > > :-) > > > > > sorry for the noise, > > > > No problem. > > > > Btw, looking at the output of the script, I noticed a > > few files where "What" doesn't point to configfs/sysfs/procfs: > > > > - Documentation/ABI/testing/sysfs-class-rtc-rtc0-device-rtc_calibration > > What: Attribute for calibrating ST-Ericsson AB8500 Real Time > > Clock > > > > This is probably a sysfs interface, so, IMHO, "What" here is wrong. > > > > From the name of the file, I suspect that it should be, instead: > > What: /sysfs/class/rtc/rtc0/device/rtc_calibration > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > Looks right, but the original authors would know more. Patch sent, c/c authors/maintainers of the driver. > > > - Documentation/ABI/stable/thermal-notification > > What: A notification mechanism for thermal related events > > > > Not sure what kind of interface this is referring to. Perhaps a > > sysctl interface? > > It's a netlink api, odd, haven't seen that before. > > - Documentation/ABI/stable/syscalls > > What: The kernel syscall interface > > > > This sounds OK, although it probably makes sense to document > > each system call. > > Heh, that's the whole man page project, I don't think we want to suck > that into the kernel just yet :) Perhaps we could prefix those "What:" entries that doesn't start with a "/" with something else. The issue with a random string at What: is that, when sorting the entries before output the database, they'll be after all others that start with "/". > > - Documentation/ABI/testing/sysfs-devices-system-ibm-rtl > > What: state > > What: version > > > > According with their descriptions, both are needed for the > > "ibm-prtm" userspace daemon, but it provides no glue where > > this should be located. Looking at its source file, it seems > > that they should be located at [1]: > > > > /sys/devices/system/ibm_rtl/state > > and > > /proc/version > > > > Btw, IMHO, it doesn't make much sense to document /proc/version > > inside sysfs-devices-system-ibm-rtl. > > That might be a sysfs version file, not /proc/version, as it doesn't > look like /proc/version is described that way. Yeah, you're right: the driver also defines a "version". Weird enough, the daemon meant to use this driver seems to look only at /proc/version. Anyway, patch sent, c/c authors/maintainers of the driver. > > thanks, > > greg k-h Thanks, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
