Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 10 Aug 2006 17:51:50 +0400 (MSD)
From:      Dmitry Marakasov <amdmi3@mail.ru>
To:        FreeBSD-gnats-submit@FreeBSD.org
Subject:   usb/101757: [patch] uhid.4: correct structure field names to match dev/usb/usb.h
Message-ID:  <20060810135150.A61D841C2@hades.panopticon>
Resent-Message-ID: <200608101400.k7AE0cHh052822@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help

>Number:         101757
>Category:       usb
>Synopsis:       [patch] uhid.4: correct structure field names to match dev/usb/usb.h
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-usb
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Aug 10 14:00:37 GMT 2006
>Closed-Date:
>Last-Modified:
>Originator:     Dmitry Marakasov
>Release:        FreeBSD 6.1-RELEASE-p3 i386
>Organization:
>Environment:
System: FreeBSD hades.panopticon 6.1-RELEASE-p3 FreeBSD 6.1-RELEASE-p3 #1: Tue Aug 8 17:11:07 MSD 2006 amdmi3@hades.panopticon:/usr/obj/usr/src/sys/HADES i386

>Description:
usb_ctl_report_desc and usb_ctl_report structures are mentioned in
uhid(4), but fields names in the man page differ from actual ones
in the header file sys/usb/usb.h (in man, fields are named size
instead of ucrd_size, data instead of ucrd_data etc.).

This introduces confusion when copypasting field names from man page.

The patch brings man page in sync with header file and also corrects identation.
>How-To-Repeat:
>Fix:

--- uhid.4.patch begins here ---
--- src/share/man/man4/uhid.4.orig	Thu Aug 10 17:41:57 2006
+++ src/share/man/man4/uhid.4	Thu Aug 10 17:43:30 2006
@@ -65,8 +65,8 @@
 without any processing.
 .Bd -literal
 struct usb_ctl_report_desc {
-    int     size;
-    u_char  data[1024];	/* filled data size will vary */
+	int	ucrd_size;
+	u_char	ucrd_data[1024];	/* filled data size will vary */
 };
 .Ed
 .It Dv USB_SET_IMMED Pq Vt int
@@ -94,8 +94,8 @@
 This call may fail if the device does not support this feature.
 .Bd -literal
 struct usb_ctl_report {
-	int report;
-	u_char	data[1024];	/* used data size will vary */
+	int	ucr_report;
+	u_char	ucr_data[1024];	/* used data size will vary */
 };
 .Ed
 .It Dv USB_SET_REPORT Pq Vt "struct usb_ctl_report"
--- uhid.4.patch ends here ---
>Release-Note:
>Audit-Trail:
>Unformatted:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060810135150.A61D841C2>