[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <dc03a954-116d-2c41-e9b6-7404652c17af@linaro.org>
Date: Fri, 19 Aug 2022 15:10:29 +0300
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Gireesh.Hiremath@...bosch.com, linux-omap@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-input@...r.kernel.org, bcousson@...libre.com,
tony@...mide.com, robh+dt@...nel.org,
krzysztof.kozlowski+dt@...aro.org, dmitry.torokhov@...il.com,
mkorpershoek@...libre.com, davidgow@...gle.com,
m.felsch@...gutronix.de, swboyd@...omium.org,
fengping.yu@...iatek.com, y.oudjana@...tonmail.com,
rdunlap@...radead.org, colin.king@...el.com
Cc: sjoerd.simons@...labora.co.uk, VinayKumar.Shettar@...bosch.com,
Govindaraji.Sivanantham@...bosch.com, anaclaudia.dias@...bosch.com
Subject: Re: [PATCH v3 2/3] driver: input: matric-keypad: add reduced matrix
support
On 19/08/2022 09:59, Gireesh.Hiremath@...bosch.com wrote:
>
> struct gpio_desc **row_gpios;
> struct gpio_desc **col_gpios;
> + struct gpio_desc **desc_gpios;
>
> - unsigned int num_row_gpios;
> - unsigned int num_col_gpios;
> + unsigned int num_row_gpios;
> + unsigned int num_col_gpios;
Do not mix up unrelated changes... One logical change, one commit.
Entire patch should be cleanup from this (it's not only here but in
several places). Except that you have build failures to fix...
Best regards,
Krzysztof
Powered by blists - more mailing lists