Pavel Machek wrote:

Hi!

quoted

This were the original plan we've discussed, back in December:

....

Seems sane.

quoted

struct keycode_table_entry {
	unsigned keycode;
	char scancode[32];	/* 32 is just an arbitrary long array - maybe shorter */
	int len;
}

What about

struct keycode_table_entry {
	unsigned keycode;
	int len;
	char scancode[];
}

? gcc extension, but commonly used around kernel.

Seems fine. Maybe we could just use "char *scancode" to avoid using a
gcc extension on a public interface.


-- 

Cheers,
Mauro