[PATCH v2 1/1] nvme: use macro definitions for setting reservation values

Christoph Hellwig hch at lst.de
Tue Oct 25 08:19:14 PDT 2022


On Mon, Oct 03, 2022 at 01:16:36PM +0300, Max Gurtovoy wrote:
>> I'd make this named nvme_pr_type
>
> Most of the enums of this header are not named.
>
> I don't understand what is the convention here.
>
> Usually, if it's not a new header file I'm trying to keep the file 
> convention and this is what I did.
>
> If all agree on named, I'll send another version.

I think named is better, nvme has some weird unnamed enums that
even combine totally unrelated fields which I always found confusing.



More information about the Linux-nvme mailing list