Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: ICzpKZuNol7nYjx5G80qZMBsZB2OnaOjir4QeT4xleVclot0wuGEADpo/FAA0ppw+mViyRLL+6
 lsUjdFQWJnfGeiXVZnWNpseiH4D0XGH0enPU0Fbj//Y3HB68duvhZZ+p11uGBSbDPs9x/W2O8P
 wYniP86nWVgdST9Xq40hm/DTSVhzLTQ6/6xiFfFJMMUEGWnfDmPO7Ds6wAfbaMU8OmaBrmTwZU
 NYtcCw77yd/4UgEpENDriWtOyno9/OiSO8waD19QWcYkMSjy7jm1XddvmO9IXBYwJFOVZYtNSB
 jLI=
Subject: Re: [PATCHv2] i2c: slave-eeprom: Make it possible to pre-load eeprom
 data
To:     Wolfram Sang <wsa@the-dreams.de>
CC:     <linux-i2c@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <patrick@stwcx.xyz>, <kernel@axis.com>
References: <20200424090443.26316-1-bjorn.ardo@axis.com>
 <20200424111337.GC1959@kunai> <5038e4c1-440b-0a56-978b-a8c9fac061cc@axis.com>
 <20200424112705.GD1959@kunai>
From:   Bjorn Ardo <bjorn.ardo@axis.com>
Message-ID: <8d1cd553-4607-4092-7020-911e43e02d11@axis.com>
Date:   Fri, 24 Apr 2020 13:31:12 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.7.0
MIME-Version: 1.0
In-Reply-To: <20200424112705.GD1959@kunai>
Content-Type: text/plain; charset="windows-1252"; format=flowed
Content-Transfer-Encoding: 7bit
Content-Language: en-US
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

You got a new patch (ignor the first one, I forgot to update the version 
number).

On 4/24/20 1:27 PM, Wolfram Sang wrote:
>> As I can see in drivers/base/firmware_loader/main.c in function
>> _request_firmware, then the fw will be released internally if it returns an
>> error value.
> Ouch, of course!
>
>>> Also, do we need 'error' and 'ret'? Can't we reuse one of them?
>> Yes, I can fix that.
> Great, thanks!
>