On Fri, May 26, 2017 at 7:09 PM, Mika Westerberg
<[email protected]> wrote:
> The device DROM contains name of the vendor and device among other
> things. Extract this information and expose it to the userspace via two
> new attributes.
> +static const char *parse_name(const u8 *data, u8 len)
Hmm... (name)
> +{
> + char *name;
> +
> + name = kmemdup(data, len, GFP_KERNEL);
Since it's ASCII by specification it may make sense to use
sw->..._name = kstrndup(entry->data, sizeof(*header), GFP_KERNEL);
if (!sw->..._name)
return -ENOMEM;
just in place, instead of this entire function.
> + if (name)
> + name[len - 1] = '\0';
> + return name;
> +}
> +
> +static int tb_drom_parse_entry_generic(struct tb_switch *sw,
> + struct tb_drom_entry_header *header)
> +{
> + const struct tb_drom_entry_generic *entry =
> + (const struct tb_drom_entry_generic *)header;
> +
> + switch (header->index) {
> + case 1:
> + /* Length includes 2 bytes header so remove it before copy */
> + sw->vendor_name = parse_name(entry->data,
> + header->len - sizeof(*header));
> + if (!sw->vendor_name)
> + return -ENOMEM;
> + break;
> +
> + case 2:
> + sw->device_name = parse_name(entry->data,
> + header->len - sizeof(*header));
> + if (!sw->device_name)
> + return -ENOMEM;
> + break;
> + }
> +
> + return 0;
> +}
--
With Best Regards,
Andy Shevchenko
