[PATCH v2 1/8] dfu: add scsi backend

Caleb Connolly caleb.connolly at linaro.org
Tue May 28 18:27:18 CEST 2024



On 28/05/2024 11:10, Mattijs Korpershoek wrote:
> Hi Caleb,
> 
> Thank you for the patch.
> 
> On lun., mai 27, 2024 at 19:17, Caleb Connolly <caleb.connolly at linaro.org> wrote:
> 
>> This is extremely similar to the MMC backend, but there are some notable
>> differences.
>>
>> Works with a DFU string like
>>
>>      scsi 4=u-boot-bin part 11
>>
>> Where "4" is the SCSI dev number (sequential LUN across all SCSI devices)
>> and "11" is the partition number >>
>> Signed-off-by: Caleb Connolly <caleb.connolly at linaro.org>
>> ---
>>   doc/usage/dfu.rst      |  31 ++++
>>   drivers/dfu/Kconfig    |   7 +
>>   drivers/dfu/Makefile   |   1 +
>>   drivers/dfu/dfu.c      |   5 +-
>>   drivers/dfu/dfu_scsi.c | 437 +++++++++++++++++++++++++++++++++++++++++++++++++
>>   include/dfu.h          |  26 +++
>>   6 files changed, 506 insertions(+), 1 deletion(-)
> 
> I reviewed this here [1]. I could not find any response on that thread.
> Could you please respond there or address the remarks for v3, please?


Hi Mattijs,

So sorry about that, will address your feedback in v3.

Thanks,
> 
> [1] https://lore.kernel.org/r/87o7a94pe1.fsf@baylibre.com
> 
>>
>> diff --git a/doc/usage/dfu.rst b/doc/usage/dfu.rst
>> index 8cc09c308d82..dc4f8d672f99 100644
>> --- a/doc/usage/dfu.rst
>> +++ b/doc/usage/dfu.rst
>> @@ -166,8 +166,38 @@ mmc
> 
> [...]
> 
>> -- 
>> 2.45.0

-- 
// Caleb (they/them)


More information about the U-Boot mailing list