Re: [Xen-devel] [PATCH v2 09/23] acpi/hvmloader: Build WAET optionally

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 09.08.16 at 17:13, <boris.ostrovsky@xxxxxxxxxx> wrote:
> On 08/09/2016 10:48 AM, Jan Beulich wrote:
>>>>> On 09.08.16 at 15:51, <boris.ostrovsky@xxxxxxxxxx> wrote:
>>> On 08/09/2016 09:29 AM, Jan Beulich wrote:
>>>>>>> On 04.08.16 at 23:06, <boris.ostrovsky@xxxxxxxxxx> wrote:
>>>>> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>
>>>>> Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
>>>>> ---
>>>>> v2:
>>>>> * Note: didn't break "if ( !waet ) return -1" line since that's the style
>>>>>   of the file.
>>>> Style of the file? At best there's a mix in the file, and hence correcting
>>>> it here is rather desirable.
>>> Style of construct_secondary_tables(), not the whole file, sorry.
>> Anyway, if you're not going to make the adjustment, and if I end
>> up committing the patches (and remember at that time), I will
>> make it nevertheless.
> 
> If you decide to go ahead with committing --- sure. Otherwise I can do
> this in the next submission.

Well, it's only going to be the first 2 or 3 patches that I can commit,
as subsequent ones have pending adjustment requests.

Jan


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