Re: PATCH v2

[Robert Middleton <robert.middleton@xxxxxxxxxx>]

On Wed, Feb 22, 2017 at 10:34 AM,  <robert.middleton@xxxxxxxxxx> wrote:
>
> This v2 patch changes the MCP23S08 interrupt logic to ensure that if multiple
> inputs are changed at the same time, the kernel will be notified of all
> the pins that changed.
>
> I added several variables to the irq handler, however this means that there
> are now quite a few local variables(possibly too many, as the kernel coding
> standard says that more than 5-10 is probably too many:
> https://www.kernel.org/doc/Documentation/process/coding-style.rst).  I'm
> not sure what the best way to fix this is, or if it has to be fixed.
>
> -Robert Middleton
> --
> To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Hi, I would like to try and get some verification here on if this patch
is a good fix or not.  I have done testing on my side, and was
wondering if there was something else that I had to do in order to have
this patched upstream.

-Robert Middleton
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html