mbox

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

Message ID 20160212120413.33e975aa@amdc2363
State Accepted
Delegated to: Marek Vasut
Headers show

Pull-request

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

Message

Łukasz Majewski Feb. 12, 2016, 11:04 a.m. UTC
The following changes since commit
7aeb1465329a997f93d00adee21a13732b8e1006:

  usb: ehci: Be explicit about the BE IO accessors (2016-02-11 14:08:37
  +0100)

are available in the git repository at:

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

for you to fetch changes up to 3e3d2ff93e966c9ace17a3d2d9abd582626af0f0:

  dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle large
  files transmission and storage (2016-02-12 12:01:14 +0100)

----------------------------------------------------------------
Łukasz Majewski (2):
      dfu: usb: f_dfu: Set deferred call for dfu_flush() function
      dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle large
files transmission and storage

 cmd/dfu.c                    | 20 ++++++++++++++++++++
 drivers/usb/gadget/f_dfu.c   | 11 +++--------
 include/configs/odroid_xu3.h |  1 +
 include/dfu.h                | 25 +++++++++++++++++++++++++
 4 files changed, 49 insertions(+), 8 deletions(-)

Comments

Marek Vasut Feb. 12, 2016, 12:32 p.m. UTC | #1
On 02/12/2016 12:04 PM, Lukasz Majewski wrote:
> The following changes since commit
> 7aeb1465329a997f93d00adee21a13732b8e1006:
> 
>   usb: ehci: Be explicit about the BE IO accessors (2016-02-11 14:08:37
>   +0100)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git 
> 
> for you to fetch changes up to 3e3d2ff93e966c9ace17a3d2d9abd582626af0f0:
> 
>   dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle large
>   files transmission and storage (2016-02-12 12:01:14 +0100)
> 
> ----------------------------------------------------------------
> Łukasz Majewski (2):
>       dfu: usb: f_dfu: Set deferred call for dfu_flush() function
>       dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle large
> files transmission and storage
> 
>  cmd/dfu.c                    | 20 ++++++++++++++++++++
>  drivers/usb/gadget/f_dfu.c   | 11 +++--------
>  include/configs/odroid_xu3.h |  1 +
>  include/dfu.h                | 25 +++++++++++++++++++++++++
>  4 files changed, 49 insertions(+), 8 deletions(-)

Merged, thanks.

Are these really _all_ of the DFU patches ? I saw quite a lot of them on
the list.
Łukasz Majewski Feb. 12, 2016, 1:51 p.m. UTC | #2
Hi Marek,

> On 02/12/2016 12:04 PM, Lukasz Majewski wrote:
> > The following changes since commit
> > 7aeb1465329a997f93d00adee21a13732b8e1006:
> > 
> >   usb: ehci: Be explicit about the BE IO accessors (2016-02-11
> > 14:08:37 +0100)
> > 
> > are available in the git repository at:
> > 
> >   git://git.denx.de/u-boot-dfu.git 
> > 
> > for you to fetch changes up to
> > 3e3d2ff93e966c9ace17a3d2d9abd582626af0f0:
> > 
> >   dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle large
> >   files transmission and storage (2016-02-12 12:01:14 +0100)
> > 
> > ----------------------------------------------------------------
> > Łukasz Majewski (2):
> >       dfu: usb: f_dfu: Set deferred call for dfu_flush() function
> >       dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle
> > large files transmission and storage
> > 
> >  cmd/dfu.c                    | 20 ++++++++++++++++++++
> >  drivers/usb/gadget/f_dfu.c   | 11 +++--------
> >  include/configs/odroid_xu3.h |  1 +
> >  include/dfu.h                | 25 +++++++++++++++++++++++++
> >  4 files changed, 49 insertions(+), 8 deletions(-)
> 
> Merged, thanks.
> 
> Are these really _all_ of the DFU patches ? I saw quite a lot of them
> on the list.
> 
> 

Some of them you have already merged (the one with Max Packet Size).

Another patch is the one posted by Heiko to display MTD partitions
larger than 2 GiB. Unfortunately if introduces regression on eMMC and I
don't have enough time to look into it.

There is also patch from Michal Simek:

[PATCH] usb: gadget: f_thor: free the allocated out request buffer

As fair as I remember you had some comments to it.
Marek Vasut Feb. 12, 2016, 4:29 p.m. UTC | #3
On 02/12/2016 02:51 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
>> On 02/12/2016 12:04 PM, Lukasz Majewski wrote:
>>> The following changes since commit
>>> 7aeb1465329a997f93d00adee21a13732b8e1006:
>>>
>>>   usb: ehci: Be explicit about the BE IO accessors (2016-02-11
>>> 14:08:37 +0100)
>>>
>>> are available in the git repository at:
>>>
>>>   git://git.denx.de/u-boot-dfu.git 
>>>
>>> for you to fetch changes up to
>>> 3e3d2ff93e966c9ace17a3d2d9abd582626af0f0:
>>>
>>>   dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle large
>>>   files transmission and storage (2016-02-12 12:01:14 +0100)
>>>
>>> ----------------------------------------------------------------
>>> Łukasz Majewski (2):
>>>       dfu: usb: f_dfu: Set deferred call for dfu_flush() function
>>>       dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle
>>> large files transmission and storage
>>>
>>>  cmd/dfu.c                    | 20 ++++++++++++++++++++
>>>  drivers/usb/gadget/f_dfu.c   | 11 +++--------
>>>  include/configs/odroid_xu3.h |  1 +
>>>  include/dfu.h                | 25 +++++++++++++++++++++++++
>>>  4 files changed, 49 insertions(+), 8 deletions(-)
>>
>> Merged, thanks.
>>
>> Are these really _all_ of the DFU patches ? I saw quite a lot of them
>> on the list.
>>
>>
> 
> Some of them you have already merged (the one with Max Packet Size).
> 
> Another patch is the one posted by Heiko to display MTD partitions
> larger than 2 GiB. Unfortunately if introduces regression on eMMC and I
> don't have enough time to look into it.

I hope you're doing fine ?

> There is also patch from Michal Simek:
> 
> [PATCH] usb: gadget: f_thor: free the allocated out request buffer
> 
> As fair as I remember you had some comments to it.

OK, understood. Thanks for the great summary! :)
Łukasz Majewski Feb. 15, 2016, 9:30 a.m. UTC | #4
Hi Marek,

> On 02/12/2016 02:51 PM, Lukasz Majewski wrote:
> > Hi Marek,
> > 
> >> On 02/12/2016 12:04 PM, Lukasz Majewski wrote:
> >>> The following changes since commit
> >>> 7aeb1465329a997f93d00adee21a13732b8e1006:
> >>>
> >>>   usb: ehci: Be explicit about the BE IO accessors (2016-02-11
> >>> 14:08:37 +0100)
> >>>
> >>> are available in the git repository at:
> >>>
> >>>   git://git.denx.de/u-boot-dfu.git 
> >>>
> >>> for you to fetch changes up to
> >>> 3e3d2ff93e966c9ace17a3d2d9abd582626af0f0:
> >>>
> >>>   dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle
> >>> large files transmission and storage (2016-02-12 12:01:14 +0100)
> >>>
> >>> ----------------------------------------------------------------
> >>> Łukasz Majewski (2):
> >>>       dfu: usb: f_dfu: Set deferred call for dfu_flush() function
> >>>       dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle
> >>> large files transmission and storage
> >>>
> >>>  cmd/dfu.c                    | 20 ++++++++++++++++++++
> >>>  drivers/usb/gadget/f_dfu.c   | 11 +++--------
> >>>  include/configs/odroid_xu3.h |  1 +
> >>>  include/dfu.h                | 25 +++++++++++++++++++++++++
> >>>  4 files changed, 49 insertions(+), 8 deletions(-)
> >>
> >> Merged, thanks.
> >>
> >> Are these really _all_ of the DFU patches ? I saw quite a lot of
> >> them on the list.
> >>
> >>
> > 
> > Some of them you have already merged (the one with Max Packet Size).
> > 
> > Another patch is the one posted by Heiko to display MTD partitions
> > larger than 2 GiB. Unfortunately if introduces regression on eMMC
> > and I don't have enough time to look into it.
> 
> I hope you're doing fine ?

It's ok, I just had less time for open source than usual (my CI setup
needs some overhaul).

> 
> > There is also patch from Michal Simek:
> > 
> > [PATCH] usb: gadget: f_thor: free the allocated out request buffer
> > 
> > As fair as I remember you had some comments to it.

I hope that Michal will submit new version of the patch soon.

> 
> OK, understood. Thanks for the great summary! :)
> 
> 

No problem :-)
Marek Vasut Feb. 15, 2016, 1:52 p.m. UTC | #5
On 02/15/2016 10:30 AM, Lukasz Majewski wrote:
> Hi Marek,
> 
>> On 02/12/2016 02:51 PM, Lukasz Majewski wrote:
>>> Hi Marek,
>>>
>>>> On 02/12/2016 12:04 PM, Lukasz Majewski wrote:
>>>>> The following changes since commit
>>>>> 7aeb1465329a997f93d00adee21a13732b8e1006:
>>>>>
>>>>>   usb: ehci: Be explicit about the BE IO accessors (2016-02-11
>>>>> 14:08:37 +0100)
>>>>>
>>>>> are available in the git repository at:
>>>>>
>>>>>   git://git.denx.de/u-boot-dfu.git 
>>>>>
>>>>> for you to fetch changes up to
>>>>> 3e3d2ff93e966c9ace17a3d2d9abd582626af0f0:
>>>>>
>>>>>   dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle
>>>>> large files transmission and storage (2016-02-12 12:01:14 +0100)
>>>>>
>>>>> ----------------------------------------------------------------
>>>>> Łukasz Majewski (2):
>>>>>       dfu: usb: f_dfu: Set deferred call for dfu_flush() function
>>>>>       dfu: odroid xu3: Define DFU_MANIFEST_POLL_TIMEOUT to handle
>>>>> large files transmission and storage
>>>>>
>>>>>  cmd/dfu.c                    | 20 ++++++++++++++++++++
>>>>>  drivers/usb/gadget/f_dfu.c   | 11 +++--------
>>>>>  include/configs/odroid_xu3.h |  1 +
>>>>>  include/dfu.h                | 25 +++++++++++++++++++++++++
>>>>>  4 files changed, 49 insertions(+), 8 deletions(-)
>>>>
>>>> Merged, thanks.
>>>>
>>>> Are these really _all_ of the DFU patches ? I saw quite a lot of
>>>> them on the list.
>>>>
>>>>
>>>
>>> Some of them you have already merged (the one with Max Packet Size).
>>>
>>> Another patch is the one posted by Heiko to display MTD partitions
>>> larger than 2 GiB. Unfortunately if introduces regression on eMMC
>>> and I don't have enough time to look into it.
>>
>> I hope you're doing fine ?
> 
> It's ok, I just had less time for open source than usual (my CI setup
> needs some overhaul).

Such task awaits me as well, good luck!

>>> There is also patch from Michal Simek:
>>>
>>> [PATCH] usb: gadget: f_thor: free the allocated out request buffer
>>>
>>> As fair as I remember you had some comments to it.
> 
> I hope that Michal will submit new version of the patch soon.

Yes.

>>
>> OK, understood. Thanks for the great summary! :)
>>
>>
> 
> No problem :-)
>