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

Re: [Xen-devel] [PATCH v2] tools/firmware: add ACPI device for Windows laptop/slate mode switch



>>> On 24.03.17 at 12:04, <Paul.Durrant@xxxxxxxxxx> wrote:
>> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
>> Sent: 24 March 2017 10:53
>> The only other concern I have here is that the abbreviation "conv"
>> used throughout the patch is sort of ambiguous. I think it means
>> "convertible" here, but without knowing the context it could easily
>> be "conventional" or "convenience". Would there be anything
>> wrong with spelling it out wherever name length limitations don't
>> require it to be just four characters?
>> 
> 
> I thought that name would be ok since it is the name of the ACPI device 
> itself but I could extend the xl.cfg option, bool and flag names to 
> 'acpi_convert' to make it more obvious.

Hmm, "convert" still leaves room for speculation (in particular, seeing
an option with this name, I'd suspect it wants to convert ACPI to
something else). Is that what "conv" stands for (and not "convertible"
or anything else)?

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.