diff options
author | Jeffrey Carlyle <jcarlyle@google.com> | 2016-05-19 04:55:13 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2016-05-19 04:59:22 +0300 |
commit | e16715c135d80aafea867849f938b080d4f4eadb (patch) | |
tree | 99ecb0aba76d6f5a0c9c369d1a6ff4796b1bd626 /drivers/staging/greybus/interface.c | |
parent | 9bc63b7ff56f7a1cad5dbf062d1d5dd7a81d7d93 (diff) | |
download | linux-e16715c135d80aafea867849f938b080d4f4eadb.tar.xz |
greybus: interface: retry enumeration of UniPro-only modules
Greybus modules will sometimes fail to send the mailbox poke and
erroneously be enumerated as UniPro-only modules. The root cause for
this on the module side is not fully understand, but it seems that this
may be due to "the bootrom bug:" a known problem with the bootrom where
linkup will occasionally fail because of a race condition.
Before the new hotplug code was implemented in the firmware, the SVC
would retry enumeration of modules that did not send the mailbox poke;
this patch ports that functionality to the AP.
Signed-off-by: Jeffrey Carlyle <jcarlyle@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/interface.c')
-rw-r--r-- | drivers/staging/greybus/interface.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/staging/greybus/interface.c b/drivers/staging/greybus/interface.c index a1435e94f5a1..3b5669375ed3 100644 --- a/drivers/staging/greybus/interface.c +++ b/drivers/staging/greybus/interface.c @@ -447,7 +447,8 @@ static int gb_interface_activate_operation(struct gb_interface *intf) return -ENODEV; case GB_SVC_INTF_TYPE_UNIPRO: dev_err(&intf->dev, "interface type UniPro not supported\n"); - return -ENODEV; + /* FIXME: check if this is a Toshiba bridge before retrying? */ + return -EAGAIN; case GB_SVC_INTF_TYPE_GREYBUS: break; default: |