mbox

[U-Boot,GIT] Pull request: u-boot-dfu

Message ID 76de457f-25e5-d591-0c6e-1ef3343fbe21@denx.de
State Accepted
Delegated to: Marek Vasut
Headers show

Pull-request

git://git.denx.de/u-boot-dfu.git

Message

Lukasz Majewski July 20, 2017, 10:03 a.m. UTC
Dear Marek,

The following changes since commit 807598a60c5bf7fdf0eb1d78625eeeacfac3c6bc:

   usb: add static to local symbols (2017-07-14 20:45:20 +0200)

are available in the git repository at:

   git://git.denx.de/u-boot-dfu.git

for you to fetch changes up to 2361a4f3be36c673fbbbd1c76bc96553e5e6635a:

   dfu: add common function to initiate transaction (2017-07-20 11:59:12 
+0200)

----------------------------------------------------------------
Patrick Delaunay (4):
       dfu: allow dfu read on partition greater than 2GB
       dfu: remove limitation on partition size
       dfu: factorize transaction cleanup
       dfu: add common function to initiate transaction

Siva Durga Prasad Paladugu (1):
       usb: gadget: f_thor: Free the allocated out request buffer

  drivers/dfu/dfu.c           | 100 
+++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------
  drivers/dfu/dfu_mmc.c       |  20 ++++++++++----------
  drivers/dfu/dfu_nand.c      |   6 ++++--
  drivers/dfu/dfu_ram.c       |   6 ++++--
  drivers/dfu/dfu_sf.c        |   6 ++++--
  drivers/usb/gadget/f_thor.c |   1 +
  include/dfu.h               |   4 ++--
  7 files changed, 70 insertions(+), 73 deletions(-)


Test HW: BBB (Beagle Bone Black)

Comments

Marek Vasut July 20, 2017, 11:07 a.m. UTC | #1
On 07/20/2017 12:03 PM, Łukasz Majewski wrote:
> Dear Marek,

Hi,

> The following changes since commit
> 807598a60c5bf7fdf0eb1d78625eeeacfac3c6bc:
> 
>   usb: add static to local symbols (2017-07-14 20:45:20 +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git
> 
> for you to fetch changes up to 2361a4f3be36c673fbbbd1c76bc96553e5e6635a:
> 
>   dfu: add common function to initiate transaction (2017-07-20 11:59:12
> +0200)
> 
> ----------------------------------------------------------------
> Patrick Delaunay (4):
>       dfu: allow dfu read on partition greater than 2GB
>       dfu: remove limitation on partition size
>       dfu: factorize transaction cleanup
>       dfu: add common function to initiate transaction
> 
> Siva Durga Prasad Paladugu (1):
>       usb: gadget: f_thor: Free the allocated out request buffer
> 
>  drivers/dfu/dfu.c           | 100
> +++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------
> 
>  drivers/dfu/dfu_mmc.c       |  20 ++++++++++----------
>  drivers/dfu/dfu_nand.c      |   6 ++++--
>  drivers/dfu/dfu_ram.c       |   6 ++++--
>  drivers/dfu/dfu_sf.c        |   6 ++++--
>  drivers/usb/gadget/f_thor.c |   1 +
>  include/dfu.h               |   4 ++--
>  7 files changed, 70 insertions(+), 73 deletions(-)
> 
> 
> Test HW: BBB (Beagle Bone Black)

I presume this is not based on u-boot-usb/master, right ? Why ?
Lukasz Majewski July 20, 2017, 12:01 p.m. UTC | #2
Hi Marek,

> On 07/20/2017 12:03 PM, Łukasz Majewski wrote:
>> Dear Marek,
>
> Hi,
>
>> The following changes since commit
>> 807598a60c5bf7fdf0eb1d78625eeeacfac3c6bc:
>>
>>   usb: add static to local symbols (2017-07-14 20:45:20 +0200)
>>
>> are available in the git repository at:
>>
>>   git://git.denx.de/u-boot-dfu.git
>>
>> for you to fetch changes up to 2361a4f3be36c673fbbbd1c76bc96553e5e6635a:
>>
>>   dfu: add common function to initiate transaction (2017-07-20 11:59:12
>> +0200)
>>
>> ----------------------------------------------------------------
>> Patrick Delaunay (4):
>>       dfu: allow dfu read on partition greater than 2GB
>>       dfu: remove limitation on partition size
>>       dfu: factorize transaction cleanup
>>       dfu: add common function to initiate transaction
>>
>> Siva Durga Prasad Paladugu (1):
>>       usb: gadget: f_thor: Free the allocated out request buffer
>>
>>  drivers/dfu/dfu.c           | 100
>> +++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------
>>
>>  drivers/dfu/dfu_mmc.c       |  20 ++++++++++----------
>>  drivers/dfu/dfu_nand.c      |   6 ++++--
>>  drivers/dfu/dfu_ram.c       |   6 ++++--
>>  drivers/dfu/dfu_sf.c        |   6 ++++--
>>  drivers/usb/gadget/f_thor.c |   1 +
>>  include/dfu.h               |   4 ++--
>>  7 files changed, 70 insertions(+), 73 deletions(-)
>>
>>
>> Test HW: BBB (Beagle Bone Black)
>
> I presume this is not based on u-boot-usb/master, right ? Why ?

This is a very good question...... :/

Nonetheless, I've uploaded correct branch to u-boot-dfu/master.


>
Marek Vasut July 20, 2017, 12:05 p.m. UTC | #3
On 07/20/2017 02:01 PM, Łukasz Majewski wrote:
> Hi Marek,
> 
>> On 07/20/2017 12:03 PM, Łukasz Majewski wrote:
>>> Dear Marek,
>>
>> Hi,
>>
>>> The following changes since commit
>>> 807598a60c5bf7fdf0eb1d78625eeeacfac3c6bc:
>>>
>>>   usb: add static to local symbols (2017-07-14 20:45:20 +0200)
>>>
>>> are available in the git repository at:
>>>
>>>   git://git.denx.de/u-boot-dfu.git
>>>
>>> for you to fetch changes up to 2361a4f3be36c673fbbbd1c76bc96553e5e6635a:
>>>
>>>   dfu: add common function to initiate transaction (2017-07-20 11:59:12
>>> +0200)
>>>
>>> ----------------------------------------------------------------
>>> Patrick Delaunay (4):
>>>       dfu: allow dfu read on partition greater than 2GB
>>>       dfu: remove limitation on partition size
>>>       dfu: factorize transaction cleanup
>>>       dfu: add common function to initiate transaction
>>>
>>> Siva Durga Prasad Paladugu (1):
>>>       usb: gadget: f_thor: Free the allocated out request buffer
>>>
>>>  drivers/dfu/dfu.c           | 100
>>> +++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------
>>>
>>>
>>>  drivers/dfu/dfu_mmc.c       |  20 ++++++++++----------
>>>  drivers/dfu/dfu_nand.c      |   6 ++++--
>>>  drivers/dfu/dfu_ram.c       |   6 ++++--
>>>  drivers/dfu/dfu_sf.c        |   6 ++++--
>>>  drivers/usb/gadget/f_thor.c |   1 +
>>>  include/dfu.h               |   4 ++--
>>>  7 files changed, 70 insertions(+), 73 deletions(-)
>>>
>>>
>>> Test HW: BBB (Beagle Bone Black)
>>
>> I presume this is not based on u-boot-usb/master, right ? Why ?
> 
> This is a very good question...... :/
> 
> Nonetheless, I've uploaded correct branch to u-boot-dfu/master.

And tested too ? Nonetheless, I applied it.