[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <JZ0U3T.53FSIQ97H39F1@gmail.com>
Date: Wed, 08 Oct 2025 23:28:31 +0200
From: Stefano Manni <stefano.manni@...il.com>
To: Bartosz Golaszewski <brgl@...ev.pl>
Cc: linus.walleij@...aro.org, linux-gpio@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] gpio: pisosr: read ngpios as U32
On lun, ago 11 2025 at 09:56:51 +02:00:00, Bartosz Golaszewski
<brgl@...ev.pl> wrote:
> On Wed, Aug 6, 2025 at 9:17 AM Stefano Manni
> <stefano.manni@...il.com> wrote:
>>
>> If of_property_read_u16() is used instead the value read
>> is always zero.
>>
>
> Please state more clearly what the problem is. This sentence is quite
> garbled,
the value returned by of_property_read_u16() is always zero, if
of_property_read_u32()
is used instead , then the returned value is the right one contained in
the dts.
>
>> Signed-off-by: Stefano Manni <stefano.manni@...il.com>
>> ---
>> drivers/gpio/gpio-pisosr.c | 6 +++++-
>> 1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpio/gpio-pisosr.c b/drivers/gpio/gpio-pisosr.c
>> index a69b74866a13..2dc1b1e021d2 100644
>> --- a/drivers/gpio/gpio-pisosr.c
>> +++ b/drivers/gpio/gpio-pisosr.c
>> @@ -117,6 +117,7 @@ static int pisosr_gpio_probe(struct spi_device
>> *spi)
>> {
>> struct device *dev = &spi->dev;
>> struct pisosr_gpio *gpio;
>> + unsigned int ngpio;
>> int ret;
>>
>> gpio = devm_kzalloc(dev, sizeof(*gpio), GFP_KERNEL);
>> @@ -125,8 +126,11 @@ static int pisosr_gpio_probe(struct spi_device
>> *spi)
>>
>> gpio->chip = template_chip;
>> gpio->chip.parent = dev;
>> - of_property_read_u16(dev->of_node, "ngpios",
>> &gpio->chip.ngpio);
>>
>> + if (of_property_read_u32(dev->of_node, "ngpios", &ngpio))
>> + ngpio = DEFAULT_NGPIO;
>> +
>
> If you're touching this, can you switch to using generic device
> properties instead?
>
what do you mean with "generic device properties"?
Best,
Stefano
>>
Powered by blists - more mailing lists