lkml.org 
[lkml]   [2017]   [Jun]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v3 10/27] thunderbolt: Fail switch adding operation if reading DROM fails
    Date
    All non-root switches are expected to have DROM so if the operation
    fails, it might be due the user unlugging the device. There is no point
    continuing adding the switch further in that case. Just bail out.

    For root switches (hosts) the DROM is either retrieved from a EFI
    variable, NVM or hard-coded.

    Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
    Reviewed-by: Yehezkel Bernat <yehezkel.bernat@intel.com>
    Reviewed-by: Michael Jamet <michael.jamet@intel.com>
    Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
    ---
    drivers/thunderbolt/switch.c | 7 +++++--
    1 file changed, 5 insertions(+), 2 deletions(-)

    diff --git a/drivers/thunderbolt/switch.c b/drivers/thunderbolt/switch.c
    index 5cd854dedeea..0ce8b600a23f 100644
    --- a/drivers/thunderbolt/switch.c
    +++ b/drivers/thunderbolt/switch.c
    @@ -535,8 +535,11 @@ int tb_switch_add(struct tb_switch *sw)
    int i, ret;

    /* read drom */
    - if (tb_drom_read(sw))
    - tb_sw_warn(sw, "tb_eeprom_read_rom failed, continuing\n");
    + ret = tb_drom_read(sw);
    + if (ret) {
    + tb_sw_warn(sw, "tb_eeprom_read_rom failed\n");
    + return ret;
    + }
    tb_sw_info(sw, "uid: %#llx\n", sw->uid);

    tb_switch_set_uuid(sw);
    --
    2.11.0
    \
     
     \ /
      Last update: 2017-06-05 00:13    [W:4.132 / U:0.096 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site