Re: [PATCH] cxl: docs/devices Fix typos and clarify wording in device-types.rst
From: Dave Jiang
Date: Mon Jun 16 2025 - 18:33:14 EST
On 6/16/25 10:27 AM, Gregory Price wrote:
> On Sun, Jun 15, 2025 at 11:07:32PM -0700, Alok Tiwari wrote:
>> Documentation/driver-api/cxl/devices/device-types.rst | 10 +++++-----
>> 1 file changed, 5 insertions(+), 5 deletions(-)
>>
> ...
>>
>> A Multi-Logical Device (MLD) is a device which may present multiple devices
>> -to one or more devices.
>> +to one or more hosts.
>
> This is subtly incorrect.
>
> A *Multi-Headed* MLD presents one or more devices to one or more hosts.
>
> A *Multi-logical Device* presents one or more devices to one or more
> upstream devices (such as a switch).
>
> You can have a Single-Headed Multi-Logical Device that presents 2
> "Logical Devices" to a single upstream device (host or switch).
>
> So please change "one or more hosts" to "one or more upstream devices".
>
>
> With that change you may add
>
> Reviewed-by: Gregory Price <gourry@xxxxxxxxxx>
I fixed it up with your suggestion and applied to cxl/next. Please check and make sure the fix is correct. Thanks!
>
> ~Gregory