Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1378548

Re: [PATCH v7 1/2] drivers: base: add support for registering notifier about deferred probe

Show all headers | View raw

Hello,

On 2016-04-13 16:12, Greg Kroah-Hartman wrote:
> On Wed, Apr 13, 2016 at 11:35:59AM +0200, Marek Szyprowski wrote:
>> This patch adds code which allow other subsystems get a notification
>> when deferred probe has been triggered. This way one can retry some
>> actions, which earlier failed with -EPROBE_DEFER error code.
>>
>> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> Why would some other subsystem want/care about this?  You aren't telling
> them what device was deferred, and you don't need to as the bus itself
> already knows this information as it did the deferring!
>
> confused,

This notifier is just to let others that the deferred probe has happened and
it is a good time to retry operation, which earlier failed due to missing
resources (i.e. power domains, clocks). Such case is with registering AMBA
device (not the driver!). During AMBA device registration, bus code has 
to read
some device's registers to get its device CID/PID. To do this, device's 
clocks
and power domain has to be turned on. Those however might not be available
that time. With this notifier, AMBA bus code is able to retry device
registration, which earlier failed due to missing clocks or power domain.

This CID/PID reading has to be done during device registration time because
of the already deployed userspace ABI. CID/PID values are reported to
userspace, which might rely on them to load proper driver modules.

Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v7 1/2] drivers: base: add support for registering notifier  about deferred probe Marek Szyprowski <m.szyprowski@samsung.com> - 2016-04-13 11:40 +0200
  Re: [PATCH v7 1/2] drivers: base: add support for registering  notifier about deferred probe Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-04-13 16:20 +0200
    Re: [PATCH v7 1/2] drivers: base: add support for registering notifier  about deferred probe Marek Szyprowski <m.szyprowski@samsung.com> - 2016-04-14 09:40 +0200
      Re: [PATCH v7 1/2] drivers: base: add support for registering  notifier about deferred probe Russell King - ARM Linux <linux@arm.linux.org.uk> - 2016-04-14 10:50 +0200
      Re: [PATCH v7 1/2] drivers: base: add support for registering  notifier about deferred probe Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-04-14 23:20 +0200
        Re: [PATCH v7 1/2] drivers: base: add support for registering  notifier about deferred probe Russell King - ARM Linux <linux@arm.linux.org.uk> - 2016-04-14 23:40 +0200
          Re: [PATCH v7 1/2] drivers: base: add support for registering  notifier about deferred probe Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-04-15 00:00 +0200

csiph-web