From: Marek Szyprowski <m.szyprowski@samsung.com>
To: Saravana Kannan <saravanak@google.com>,
Ulf Hansson <ulf.hansson@linaro.org>
Cc: Rob Herring <robh@kernel.org>,
Geert Uytterhoeven <geert+renesas@glider.be>,
Linus Walleij <linus.walleij@linaro.org>,
Russell King <linux@armlinux.org.uk>,
John Stultz <john.stultz@linaro.org>,
Nicolas Saenz Julienne <nsaenzjulienne@suse.de>,
Sudeep Holla <sudeep.holla@arm.com>,
Linux ARM <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH] amba: Retry adding deferred devices at late_initcall
Date: Thu, 7 May 2020 13:44:23 +0200 [thread overview]
Message-ID: <c9a2dccd-50c0-5e9a-5ca7-69ec063e3630@samsung.com> (raw)
In-Reply-To: <CAGETcx-kC18pwF7uDYE4XwVOWBJN8Lx48LD74LECgDts+VB0gA@mail.gmail.com>
Hi Saravana,
On 04.05.2020 21:28, Saravana Kannan wrote:
> On Mon, May 4, 2020 at 12:11 PM Ulf Hansson <ulf.hansson@linaro.org> wrote:
>> In principle we should then be able to rely on the regular defered
>> probe mechanism, just that it's the dummy device that is being defered
>> probed (if we fail to read PID/CID).
>>> Any reason to not do something like that? I'd think that should clean
>>> up a whole lot of this code. Also, if we are primarily dealing with
>>> AMBA devices created from DT, then we might even be able to massage
>>> the fw_devlink feature to optimize this even more when fw_devlink=on.
>>>
>>> Just my 2 cents.
>> Someone should try to implement this to see if it fits well.
> I don't mind taking a stab at this if people are actually okay with
> this approach and will test and merge it if it works. I have no
> platform to test this. I'll wait to hear what others think before I
> jump on this.
The time I've prepared my patch I've also considered something like
that, but I gave up because timer or notifier based approach was much
simpler. If you have some time to implement your idea I would be happy
to test it.
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-05-07 11:44 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-27 21:25 [PATCH] amba: Retry adding deferred devices at late_initcall Rob Herring
2020-04-28 15:51 ` Sudeep Holla
2020-04-28 20:39 ` Linus Walleij
2020-04-29 6:06 ` Marek Szyprowski
2020-04-29 7:33 ` Saravana Kannan
2020-05-04 19:10 ` Ulf Hansson
2020-05-04 19:28 ` Saravana Kannan
2020-05-07 11:44 ` Marek Szyprowski [this message]
2020-05-07 17:39 ` Saravana Kannan
2020-05-08 13:41 ` Rob Herring
2020-05-08 19:19 ` Saravana Kannan
2020-04-29 12:26 ` Linus Walleij
2020-04-29 14:01 ` Rob Herring
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=c9a2dccd-50c0-5e9a-5ca7-69ec063e3630@samsung.com \
--to=m.szyprowski@samsung.com \
--cc=geert+renesas@glider.be \
--cc=john.stultz@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux@armlinux.org.uk \
--cc=nsaenzjulienne@suse.de \
--cc=robh@kernel.org \
--cc=saravanak@google.com \
--cc=sudeep.holla@arm.com \
--cc=ulf.hansson@linaro.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.