On Wed, 14 Nov 2012 08:48:07 -0800
Dmitry Torokhov <[email protected]> wrote:

> Failures to build a keymap, request an IRQ, or register input device are
> fatal for the driver, therefore the messages should have "error" severity
> instead of "debug".
> 
> Signed-off-by: Dmitry Torokhov <[email protected]>

Reviewed-by: Alban Bedel <[email protected]>

> ---
>  drivers/input/keyboard/tca8418_keypad.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/input/keyboard/tca8418_keypad.c 
> b/drivers/input/keyboard/tca8418_keypad.c
> index 35340c3..10047eb 100644
> --- a/drivers/input/keyboard/tca8418_keypad.c
> +++ b/drivers/input/keyboard/tca8418_keypad.c
> @@ -348,7 +348,7 @@ static int __devinit tca8418_keypad_probe(struct 
> i2c_client *client,
>       error = matrix_keypad_build_keymap(keymap_data, NULL, rows, cols,
>                                          keypad_data->keymap, input);
>       if (error) {
> -             dev_dbg(dev, "Failed to build keymap\n");
> +             dev_err(dev, "Failed to build keymap\n");
>               goto fail2;
>       }
>  
> @@ -367,14 +367,14 @@ static int __devinit tca8418_keypad_probe(struct 
> i2c_client *client,
>                                    IRQF_ONESHOT,
>                                    client->name, keypad_data);
>       if (error) {
> -             dev_dbg(dev, "Unable to claim irq %d; error %d\n",
> +             dev_err(dev, "Unable to claim irq %d; error %d\n",
>                       client->irq, error);
>               goto fail2;
>       }
>  
>       error = input_register_device(input);
>       if (error) {
> -             dev_dbg(dev, "Unable to register input device, error: %d\n",
> +             dev_err(dev, "Unable to register input device, error: %d\n",
>                       error);
>               goto fail3;
>       }

--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to