Re: [PATCH] Update USB/IP OP_REP_IMPORT documentation.

[Gordon -]

On Tue, Dec 24, 2024 at 1:11 AM Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx> wrote:
>
> On 12/23/24 02:04, Gordon Ou wrote:
> > Corrects byte offsets for OP_REP_IMPORT.
>
> Add more information on why this change is needed.
>

The change is needed to correct the mistaken byte offset of the field
bDeviceClass in OP_REP_IMPORT documentation. The previous field bcdDevice
has length 2 and the offset for bDeviceClass should be 0x138 + 2 = 0x13A
instead of 0x139. Offsets for subsequent fields are also affected and fixed
in this patch.

> >
> > Signed-off-by: Gordon Ou <gordon.xwj@xxxxxxxxx>
> > ---
> >   Documentation/usb/usbip_protocol.rst | 12 ++++++------
> >   1 file changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/Documentation/usb/usbip_protocol.rst b/Documentation/usb/usbip_protocol.rst
> > index adc158967cc6..3da1df3d94f5 100644
> > --- a/Documentation/usb/usbip_protocol.rst
> > +++ b/Documentation/usb/usbip_protocol.rst
> > @@ -285,17 +285,17 @@ OP_REP_IMPORT:
> >   +-----------+--------+------------+---------------------------------------------------+
> >   | 0x138     | 2      |            | bcdDevice                                         |
> >   +-----------+--------+------------+---------------------------------------------------+
> > -| 0x139     | 1      |            | bDeviceClass                                      |
> > +| 0x13A     | 1      |            | bDeviceClass                                      |
> >   +-----------+--------+------------+---------------------------------------------------+
> > -| 0x13A     | 1      |            | bDeviceSubClass                                   |
> > +| 0x13B     | 1      |            | bDeviceSubClass                                   |
> >   +-----------+--------+------------+---------------------------------------------------+
> > -| 0x13B     | 1      |            | bDeviceProtocol                                   |
> > +| 0x13C     | 1      |            | bDeviceProtocol                                   |
> >   +-----------+--------+------------+---------------------------------------------------+
> > -| 0x13C     | 1      |            | bConfigurationValue                               |
> > +| 0x13D     | 1      |            | bConfigurationValue                               |
> >   +-----------+--------+------------+---------------------------------------------------+
> > -| 0x13D     | 1      |            | bNumConfigurations                                |
> > +| 0x13E     | 1      |            | bNumConfigurations                                |
> >   +-----------+--------+------------+---------------------------------------------------+
> > -| 0x13E     | 1      |            | bNumInterfaces                                    |
> > +| 0x13F     | 1      |            | bNumInterfaces                                    |
> >   +-----------+--------+------------+---------------------------------------------------+
> >
> >   The following four commands have a common basic header called
>
> thanks,
> -- Shuah

Thanks for the reply, please find my update inline, and let me know if an
updated patch is needed.

Gordon