lkml.org 
[lkml]   [2015]   [Jun]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2 1/2] vTPM: support little endian guests
From
Date
Hi Ashley,

Ah, good catch. I think I can only join the first two lines (where the
assignments are) and will have to leave the rest splitted. I'll resubmit
this one soon.

Thanks for the review!


Vicky
On Tue, 2015-06-16 at 20:17 -0500, Ashley Lai wrote:
> Just a small comment otherwise it looks good.
>
> On Tue, 9 Jun 2015, Hon Ching(Vicky) Lo wrote:
> > case NONHOST_INFO:
> > - name = tcpa_event_type_strings[event->event_type];
> > + name =
> > + tcpa_event_type_strings[do_endian_conversion
> > + (event->event_type)];
> Not being picky but if it does not exceed 80 characters it looks better
> to join the line above.
> name = tcpa_event_type_strings[do_endian_conversion
> (event->event_type)];
>
> > case POST_CONTENTS:
> > - name = tcpa_pc_event_id_strings[pc_event->event_id];
> > + name =
> > + tcpa_pc_event_id_strings[do_endian_conversion
> > + (pc_event->event_id)];
> Same as above.
>
> Thanks,
> --Ashley Lai
>




\
 
 \ /
  Last update: 2015-06-17 23:41    [W:0.053 / U:1.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site