Subject: Re: [PATCH v3 1/5] Docs: usb: update usb_bulk_msg receiving example
To:     Philipp Hortmann <philipp.g.hortmann@gmail.com>, corbet@lwn.net,
        linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
CC:     linux-usb@vger.kernel.org, gregkh@linuxfoundation.org
References: <cover.1638771720.git.philipp.g.hortmann@gmail.com>
 <3b794ef1936eb410b60cb536e47a0a00e36611d4.1638771720.git.philipp.g.hortmann@gmail.com>
From:   Oliver Neukum <oneukum@suse.com>
Message-ID: <ec1a30b7-941c-331a-fbc1-02b907cb5788@suse.com>
Date:   Tue, 7 Dec 2021 09:55:54 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.12.0
In-Reply-To: <3b794ef1936eb410b60cb536e47a0a00e36611d4.1638771720.git.philipp.g.hortmann@gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Content-Language: en-US
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Wg73pipS89jfgStU6YPAkbVvkSzVc12ey8ZnQD1wZug2KpdgRpAS6KYS/rmI?=
 =?us-ascii?Q?rdwZcyjbw3Y2d95F3JAk4RHocQq6IP8YdtX7WdhiInOZc5BqNnpWpv1kUm8W?=
 =?us-ascii?Q?eHNIo6W8Kh09KE/Sc2MX2jdmW9vkrwdX6c2mKIC1CJxDINJQdtFmeW1j8W5I?=
 =?us-ascii?Q?oFBW2CrtLdR7szeKuM21xv/cd9edjVn9n23BlDlYj+JAGBxf/Q0OY6vdYeDv?=
 =?us-ascii?Q?jQVfiUUFmaeSn78k/ZHV7nMEKXNAHcCDa5BBs2OqHw9dTfZFMXdAUGH1vc+n?=
 =?us-ascii?Q?9065p67WfYxnC1eH3g3a3p3E44++nyb0Prx2Qyx1EC+ZTWAMJE3AOA46WDYh?=
 =?us-ascii?Q?aT/r0HmNY6oOwbVgysWW6YPiVnuj6XZlXf11xLMqCCAGBLePmu2j0jDBYJIU?=
 =?us-ascii?Q?qw1AxgaNzVduLmyyHmxclGo/xWnGgtTg0DYvR8lpYf1yJkCDjHMohpnZbVsu?=
 =?us-ascii?Q?cJpGhaWWgIkyjMo0LmTDoYQdb7kWXABf5gigK15P8wpUsWut9Cfun224jaGB?=
 =?us-ascii?Q?2yOxJpfvgBzOoUzENFZpiUsNTbXEFsktAVhsUqcDBj2TfF5Z2y3Y1s4P1G+o?=
 =?us-ascii?Q?HiHFDx/ltM3HN7VdQPjP67RIZ80oVbiZciBtyPLeoOkGxHHSzOUP8D9aoLWZ?=
 =?us-ascii?Q?V8SPSBhD4oV318jrTIBP3t1zTBZToHVg039IsDnq8oSYkexd8YIg81v1nk9+?=
 =?us-ascii?Q?TaB+afoHzfxwDXcDucvUr73vcSLoRiUziqSSWfiy/R7FaD9i+uPG7rtxtodM?=
 =?us-ascii?Q?A7bvG0MjwUJ2YCfIR3MhMBaqOcZZIdgP6h6/LhQJxSIP+dGChNA6lAfmBiGm?=
 =?us-ascii?Q?v4Bjf95orBYiYpOo0Np77U40uIxSpPsbvscG9FtTvhRUxqkko+eEr36KP0tu?=
 =?us-ascii?Q?BZzuvoiaY9LPWpTG9iwMLm/cZhLqDzsMz4Bmvd6ZkNnmbb/GD+VNq+OH2Vqn?=
 =?us-ascii?Q?ocGZhiru1rf5KvA5wH+BG//6Z6eVXjhX7anRf2vtjTyuugrP546YCa4omNFk?=
 =?us-ascii?Q?7xT7g6w/mYncwx45y5bUh1Tv5kp7Y88+vtRNCww2mJaNDRs07kUzKkv2NJjd?=
 =?us-ascii?Q?tJSTwgFlbPFtg7FWaePSJ/FU4QkGVBtsdlmZ33JpGw4kFsVs6O7bb1y2CmY2?=
 =?us-ascii?Q?3Pfcd7Ul3ti49qPWzqAudHmUkxXw7TVVGx+M3ufqyOsl23PW0NwHGA+FWUTm?=
 =?us-ascii?Q?EM/byKT+hy6QIaz4+7033xcIToS06YTQxZM3BBhRTGbJwjptCEY5QBCmg7tW?=
 =?us-ascii?Q?Aljrrn+deAbjtY4Dqkpkg1WTQ1pbg0wlPW7h3Lwk3p70DOlCDeJnOp5DqAhX?=
 =?us-ascii?Q?HGaraublOaX9TcdaYG+jzHjznW2bPAfVkXMOX3j9/+VCxMBUXr/Wm0GV7tuy?=
 =?us-ascii?Q?D7B1FZKsR9C5xu1a/VnZ0DvTNN6YR+yWop18ZXXm572b/37yA/WI2Ya/5eww?=
 =?us-ascii?Q?9EitgJ4muQTSlaaXKqWxPtQLsJ2L72KDIdkOf/7CpqbSS9UtTakoRBajCYt5?=
 =?us-ascii?Q?KM+6ism9+iMKMlZbBpjxcBmLG5euJHF3/ivYpK6MZUUGo4gJGlL71IN2il2R?=
 =?us-ascii?Q?UPYbufecFEeWM3RYe5Q8JGriGnp876P0svCQHgVn/XDg9nx7JqrGg0qP977s?=
 =?us-ascii?Q?08tn2p1ry88YSg06Cp2rQZpJTFCe5GhmuWloTsXzii7zH69zqhAxvnIfVAGg?=
 =?us-ascii?Q?eunoUpKpchputQYwd1XgXJy8J+Q=3D?=
X-OriginatorOrg: suse.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 14172f4d-1524-4d91-6df8-08d9b95f6219
X-MS-Exchange-CrossTenant-AuthSource: DB7PR04MB5050.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Dec 2021 08:55:57.0040
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: o1CACzwSkKfPKqei1X4gs9YbQmtH4KXOUvboCUtaNSnr821c/Jk800k6bC/GJpDngo0apF2SYDDKpfrdxrQ3RQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB3PR0402MB3882
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 06.12.21 21:57, Philipp Hortmann wrote:
>  The :c:func:`usb_bulk_msg` function can be very useful for doing single =
reads
>  or writes to a device; however, if you need to read or write constantly =
to
>  a device, it is recommended to set up your own urbs and submit them to
> -the USB subsystem.
> +the USB subsystem. The template uses urbs for read and write.

Hi,

now that I read this, shouldn't we tell the reader why exactly the use
of URBs is a good idea at that place?

=C2=A0=C2=A0=C2=A0 Regards
=C2=A0=C2=A0=C2=A0 =C2=A0=C2=A0=C2=A0 Oliver