Olof Johansson wrote at Tuesday, January 03, 2012 2:37 PM:
> This adds a simple device tree binding for simple key matrix data and
> a helper to fill in the platform data.
>
> The implementation is in a shared file outside if drivers/input/keyboard
> since some drivers in drivers/input/misc might be making use of it
> as well.
>
> Changes since v3:
> * Dropped compatible field in matrix-keymap.txt
> * Dropped linux,fn-key
> * Dropped linux,fn-keymap optional property but included guideline on
> naming convention
> * Now passing property name in to the helper function (or will assume
> "linux,keymap" if passed NULL)
> diff --git a/drivers/input/of_keymap.c b/drivers/input/of_keymap.c
> +struct matrix_keymap_data *
> +matrix_keyboard_of_fill_keymap(struct device_node *np, char *propname)
> +{
...
> + kd = kmalloc(sizeof(*kd), GFP_KERNEL);
> + if (!kd)
> + return NULL;
Should that use kzalloc in case struct matrix_keymap_data grows some
new fields that people will assume are set to zero since the struct
would usually be in .data? Still, people should probably grep the code
when making such changes...
Otherwise,
Acked-by: Stephen Warren <[email protected]>
--
nvpublic
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss
