At Tue, 20 Sep 2011 13:19:14 +0100, Mark Brown wrote:
On Tue, Sep 20, 2011 at 02:12:15PM +0200, Takashi Iwai wrote:
Mark Brown wrote:
Better to replace any letter except [a-zA-Z0-9_] with '_' or drop them. Use isalnum(), for example.
What *are* the rules? The documentation says "driver name" which isn't verbose...
It's used as an id string, so you can guess. Practically, it should be a single word, contain no dangerous letters that may screw up the parser.
I'm inclined to just stick with the original patch then, I'd expect that anything that's problematic for the driver name is also going to be an issue in the main name field and it's a lot simpler to implement (we don't have isalnum() in kernel right now and you still need to handle _ and possibly -).
The main name field isn't used by the parser. So it may contain special letters like parentheses or such.
And we have isalnum() in kernel, see include/linux/ctype.h.
Takashi