All of lore.kernel.org
 help / color / mirror / Atom feed
From: <Ryan.Wanner@microchip.com>
To: <alexandre.belloni@bootlin.com>, <linux-arm-kernel@lists.infradead.org>
Cc: robh@kernel.org, linus.walleij@linaro.org
Subject: Re: [PATCH] pinctrl: at91: fix deferred probing support
Date: Mon, 21 Nov 2022 21:56:05 +0000	[thread overview]
Message-ID: <37cca51c-7b3f-1c39-925c-839e84913b38@microchip.com> (raw)
In-Reply-To: <20180713094717.GQ16084@piout.net>

On 7/13/18 02:47, Alexandre Belloni wrote:
> Hi Rob,
> 
> On 12/07/2018 13:22:22-0600, Rob Herring wrote:
>> AT91 pinctrl deferred probing support is broken if the GPIO devices
>> haven't probed first and set gpio_banks to non-zero. The later condition
>> that only some GPIO devices haven't probed can't actually happen as
>> either all the GPIO devices have probed first or none of them have. Plus
>> the pinctrl driver has already returned -EINVAL, so it's not on the
>> deferred list to retry probing.
>>
>> Fix this by consolidating the checking that all GPIO devices are probed.
>>
>> Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
>> Cc: Linus Walleij <linus.walleij@linaro.org>
>> Cc: Nicolas Ferre <nicolas.ferre@microchip.com>
>> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
>> Cc: linux-arm-kernel at lists.infradead.org
>> Cc: linux-gpio at vger.kernel.org
>> Signed-off-by: Rob Herring <robh@kernel.org>
>> ---
>> This is a result of trying to remove of_platform_default_populate from 
>> at91 code and relying on the DT core handling populating devices. That 
>> changed the probe order and broke booting.
>>
> 
> This solves part of the issue but when tested with the
> of_platform_default_populate removal, many drivers will fail with
> gpiod_set_value: invalid GPIO (errorpointer)
> 
> or 
> 
> gpiod_get_value: invalid GPIO (errorpointer)
> 
> This happens both before and after the pinctrl driver probed.
> 
> I didn't investigate much yet.
> 
I applied this patch as well as of_platform_default_populate removal patch
on v6.1.0-rc6 with the at91-sam9x60-ek I did not experience the issues that
are mentioned on this thread or 
ARM: at91: remove unnecessary of_platform_default_populate calls, thread.

Has this been fixed since 2018?

Best, 
Ryan
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2022-11-21 21:58 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-12 19:22 [PATCH] pinctrl: at91: fix deferred probing support Rob Herring
2018-07-12 19:22 ` Rob Herring
2018-07-13  9:24 ` Ludovic Desroches
2018-07-13  9:24   ` Ludovic Desroches
2018-07-13  9:47 ` Alexandre Belloni
2018-07-13  9:47   ` Alexandre Belloni
2018-07-13 15:27   ` Rob Herring
2018-07-13 15:27     ` Rob Herring
2018-07-16 13:58     ` Linus Walleij
2018-07-16 13:58       ` Linus Walleij
2018-07-16 14:31       ` Rob Herring
2018-07-16 14:31         ` Rob Herring
2022-11-21 21:56   ` Ryan.Wanner [this message]
2022-11-22  8:32     ` Linus Walleij
2023-01-25 11:35       ` Nicolas Ferre
2023-01-27 12:49         ` Linus Walleij

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=37cca51c-7b3f-1c39-925c-839e84913b38@microchip.com \
    --to=ryan.wanner@microchip.com \
    --cc=alexandre.belloni@bootlin.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=robh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.