Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: g/o0rrupzx140CJS7pDJtR1/Mh1pjHVrMxzBbBMR4E/jaI6Izbe79+Hi1IxGLiGkEy3uAzHWTw
 fC+KEjiaRp+A==
IronPort-SDR: 0tve1CYzhTsytisDy1TclHePtLyVjyq3/rDx/tErL8bdbcltl3BLKazn/LxTYXfTz+kvvbMY13
 9vpMtXBxBTpQ==
Date:   Wed, 9 Sep 2020 13:34:40 +0300
From:   Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To:     Bartosz Golaszewski <brgl@bgdev.pl>
Cc:     Linus Walleij <linus.walleij@linaro.org>,
        Mika Westerberg <mika.westerberg@linux.intel.com>,
        Kent Gibson <warthog618@gmail.com>, linux-gpio@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org,
        Bartosz Golaszewski <bgolaszewski@baylibre.com>
Subject: Re: [PATCH v2 0/3] gpiolib: generalize GPIO line names property
Message-ID: <20200909103440.GM1891694@smile.fi.intel.com>
References: <20200909085426.19862-1-brgl@bgdev.pl>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200909085426.19862-1-brgl@bgdev.pl>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Sep 09, 2020 at 10:54:23AM +0200, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> 
> I initially sent this as part of the gpio-mockup overhaul but since
> these patches are indepentent and the work on gpio-mockup may become
> more complicated - I'm sending these separately.
> 
> The only change is adding additional property helpers to count strings
> in array.

Seems it lost my tag for patch 1/3, but in any case now I'm good with it
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> v1 -> v2:
> - actually remove the previous devprop source file in patch 3
> - rename the string counting functions to something more explicit
> 
> Bartosz Golaszewski (3):
>   device: property: add helpers to count items in string arrays
>   gpiolib: generalize devprop_gpiochip_set_names() for device properties
>   gpiolib: unexport devprop_gpiochip_set_names()
> 
>  drivers/gpio/Makefile          |  1 -
>  drivers/gpio/gpiolib-acpi.c    |  3 --
>  drivers/gpio/gpiolib-devprop.c | 63 ----------------------------------
>  drivers/gpio/gpiolib-of.c      |  5 ---
>  drivers/gpio/gpiolib.c         | 55 ++++++++++++++++++++++++++---
>  include/linux/gpio/driver.h    |  3 --
>  include/linux/property.h       | 13 +++++++
>  7 files changed, 64 insertions(+), 79 deletions(-)
>  delete mode 100644 drivers/gpio/gpiolib-devprop.c
> 
> -- 
> 2.26.1
> 

-- 
With Best Regards,
Andy Shevchenko