> ret = si2157_load_firmware(fe, fw_name); > if (ret) { > + if (!fw_required) > + goto skip_fw_download; > + In conjunction with my proposal for PATCH 1/3, this can be simplified to: ret = si2157_load_firmware(fe, fw_name); if (ret && fw_required) goto err; Best Regards, -Robert Schlabbach