Re: [PATCH v6 2/5] rust: pci: provide access to PCI Vendor values

From: John Hubbard
Date: Mon Aug 25 2025 - 22:20:06 EST


On 8/25/25 5:33 AM, Alexandre Courbot wrote:
>> +/// Once constructed, a `Vendor` contains a valid PCI Vendor ID.
>> +impl Vendor {
>> + /// Create a new Vendor from a raw 16-bit vendor ID.
>
> The argument is 32-bit. :) Which triggers the question: why store these
> as u32 if a u16 is the right size?

Good point! I'll change it to u16.


thanks,
--
John Hubbard