From: Thomas Richard <thomas.richard@bootlin.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Linus Walleij <linus.walleij@linaro.org>,
Bartosz Golaszewski <brgl@bgdev.pl>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>,
linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org,
kernel test robot <oliver.sang@intel.com>
Subject: Re: [PATCH] gpio: aggregator: restore the set_config operation
Date: Fri, 3 Oct 2025 15:59:46 +0200 [thread overview]
Message-ID: <4db46e2b-47c4-4c81-a36f-8b195b090d2f@bootlin.com> (raw)
In-Reply-To: <CAMuHMdX5HXx2HSAHP-H1EEKO-csBku_cMm-OaacE7GZLXwBxOg@mail.gmail.com>
>
> Is there any specific reason why you are doing this unconditionally,
> instead of only when any of its parents support .set_config(), like
> was done before?
>
My idea was: it will be handled by the core, so the if statement is not
needed. But if we conditionally add the operation we can save some time
in case there is no chip supporting set_config().
Thomas
next prev parent reply other threads:[~2025-10-03 13:59 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-29 10:03 [PATCH] gpio: aggregator: restore the set_config operation Thomas Richard
2025-09-29 10:15 ` Geert Uytterhoeven
2025-10-03 13:59 ` Thomas Richard [this message]
2025-10-03 14:30 ` Thomas Richard
2025-10-06 7:42 ` Geert Uytterhoeven
2025-10-06 12:26 ` Thomas Richard
2025-09-29 12:45 ` Bartosz Golaszewski
2025-11-05 9:59 ` Thomas Richard
2025-11-05 10:39 ` Bartosz Golaszewski
2025-11-05 10:39 ` Bartosz Golaszewski
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=4db46e2b-47c4-4c81-a36f-8b195b090d2f@bootlin.com \
--to=thomas.richard@bootlin.com \
--cc=bartosz.golaszewski@linaro.org \
--cc=brgl@bgdev.pl \
--cc=geert@linux-m68k.org \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=oliver.sang@intel.com \
--cc=thomas.petazzoni@bootlin.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).