Re: [PATCH] drivers: block: save return value of pci_find_capability() in u8

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Bjorn,
On 12/6/20 17:26, Bjorn Helgaas wrote:
> There's a patch pending via the PCI tree to change the return type to
> u8.  We can do one of:
I did not know about the pending patch, if that is going to change the
return

type then this patch makes sense.

>
>   - Ignore this.  It only changes something on the stack, so no real
>     space saving and there's no problem assigning the u8 return value
>     to the "int".
>   - The maintainer could ack it and I could merge it via the PCI tree
>     so it happens in the correct order (after the interface change).
If we want it in 5.11 then I think PCI tree is a right way go about it.
>   - The PCI core interface change will be merged for v5.11, so we
>     could hold this until v5.12.
> I don't really have a preference.  The only place there would really
> be a benefit would be if we store the return value in a struct, where
> we could potentially save three bytes.
Totally agree.
> Bjorn

Whoever is going to apply please add :-

Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@xxxxxxx>






[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux