lkml.org 
[lkml]   [2021]   [Oct]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v5 4/4] tools: Introduce power/acpi/pfru/pfru
On Fri, Oct 22, 2021 at 11:12:20PM +0300, Andy Shevchenko wrote:
> On Sat, Oct 23, 2021 at 01:10:32AM +0800, Chen Yu wrote:
> > Introduce a user space tool to make use of the interface exposed by
> > Platform Firmware Runtime Update and Telemetry drivers. The users
> > can use this tool to do firmware code injection, driver update and
> > to retrieve the telemetry data.
>
> ...
>
> > #include <linux/ioctl.h>
> > +#ifdef __KERNEL__
> > #include <linux/uuid.h>
> > +#else
> > +#include <uuid/uuid.h>
> > +#include <linux/types.h>
> > +#endif
>
> This is exactly my point why you mustn't use uuid.h in uAPI headers.
>
Ok, will remove uuid.h.
> > + while (1) {
>
> Why not while ((c = getopt_long(...)) != -1) ?
>
Ok, changed it to this form.
> > + int option_index = 0;
> > +
> > + c = getopt_long(argc, argv, option_string,
> > + long_options, &option_index);
> > + if (c == -1)
> > + break;
>
> > + }
> > + }
> > +}
>
> ...
>
> > + char *uuid = malloc(37);
>
> libuuid doesn't have helpers for this?
>
There's no helper for uuid allocation AFAIK.
> Split assignment so if will be closer to its user.
>
Ok, will do.

thanks,
Chenyu

\
 
 \ /
  Last update: 2021-10-23 15:19    [W:0.057 / U:23.376 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site