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

[Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>]

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.

(I actually backed out this change at the last moment before sending the
series when I noticed routine style)

-boris


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