RE: [PATCHv2] usb: gadget: serial: replace hardcoded ttyGS with PREFIX

From: David Laight
Date: Mon Jun 30 2014 - 04:42:24 EST


From: Of Richard Leitner
> Replaces all hardcoded ttyGS strings with the PREFIX macro.
> Due to the fact the strings are spread over different source files the
> PREFIX definition is moved to u_serial.h

Lots of changes like:
> - DBG(cdev, "acm ttyGS%d completion, err %d\n",
> - acm->port_num, req->status);
> + DBG(cdev, "acm %s%d completion, err %d\n",
> + PREFIX, acm->port_num, req->status);

I'm not sure this improves the code.
Do you see a need to ever change PREFIX?

Even if it is a reasonable change, the name PREFIX is hopeless.
It would have to be something like ACM_TTYNAME_PREFIX.

David



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/