On Mon, Mar 14, 2016 at 05:35:01PM -0700, Andrew Duggan wrote:
> Originally the irq_mask member of rmi_function was a pointer. Then it was
> switched to being a zero length array. However, the checks for a NULL
> pointer where not removed.
>
> Signed-off-by: Andrew Duggan <adug...@synaptics.com>
Applied, thank you.
> ---
> drivers/input/rmi4/rmi_driver.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/input/rmi4/rmi_driver.c b/drivers/input/rmi4/rmi_driver.c
> index da38f0a..faa295e 100644
> --- a/drivers/input/rmi4/rmi_driver.c
> +++ b/drivers/input/rmi4/rmi_driver.c
> @@ -126,7 +126,7 @@ static void process_one_interrupt(struct rmi_driver_data
> *data,
> return;
>
> fh = to_rmi_function_handler(fn->dev.driver);
> - if (fn->irq_mask && fh->attention) {
> + if (fh->attention) {
> bitmap_and(data->fn_irq_bits, data->irq_status, fn->irq_mask,
> data->irq_count);
> if (!bitmap_empty(data->fn_irq_bits, data->irq_count))
> @@ -172,8 +172,7 @@ int rmi_process_interrupt_requests(struct rmi_device
> *rmi_dev)
> * use irq_chip.
> */
> list_for_each_entry(entry, &data->function_list, node)
> - if (entry->irq_mask)
> - process_one_interrupt(data, entry);
> + process_one_interrupt(data, entry);
>
> if (data->input)
> input_sync(data->input);
> --
> 2.5.0
>
--
Dmitry
