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

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

[Lukasz Majewski]

Hi Marek,

Please find following code for u-boot-dfu repository.

The following changes since commit 76b2fad775ee3cb58788b11454655ba5a244ac56:

  eth: asix88179: Add support for the driver model (2016-08-09 12:52:05 +0200)

are available in the git repository at:

  ssh://gu-dfu at git.denx.de/u-boot-dfu e8091b356dd0f1e8e742a5da7a5e8256744b39b7

for you to fetch changes up to e8091b356dd0f1e8e742a5da7a5e8256744b39b7:

  cmd: dfu: Add error handling for failed registration (2016-08-16 13:06:04 +0200)

----------------------------------------------------------------
B, Ravi (5):
      spl: dfu: add dfu support in SPL
      common: dfu: saperate the dfu common functionality
      spl: dfu: adding dfu support functions for SPL-DFU
      dra7x: boot: add dfu bootmode support
      dra7x: configs: enable SPL-DFU support

Sanchayan Maity (1):
      cmd: dfu: Add error handling for failed registration

 Kconfig                                      | 27 +++++++++++++++++++++++++++
 arch/arm/cpu/armv7/omap-common/boot-common.c |  5 +++++
 arch/arm/include/asm/arch-omap5/spl.h        |  2 +-
 cmd/dfu.c                                    | 61 ++-----------------------------------------------------------
 common/Makefile                              |  3 +++
 common/dfu.c                                 | 92 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 common/spl/Makefile                          |  1 +
 common/spl/spl.c                             | 10 +++++++++-
 common/spl/spl_dfu.c                         | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 include/configs/dra7xx_evm.h                 | 20 +++++++++++++++++++-
 include/configs/ti_omap5_common.h            |  2 --
 include/g_dnl.h                              |  1 +
 include/spl.h                                |  8 ++++++++
 scripts/Makefile.spl                         |  4 ++++
 14 files changed, 229 insertions(+), 64 deletions(-)
 create mode 100644 common/dfu.c
 create mode 100644 common/spl/spl_dfu.c


Test HW: Exynos 4412 Odroid U3, Exynos5422 Odroid XU3
Tested-by: Lukasz Majewski <l.majewski@samsung.com>

by HWT system

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On 08/16/2016 03:41 PM, Lukasz Majewski wrote:
> Hi Marek,

Hi,

is that for master or next ? Was this build tested ?

> Please find following code for u-boot-dfu repository.
> 
> The following changes since commit 76b2fad775ee3cb58788b11454655ba5a244ac56:
> 
>   eth: asix88179: Add support for the driver model (2016-08-09 12:52:05 +0200)
> 
> are available in the git repository at:
> 
>   ssh://gu-dfu at git.denx.de/u-boot-dfu e8091b356dd0f1e8e742a5da7a5e8256744b39b7
> 
> for you to fetch changes up to e8091b356dd0f1e8e742a5da7a5e8256744b39b7:
> 
>   cmd: dfu: Add error handling for failed registration (2016-08-16 13:06:04 +0200)
> 
> ----------------------------------------------------------------
> B, Ravi (5):
>       spl: dfu: add dfu support in SPL
>       common: dfu: saperate the dfu common functionality
>       spl: dfu: adding dfu support functions for SPL-DFU
>       dra7x: boot: add dfu bootmode support
>       dra7x: configs: enable SPL-DFU support
> 
> Sanchayan Maity (1):
>       cmd: dfu: Add error handling for failed registration
> 
>  Kconfig                                      | 27 +++++++++++++++++++++++++++
>  arch/arm/cpu/armv7/omap-common/boot-common.c |  5 +++++
>  arch/arm/include/asm/arch-omap5/spl.h        |  2 +-
>  cmd/dfu.c                                    | 61 ++-----------------------------------------------------------
>  common/Makefile                              |  3 +++
>  common/dfu.c                                 | 92 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  common/spl/Makefile                          |  1 +
>  common/spl/spl.c                             | 10 +++++++++-
>  common/spl/spl_dfu.c                         | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  include/configs/dra7xx_evm.h                 | 20 +++++++++++++++++++-
>  include/configs/ti_omap5_common.h            |  2 --
>  include/g_dnl.h                              |  1 +
>  include/spl.h                                |  8 ++++++++
>  scripts/Makefile.spl                         |  4 ++++
>  14 files changed, 229 insertions(+), 64 deletions(-)
>  create mode 100644 common/dfu.c
>  create mode 100644 common/spl/spl_dfu.c
> 
> 
> Test HW: Exynos 4412 Odroid U3, Exynos5422 Odroid XU3
> Tested-by: Lukasz Majewski <l.majewski@samsung.com>
> 
> by HWT system
> 


-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

> On 08/16/2016 03:41 PM, Lukasz Majewski wrote:
> > Hi Marek,
> 
> Hi,
> 
> is that for master or next ? 

This patch _was_ supposed to go to "master"

> Was this build tested ?

Unfortunately, not so thoroughly as I thought. 

Moving dfu code to SPL causes following error on some boards:

       arm:  +   smartweb
+In file included from ../include/dfu.h:18:0,
+                 from ../common/dfu.c:16:
+../include/linux/usb/composite.h:331:9: error: requested alignment is not an integer constant
+  struct usb_device_descriptor __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
+         ^
+make[3]: *** [spl/common/dfu.o] Error 1
+make[2]: *** [spl/common] Error 2
+make[1]: *** [spl/u-boot-spl] Error 2
+make: *** [sub-make] Error 2


Ravi, to reproduce you need to fetch u-boot-dfu/test

and run buildman:


This is only a small subset of affected boards, so please test also "arm" (which might take long time).

./tools/buildman/buildman.py --branch=HEAD siemens --detail --verbose
--show_errors --force-build --count=5 --output-dir=./BUILD/


Thanks Marek, for pointing out.


> 
> > Please find following code for u-boot-dfu repository.
> > 
> > The following changes since commit
> > 76b2fad775ee3cb58788b11454655ba5a244ac56:
> > 
> >   eth: asix88179: Add support for the driver model (2016-08-09
> > 12:52:05 +0200)
> > 
> > are available in the git repository at:
> > 
> >   ssh://gu-dfu at git.denx.de/u-boot-dfu
> > e8091b356dd0f1e8e742a5da7a5e8256744b39b7
> > 
> > for you to fetch changes up to
> > e8091b356dd0f1e8e742a5da7a5e8256744b39b7:
> > 
> >   cmd: dfu: Add error handling for failed registration (2016-08-16
> > 13:06:04 +0200)
> > 
> > ----------------------------------------------------------------
> > B, Ravi (5):
> >       spl: dfu: add dfu support in SPL
> >       common: dfu: saperate the dfu common functionality
> >       spl: dfu: adding dfu support functions for SPL-DFU
> >       dra7x: boot: add dfu bootmode support
> >       dra7x: configs: enable SPL-DFU support
> > 
> > Sanchayan Maity (1):
> >       cmd: dfu: Add error handling for failed registration
> > 
> >  Kconfig                                      | 27
> > +++++++++++++++++++++++++++
> > arch/arm/cpu/armv7/omap-common/boot-common.c |  5 +++++
> > arch/arm/include/asm/arch-omap5/spl.h        |  2 +-
> > cmd/dfu.c                                    | 61
> > ++-----------------------------------------------------------
> > common/Makefile                              |  3 +++
> > common/dfu.c                                 | 92
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > common/spl/Makefile                          |  1 +
> > common/spl/spl.c                             | 10 +++++++++-
> > common/spl/spl_dfu.c                         | 57
> > +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > include/configs/dra7xx_evm.h                 | 20
> > +++++++++++++++++++- include/configs/ti_omap5_common.h
> > |  2 -- include/g_dnl.h                              |  1 +
> > include/spl.h                                |  8 ++++++++
> > scripts/Makefile.spl                         |  4 ++++ 14 files
> > changed, 229 insertions(+), 64 deletions(-) create mode 100644
> > common/dfu.c create mode 100644 common/spl/spl_dfu.c
> > 
> > 
> > Test HW: Exynos 4412 Odroid U3, Exynos5422 Odroid XU3
> > Tested-by: Lukasz Majewski <l.majewski@samsung.com>
> > 
> > by HWT system
> > 
> 
> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[B, Ravi]

Hi Lukasz, Marek

>> 
>> is that for master or next ? 

>This patch _was_ supposed to go to "master"

>> Was this build tested ?

>Unfortunately, not so thoroughly as I thought. 

>Moving dfu code to SPL causes following error on some boards:

>       arm:  +   smartweb
>+In file included from ../include/dfu.h:18:0,
>+                 from ../common/dfu.c:16:
>+../include/linux/usb/composite.h:331:9: error: requested alignment is 
>+not an integer constant
>+  struct usb_device_descriptor __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
>+         ^
>+make[3]: *** [spl/common/dfu.o] Error 1
>+make[2]: *** [spl/common] Error 2
>+make[1]: *** [spl/u-boot-spl] Error 2
>+make: *** [sub-make] Error 2


>Ravi, to reproduce you need to fetch u-boot-dfu/test

>and run buildman:


>This is only a small subset of affected boards, so please test also "arm" (which might take long time).

>./tools/buildman/buildman.py --branch=HEAD siemens --detail --verbose --show_errors --force-build --count=5 --output-dir=./BUILD/


>Thanks Marek, for pointing out.

I have test build for all ti-platforms, not checked for rest. My bad ..!
I could reproduce this error, for some platform CONFIG_SYS_CACHELINE_SIZE is not defined. (eg. configs/smartweb_defconfig).
I will check with "arm" as well, let you know.

Regards
Ravi 

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

[B, Ravi]

Hi Heiko 

>>> 
>>> is that for master or next ? 

>>This patch _was_ supposed to go to "master"

>>> Was this build tested ?

>>Unfortunately, not so thoroughly as I thought. 

>>Moving dfu code to SPL causes following error on some boards:

>>       arm:  +   smartweb
>>+In file included from ../include/dfu.h:18:0,
>>+                 from ../common/dfu.c:16:
>>+../include/linux/usb/composite.h:331:9: error: requested alignment is 
>>+not an integer constant
>>+  struct usb_device_descriptor __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
>>+         ^
>>+make[3]: *** [spl/common/dfu.o] Error 1
>>+make[2]: *** [spl/common] Error 2
>+make[1]: *** [spl/u-boot-spl] Error 2
>>+make: *** [sub-make] Error 2

The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb platform which is causing build error.
By defining this error goes away. What would be value for cache line size for smartweb platform? (32/64/..?)

>>Ravi, to reproduce you need to fetch u-boot-dfu/test

>>and run buildman:


>>This is only a small subset of affected boards, so please test also "arm" (which might take long time).

>>./tools/buildman/buildman.py --branch=HEAD siemens --detail --verbose 
>>--show_errors --force-build --count=5 --output-dir=./BUILD/


>>Thanks Marek, for pointing out.

>I have test build for all ti-platforms, not checked for rest. My bad ..!
>I could reproduce this error, for some platform CONFIG_SYS_CACHELINE_SIZE is not defined. (eg. configs/smartweb_defconfig).
>I will check with "arm" as well, let you know.

Regards
Ravi 

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

[Heiko Schocher]

Hello B, Ravi,

Am 17.08.2016 um 09:40 schrieb B, Ravi:
> Hi Heiko
>
>>>>
>>>> is that for master or next ?
>
>>> This patch _was_ supposed to go to "master"
>
>>>> Was this build tested ?
>
>>> Unfortunately, not so thoroughly as I thought.
>
>>> Moving dfu code to SPL causes following error on some boards:
>
>>>        arm:  +   smartweb
>>> +In file included from ../include/dfu.h:18:0,
>>> +                 from ../common/dfu.c:16:
>>> +../include/linux/usb/composite.h:331:9: error: requested alignment is
>>> +not an integer constant
>>> +  struct usb_device_descriptor __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
>>> +         ^
>>> +make[3]: *** [spl/common/dfu.o] Error 1
>>> +make[2]: *** [spl/common] Error 2
>> +make[1]: *** [spl/u-boot-spl] Error 2
>>> +make: *** [sub-make] Error 2
>
> The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb platform which is causing build error.
> By defining this error goes away. What would be value for cache line size for smartweb platform? (32/64/..?)

Hups? Which patch introduced this? I wonder if other at91
based boards does not show the same error?

The RM for the at91sam9260 says:  8 words cache line size

So 32 would be the correct value.

bye,
Heiko
>
>>> Ravi, to reproduce you need to fetch u-boot-dfu/test
>
>>> and run buildman:
>
>
>>> This is only a small subset of affected boards, so please test also "arm" (which might take long time).
>
>>> ./tools/buildman/buildman.py --branch=HEAD siemens --detail --verbose
>>> --show_errors --force-build --count=5 --output-dir=./BUILD/
>
>
>>> Thanks Marek, for pointing out.
>
>> I have test build for all ti-platforms, not checked for rest. My bad ..!
>> I could reproduce this error, for some platform CONFIG_SYS_CACHELINE_SIZE is not defined. (eg. configs/smartweb_defconfig).
>> I will check with "arm" as well, let you know.
>
> Regards
> Ravi
>

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

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

[B, Ravi]

>>>> +is not an integer constant
>>>> +  struct usb_device_descriptor __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
>>>> +         ^
>>>> +make[3]: *** [spl/common/dfu.o] Error 1
>>>> +make[2]: *** [spl/common] Error 2
>>> +make[1]: *** [spl/u-boot-spl] Error 2
>>>> +make: *** [sub-make] Error 2
>>
>> The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb platform which is causing build error.
>> By defining this error goes away. What would be value for cache line 
>> size for smartweb platform? (32/64/..?)

>Hups? Which patch introduced this? I wonder if other at91 based boards does not show the same error?

The spl-dfu patch series include usb header files in SPL for DFU, since cache line is not defined which is causing
the build failure for smartweb platform.

>The RM for the at91sam9260 says:  8 words cache line size

>So 32 would be the correct value.

Ok thanks. 

Regards
Ravi

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

[Lukasz Majewski]

Hi Heiko, Ravi,

> Hello B, Ravi,
> 
> Am 17.08.2016 um 09:40 schrieb B, Ravi:
> > Hi Heiko
> >
> >>>>
> >>>> is that for master or next ?
> >
> >>> This patch _was_ supposed to go to "master"
> >
> >>>> Was this build tested ?
> >
> >>> Unfortunately, not so thoroughly as I thought.
> >
> >>> Moving dfu code to SPL causes following error on some boards:
> >
> >>>        arm:  +   smartweb
> >>> +In file included from ../include/dfu.h:18:0,
> >>> +                 from ../common/dfu.c:16:
> >>> +../include/linux/usb/composite.h:331:9: error: requested
> >>> alignment is +not an integer constant
> >>> +  struct usb_device_descriptor
> >>> __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
> >>> +         ^
> >>> +make[3]: *** [spl/common/dfu.o] Error 1
> >>> +make[2]: *** [spl/common] Error 2
> >> +make[1]: *** [spl/u-boot-spl] Error 2
> >>> +make: *** [sub-make] Error 2
> >
> > The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb platform
> > which is causing build error. By defining this error goes away.
> > What would be value for cache line size for smartweb platform?
> > (32/64/..?)
> 
> Hups? Which patch introduced this? I wonder if other at91
> based boards does not show the same error?
> 
> The RM for the at91sam9260 says:  8 words cache line size
> 
> So 32 would be the correct value.

Heiko, thanks for your support.

Ravi, Marek Vasut before accepting PR tests following archs with
buildman: arm, aarch64, mips, powerpc, nios2 [1]

Regarding the CONFIG_SYS_CACHELINE_SIZE, some older boards do not
define it. Hence we have several problems (also with other patches
http://patchwork.ozlabs.org/patch/657216/).

IMHO, it would be best to check [1] and then add this define if
required.

Or does anybody have any better idea?

> 
> bye,
> Heiko
> >
> >>> Ravi, to reproduce you need to fetch u-boot-dfu/test
> >
> >>> and run buildman:
> >
> >
> >>> This is only a small subset of affected boards, so please test
> >>> also "arm" (which might take long time).
> >
> >>> ./tools/buildman/buildman.py --branch=HEAD siemens --detail
> >>> --verbose --show_errors --force-build --count=5
> >>> --output-dir=./BUILD/
> >
> >
> >>> Thanks Marek, for pointing out.
> >
> >> I have test build for all ti-platforms, not checked for rest. My
> >> bad ..! I could reproduce this error, for some platform
> >> CONFIG_SYS_CACHELINE_SIZE is not defined. (eg.
> >> configs/smartweb_defconfig). I will check with "arm" as well, let
> >> you know.
> >
> > Regards
> > Ravi
> >
> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Tom Rini]

On Wed, Aug 17, 2016 at 11:56:59AM +0200, Lukasz Majewski wrote:
> Hi Heiko, Ravi,
> 
> > Hello B, Ravi,
> > 
> > Am 17.08.2016 um 09:40 schrieb B, Ravi:
> > > Hi Heiko
> > >
> > >>>>
> > >>>> is that for master or next ?
> > >
> > >>> This patch _was_ supposed to go to "master"
> > >
> > >>>> Was this build tested ?
> > >
> > >>> Unfortunately, not so thoroughly as I thought.
> > >
> > >>> Moving dfu code to SPL causes following error on some boards:
> > >
> > >>>        arm:  +   smartweb
> > >>> +In file included from ../include/dfu.h:18:0,
> > >>> +                 from ../common/dfu.c:16:
> > >>> +../include/linux/usb/composite.h:331:9: error: requested
> > >>> alignment is +not an integer constant
> > >>> +  struct usb_device_descriptor
> > >>> __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
> > >>> +         ^
> > >>> +make[3]: *** [spl/common/dfu.o] Error 1
> > >>> +make[2]: *** [spl/common] Error 2
> > >> +make[1]: *** [spl/u-boot-spl] Error 2
> > >>> +make: *** [sub-make] Error 2
> > >
> > > The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb platform
> > > which is causing build error. By defining this error goes away.
> > > What would be value for cache line size for smartweb platform?
> > > (32/64/..?)
> > 
> > Hups? Which patch introduced this? I wonder if other at91
> > based boards does not show the same error?
> > 
> > The RM for the at91sam9260 says:  8 words cache line size
> > 
> > So 32 would be the correct value.
> 
> Heiko, thanks for your support.
> 
> Ravi, Marek Vasut before accepting PR tests following archs with
> buildman: arm, aarch64, mips, powerpc, nios2 [1]
> 
> Regarding the CONFIG_SYS_CACHELINE_SIZE, some older boards do not
> define it. Hence we have several problems (also with other patches
> http://patchwork.ozlabs.org/patch/657216/).
> 
> IMHO, it would be best to check [1] and then add this define if
> required.
> 
> Or does anybody have any better idea?

This is something we can get right in Kconfig, I believe.  Looking at
the Linux Kernel, the relevant bits I think are in arch/arm/mm/Kconfig:
config ARM_L1_CACHE_SHIFT_6
        bool
        default y if CPU_V7
        help
          Setting ARM L1 cache line size to 64 Bytes.

config ARM_L1_CACHE_SHIFT
        int
        default 6 if ARM_L1_CACHE_SHIFT_6
        default 5

I only hesitate a little as there's not a 1:1 match-up between those
values and all ARMv7 platforms we have today for example.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160817/f70bd07d/attachment.sig>

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

[Lukasz Majewski]

Hi Tom,

> On Wed, Aug 17, 2016 at 11:56:59AM +0200, Lukasz Majewski wrote:
> > Hi Heiko, Ravi,
> > 
> > > Hello B, Ravi,
> > > 
> > > Am 17.08.2016 um 09:40 schrieb B, Ravi:
> > > > Hi Heiko
> > > >
> > > >>>>
> > > >>>> is that for master or next ?
> > > >
> > > >>> This patch _was_ supposed to go to "master"
> > > >
> > > >>>> Was this build tested ?
> > > >
> > > >>> Unfortunately, not so thoroughly as I thought.
> > > >
> > > >>> Moving dfu code to SPL causes following error on some boards:
> > > >
> > > >>>        arm:  +   smartweb
> > > >>> +In file included from ../include/dfu.h:18:0,
> > > >>> +                 from ../common/dfu.c:16:
> > > >>> +../include/linux/usb/composite.h:331:9: error: requested
> > > >>> alignment is +not an integer constant
> > > >>> +  struct usb_device_descriptor
> > > >>> __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
> > > >>> +         ^
> > > >>> +make[3]: *** [spl/common/dfu.o] Error 1
> > > >>> +make[2]: *** [spl/common] Error 2
> > > >> +make[1]: *** [spl/u-boot-spl] Error 2
> > > >>> +make: *** [sub-make] Error 2
> > > >
> > > > The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb
> > > > platform which is causing build error. By defining this error
> > > > goes away. What would be value for cache line size for smartweb
> > > > platform? (32/64/..?)
> > > 
> > > Hups? Which patch introduced this? I wonder if other at91
> > > based boards does not show the same error?
> > > 
> > > The RM for the at91sam9260 says:  8 words cache line size
> > > 
> > > So 32 would be the correct value.
> > 
> > Heiko, thanks for your support.
> > 
> > Ravi, Marek Vasut before accepting PR tests following archs with
> > buildman: arm, aarch64, mips, powerpc, nios2 [1]
> > 
> > Regarding the CONFIG_SYS_CACHELINE_SIZE, some older boards do not
> > define it. Hence we have several problems (also with other patches
> > http://patchwork.ozlabs.org/patch/657216/).
> > 
> > IMHO, it would be best to check [1] and then add this define if
> > required.
> > 
> > Or does anybody have any better idea?
> 
> This is something we can get right in Kconfig, I believe.  Looking at
> the Linux Kernel, the relevant bits I think are in
> arch/arm/mm/Kconfig: config ARM_L1_CACHE_SHIFT_6
>         bool
>         default y if CPU_V7
>         help
>           Setting ARM L1 cache line size to 64 Bytes.
> 
> config ARM_L1_CACHE_SHIFT
>         int
>         default 6 if ARM_L1_CACHE_SHIFT_6
>         default 5
> 
> I only hesitate a little as there's not a 1:1 match-up between those
> values and all ARMv7 platforms we have today for example.

True.

I do recall that some Cortex-A9 had 32B L1 cache and Cortex-A8 had 64B
L1 cache line size.

And both were ARMv7 cores.

> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Tom Rini]

On Wed, Aug 17, 2016 at 04:55:52PM +0200, Lukasz Majewski wrote:
> Hi Tom,
> 
> > On Wed, Aug 17, 2016 at 11:56:59AM +0200, Lukasz Majewski wrote:
> > > Hi Heiko, Ravi,
> > > 
> > > > Hello B, Ravi,
> > > > 
> > > > Am 17.08.2016 um 09:40 schrieb B, Ravi:
> > > > > Hi Heiko
> > > > >
> > > > >>>>
> > > > >>>> is that for master or next ?
> > > > >
> > > > >>> This patch _was_ supposed to go to "master"
> > > > >
> > > > >>>> Was this build tested ?
> > > > >
> > > > >>> Unfortunately, not so thoroughly as I thought.
> > > > >
> > > > >>> Moving dfu code to SPL causes following error on some boards:
> > > > >
> > > > >>>        arm:  +   smartweb
> > > > >>> +In file included from ../include/dfu.h:18:0,
> > > > >>> +                 from ../common/dfu.c:16:
> > > > >>> +../include/linux/usb/composite.h:331:9: error: requested
> > > > >>> alignment is +not an integer constant
> > > > >>> +  struct usb_device_descriptor
> > > > >>> __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
> > > > >>> +         ^
> > > > >>> +make[3]: *** [spl/common/dfu.o] Error 1
> > > > >>> +make[2]: *** [spl/common] Error 2
> > > > >> +make[1]: *** [spl/u-boot-spl] Error 2
> > > > >>> +make: *** [sub-make] Error 2
> > > > >
> > > > > The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb
> > > > > platform which is causing build error. By defining this error
> > > > > goes away. What would be value for cache line size for smartweb
> > > > > platform? (32/64/..?)
> > > > 
> > > > Hups? Which patch introduced this? I wonder if other at91
> > > > based boards does not show the same error?
> > > > 
> > > > The RM for the at91sam9260 says:  8 words cache line size
> > > > 
> > > > So 32 would be the correct value.
> > > 
> > > Heiko, thanks for your support.
> > > 
> > > Ravi, Marek Vasut before accepting PR tests following archs with
> > > buildman: arm, aarch64, mips, powerpc, nios2 [1]
> > > 
> > > Regarding the CONFIG_SYS_CACHELINE_SIZE, some older boards do not
> > > define it. Hence we have several problems (also with other patches
> > > http://patchwork.ozlabs.org/patch/657216/).
> > > 
> > > IMHO, it would be best to check [1] and then add this define if
> > > required.
> > > 
> > > Or does anybody have any better idea?
> > 
> > This is something we can get right in Kconfig, I believe.  Looking at
> > the Linux Kernel, the relevant bits I think are in
> > arch/arm/mm/Kconfig: config ARM_L1_CACHE_SHIFT_6
> >         bool
> >         default y if CPU_V7
> >         help
> >           Setting ARM L1 cache line size to 64 Bytes.
> > 
> > config ARM_L1_CACHE_SHIFT
> >         int
> >         default 6 if ARM_L1_CACHE_SHIFT_6
> >         default 5
> > 
> > I only hesitate a little as there's not a 1:1 match-up between those
> > values and all ARMv7 platforms we have today for example.
> 
> True.
> 
> I do recall that some Cortex-A9 had 32B L1 cache and Cortex-A8 had 64B
> L1 cache line size.
> 
> And both were ARMv7 cores.

I had a quick conversation with some folks and the answer is that it's
OK to round up (as it appears that our uses of CONFIG_SYS_CACHELINE_SIZE
are) so long as the mechanics of the flushes work at the smallest
implemented increment, which is what the Linux Kernel does and we're
using that code.  So we should be safe to mirror the kernel logic here.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160817/99f4ed79/attachment.sig>

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

[Lukasz Majewski]

Hi Tom,

> On Wed, Aug 17, 2016 at 04:55:52PM +0200, Lukasz Majewski wrote:
> > Hi Tom,
> > 
> > > On Wed, Aug 17, 2016 at 11:56:59AM +0200, Lukasz Majewski wrote:
> > > > Hi Heiko, Ravi,
> > > > 
> > > > > Hello B, Ravi,
> > > > > 
> > > > > Am 17.08.2016 um 09:40 schrieb B, Ravi:
> > > > > > Hi Heiko
> > > > > >
> > > > > >>>>
> > > > > >>>> is that for master or next ?
> > > > > >
> > > > > >>> This patch _was_ supposed to go to "master"
> > > > > >
> > > > > >>>> Was this build tested ?
> > > > > >
> > > > > >>> Unfortunately, not so thoroughly as I thought.
> > > > > >
> > > > > >>> Moving dfu code to SPL causes following error on some
> > > > > >>> boards:
> > > > > >
> > > > > >>>        arm:  +   smartweb
> > > > > >>> +In file included from ../include/dfu.h:18:0,
> > > > > >>> +                 from ../common/dfu.c:16:
> > > > > >>> +../include/linux/usb/composite.h:331:9: error: requested
> > > > > >>> alignment is +not an integer constant
> > > > > >>> +  struct usb_device_descriptor
> > > > > >>> __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
> > > > > >>> +         ^
> > > > > >>> +make[3]: *** [spl/common/dfu.o] Error 1
> > > > > >>> +make[2]: *** [spl/common] Error 2
> > > > > >> +make[1]: *** [spl/u-boot-spl] Error 2
> > > > > >>> +make: *** [sub-make] Error 2
> > > > > >
> > > > > > The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb
> > > > > > platform which is causing build error. By defining this
> > > > > > error goes away. What would be value for cache line size
> > > > > > for smartweb platform? (32/64/..?)
> > > > > 
> > > > > Hups? Which patch introduced this? I wonder if other at91
> > > > > based boards does not show the same error?
> > > > > 
> > > > > The RM for the at91sam9260 says:  8 words cache line size
> > > > > 
> > > > > So 32 would be the correct value.
> > > > 
> > > > Heiko, thanks for your support.
> > > > 
> > > > Ravi, Marek Vasut before accepting PR tests following archs with
> > > > buildman: arm, aarch64, mips, powerpc, nios2 [1]
> > > > 
> > > > Regarding the CONFIG_SYS_CACHELINE_SIZE, some older boards do
> > > > not define it. Hence we have several problems (also with other
> > > > patches http://patchwork.ozlabs.org/patch/657216/).
> > > > 
> > > > IMHO, it would be best to check [1] and then add this define if
> > > > required.
> > > > 
> > > > Or does anybody have any better idea?
> > > 
> > > This is something we can get right in Kconfig, I believe.
> > > Looking at the Linux Kernel, the relevant bits I think are in
> > > arch/arm/mm/Kconfig: config ARM_L1_CACHE_SHIFT_6
> > >         bool
> > >         default y if CPU_V7
> > >         help
> > >           Setting ARM L1 cache line size to 64 Bytes.
> > > 
> > > config ARM_L1_CACHE_SHIFT
> > >         int
> > >         default 6 if ARM_L1_CACHE_SHIFT_6
> > >         default 5
> > > 
> > > I only hesitate a little as there's not a 1:1 match-up between
> > > those values and all ARMv7 platforms we have today for example.
> > 
> > True.
> > 
> > I do recall that some Cortex-A9 had 32B L1 cache and Cortex-A8 had
> > 64B L1 cache line size.
> > 
> > And both were ARMv7 cores.
> 
> I had a quick conversation with some folks and the answer is that it's
> OK to round up (as it appears that our uses of
> CONFIG_SYS_CACHELINE_SIZE are) so long as the mechanics of the
> flushes work at the smallest implemented increment, which is what the
> Linux Kernel does and we're using that code.  So we should be safe to
> mirror the kernel logic here.

+1

If I understood you correctly, for armv7 boards with undefined
CONFIG_SYS_CACHELINE_SIZE we should use the largest cache line (IIRC
64B).

> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Tom Rini]

On Wed, Aug 17, 2016 at 06:17:44PM +0200, Lukasz Majewski wrote:
> Hi Tom,
> 
> > On Wed, Aug 17, 2016 at 04:55:52PM +0200, Lukasz Majewski wrote:
> > > Hi Tom,
> > > 
> > > > On Wed, Aug 17, 2016 at 11:56:59AM +0200, Lukasz Majewski wrote:
> > > > > Hi Heiko, Ravi,
> > > > > 
> > > > > > Hello B, Ravi,
> > > > > > 
> > > > > > Am 17.08.2016 um 09:40 schrieb B, Ravi:
> > > > > > > Hi Heiko
> > > > > > >
> > > > > > >>>>
> > > > > > >>>> is that for master or next ?
> > > > > > >
> > > > > > >>> This patch _was_ supposed to go to "master"
> > > > > > >
> > > > > > >>>> Was this build tested ?
> > > > > > >
> > > > > > >>> Unfortunately, not so thoroughly as I thought.
> > > > > > >
> > > > > > >>> Moving dfu code to SPL causes following error on some
> > > > > > >>> boards:
> > > > > > >
> > > > > > >>>        arm:  +   smartweb
> > > > > > >>> +In file included from ../include/dfu.h:18:0,
> > > > > > >>> +                 from ../common/dfu.c:16:
> > > > > > >>> +../include/linux/usb/composite.h:331:9: error: requested
> > > > > > >>> alignment is +not an integer constant
> > > > > > >>> +  struct usb_device_descriptor
> > > > > > >>> __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
> > > > > > >>> +         ^
> > > > > > >>> +make[3]: *** [spl/common/dfu.o] Error 1
> > > > > > >>> +make[2]: *** [spl/common] Error 2
> > > > > > >> +make[1]: *** [spl/u-boot-spl] Error 2
> > > > > > >>> +make: *** [sub-make] Error 2
> > > > > > >
> > > > > > > The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb
> > > > > > > platform which is causing build error. By defining this
> > > > > > > error goes away. What would be value for cache line size
> > > > > > > for smartweb platform? (32/64/..?)
> > > > > > 
> > > > > > Hups? Which patch introduced this? I wonder if other at91
> > > > > > based boards does not show the same error?
> > > > > > 
> > > > > > The RM for the at91sam9260 says:  8 words cache line size
> > > > > > 
> > > > > > So 32 would be the correct value.
> > > > > 
> > > > > Heiko, thanks for your support.
> > > > > 
> > > > > Ravi, Marek Vasut before accepting PR tests following archs with
> > > > > buildman: arm, aarch64, mips, powerpc, nios2 [1]
> > > > > 
> > > > > Regarding the CONFIG_SYS_CACHELINE_SIZE, some older boards do
> > > > > not define it. Hence we have several problems (also with other
> > > > > patches http://patchwork.ozlabs.org/patch/657216/).
> > > > > 
> > > > > IMHO, it would be best to check [1] and then add this define if
> > > > > required.
> > > > > 
> > > > > Or does anybody have any better idea?
> > > > 
> > > > This is something we can get right in Kconfig, I believe.
> > > > Looking at the Linux Kernel, the relevant bits I think are in
> > > > arch/arm/mm/Kconfig: config ARM_L1_CACHE_SHIFT_6
> > > >         bool
> > > >         default y if CPU_V7
> > > >         help
> > > >           Setting ARM L1 cache line size to 64 Bytes.
> > > > 
> > > > config ARM_L1_CACHE_SHIFT
> > > >         int
> > > >         default 6 if ARM_L1_CACHE_SHIFT_6
> > > >         default 5
> > > > 
> > > > I only hesitate a little as there's not a 1:1 match-up between
> > > > those values and all ARMv7 platforms we have today for example.
> > > 
> > > True.
> > > 
> > > I do recall that some Cortex-A9 had 32B L1 cache and Cortex-A8 had
> > > 64B L1 cache line size.
> > > 
> > > And both were ARMv7 cores.
> > 
> > I had a quick conversation with some folks and the answer is that it's
> > OK to round up (as it appears that our uses of
> > CONFIG_SYS_CACHELINE_SIZE are) so long as the mechanics of the
> > flushes work at the smallest implemented increment, which is what the
> > Linux Kernel does and we're using that code.  So we should be safe to
> > mirror the kernel logic here.
> 
> +1
> 
> If I understood you correctly, for armv7 boards with undefined
> CONFIG_SYS_CACHELINE_SIZE we should use the largest cache line (IIRC
> 64B).

I'm saying we should just mirror the kernel and universally use 64B on
CPU_V7 (and note, not V7M) and 32B otherwise, wrt ARM at least.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160817/6ecc6310/attachment.sig>

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

[Lukasz Majewski]

Hi Tom,

> On Wed, Aug 17, 2016 at 06:17:44PM +0200, Lukasz Majewski wrote:
> > Hi Tom,
> > 
> > > On Wed, Aug 17, 2016 at 04:55:52PM +0200, Lukasz Majewski wrote:
> > > > Hi Tom,
> > > > 
> > > > > On Wed, Aug 17, 2016 at 11:56:59AM +0200, Lukasz Majewski
> > > > > wrote:
> > > > > > Hi Heiko, Ravi,
> > > > > > 
> > > > > > > Hello B, Ravi,
> > > > > > > 
> > > > > > > Am 17.08.2016 um 09:40 schrieb B, Ravi:
> > > > > > > > Hi Heiko
> > > > > > > >
> > > > > > > >>>>
> > > > > > > >>>> is that for master or next ?
> > > > > > > >
> > > > > > > >>> This patch _was_ supposed to go to "master"
> > > > > > > >
> > > > > > > >>>> Was this build tested ?
> > > > > > > >
> > > > > > > >>> Unfortunately, not so thoroughly as I thought.
> > > > > > > >
> > > > > > > >>> Moving dfu code to SPL causes following error on some
> > > > > > > >>> boards:
> > > > > > > >
> > > > > > > >>>        arm:  +   smartweb
> > > > > > > >>> +In file included from ../include/dfu.h:18:0,
> > > > > > > >>> +                 from ../common/dfu.c:16:
> > > > > > > >>> +../include/linux/usb/composite.h:331:9: error:
> > > > > > > >>> requested alignment is +not an integer constant
> > > > > > > >>> +  struct usb_device_descriptor
> > > > > > > >>> __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
> > > > > > > >>> +         ^
> > > > > > > >>> +make[3]: *** [spl/common/dfu.o] Error 1
> > > > > > > >>> +make[2]: *** [spl/common] Error 2
> > > > > > > >> +make[1]: *** [spl/u-boot-spl] Error 2
> > > > > > > >>> +make: *** [sub-make] Error 2
> > > > > > > >
> > > > > > > > The CONFIG_SYS_CACHELINE_SIZE is not defined for
> > > > > > > > smartweb platform which is causing build error. By
> > > > > > > > defining this error goes away. What would be value for
> > > > > > > > cache line size for smartweb platform? (32/64/..?)
> > > > > > > 
> > > > > > > Hups? Which patch introduced this? I wonder if other at91
> > > > > > > based boards does not show the same error?
> > > > > > > 
> > > > > > > The RM for the at91sam9260 says:  8 words cache line size
> > > > > > > 
> > > > > > > So 32 would be the correct value.
> > > > > > 
> > > > > > Heiko, thanks for your support.
> > > > > > 
> > > > > > Ravi, Marek Vasut before accepting PR tests following archs
> > > > > > with buildman: arm, aarch64, mips, powerpc, nios2 [1]
> > > > > > 
> > > > > > Regarding the CONFIG_SYS_CACHELINE_SIZE, some older boards
> > > > > > do not define it. Hence we have several problems (also with
> > > > > > other patches http://patchwork.ozlabs.org/patch/657216/).
> > > > > > 
> > > > > > IMHO, it would be best to check [1] and then add this
> > > > > > define if required.
> > > > > > 
> > > > > > Or does anybody have any better idea?
> > > > > 
> > > > > This is something we can get right in Kconfig, I believe.
> > > > > Looking at the Linux Kernel, the relevant bits I think are in
> > > > > arch/arm/mm/Kconfig: config ARM_L1_CACHE_SHIFT_6
> > > > >         bool
> > > > >         default y if CPU_V7
> > > > >         help
> > > > >           Setting ARM L1 cache line size to 64 Bytes.
> > > > > 
> > > > > config ARM_L1_CACHE_SHIFT
> > > > >         int
> > > > >         default 6 if ARM_L1_CACHE_SHIFT_6
> > > > >         default 5
> > > > > 
> > > > > I only hesitate a little as there's not a 1:1 match-up between
> > > > > those values and all ARMv7 platforms we have today for
> > > > > example.
> > > > 
> > > > True.
> > > > 
> > > > I do recall that some Cortex-A9 had 32B L1 cache and Cortex-A8
> > > > had 64B L1 cache line size.
> > > > 
> > > > And both were ARMv7 cores.
> > > 
> > > I had a quick conversation with some folks and the answer is that
> > > it's OK to round up (as it appears that our uses of
> > > CONFIG_SYS_CACHELINE_SIZE are) so long as the mechanics of the
> > > flushes work at the smallest implemented increment, which is what
> > > the Linux Kernel does and we're using that code.  So we should be
> > > safe to mirror the kernel logic here.
> > 
> > +1
> > 
> > If I understood you correctly, for armv7 boards with undefined
> > CONFIG_SYS_CACHELINE_SIZE we should use the largest cache line (IIRC
> > 64B).
> 
> I'm saying we should just mirror the kernel and universally use 64B on
> CPU_V7 (and note, not V7M) and 32B otherwise, wrt ARM at least.
> 

Ok, now it is clear :-)

Thanks for providing solution.

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Łukasz Majewski]

Dear Marek,

The following changes since commit d81a1de96e2636163783c342b8fda965e696e382:

   misc: Kconfig: Add SPL_I2C_EEPROM option (2017-09-11 22:44:30 -0400)

are available in the git repository at:

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

for you to fetch changes up to 9bfa5a43ed4faeda391be5015a0d9832701102ae:

   usb: dwc2: Align size of invalidating dcache before starting DMA 
(2017-09-13 12:54:14 +0200)

----------------------------------------------------------------
Eddie Cai (4):
       usb: rockchip: add the rockusb gadget
       usb: rockchip: add rockusb command
       rockchip:usb: add a simple readme for rockusb
       rockchip: rk3288: enable rockusb support on rk3288 based device

Seung-Woo Kim (1):
       usb: dwc2: Align size of invalidating dcache before starting DMA

  arch/arm/include/asm/arch-rockchip/f_rockusb.h | 132 
+++++++++++++++++++++++++++++++++++
  arch/arm/mach-rockchip/Kconfig                 |   2 +
  cmd/Kconfig                                    |   9 +++
  cmd/Makefile                                   |   1 +
  cmd/rockusb.c                                  |  74 ++++++++++++++++++++
  configs/chromebit_mickey_defconfig             |   8 +++
  configs/chromebook_jerry_defconfig             |   8 +++
  configs/chromebook_minnie_defconfig            |   8 +++
  configs/evb-rk3288_defconfig                   |  10 +++
  configs/fennec-rk3288_defconfig                |   6 ++
  configs/firefly-rk3288_defconfig               |   6 ++
  configs/miqi-rk3288_defconfig                  |   6 ++
  configs/phycore-rk3288_defconfig               |   7 ++
  configs/popmetal-rk3288_defconfig              |   6 ++
  configs/rock2_defconfig                        |   9 +++
  configs/tinker-rk3288_defconfig                |   6 ++
  doc/README.rockusb                             |  51 ++++++++++++++
  drivers/usb/gadget/Kconfig                     |   8 +++
  drivers/usb/gadget/Makefile                    |   1 +
  drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c     |   3 +-
  drivers/usb/gadget/f_rockusb.c                 | 691 
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
  include/configs/rk3288_common.h                |  12 ++++
  22 files changed, 1063 insertions(+), 1 deletion(-)
  create mode 100644 arch/arm/include/asm/arch-rockchip/f_rockusb.h
  create mode 100644 cmd/rockusb.c
  create mode 100644 doc/README.rockusb
  create mode 100644 drivers/usb/gadget/f_rockusb.c


Travis CI tested:

https://travis-ci.org/lmajewski/u-boot-dfu/builds/274944050
https://travis-ci.org/lmajewski/u-boot-dfu/builds/274944178
https://travis-ci.org/lmajewski/u-boot-dfu/builds/274945208
https://travis-ci.org/lmajewski/u-boot-dfu/builds/274965975


-- 
Best regards.

Łukasz Majewski

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

[Marek Vasut]

On 09/13/2017 01:07 PM, Łukasz Majewski wrote:
> Dear Marek,
> 
> The following changes since commit
> d81a1de96e2636163783c342b8fda965e696e382:
> 
>   misc: Kconfig: Add SPL_I2C_EEPROM option (2017-09-11 22:44:30 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git
> 
> for you to fetch changes up to 9bfa5a43ed4faeda391be5015a0d9832701102ae:
> 
>   usb: dwc2: Align size of invalidating dcache before starting DMA
> (2017-09-13 12:54:14 +0200)
> 
> ----------------------------------------------------------------
> Eddie Cai (4):
>       usb: rockchip: add the rockusb gadget
>       usb: rockchip: add rockusb command
>       rockchip:usb: add a simple readme for rockusb
>       rockchip: rk3288: enable rockusb support on rk3288 based device
> 
> Seung-Woo Kim (1):
>       usb: dwc2: Align size of invalidating dcache before starting DMA
> 
>  arch/arm/include/asm/arch-rockchip/f_rockusb.h | 132
> +++++++++++++++++++++++++++++++++++
>  arch/arm/mach-rockchip/Kconfig                 |   2 +
>  cmd/Kconfig                                    |   9 +++
>  cmd/Makefile                                   |   1 +
>  cmd/rockusb.c                                  |  74 ++++++++++++++++++++
>  configs/chromebit_mickey_defconfig             |   8 +++
>  configs/chromebook_jerry_defconfig             |   8 +++
>  configs/chromebook_minnie_defconfig            |   8 +++
>  configs/evb-rk3288_defconfig                   |  10 +++
>  configs/fennec-rk3288_defconfig                |   6 ++
>  configs/firefly-rk3288_defconfig               |   6 ++
>  configs/miqi-rk3288_defconfig                  |   6 ++
>  configs/phycore-rk3288_defconfig               |   7 ++
>  configs/popmetal-rk3288_defconfig              |   6 ++
>  configs/rock2_defconfig                        |   9 +++
>  configs/tinker-rk3288_defconfig                |   6 ++
>  doc/README.rockusb                             |  51 ++++++++++++++
>  drivers/usb/gadget/Kconfig                     |   8 +++
>  drivers/usb/gadget/Makefile                    |   1 +
>  drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c     |   3 +-
>  drivers/usb/gadget/f_rockusb.c                 | 691
> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> 
>  include/configs/rk3288_common.h                |  12 ++++
>  22 files changed, 1063 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm/include/asm/arch-rockchip/f_rockusb.h
>  create mode 100644 cmd/rockusb.c
>  create mode 100644 doc/README.rockusb
>  create mode 100644 drivers/usb/gadget/f_rockusb.c
> 
> 
> Travis CI tested:
> 
> https://travis-ci.org/lmajewski/u-boot-dfu/builds/274944050
> https://travis-ci.org/lmajewski/u-boot-dfu/builds/274944178
> https://travis-ci.org/lmajewski/u-boot-dfu/builds/274945208
> https://travis-ci.org/lmajewski/u-boot-dfu/builds/274965975

Applied, thanks



-- 
Best regards,
Marek Vasut

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

[Łukasz Majewski]

Dear Marek,

Those patches are preferably for -next merge widow.



The following changes since commit ae61b5353b48fa49c66376f273cc53cd3c06b656:

   usb: dwc3: fix Kconfig dependency to accept host driver in 
drivers/usb/dwc3 (2017-08-25 11:23:27 +0200)

are available in the git repository at:

   u-boot-denx-dfu/master

for you to fetch changes up to 6496bc35d93975f01551dc3960b88807eae9d6fb:

   rockchip: rk3288: enable rockusb support on rk3288 based device 
(2017-08-28 21:20:50 +0200)

----------------------------------------------------------------
Eddie Cai (4):
       usb: rockchip: add the rockusb gadget
       usb: rockchip: add rockusb command
       rockchip:usb: add a simple readme for rockusb
       rockchip: rk3288: enable rockusb support on rk3288 based device

  arch/arm/include/asm/arch-rockchip/f_rockusb.h | 132 ++++++++++++++++++++
  arch/arm/mach-rockchip/Kconfig                 |   2 +
  cmd/Kconfig                                    |   9 ++
  cmd/Makefile                                   |   1 +
  cmd/rockusb.c                                  |  74 ++++++++++++
  configs/chromebit_mickey_defconfig             |   9 ++
  configs/chromebook_jerry_defconfig             |   9 ++
  configs/chromebook_minnie_defconfig            |   9 ++
  configs/evb-rk3288_defconfig                   |  10 ++
  configs/fennec-rk3288_defconfig                |   6 +
  configs/firefly-rk3288_defconfig               |   6 +
  configs/miqi-rk3288_defconfig                  |   6 +
  configs/phycore-rk3288_defconfig               |   7 ++
  configs/popmetal-rk3288_defconfig              |   6 +
  configs/rock2_defconfig                        |  10 ++
  configs/tinker-rk3288_defconfig                |   6 +
  doc/README.rockusb                             |  51 ++++++++
  drivers/usb/gadget/Kconfig                     |   8 ++
  drivers/usb/gadget/Makefile                    |   1 +
  drivers/usb/gadget/f_rockusb.c                 | 691 
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
  include/configs/rk3288_common.h                |   7 --
  21 files changed, 1053 insertions(+), 7 deletions(-)
  create mode 100644 arch/arm/include/asm/arch-rockchip/f_rockusb.h
  create mode 100644 cmd/rockusb.c
  create mode 100644 doc/README.rockusb
  create mode 100644 drivers/usb/gadget/f_rockusb.c


Travis-CI output (all clean):

https://travis-ci.org/lmajewski/u-boot-dfu/builds/269119055
https://travis-ci.org/lmajewski/u-boot-dfu/builds/269118570
https://travis-ci.org/lmajewski/u-boot-dfu/builds/269118344
https://travis-ci.org/lmajewski/u-boot-dfu/builds/269117873
https://travis-ci.org/lmajewski/u-boot-dfu/builds/269099287

-- 
Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

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

[Marek Vasut]

On 08/28/2017 09:28 PM, Łukasz Majewski wrote:
> Dear Marek,
> 
> Those patches are preferably for -next merge widow.
> 
> 
> 
> The following changes since commit
> ae61b5353b48fa49c66376f273cc53cd3c06b656:
> 
>   usb: dwc3: fix Kconfig dependency to accept host driver in
> drivers/usb/dwc3 (2017-08-25 11:23:27 +0200)
> 
> are available in the git repository at:
> 
>   u-boot-denx-dfu/master

Might be a good idea to put it into -next branch then.

Applied

> for you to fetch changes up to 6496bc35d93975f01551dc3960b88807eae9d6fb:
> 
>   rockchip: rk3288: enable rockusb support on rk3288 based device
> (2017-08-28 21:20:50 +0200)
> 
> ----------------------------------------------------------------
> Eddie Cai (4):
>       usb: rockchip: add the rockusb gadget
>       usb: rockchip: add rockusb command
>       rockchip:usb: add a simple readme for rockusb
>       rockchip: rk3288: enable rockusb support on rk3288 based device
> 
>  arch/arm/include/asm/arch-rockchip/f_rockusb.h | 132 ++++++++++++++++++++
>  arch/arm/mach-rockchip/Kconfig                 |   2 +
>  cmd/Kconfig                                    |   9 ++
>  cmd/Makefile                                   |   1 +
>  cmd/rockusb.c                                  |  74 ++++++++++++
>  configs/chromebit_mickey_defconfig             |   9 ++
>  configs/chromebook_jerry_defconfig             |   9 ++
>  configs/chromebook_minnie_defconfig            |   9 ++
>  configs/evb-rk3288_defconfig                   |  10 ++
>  configs/fennec-rk3288_defconfig                |   6 +
>  configs/firefly-rk3288_defconfig               |   6 +
>  configs/miqi-rk3288_defconfig                  |   6 +
>  configs/phycore-rk3288_defconfig               |   7 ++
>  configs/popmetal-rk3288_defconfig              |   6 +
>  configs/rock2_defconfig                        |  10 ++
>  configs/tinker-rk3288_defconfig                |   6 +
>  doc/README.rockusb                             |  51 ++++++++
>  drivers/usb/gadget/Kconfig                     |   8 ++
>  drivers/usb/gadget/Makefile                    |   1 +
>  drivers/usb/gadget/f_rockusb.c                 | 691
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> 
>  include/configs/rk3288_common.h                |   7 --
>  21 files changed, 1053 insertions(+), 7 deletions(-)
>  create mode 100644 arch/arm/include/asm/arch-rockchip/f_rockusb.h
>  create mode 100644 cmd/rockusb.c
>  create mode 100644 doc/README.rockusb
>  create mode 100644 drivers/usb/gadget/f_rockusb.c
> 
> 
> Travis-CI output (all clean):
> 
> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269119055
> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269118570
> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269118344
> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269117873
> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269099287
> 


-- 
Best regards,
Marek Vasut

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

[Łukasz Majewski]

On 08/28/2017 10:32 PM, Marek Vasut wrote:
> On 08/28/2017 09:28 PM, Łukasz Majewski wrote:
>> Dear Marek,
>>
>> Those patches are preferably for -next merge widow.
>>
>>
>>
>> The following changes since commit
>> ae61b5353b48fa49c66376f273cc53cd3c06b656:
>>
>>   usb: dwc3: fix Kconfig dependency to accept host driver in
>> drivers/usb/dwc3 (2017-08-25 11:23:27 +0200)
>>
>> are available in the git repository at:
>>
>>   u-boot-denx-dfu/master
>
> Might be a good idea to put it into -next branch then.

+1.

I will create -next branch :-)

BR,
Łukasz

>
> Applied
>
>> for you to fetch changes up to 6496bc35d93975f01551dc3960b88807eae9d6fb:
>>
>>   rockchip: rk3288: enable rockusb support on rk3288 based device
>> (2017-08-28 21:20:50 +0200)
>>
>> ----------------------------------------------------------------
>> Eddie Cai (4):
>>       usb: rockchip: add the rockusb gadget
>>       usb: rockchip: add rockusb command
>>       rockchip:usb: add a simple readme for rockusb
>>       rockchip: rk3288: enable rockusb support on rk3288 based device
>>
>>  arch/arm/include/asm/arch-rockchip/f_rockusb.h | 132 ++++++++++++++++++++
>>  arch/arm/mach-rockchip/Kconfig                 |   2 +
>>  cmd/Kconfig                                    |   9 ++
>>  cmd/Makefile                                   |   1 +
>>  cmd/rockusb.c                                  |  74 ++++++++++++
>>  configs/chromebit_mickey_defconfig             |   9 ++
>>  configs/chromebook_jerry_defconfig             |   9 ++
>>  configs/chromebook_minnie_defconfig            |   9 ++
>>  configs/evb-rk3288_defconfig                   |  10 ++
>>  configs/fennec-rk3288_defconfig                |   6 +
>>  configs/firefly-rk3288_defconfig               |   6 +
>>  configs/miqi-rk3288_defconfig                  |   6 +
>>  configs/phycore-rk3288_defconfig               |   7 ++
>>  configs/popmetal-rk3288_defconfig              |   6 +
>>  configs/rock2_defconfig                        |  10 ++
>>  configs/tinker-rk3288_defconfig                |   6 +
>>  doc/README.rockusb                             |  51 ++++++++
>>  drivers/usb/gadget/Kconfig                     |   8 ++
>>  drivers/usb/gadget/Makefile                    |   1 +
>>  drivers/usb/gadget/f_rockusb.c                 | 691
>> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>
>>  include/configs/rk3288_common.h                |   7 --
>>  21 files changed, 1053 insertions(+), 7 deletions(-)
>>  create mode 100644 arch/arm/include/asm/arch-rockchip/f_rockusb.h
>>  create mode 100644 cmd/rockusb.c
>>  create mode 100644 doc/README.rockusb
>>  create mode 100644 drivers/usb/gadget/f_rockusb.c
>>
>>
>> Travis-CI output (all clean):
>>
>> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269119055
>> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269118570
>> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269118344
>> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269117873
>> https://travis-ci.org/lmajewski/u-boot-dfu/builds/269099287
>>
>
>


-- 
Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

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

[Marek Vasut]

On 08/28/2017 11:33 PM, Łukasz Majewski wrote:
> On 08/28/2017 10:32 PM, Marek Vasut wrote:
>> On 08/28/2017 09:28 PM, Łukasz Majewski wrote:
>>> Dear Marek,
>>>
>>> Those patches are preferably for -next merge widow.
>>>
>>>
>>>
>>> The following changes since commit
>>> ae61b5353b48fa49c66376f273cc53cd3c06b656:
>>>
>>>   usb: dwc3: fix Kconfig dependency to accept host driver in
>>> drivers/usb/dwc3 (2017-08-25 11:23:27 +0200)
>>>
>>> are available in the git repository at:
>>>
>>>   u-boot-denx-dfu/master
>>
>> Might be a good idea to put it into -next branch then.
> 
> +1.
> 
> I will create -next branch :-)
Don't bother for this PR, I pulled it.
Please do so in the future.

-- 
Best regards,
Marek Vasut

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

[Łukasz Majewski]

Dear Marek,

The following changes since commit 5b70b1688cdf677563096063e4a48e4bed250a10:

   usb: ehci: Convert CONFIG_USB_EHCI_PCI to Kconfig (2017-08-09 
10:50:08 +0200)

are available in the git repository at:

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

for you to fetch changes up to b14b19e1a96f8b891ed094009204e07af798b137:

   doc: license: Add license header to the README.dfutftp file 
(2017-08-21 11:07:26 +0200)

----------------------------------------------------------------
John Keeping (1):
       fastboot: avoid printing invalid data

Lukasz Majewski (1):
       doc: license: Add license header to the README.dfutftp file

Peter Chubb (1):
       Fix fastboot boot address

Tom Rini (2):
       common/fb_mmc.c: Fix warnings about casts
       fb_mmc.c: Correct blk_dread() return value checks

  common/fb_mmc.c                 | 8 ++++----
  doc/README.dfutftp              | 8 ++++++++
  drivers/usb/gadget/f_fastboot.c | 4 ++--
  3 files changed, 14 insertions(+), 6 deletions(-)



Test HW: BBB (Beagle Bone Black) (Old dfu tests passed)

Build test:
https://travis-ci.org/lmajewski/u-boot-dfu/builds/266707409

-- 
Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

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

[Marek Vasut]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="windows-1254", Size: 196 bytes --]

On 08/21/2017 11:16 AM, Łukasz Majewski wrote:
> Dear Marek,

Hi,

> The following changes since commit
> 5b70b1688cdf677563096063e4a48e4bed250a10:
Pulled, thanks.

-- 
Best regards,
Marek Vasut

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

[Łukasz Majewski]

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)


-- 
Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

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

[Marek Vasut]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="windows-1254", Size: 1479 bytes --]

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 ?

-- 
Best regards,
Marek Vasut

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

[Łukasz Majewski]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="windows-1254", Size: 1858 bytes --]

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.


>


-- 
Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

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

[Marek Vasut]

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.

-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Dear Marek,

Please find pull request for dfu repository:

The following changes since commit
9000eddbae0d292331ef403b0a1ff7ec7a983a79:

  drivers/usb/ehci: Use platform-specific accessors (2017-06-17
  17:59:03 +0200)

are available in the git repository at:

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

for you to fetch changes up to b9203429a04402bd03c48fa87670686ef7e7891d:

  cmd: usb_mass_storage: Staticize do_usb_mass_storage() (2017-06-18
  21:11:10 +0200)

----------------------------------------------------------------
Fabio Estevam (2):
      cmd: usb_mass_storage: Use NULL for pointer
      cmd: usb_mass_storage: Staticize do_usb_mass_storage()

Tom Rini (1):
      at91_udc.c: Fix unused variable warning

 cmd/usb_mass_storage.c        |  4 ++--
 drivers/usb/gadget/at91_udc.c | 10 ----------
 2 files changed, 2 insertions(+), 12 deletions(-)


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

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

[Marek Vasut]

On 06/18/2017 09:16 PM, Lukasz Majewski wrote:
> Dear Marek,
> 
> Please find pull request for dfu repository:

Would be nice if you specified the branch next time. Pulled.

> The following changes since commit
> 9000eddbae0d292331ef403b0a1ff7ec7a983a79:
> 
>   drivers/usb/ehci: Use platform-specific accessors (2017-06-17
>   17:59:03 +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git 
> 
> for you to fetch changes up to b9203429a04402bd03c48fa87670686ef7e7891d:
> 
>   cmd: usb_mass_storage: Staticize do_usb_mass_storage() (2017-06-18
>   21:11:10 +0200)
> 
> ----------------------------------------------------------------
> Fabio Estevam (2):
>       cmd: usb_mass_storage: Use NULL for pointer
>       cmd: usb_mass_storage: Staticize do_usb_mass_storage()
> 
> Tom Rini (1):
>       at91_udc.c: Fix unused variable warning
> 
>  cmd/usb_mass_storage.c        |  4 ++--
>  drivers/usb/gadget/at91_udc.c | 10 ----------
>  2 files changed, 2 insertions(+), 12 deletions(-)
> 
> 
> Best regards,
> 
> Lukasz Majewski
> 
> --
> 
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> 


-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Dear Marek,

Please find pull request for dfu repository:

The following changes since commit 46bac66b20da6d50e757bdca74703153f233090b:

  sandbox: Move to use live tree (2017-06-01 07:03:17 -0600)

are available in the git repository at:

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

for you to fetch changes up to efeccfe7f3b3d45f9a94f4eb62f4548b1a2fbfb5:

  fastboot: Add support for flashing zImage (2017-06-03 19:08:31 +0200)

----------------------------------------------------------------
Phil Edworthy (1):
      dfu: dfu_sf: Fix read offset

Sam Protsenko (1):
      fastboot: Add support for flashing zImage

 common/fb_mmc.c      | 168 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/dfu/dfu_sf.c |   3 ++-
 2 files changed, 170 insertions(+), 1 deletion(-)



Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

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

[Marek Vasut]

On 06/03/2017 07:18 PM, Lukasz Majewski wrote:
> Dear Marek,
> 
> Please find pull request for dfu repository:

Pulled, thanks.

> The following changes since commit 46bac66b20da6d50e757bdca74703153f233090b:
> 
>   sandbox: Move to use live tree (2017-06-01 07:03:17 -0600)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot.git 
> 
> for you to fetch changes up to efeccfe7f3b3d45f9a94f4eb62f4548b1a2fbfb5:
> 
>   fastboot: Add support for flashing zImage (2017-06-03 19:08:31 +0200)
> 
> ----------------------------------------------------------------
> Phil Edworthy (1):
>       dfu: dfu_sf: Fix read offset
> 
> Sam Protsenko (1):
>       fastboot: Add support for flashing zImage
> 
>  common/fb_mmc.c      | 168 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/dfu/dfu_sf.c |   3 ++-
>  2 files changed, 170 insertions(+), 1 deletion(-)
> 
> 
> 
> Best regards,
> 
> Lukasz Majewski
> 
> --
> 
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> 


-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Dear Marek,

Please find pull request for dfu repository:

The following changes since commit
8fb397da91eefe472f25c0a3c7d910fb8f9be129:

  drivers/usb/ehci: Use platform-specific accessors (2017-03-24
  13:55:45 +0100)

are available in the git repository at:

  ssh://gu-dfu at git.denx.de/u-boot-dfu/master 

for you to fetch changes up to f51a176d749ca38c3e8ca5fc8dada26bb2dd8877:

  usb: gadget: g_dnl: don't set iProduct nor iSerialNumber (2017-03-27
  13:09:51 +0200)

----------------------------------------------------------------
Felipe Balbi (3):
      usb: gadget: g_dnl: hold maximum string descriptor
      usb: gadget: g_dnl: only set iSerialNumber if we have a serial#
      usb: gadget: g_dnl: don't set iProduct nor iSerialNumber

 drivers/usb/gadget/g_dnl.c | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

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

[Marek Vasut]

On 03/27/2017 01:23 PM, Lukasz Majewski wrote:
> Dear Marek,
> 
> Please find pull request for dfu repository:
> 
> The following changes since commit
> 8fb397da91eefe472f25c0a3c7d910fb8f9be129:
> 
>   drivers/usb/ehci: Use platform-specific accessors (2017-03-24
>   13:55:45 +0100)
> 
> are available in the git repository at:
> 
>   ssh://gu-dfu at git.denx.de/u-boot-dfu/master 

Applied, thanks


-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Dear Marek,

Please find pull request for dfu repository:

The following changes since commit cc84f46800752edefb45a8a2de59ab6650dc4840:

  usb: dwc3: gadget: Remove unused header inclusion (2017-02-14 17:19:20 +0100)

are available in the git repository at:

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

for you to fetch changes up to 97786cc0ee07dabfbbd471ee686be1df82d3db64:

  usb: gadget: f_dfu: set serial number if serial# is valid (2017-02-22 13:22:05 +0100)

----------------------------------------------------------------
Felipe Balbi (3):
      usb: gadget: f_dfu: write req->actual bytes
      usb: gadget: g_dnl: fix g_dnl_set_serialnumber()
      usb: gadget: f_dfu: set serial number if serial# is valid

Patrick Delaunay (3):
      usb: gadget: dfu: add functional descriptor in descriptor set
      usb: gadget: dfu: correct size for USB_REQ_DFU_GETSTATE result
      usb: gadget: dfu: add result for handle_getstatus()

 drivers/usb/gadget/f_dfu.c | 73 +++++++++++++++++++++++++++++++++++++++----------------------------------
 drivers/usb/gadget/f_dfu.h |  1 -
 drivers/usb/gadget/g_dnl.c |  3 +--
 3 files changed, 40 insertions(+), 37 deletions(-)



Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

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

[Marek Vasut]

On 02/22/2017 01:36 PM, Lukasz Majewski wrote:
> Dear Marek,

Pulled, thanks.

> Please find pull request for dfu repository:
> 
> The following changes since commit cc84f46800752edefb45a8a2de59ab6650dc4840:
> 
>   usb: dwc3: gadget: Remove unused header inclusion (2017-02-14 17:19:20 +0100)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git 
> 
> for you to fetch changes up to 97786cc0ee07dabfbbd471ee686be1df82d3db64:
> 
>   usb: gadget: f_dfu: set serial number if serial# is valid (2017-02-22 13:22:05 +0100)
> 
> ----------------------------------------------------------------
> Felipe Balbi (3):
>       usb: gadget: f_dfu: write req->actual bytes
>       usb: gadget: g_dnl: fix g_dnl_set_serialnumber()
>       usb: gadget: f_dfu: set serial number if serial# is valid
> 
> Patrick Delaunay (3):
>       usb: gadget: dfu: add functional descriptor in descriptor set
>       usb: gadget: dfu: correct size for USB_REQ_DFU_GETSTATE result
>       usb: gadget: dfu: add result for handle_getstatus()
> 
>  drivers/usb/gadget/f_dfu.c | 73 +++++++++++++++++++++++++++++++++++++++----------------------------------
>  drivers/usb/gadget/f_dfu.h |  1 -
>  drivers/usb/gadget/g_dnl.c |  3 +--
>  3 files changed, 40 insertions(+), 37 deletions(-)
> 
> 
> 
> Best regards,
> 
> Lukasz Majewski
> 
> --
> 
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> 


-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

Please find following code for u-boot-dfu repository.

The following changes since commit 57faca19a82fc9b43a227824c30aeb76a43d1957:

  drivers: USB: OHCI: allow compilation for 64-bit targets (2016-10-29 19:45:40 +0200)

are available in the git repository at:

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

for you to fetch changes up to 1cccd1d11f33ff380b1e57700044fdee3fd6e044:

  dfu: align array in dfu_get_dev_type with enum dfu_device_type (2016-11-02 14:42:57 +0100)

----------------------------------------------------------------
Michal Simek (1):
      cmd: dfu: Add error handling for board_usb_init

Patrick Delaunay (1):
      dfu: align array in dfu_get_dev_type with enum dfu_device_type

 common/dfu.c      | 6 +++++-
 drivers/dfu/dfu.c | 2 +-
 2 files changed, 6 insertions(+), 2 deletions(-)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On 11/02/2016 04:28 PM, Lukasz Majewski wrote:
> Hi Marek,

Is this for 2016.11 or next ?

> Please find following code for u-boot-dfu repository.
> 
> The following changes since commit 57faca19a82fc9b43a227824c30aeb76a43d1957:
> 
>   drivers: USB: OHCI: allow compilation for 64-bit targets (2016-10-29 19:45:40 +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git 
> 
> for you to fetch changes up to 1cccd1d11f33ff380b1e57700044fdee3fd6e044:
> 
>   dfu: align array in dfu_get_dev_type with enum dfu_device_type (2016-11-02 14:42:57 +0100)
> 
> ----------------------------------------------------------------
> Michal Simek (1):
>       cmd: dfu: Add error handling for board_usb_init
> 
> Patrick Delaunay (1):
>       dfu: align array in dfu_get_dev_type with enum dfu_device_type
> 
>  common/dfu.c      | 6 +++++-
>  drivers/dfu/dfu.c | 2 +-
>  2 files changed, 6 insertions(+), 2 deletions(-)
> 
> 


-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

> On 11/02/2016 04:28 PM, Lukasz Majewski wrote:
> > Hi Marek,
> 
> Is this for 2016.11 or next ?

We can go with v2016.11, since those changes are fixes.

> 
> > Please find following code for u-boot-dfu repository.
> > 
> > The following changes since commit
> > 57faca19a82fc9b43a227824c30aeb76a43d1957:
> > 
> >   drivers: USB: OHCI: allow compilation for 64-bit targets
> > (2016-10-29 19:45:40 +0200)
> > 
> > are available in the git repository at:
> > 
> >   git://git.denx.de/u-boot-dfu.git 
> > 
> > for you to fetch changes up to
> > 1cccd1d11f33ff380b1e57700044fdee3fd6e044:
> > 
> >   dfu: align array in dfu_get_dev_type with enum dfu_device_type
> > (2016-11-02 14:42:57 +0100)
> > 
> > ----------------------------------------------------------------
> > Michal Simek (1):
> >       cmd: dfu: Add error handling for board_usb_init
> > 
> > Patrick Delaunay (1):
> >       dfu: align array in dfu_get_dev_type with enum dfu_device_type
> > 
> >  common/dfu.c      | 6 +++++-
> >  drivers/dfu/dfu.c | 2 +-
> >  2 files changed, 6 insertions(+), 2 deletions(-)
> > 
> > 
> 
> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On 11/03/2016 09:24 AM, Lukasz Majewski wrote:
> Hi Marek,
> 
>> On 11/02/2016 04:28 PM, Lukasz Majewski wrote:
>>> Hi Marek,
>>
>> Is this for 2016.11 or next ?
> 
> We can go with v2016.11, since those changes are fixes.

Applied, thanks.

>>
>>> Please find following code for u-boot-dfu repository.
>>>
>>> The following changes since commit
>>> 57faca19a82fc9b43a227824c30aeb76a43d1957:
>>>
>>>   drivers: USB: OHCI: allow compilation for 64-bit targets
>>> (2016-10-29 19:45:40 +0200)
>>>
>>> are available in the git repository at:
>>>
>>>   git://git.denx.de/u-boot-dfu.git 
>>>
>>> for you to fetch changes up to
>>> 1cccd1d11f33ff380b1e57700044fdee3fd6e044:
>>>
>>>   dfu: align array in dfu_get_dev_type with enum dfu_device_type
>>> (2016-11-02 14:42:57 +0100)
>>>
>>> ----------------------------------------------------------------
>>> Michal Simek (1):
>>>       cmd: dfu: Add error handling for board_usb_init
>>>
>>> Patrick Delaunay (1):
>>>       dfu: align array in dfu_get_dev_type with enum dfu_device_type
>>>
>>>  common/dfu.c      | 6 +++++-
>>>  drivers/dfu/dfu.c | 2 +-
>>>  2 files changed, 6 insertions(+), 2 deletions(-)
>>>
>>>
>>
>>
> 
> 
> 


-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

Please find following PR for u-boot dfu.

Those changes are aimed to be placed on top of u-boot-usb/next tree.

Moreover, they depend on already mainlined patch (but not yet available in your tree):
https://patchwork.ozlabs.org/patch/661424/

With the above patch in place, no build errors are introduced.


The following changes since commit fd69c4277c501a1082d118bbf8285db5f730ccdb:

  drivers: usb: xhci-fsl: Change burst beat and outstanding pipelined transfers requests (2016-08-26 14:05:03 +0200)

are available in the git repository at:

  ssh://gu-dfu at git.denx.de/u-boot-dfu 02f0f6d276012f384620174bf7f72bf56785934f

for you to fetch changes up to 02f0f6d276012f384620174bf7f72bf56785934f:

  cmd: dfu: Add error handling for failed registration (2016-08-29 14:36:35 +0200)

----------------------------------------------------------------
B, Ravi (5):
      spl: dfu: add dfu support in SPL
      common: dfu: saperate the dfu common functionality
      spl: dfu: adding dfu support functions for SPL-DFU
      dra7x: boot: add dfu bootmode support
      dra7x: configs: enable SPL-DFU support

Sanchayan Maity (1):
      cmd: dfu: Add error handling for failed registration

 Kconfig                                      | 27 +++++++++++++++++++++++++++
 arch/arm/cpu/armv7/omap-common/boot-common.c |  5 +++++
 arch/arm/include/asm/arch-omap5/spl.h        |  2 +-
 cmd/dfu.c                                    | 61 ++-----------------------------------------------------------
 common/Makefile                              |  3 +++
 common/dfu.c                                 | 92 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 common/spl/Makefile                          |  1 +
 common/spl/spl.c                             | 10 +++++++++-
 common/spl/spl_dfu.c                         | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 include/configs/dra7xx_evm.h                 | 20 +++++++++++++++++++-
 include/configs/ti_omap5_common.h            |  2 --
 include/g_dnl.h                              |  1 +
 include/spl.h                                |  8 ++++++++
 scripts/Makefile.spl                         |  4 ++++
 14 files changed, 229 insertions(+), 64 deletions(-)
 create mode 100644 common/dfu.c
 create mode 100644 common/spl/spl_dfu.c

Tested with HWT system at Exynos4412 and Exynos5422


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On 08/29/2016 04:24 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
> Please find following PR for u-boot dfu.
> 
> Those changes are aimed to be placed on top of u-boot-usb/next tree.
> 

Applied, thanks


-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

Please find following pull request:

The following changes since commit e96e064f51139c4af39f14499564ef76e40bbc29:

  usb: dwc2: Init desc_before_addr (2016-05-03 19:21:18 +0200)

are available in the git repository at:

  u-boot-dfu/master 

for you to fetch changes up to c3fdefb26d713eb34e506761be72395dec1cc4ab:

  usb: gadget: dfu: discard dead code (2016-05-06 09:58:56 +0200)

----------------------------------------------------------------
Peng Fan (2):
      dfu: avoid memory leak
      usb: gadget: dfu: discard dead code

 drivers/dfu/dfu.c          |  4 +++-
 drivers/usb/gadget/f_dfu.c | 10 +---------
 2 files changed, 4 insertions(+), 10 deletions(-)


Tested at Odroid U3 -> ./test/py {dfu|ums}

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On 05/06/2016 11:17 AM, Lukasz Majewski wrote:
> Hi Marek,

Hi!

> Please find following pull request:
> 
> The following changes since commit e96e064f51139c4af39f14499564ef76e40bbc29:
> 
>   usb: dwc2: Init desc_before_addr (2016-05-03 19:21:18 +0200)
> 
> are available in the git repository at:
> 
>   u-boot-dfu/master 
> 
> for you to fetch changes up to c3fdefb26d713eb34e506761be72395dec1cc4ab:
> 
>   usb: gadget: dfu: discard dead code (2016-05-06 09:58:56 +0200)
> 
> ----------------------------------------------------------------
> Peng Fan (2):
>       dfu: avoid memory leak
>       usb: gadget: dfu: discard dead code
> 
>  drivers/dfu/dfu.c          |  4 +++-
>  drivers/usb/gadget/f_dfu.c | 10 +---------
>  2 files changed, 4 insertions(+), 10 deletions(-)
> 
> 
> Tested at Odroid U3 -> ./test/py {dfu|ums}
> 
Applied, thanks!

-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

Please find DFU related code to be pulled to u-boot-usb branch.

The following changes since commit e6c0bc0643e5a4387fecbcf83080d0b796eb067c:

  usb: gadget Move: CONFIG_G_DNL_* to Kconfig (2016-04-20 11:43:28 +0200)

are available in the git repository at:

  u-boot-dfu/master 

for you to fetch changes up to 6cb2ff6bd0f0876a352c3020098b5ef71086403c:

  dfu: ram: fix number base of RAM entity parameters (2016-04-25 17:20:19 +0200)

----------------------------------------------------------------
Mugunthan V N (1):
      drivers: dfu: ram: fix a crash with dfu ram with invalid dfu_alt_info env

Roger Quadros (5):
      fastboot: Fix wMaxPacketSize for High-Speed IN endpoint
      fastboot: Enable the respective speed endpoints at runtime
      fastboot: Clean up bulk-out logic
      usb: s3c-otg: Fix short packet for request size > ep.maxpacket
      usb: s3c-otg: Fix remaining bytes in debug messages

Stephen Warren (1):
      dfu: ram: fix number base of RAM entity parameters

?ukasz Majewski (4):
      usb: mass storage: Initialize ret variable to zero
      tests: py: dfu: Add variables to store dfu alt numbers for test and dummy files
      tests: py: dfu: Add functionality to set different u-boot's dfu env variable
      tests: py: dfu: Provide functionality to set test and dummy files alt settings

 cmd/usb_mass_storage.c                     |   2 +-
 drivers/dfu/dfu_ram.c                      |  21 ++++++++++++++-------
 drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c |   6 +++---
 drivers/usb/gadget/f_fastboot.c            | 104 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------
 test/py/tests/test_dfu.py                  |  44 ++++++++++++++++++++++++++++++++++++++------
 5 files changed, 120 insertions(+), 57 deletions(-)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On 04/25/2016 05:40 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
> Please find DFU related code to be pulled to u-boot-usb branch.
> 
> The following changes since commit e6c0bc0643e5a4387fecbcf83080d0b796eb067c:
> 
>   usb: gadget Move: CONFIG_G_DNL_* to Kconfig (2016-04-20 11:43:28 +0200)
> 
> are available in the git repository at:
> 
>   u-boot-dfu/master 
> 
> for you to fetch changes up to 6cb2ff6bd0f0876a352c3020098b5ef71086403c:
> 
>   dfu: ram: fix number base of RAM entity parameters (2016-04-25 17:20:19 +0200)
> 
> ----------------------------------------------------------------
> Mugunthan V N (1):
>       drivers: dfu: ram: fix a crash with dfu ram with invalid dfu_alt_info env
> 
> Roger Quadros (5):
>       fastboot: Fix wMaxPacketSize for High-Speed IN endpoint
>       fastboot: Enable the respective speed endpoints at runtime
>       fastboot: Clean up bulk-out logic
>       usb: s3c-otg: Fix short packet for request size > ep.maxpacket
>       usb: s3c-otg: Fix remaining bytes in debug messages
> 
> Stephen Warren (1):
>       dfu: ram: fix number base of RAM entity parameters
> 
> ?ukasz Majewski (4):
>       usb: mass storage: Initialize ret variable to zero

I dropped this patch in favor of:

commit a238b0dacffd305f78fc23b2151521cce2384311
Author: Tom Rini <xxx>
Date:   Wed Apr 20 10:59:32 2016 -0400

    cmd/usb_mass_storage.c: Rework ums_init() ret logic slightly

>       tests: py: dfu: Add variables to store dfu alt numbers for test and dummy files
>       tests: py: dfu: Add functionality to set different u-boot's dfu env variable
>       tests: py: dfu: Provide functionality to set test and dummy files alt settings
> 
>  cmd/usb_mass_storage.c                     |   2 +-
>  drivers/dfu/dfu_ram.c                      |  21 ++++++++++++++-------
>  drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c |   6 +++---
>  drivers/usb/gadget/f_fastboot.c            | 104 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------
>  test/py/tests/test_dfu.py                  |  44 ++++++++++++++++++++++++++++++++++++++------
>  5 files changed, 120 insertions(+), 57 deletions(-)
> 
> 
Applied, thanks!

-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

> On 04/25/2016 05:40 PM, Lukasz Majewski wrote:
> > Hi Marek,
> > 
> > Please find DFU related code to be pulled to u-boot-usb branch.
> > 
> > The following changes since commit
> > e6c0bc0643e5a4387fecbcf83080d0b796eb067c:
> > 
> >   usb: gadget Move: CONFIG_G_DNL_* to Kconfig (2016-04-20 11:43:28
> > +0200)
> > 
> > are available in the git repository at:
> > 
> >   u-boot-dfu/master 
> > 
> > for you to fetch changes up to
> > 6cb2ff6bd0f0876a352c3020098b5ef71086403c:
> > 
> >   dfu: ram: fix number base of RAM entity parameters (2016-04-25
> > 17:20:19 +0200)
> > 
> > ----------------------------------------------------------------
> > Mugunthan V N (1):
> >       drivers: dfu: ram: fix a crash with dfu ram with invalid
> > dfu_alt_info env
> > 
> > Roger Quadros (5):
> >       fastboot: Fix wMaxPacketSize for High-Speed IN endpoint
> >       fastboot: Enable the respective speed endpoints at runtime
> >       fastboot: Clean up bulk-out logic
> >       usb: s3c-otg: Fix short packet for request size > ep.maxpacket
> >       usb: s3c-otg: Fix remaining bytes in debug messages
> > 
> > Stephen Warren (1):
> >       dfu: ram: fix number base of RAM entity parameters
> > 
> > ?ukasz Majewski (4):
> >       usb: mass storage: Initialize ret variable to zero
> 
> I dropped this patch in favor of:
> 
> commit a238b0dacffd305f78fc23b2151521cce2384311
> Author: Tom Rini <xxx>
> Date:   Wed Apr 20 10:59:32 2016 -0400
> 
>     cmd/usb_mass_storage.c: Rework ums_init() ret logic slightly

Thanks for applying this patch. :-)

> 
> >       tests: py: dfu: Add variables to store dfu alt numbers for
> > test and dummy files tests: py: dfu: Add functionality to set
> > different u-boot's dfu env variable tests: py: dfu: Provide
> > functionality to set test and dummy files alt settings
> > 
> >  cmd/usb_mass_storage.c                     |   2 +-
> >  drivers/dfu/dfu_ram.c                      |  21
> > ++++++++++++++------- drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c
> > |   6 +++--- drivers/usb/gadget/f_fastboot.c            | 104
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------
> > test/py/tests/test_dfu.py                  |  44
> > ++++++++++++++++++++++++++++++++++++++------ 5 files changed, 120
> > insertions(+), 57 deletions(-)
> > 
> > 
> Applied, thanks!
> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Lukasz Majewski]

+CC ing u-boot mailing list.

The following changes since commit
e6c0bc0643e5a4387fecbcf83080d0b796eb067c:

  usb: gadget Move: CONFIG_G_DNL_* to Kconfig (2016-04-20 11:43:28
  +0200)

are available in the git repository at:

  u-boot-dfu/master 

for you to fetch changes up to 6aeb877afef0c2d6cd3ea74e1b7d3a061d7cbd85:

  drivers: dfu: ram: fix a crash with dfu ram with invalid dfu_alt_info
  env (2016-04-22 17:03:54 +0200)

----------------------------------------------------------------
Mugunthan V N (1):
      drivers: dfu: ram: fix a crash with dfu ram with invalid
	dfu_alt_info env

Roger Quadros (5):
      fastboot: Fix wMaxPacketSize for High-Speed IN endpoint
      fastboot: Enable the respective speed endpoints at runtime
      fastboot: Clean up bulk-out logic
      usb: s3c-otg: Fix short packet for request size > ep.maxpacket
      usb: s3c-otg: Fix remaining bytes in debug messages

?ukasz Majewski (4):
      usb: mass storage: Initialize ret variable to zero
      tests: py: dfu: Add variables to store dfu alt numbers for test
	and dummy files tests: py: dfu: Add functionality to set
	different u-boot's dfu env variable
      tests: py: dfu: Provide
	functionality to set test and dummy files alt settings

 cmd/usb_mass_storage.c                     |   2 +-
 drivers/dfu/dfu_ram.c                      |  21 ++++++++++++++-------
 drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c |   6 +++---
 drivers/usb/gadget/f_fastboot.c            | 104++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------
 test/py/tests/test_dfu.py                  |  44 ++++++++++++++++++++++++++++++++++++++------ 

5 files changed, 120
 insertions(+), 57 deletions(-)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Stephen Warren]

On 04/22/2016 10:33 AM, Lukasz Majewski wrote:
> +CC ing u-boot mailing list.
>
> The following changes since commit
> e6c0bc0643e5a4387fecbcf83080d0b796eb067c:
>
>    usb: gadget Move: CONFIG_G_DNL_* to Kconfig (2016-04-20 11:43:28
>    +0200)
>
> are available in the git repository at:
>
>    u-boot-dfu/master
>
> for you to fetch changes up to 6aeb877afef0c2d6cd3ea74e1b7d3a061d7cbd85:
>
>    drivers: dfu: ram: fix a crash with dfu ram with invalid dfu_alt_info
>    env (2016-04-22 17:03:54 +0200)

Given that this currently fails DFU RAM testing on Tegra, let's hold off 
until that's fixed. I plan to investigate later today after I fix an 
unrelated emergency.

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

[Marek Vasut]

On 04/22/2016 06:38 PM, Stephen Warren wrote:
> On 04/22/2016 10:33 AM, Lukasz Majewski wrote:
>> +CC ing u-boot mailing list.
>>
>> The following changes since commit
>> e6c0bc0643e5a4387fecbcf83080d0b796eb067c:
>>
>>    usb: gadget Move: CONFIG_G_DNL_* to Kconfig (2016-04-20 11:43:28
>>    +0200)
>>
>> are available in the git repository at:
>>
>>    u-boot-dfu/master
>>
>> for you to fetch changes up to 6aeb877afef0c2d6cd3ea74e1b7d3a061d7cbd85:
>>
>>    drivers: dfu: ram: fix a crash with dfu ram with invalid dfu_alt_info
>>    env (2016-04-22 17:03:54 +0200)
> 
> Given that this currently fails DFU RAM testing on Tegra, let's hold off
> until that's fixed. I plan to investigate later today after I fix an
> unrelated emergency.

OK, will wait

-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

Dear Marek,

The following changes since commit
bbf8bef196ae52517284a0a2e2ffc94a4ff89f5f:

  Prepare v2016.05-rc1 (2016-04-11 22:22:25 -0400)

are available in the git repository at:

  u-boot-dfu/master 

for you to fetch changes up to e867669afd0478b1582b67e0c08ad047f1f49ec1:

  usb: gadget Move: CONFIG_G_DNL_* to Kconfig (2016-04-14 11:55:30
  +0200)

----------------------------------------------------------------
John Tobias (2):
      USB: g_dnl: Change device class
      usb: ums - expose selected partition/s

Sam Protsenko (7):
      usb: gadget: Move CONFIG_USB_GADGET_VBUS_DRAW to Kconfig
      usb: gadget: Move CONFIG_USB_GADGET_DUALSPEED to Kconfig
      usb: gadget: Move CONFIG_USB_GADGET_DOWNLOAD to Kconfig
      usb: dwc3: Move CONFIG_USB_DWC3_GADGET/HOST to Kconfig
      usb: dwc3: Move CONFIG_USB_DWC3_OMAP to Kconfig
      usb: dwc3: Move CONFIG_USB_DWC3_PHY_* to Kconfig
      usb: gadget Move: CONFIG_G_DNL_* to Kconfig

 cmd/usb_mass_storage.c                       | 39
 +++++++++++++++++++++++++++++----------
 configs/A13-OLinuXino_defconfig              |  4 ++++
 configs/CHIP_defconfig                       |  4 ++++
 configs/Cubietruck_defconfig                 |  4 ++++
 configs/am335x_baltos_defconfig              |  5 +++++
 configs/am335x_boneblack_defconfig           |  5 +++++
 configs/am335x_boneblack_vboot_defconfig     |  5 +++++
 configs/am335x_evm_defconfig                 |  5 +++++
 configs/am335x_evm_nor_defconfig             |  5 +++++
 configs/am335x_evm_norboot_defconfig         |  5 +++++
 configs/am335x_evm_spiboot_defconfig         |  5 +++++
 configs/am335x_evm_usbspl_defconfig          |  5 +++++
 configs/am335x_gp_evm_defconfig              |  5 +++++
 configs/am437x_gp_evm_defconfig              |  8 ++++++++
 configs/am437x_sk_evm_defconfig              |  8 ++++++++
 configs/am43xx_evm_defconfig                 |  8 ++++++++
 configs/am43xx_evm_ethboot_defconfig         |  8 ++++++++
 configs/am43xx_evm_qspiboot_defconfig        |  8 ++++++++
 configs/am43xx_evm_usbhost_boot_defconfig    |  8 ++++++++
 configs/apalis_t30_defconfig                 |  5 +++++
 configs/bcm11130_defconfig                   |  5 +++++
 configs/bcm11130_nand_defconfig              |  5 +++++
 configs/bcm28155_ap_defconfig                |  5 +++++
 configs/bcm28155_w1d_defconfig               |  5 +++++
 configs/beaver_defconfig                     |  5 +++++
 configs/birdland_bav335a_defconfig           |  5 +++++
 configs/birdland_bav335b_defconfig           |  5 +++++
 configs/cgtqmx6eval_defconfig                |  5 +++++
 configs/colibri_t20_defconfig                |  9 +++++++--
 configs/colibri_t30_defconfig                |  5 +++++
 configs/colibri_vf_defconfig                 |  5 +++++
 configs/corvus_defconfig                     |  5 +++++
 configs/dalmore_defconfig                    |  5 +++++
 configs/dra72_evm_defconfig                  |  8 ++++++++
 configs/dra74_evm_defconfig                  |  8 ++++++++
 configs/dra7xx_evm_defconfig                 |  8 ++++++++
 configs/dra7xx_evm_qspiboot_defconfig        |  8 ++++++++
 configs/dra7xx_evm_uart3_defconfig           |  8 ++++++++
 configs/draco_defconfig                      |  5 +++++
 configs/e2220-1170_defconfig                 |  5 +++++
 configs/gwventana_defconfig                  |  5 +++++
 configs/jetson-tk1_defconfig                 |  5 +++++
 configs/kc1_defconfig                        |  4 ++++
 configs/ma5d4evk_defconfig                   |  1 +
 configs/mx6dlsabreauto_defconfig             |  5 +++++
 configs/mx6dlsabresd_defconfig               |  5 +++++
 configs/mx6qpsabreauto_defconfig             |  5 +++++
 configs/mx6qsabreauto_defconfig              |  5 +++++
 configs/mx6qsabrelite_defconfig              |  5 +++++
 configs/mx6qsabresd_defconfig                |  5 +++++
 configs/mx6sabresd_spl_defconfig             |  5 +++++
 configs/mx7dsabresd_defconfig                |  5 +++++
 configs/nitrogen6dl2g_defconfig              |  5 +++++
 configs/nitrogen6dl_defconfig                |  5 +++++
 configs/nitrogen6q2g_defconfig               |  5 +++++
 configs/nitrogen6q_defconfig                 |  5 +++++
 configs/nitrogen6s1g_defconfig               |  5 +++++
 configs/nitrogen6s_defconfig                 |  5 +++++
 configs/novena_defconfig                     |  3 +++
 configs/nyan-big_defconfig                   |  5 +++++
 configs/odroid-xu3_defconfig                 |  7 +++++++
 configs/odroid_defconfig                     |  5 +++++
 configs/omap3_beagle_defconfig               |  5 +++++
 configs/omap3_logic_defconfig                |  5 +++++
 configs/omap5_uevm_defconfig                 |  8 ++++++++
 configs/origen_defconfig                     |  5 +++++
 configs/p2371-0000_defconfig                 |  5 +++++
 configs/p2371-2180_defconfig                 |  5 +++++
 configs/p2571_defconfig                      |  5 +++++
 configs/pcm051_rev1_defconfig                |  3 +++
 configs/pcm051_rev3_defconfig                |  3 +++
 configs/pengwyn_defconfig                    |  1 +
 configs/pxm2_defconfig                       |  5 +++++
 configs/rastaban_defconfig                   |  5 +++++
 configs/rut_defconfig                        |  5 +++++
 configs/s5p_goni_defconfig                   |  5 +++++
 configs/s5pc210_universal_defconfig          |  5 +++++
 configs/sama5d2_xplained_mmc_defconfig       |  1 +
 configs/sama5d2_xplained_spiflash_defconfig  |  1 +
 configs/sama5d3xek_mmc_defconfig             |  1 +
 configs/sama5d3xek_nandflash_defconfig       |  1 +
 configs/sama5d3xek_spiflash_defconfig        |  1 +
 configs/sama5d4_xplained_mmc_defconfig       |  1 +
 configs/sama5d4_xplained_nandflash_defconfig |  1 +
 configs/sama5d4_xplained_spiflash_defconfig  |  1 +
 configs/sama5d4ek_mmc_defconfig              |  1 +
 configs/sama5d4ek_nandflash_defconfig        |  1 +
 configs/sama5d4ek_spiflash_defconfig         |  1 +
 configs/sansa_fuze_plus_defconfig            |  3 +++
 configs/smartweb_defconfig                   |  4 ++++
 configs/sniper_defconfig                     |  4 ++++
 configs/socfpga_arria5_defconfig             |  5 +++++
 configs/socfpga_cyclone5_defconfig           |  5 +++++
 configs/socfpga_de0_nano_soc_defconfig       |  5 +++++
 configs/socfpga_mcvevk_defconfig             |  5 +++++
 configs/socfpga_sockit_defconfig             |  5 +++++
 configs/socfpga_socrates_defconfig           |  5 +++++
 configs/taurus_defconfig                     |  4 ++++
 configs/tbs2910_defconfig                    |  5 +++++
 configs/thuban_defconfig                     |  5 +++++
 configs/trats2_defconfig                     |  5 +++++
 configs/trats_defconfig                      |  5 +++++
 configs/venice2_defconfig                    |  5 +++++
 configs/vinco_defconfig                      |  1 +
 configs/xfi3_defconfig                       |  3 +++
 configs/xilinx_zynqmp_ep_defconfig           |  6 ++++++
 configs/zynq_picozed_defconfig               |  9 +++++++--
 configs/zynq_zc702_defconfig                 |  9 +++++++--
 configs/zynq_zc706_defconfig                 |  9 +++++++--
 configs/zynq_zed_defconfig                   |  9 +++++++--
 configs/zynq_zybo_defconfig                  |  9 +++++++--
 drivers/usb/Kconfig                          |  2 ++
 drivers/usb/dwc3/Kconfig                     | 57
 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/usb/gadget/Kconfig                   | 67
 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/usb/gadget/g_dnl.c                   |  6 +++---
 drivers/usb/musb-new/Kconfig                 |  1 +
 include/configs/am335x_evm.h                 |  9 ---------
 include/configs/am3517_evm.h                 |  1 -
 include/configs/am43xx_evm.h                 | 26
 ++++++++++++--------------
 include/configs/baltos.h                     |  9 ---------
 include/configs/bav335x.h                    |  9 ---------
 include/configs/bcm28155_ap.h                |  7 -------
 include/configs/cgtqmx6eval.h                |  8 --------
 include/configs/colibri_vf.h                 |  7 -------
 include/configs/corvus.h                     | 10 ----------
 include/configs/dra7xx_evm.h                 | 13 -------------
 include/configs/exynos4-common.h             | 11 +----------
 include/configs/gw_ventana.h                 |  9 ---------
 include/configs/kc1.h                        | 13 -------------
 include/configs/kwb.h                        |  2 --
 include/configs/ma5d4evk.h                   |  2 --
 include/configs/mx6sabre_common.h            |  8 --------
 include/configs/mx7dsabresd.h                |  8 --------
 include/configs/nitrogen6x.h                 |  9 ---------
 include/configs/novena.h                     |  2 --
 include/configs/odroid_xu3.h                 | 15 ---------------
 include/configs/omap3_beagle.h               |  7 -------
 include/configs/omap3_logic.h                |  7 -------
 include/configs/omap5_uevm.h                 | 13 -------------
 include/configs/pcm051.h                     |  2 --
 include/configs/pengwyn.h                    |  3 ---
 include/configs/s5p_goni.h                   | 10 ++--------
 include/configs/s5pc210_universal.h          |  2 --
 include/configs/sama5d2_xplained.h           |  2 --
 include/configs/sama5d3xek.h                 |  2 --
 include/configs/sama5d4_xplained.h           |  2 --
 include/configs/sama5d4ek.h                  |  2 --
 include/configs/sansa_fuze_plus.h            |  3 ---
 include/configs/siemens-am33x-common.h       |  7 -------
 include/configs/smartweb.h                   |  7 -------
 include/configs/sniper.h                     | 13 -------------
 include/configs/socfpga_common.h             | 15 ++-------------
 include/configs/sunxi-common.h               | 10 ----------
 include/configs/taurus.h                     |  7 -------
 include/configs/tbs2910.h                    |  7 -------
 include/configs/tegra-common-usb-gadget.h    | 13 -------------
 include/configs/tseries.h                    |  2 --
 include/configs/vinco.h                      |  2 --
 include/configs/warp.h                       |  8 --------
 include/configs/xfi3.h                       |  3 ---
 include/configs/xilinx_zynqmp.h              | 10 ----------
 include/configs/zynq-common.h                |  7 ------- 163 files
 changed, 709 insertions(+), 352 deletions(-) create mode 100644
 drivers/usb/dwc3/Kconfig


Tested with tests/py DFU and UMS on Odroid XU3 target board.

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On 04/14/2016 12:04 PM, Lukasz Majewski wrote:
> Dear Marek,

Hi!

> The following changes since commit
> bbf8bef196ae52517284a0a2e2ffc94a4ff89f5f:
> 
>   Prepare v2016.05-rc1 (2016-04-11 22:22:25 -0400)
> 
> are available in the git repository at:
> 
>   u-boot-dfu/master 
> 
> for you to fetch changes up to e867669afd0478b1582b67e0c08ad047f1f49ec1:
> 
>   usb: gadget Move: CONFIG_G_DNL_* to Kconfig (2016-04-14 11:55:30
>   +0200)

Applied, thanks!

-- 
Best regards,
Marek Vasut

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

[Lukasz Majewski]

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(-)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

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.

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

[Lukasz Majewski]

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.

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

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! :)

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

[Lukasz Majewski]

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 :-)

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

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 :-)
> 

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

[Lukasz Majewski]

The following changes since commit
517173d9652957e99598a34b5752dae21a5fea53:

  ohci: Add missing cache-flush for hcca area (2015-10-21 00:25:53
  +0200)

are available in the git repository at:

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

for you to fetch changes up to a468bc844eb191b82e3f5af2f277854eb6557394:

  ARM: dra7xx_evm: Add DFU support for qspi flash (2015-10-30 10:50:12
  +0100)

----------------------------------------------------------------
Vignesh R (2):
      dfu: dfu_sf: Pass duplicate devstr to parse_dev
      ARM: dra7xx_evm: Add DFU support for qspi flash

 drivers/dfu/dfu_sf.c         |  4 +++-
 include/configs/dra7xx_evm.h | 16 +++++++++++++++-
 2 files changed, 18 insertions(+), 2 deletions(-)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Friday, October 30, 2015 at 11:40:07 AM, Lukasz Majewski wrote:
> The following changes since commit
> 517173d9652957e99598a34b5752dae21a5fea53:
> 
>   ohci: Add missing cache-flush for hcca area (2015-10-21 00:25:53
>   +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git
> 
> for you to fetch changes up to a468bc844eb191b82e3f5af2f277854eb6557394:
> 
>   ARM: dra7xx_evm: Add DFU support for qspi flash (2015-10-30 10:50:12
>   +0100)
> 
> ----------------------------------------------------------------
> Vignesh R (2):
>       dfu: dfu_sf: Pass duplicate devstr to parse_dev
>       ARM: dra7xx_evm: Add DFU support for qspi flash
> 
>  drivers/dfu/dfu_sf.c         |  4 +++-
>  include/configs/dra7xx_evm.h | 16 +++++++++++++++-
>  2 files changed, 18 insertions(+), 2 deletions(-)

Applied, thanks!

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

The following changes since commit
25215622a3af013e28f16d1dd54a10ad8f73d15d:

  include: configs: am43xx_evm: add 'usb stop' in usbboot env
  (2015-08-24 20:43:38 +0200)

are available in the git repository at:

  ssh://gu-dfu at git.denx.de/u-boot-dfu/master 

for you to fetch changes up to 0e3168df70fee2b3b45c571356d56463bd5d34e4:

  dfu: tftp: Kconfig: Enable DFU_TFTP support on the
  am335x_boneblack_defconfig (2015-08-26 17:01:48 +0200)

----------------------------------------------------------------
Lukasz Majewski (9):
      doc: dfu: tftp: README entry for TFTP extension of DFU
      net: tftp: Move tftp.h file from ./net to ./include/net
      tftp: update: Allow some parts of the code to be reused when
CONFIG_SYS_NO_FLASH is set dfu: tftp: update: Provide tftp support for
the DFU subsystem dfu: tftp: update: Add dfu_write_from_mem_addr()
function update: tftp: dfu: Extend update_tftp() function to support DFU
      dfu: command: Extend "dfu" command to handle receiving data via
TFTP dfu: tftp: Kconfig: Add Kconfig entry for dfu tftp feature
      dfu: tftp: Kconfig: Enable DFU_TFTP support on the
am335x_boneblack_defconfig

 common/Makefile                    |   1 +
 common/cmd_dfu.c                   |  20 ++++++++++++++++++++
 common/cmd_fitupd.c                |   2 +-
 common/main.c                      |   2 +-
 common/update.c                    |  58
 ++++++++++++++++++++++++++++++++++++++++++----------------
 configs/am335x_boneblack_defconfig |   1 +
 doc/README.dfutftp                 | 114
 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 doc/README.update                  |   7 +++++++
 drivers/dfu/Kconfig                |  10 ++++++++++
 drivers/dfu/Makefile               |   1 +
 drivers/dfu/dfu.c                  |  37
 +++++++++++++++++++++++++++++++++++++
 drivers/dfu/dfu_tftp.c             |  65
 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 include/dfu.h                      |  41
 +++++++++++++++++++++++++++++++++++++++++
 include/net.h                      |  14 ++++++++++++-- {net =>
 include/net}/tftp.h        |   0 net/bootp.c
 |   2 +- net/net.c                          |   2 +-
 net/rarp.c                         |   2 +-
 net/tftp.c                         |   2 +- 19 files changed, 357
 insertions(+), 24 deletions(-) create mode 100644 doc/README.dfutftp
 create mode 100644 drivers/dfu/dfu_tftp.c rename {net =>
 include/net}/tftp.h (100%)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Wednesday, August 26, 2015 at 05:32:41 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
> The following changes since commit
> 25215622a3af013e28f16d1dd54a10ad8f73d15d:
> 
>   include: configs: am43xx_evm: add 'usb stop' in usbboot env
>   (2015-08-24 20:43:38 +0200)
> 
> are available in the git repository at:
> 
>   ssh://gu-dfu at git.denx.de/u-boot-dfu/master
> 
> for you to fetch changes up to 0e3168df70fee2b3b45c571356d56463bd5d34e4:
> 
>   dfu: tftp: Kconfig: Enable DFU_TFTP support on the
>   am335x_boneblack_defconfig (2015-08-26 17:01:48 +0200)

Applied, thanks!

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

The following changes since commit
8d77576371381ade83de475bb639949b44941e8c:

  ARM: davinci: remove support for cam_enc_4xx (2015-08-20 12:55:50
  -0400)

are available in the git repository at:

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

for you to fetch changes up to 41f25a67cd3ec195eb3cd07f67171b7bd702cc17:

  dfu: Delete superfluous initialization of the dfu_buf_size static
  variable (2015-08-24 15:18:09 +0200)

----------------------------------------------------------------
Jiandong Zheng (1):
      implement Fastboot via USB OTG on bcm28155_ap boards

Lukasz Majewski (2):
      dfu:tests: Modify dfu_gadget_test.sh to accept USB device
vendor:product ID dfu: Delete superfluous initialization of the
dfu_buf_size static variable

Siva Durga Prasad Paladugu (2):
      f_thor: Dont perform reset at the end of thor
      usb: gadget: f_thor: Allocate request up to THOR_PACKET_SIZE

 drivers/dfu/dfu.c             |  2 +-
 drivers/usb/gadget/f_thor.c   |  9 ++++++++-
 drivers/usb/gadget/f_thor.h   |  3 +++
 include/configs/bcm28155_ap.h | 19 +++++++++++++++++++
 test/dfu/README               |  9 ++++++++-
 test/dfu/dfu_gadget_test.sh   | 18 +++++++++++-------
 6 files changed, 50 insertions(+), 10 deletions(-)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Monday, August 24, 2015 at 03:21:41 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
> The following changes since commit
> 8d77576371381ade83de475bb639949b44941e8c:
> 
>   ARM: davinci: remove support for cam_enc_4xx (2015-08-20 12:55:50
>   -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git
> 
> for you to fetch changes up to 41f25a67cd3ec195eb3cd07f67171b7bd702cc17:
> 
>   dfu: Delete superfluous initialization of the dfu_buf_size static
>   variable (2015-08-24 15:18:09 +0200)
> 
> ----------------------------------------------------------------
> Jiandong Zheng (1):
>       implement Fastboot via USB OTG on bcm28155_ap boards
> 
> Lukasz Majewski (2):
>       dfu:tests: Modify dfu_gadget_test.sh to accept USB device
> vendor:product ID dfu: Delete superfluous initialization of the
> dfu_buf_size static variable
> 
> Siva Durga Prasad Paladugu (2):
>       f_thor: Dont perform reset at the end of thor
>       usb: gadget: f_thor: Allocate request up to THOR_PACKET_SIZE
> 
>  drivers/dfu/dfu.c             |  2 +-
>  drivers/usb/gadget/f_thor.c   |  9 ++++++++-
>  drivers/usb/gadget/f_thor.h   |  3 +++
>  include/configs/bcm28155_ap.h | 19 +++++++++++++++++++
>  test/dfu/README               |  9 ++++++++-
>  test/dfu/dfu_gadget_test.sh   | 18 +++++++++++-------
>  6 files changed, 50 insertions(+), 10 deletions(-)

Applied, thanks!

Best regards,
Marek Vasut

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

[Lukasz Majewski]

The following changes since commit
8c4735c56b93019916e391cc3542cea14763d9fb:
[2/1839]

  drivers: usb: fsl: Remove LS102XA immap inclusion (2015-06-26
  16:33:56 +0200)

are available in the git repository at:

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

for you to fetch changes up to 2e57ed002438162dda78241f6fed9860a7d58635:

  implement Fastboot via USB OTG on bcm28155_ap boards (2015-07-20
  09:21:00 +0200)

----------------------------------------------------------------
Jiandong Zheng (2):
      usb: gadget: bcm_udc_otg files
      implement Fastboot via USB OTG on bcm28155_ap boards

Paul Kocialkowski (7):
      usb: USB download gadget and functions config options coherent
naming 
      usb: Fastboot function config for better consistency with other
functions 
      usb: board_usb_init and board_usb_cleanup calls in the
fastboot command 
      usb: gadget: Weak board_usb_init/cleanup definitions
in USB download gadget code 
      fastboot: Dynamic controller index for
usb_gadget_handle_interrupts 
      usb: gadget: fastboot: Request status and length check in rx
handler 
      usb: gadget: fastboot: Dequeue the previous IN request for the
current request

Steve Rae (1):
      g_dnl: add missing declaration

 README                                       |  5 ++++-
 arch/arm/cpu/armv7/omap-common/boot-common.c |  2 +-
 arch/arm/include/asm/arch-bcm281xx/sysmap.h  |  7 +++++++
 board/broadcom/bcm28155_ap/bcm28155_ap.c     | 40
 ++++++++++++++++++++++++++++++++++++++++
 board/samsung/common/Makefile                |  2 +-
 board/siemens/common/factoryset.c            |  4 ++--
 common/cmd_fastboot.c                        | 33
 ++++++++++++++++++++++++++-------
 doc/README.android-fastboot                  |  6 +++---
 drivers/dfu/Makefile                         |  2 +-
 drivers/usb/gadget/Makefile                  | 11 ++++++-----
 drivers/usb/gadget/bcm_udc_otg.h             | 22
 ++++++++++++++++++++++ 
 drivers/usb/gadget/bcm_udc_otg_phy.c         | 51
 +++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/usb/gadget/f_fastboot.c              | 14 +++++++++-----
 drivers/usb/gadget/g_dnl.c                   | 13 +++++++++++++
 include/configs/am335x_evm.h                 |  5 +++--
 include/configs/am43xx_evm.h                 |  4 ++--
 include/configs/bav335x.h                    |  7 ++++---
 include/configs/bcm28155_ap.h                | 20 ++++++++++++++++++++
 include/configs/colibri_vf.h                 |  6 +++---
 include/configs/dra7xx_evm.h                 |  5 +++--
 include/configs/exynos4-common.h             |  8 ++++----
 include/configs/gw_ventana.h                 |  4 ++--
 include/configs/mx6sabre_common.h            |  4 ++--
 include/configs/nitrogen6x.h                 |  5 +++--
 include/configs/odroid_xu3.h                 |  8 ++++----
 include/configs/omap3_beagle.h               |  3 ++-
 include/configs/s5p_goni.h                   |  8 ++++----
 include/configs/siemens-am33x-common.h       |  4 ++--
 include/configs/socfpga_common.h             |  6 +++---
 include/configs/tbs2910.h                    |  4 ++--
 include/configs/tegra-common-usb-gadget.h    |  6 +++---
 include/configs/ti_omap5_common.h            |  2 +-
 include/configs/warp.h                       |  6 +++---
 include/configs/zynq-common.h                |  6 +++---
 include/g_dnl.h                              |  1 + 35 files changed,
 260 insertions(+), 74 deletions(-) create mode 100644
 drivers/usb/gadget/bcm_udc_otg.h create mode 100644
 drivers/usb/gadget/bcm_udc_otg_phy.c


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Monday, July 20, 2015 at 11:51:36 AM, Lukasz Majewski wrote:
> The following changes since commit
> 8c4735c56b93019916e391cc3542cea14763d9fb:
> [2/1839]
> 
>   drivers: usb: fsl: Remove LS102XA immap inclusion (2015-06-26
>   16:33:56 +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git
> 
> for you to fetch changes up to 2e57ed002438162dda78241f6fed9860a7d58635:
> 
>   implement Fastboot via USB OTG on bcm28155_ap boards (2015-07-20
>   09:21:00 +0200)

Applied, thanks !

Best regards,
Marek Vasut

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

[Lukasz Majewski]

The following changes since commit
19b17d12549fbd670e516e40757265e406153da3:

  drivers: usb: fsl: Return if USB_MAX_CONTROLLER_COUNT is incorrect
  (2015-06-07 15:46:02 +0200)

are available in the git repository at:

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

for you to fetch changes up to 2911bd1858d11308164b0b33de4ea0e8994d3edf:

  samsung: common: add example boot scripts (2015-06-08 15:26:11 +0200)

----------------------------------------------------------------
Inha Song (2):
      smdk5420: board: add functions to init dfu environment settings
      usb: f_mass_storage: sleep_thread: decrease the interval for
check ctrlc()

Joonyoung Shim (1):
      smdk5420: board: add functions required to enable USB DWC3

Przemyslaw Marczak (2):
      odroid-xu3: config: enable DFU/THOR/UMS by add configs and
environment samsung: common: add example boot scripts

?ukasz Majewski (2):
      arm: exynos: USB3 PHY base definition for Exynos5 SoCs
      samsung: usb: phy: Support for DWC3 PHY

 arch/arm/include/asm/arch-exynos/cpu.h        |  2 +-
 board/samsung/common/board.c                  |  6 +++++-
 board/samsung/common/bootscripts/autoboot.cmd | 92
 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 board/samsung/common/bootscripts/bootzimg.cmd | 10 ++++++++++
 board/samsung/smdk5420/smdk5420.c             | 80
 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-------
 drivers/usb/dwc3/Makefile                     |  1 +
 drivers/usb/dwc3/samsung_usb_phy.c            | 78
 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/usb/gadget/f_mass_storage.c           |  2 +-
 include/configs/odroid_xu3.h                  | 88
 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--
 include/samsung-usb-phy-uboot.h               | 16 ++++++++++++++++ 10
 files changed, 363 insertions(+), 12 deletions(-) create mode 100644
 board/samsung/common/bootscripts/autoboot.cmd create mode 100644
 board/samsung/common/bootscripts/bootzimg.cmd create mode 100644
 drivers/usb/dwc3/samsung_usb_phy.c create mode 100644
 include/samsung-usb-phy-uboot.h


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Monday, June 08, 2015 at 04:28:51 PM, Lukasz Majewski wrote:
> The following changes since commit
> 19b17d12549fbd670e516e40757265e406153da3:
> 
>   drivers: usb: fsl: Return if USB_MAX_CONTROLLER_COUNT is incorrect
>   (2015-06-07 15:46:02 +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git
> 
> for you to fetch changes up to 2911bd1858d11308164b0b33de4ea0e8994d3edf:
> 
>   samsung: common: add example boot scripts (2015-06-08 15:26:11 +0200)

Pulled, thanks!

Best regards,
Marek Vasut

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

[Lukasz Majewski]

The following changes since commit
7704fdbda3afb3d3bb0749378f444c71f92fb9ca:

  usb: gadget: thor: Claim EP after allocating it in thor gadget
  (2015-04-14 05:48:12 +0200)

are available in the git repository at:

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

for you to fetch changes up to a32d27912961040230320268d6cde80e9442dc33:

  fastboot: Proper download gadget unregister and clear when cable is
  missing (2015-04-14 10:19:05 +0200)

----------------------------------------------------------------
Alexey Firago (1):
      fastboot: add support for reboot-bootloader command

Fabio Estevam (1):
      cmd_usb_mass_storage: Use 'USB Mass Storage' in the help text

Heiko Schocher (1):
      dfu: cmd: trigger watchdog before calling
      usb_gadget_handle_interrupts

Michael Scott (1):
      fastboot: check for alias when looking up partition by name

Paul Kocialkowski (1):
      fastboot: Proper download gadget unregister and clear when cable
      is missing

 common/cmd_dfu.c                |  2 ++
 common/cmd_fastboot.c           |  2 ++
 common/cmd_usb_mass_storage.c   |  2 +-
 common/fb_mmc.c                 | 26 ++++++++++++++++++++++++--
 doc/README.android-fastboot     |  9 +++++++++
 drivers/usb/gadget/f_fastboot.c | 13 +++++++++++++
 6 files changed, 51 insertions(+), 3 deletions(-)

Tested at: Exynos4210 - Trats board


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Tom Rini]

On Tue, Apr 14, 2015 at 10:22:37AM +0200, Lukasz Majewski wrote:

> The following changes since commit
> 7704fdbda3afb3d3bb0749378f444c71f92fb9ca:
> 
>   usb: gadget: thor: Claim EP after allocating it in thor gadget
>   (2015-04-14 05:48:12 +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git 
> 
> for you to fetch changes up to a32d27912961040230320268d6cde80e9442dc33:
> 
>   fastboot: Proper download gadget unregister and clear when cable is
>   missing (2015-04-14 10:19:05 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150415/3403b9b7/attachment.sig>

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

[Lukasz Majewski]

Dear Marek,

The following changes since commit
d3d087c29c59dbae98c02de098e7e03f29320fe0:

  usb: musb-new: omap2430: Reset the MUSB controller early (2015-01-22
  08:19:18 +0100)

are available in the git repository at:

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

for you to fetch changes up to 934d274534275f5f4bbb45697a13c50ae54be346:

  odroid: adjust get_dfu_alt_*() functions to new declarations
  (2015-02-23 09:48:22 +0100)

----------------------------------------------------------------
Inha Song (1):
      odroid: adjust get_dfu_alt_*() functions to new declarations

Michael Scott (1):
      fastboot: add "fastboot oem" command support

Przemyslaw Marczak (1):
      dfu: samsung: move call to set_dfu_alt_info() to dfu common code

Rob Herring (1):
      fastboot: add support for "oem format" command

 board/samsung/common/board.c    |  3 ---
 board/samsung/common/misc.c     |  6 +++---
 board/samsung/odroid/odroid.c   | 31 +++++++++++++++++--------------
 drivers/dfu/dfu.c               |  3 +++
 drivers/usb/gadget/f_fastboot.c | 26 ++++++++++++++++++++++++++
 include/dfu.h                   |  3 +++
 include/samsung/misc.h          |  5 ++---
 7 files changed, 54 insertions(+), 23 deletions(-)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Monday, February 23, 2015 at 10:06:08 AM, Lukasz Majewski wrote:
> Dear Marek,

Hi!

> The following changes since commit
> d3d087c29c59dbae98c02de098e7e03f29320fe0:
> 
>   usb: musb-new: omap2430: Reset the MUSB controller early (2015-01-22
>   08:19:18 +0100)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git master
> 
> for you to fetch changes up to 934d274534275f5f4bbb45697a13c50ae54be346:
> 
>   odroid: adjust get_dfu_alt_*() functions to new declarations
>   (2015-02-23 09:48:22 +0100)

Is this PR still valid please ? Did you change something on your DFU/master
by any chance , like applying additional patches or somesuch please ?

Best regards,
Marek Vasut

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

[Lukasz Majewski]

On Tue, 24 Feb 2015 18:56:59 +0100
Marek Vasut <marex@denx.de> wrote:

> On Monday, February 23, 2015 at 10:06:08 AM, Lukasz Majewski wrote:
> > Dear Marek,
> 
> Hi!
> 
> > The following changes since commit
> > d3d087c29c59dbae98c02de098e7e03f29320fe0:
> > 
> >   usb: musb-new: omap2430: Reset the MUSB controller early
> > (2015-01-22 08:19:18 +0100)
> > 
> > are available in the git repository at:
> > 
> >   git://git.denx.de/u-boot-dfu.git master
> > 
> > for you to fetch changes up to
> > 934d274534275f5f4bbb45697a13c50ae54be346:
> > 
> >   odroid: adjust get_dfu_alt_*() functions to new declarations
> >   (2015-02-23 09:48:22 +0100)
> 
> Is this PR still valid please ? Did you change something on your
> DFU/master by any chance , like applying additional patches or
> somesuch please ?

I've applied some fastboot related patches on the -dfu branch today.

However, the PR is still valid and patches mentioned in it have been
tested and validated.

If possible please pull this PR.

Thanks in advance,
Lukasz Majewski

> 
> Best regards,
> Marek Vasut
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150224/b6046241/attachment.sig>

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

[Marek Vasut]

On Tuesday, February 24, 2015 at 11:29:39 PM, Lukasz Majewski wrote:
> On Tue, 24 Feb 2015 18:56:59 +0100
> 
> Marek Vasut <marex@denx.de> wrote:
> > On Monday, February 23, 2015 at 10:06:08 AM, Lukasz Majewski wrote:
> > > Dear Marek,
> > 
> > Hi!
> > 
> > > The following changes since commit
> > > 
> > > d3d087c29c59dbae98c02de098e7e03f29320fe0:
> > >   usb: musb-new: omap2430: Reset the MUSB controller early
> > > 
> > > (2015-01-22 08:19:18 +0100)
> > > 
> > > are available in the git repository at:
> > >   git://git.denx.de/u-boot-dfu.git master
> > > 
> > > for you to fetch changes up to
> > > 
> > > 934d274534275f5f4bbb45697a13c50ae54be346:
> > >   odroid: adjust get_dfu_alt_*() functions to new declarations
> > >   (2015-02-23 09:48:22 +0100)
> > 
> > Is this PR still valid please ? Did you change something on your
> > DFU/master by any chance , like applying additional patches or
> > somesuch please ?
> 
> I've applied some fastboot related patches on the -dfu branch today.
> 
> However, the PR is still valid and patches mentioned in it have been
> tested and validated.
> 
> If possible please pull this PR.

Hi!

In case I pull u-boot-dfu/master , I pull 7 patches though, is that right?

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

> On Tuesday, February 24, 2015 at 11:29:39 PM, Lukasz Majewski wrote:
> > On Tue, 24 Feb 2015 18:56:59 +0100
> > 
> > Marek Vasut <marex@denx.de> wrote:
> > > On Monday, February 23, 2015 at 10:06:08 AM, Lukasz Majewski
> > > wrote:
> > > > Dear Marek,
> > > 
> > > Hi!
> > > 
> > > > The following changes since commit
> > > > 
> > > > d3d087c29c59dbae98c02de098e7e03f29320fe0:
> > > >   usb: musb-new: omap2430: Reset the MUSB controller early
> > > > 
> > > > (2015-01-22 08:19:18 +0100)
> > > > 
> > > > are available in the git repository at:
> > > >   git://git.denx.de/u-boot-dfu.git master
> > > > 
> > > > for you to fetch changes up to
> > > > 
> > > > 934d274534275f5f4bbb45697a13c50ae54be346:
> > > >   odroid: adjust get_dfu_alt_*() functions to new declarations
> > > >   (2015-02-23 09:48:22 +0100)
> > > 
> > > Is this PR still valid please ? Did you change something on your
> > > DFU/master by any chance , like applying additional patches or
> > > somesuch please ?
> > 
> > I've applied some fastboot related patches on the -dfu branch today.
> > 
> > However, the PR is still valid and patches mentioned in it have been
> > tested and validated.
> > 
> > If possible please pull this PR.
> 
> Hi!
> 
> In case I pull u-boot-dfu/master , I pull 7 patches though, is that
> right?
> 

There are 9 patches actually. It's OK to pull them. 

> Best regards,
> Marek Vasut



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Wednesday, February 25, 2015 at 02:09:45 PM, Lukasz Majewski wrote:
> Hi Marek,

Hi!

> > On Tuesday, February 24, 2015 at 11:29:39 PM, Lukasz Majewski wrote:
> > > On Tue, 24 Feb 2015 18:56:59 +0100
> > > 
> > > Marek Vasut <marex@denx.de> wrote:
> > > > On Monday, February 23, 2015 at 10:06:08 AM, Lukasz Majewski
> > > > 
> > > > wrote:
> > > > > Dear Marek,
> > > > 
> > > > Hi!
> > > > 
> > > > > The following changes since commit
> > > > > 
> > > > > d3d087c29c59dbae98c02de098e7e03f29320fe0:
> > > > >   usb: musb-new: omap2430: Reset the MUSB controller early
> > > > > 
> > > > > (2015-01-22 08:19:18 +0100)
> > > > > 
> > > > > are available in the git repository at:
> > > > >   git://git.denx.de/u-boot-dfu.git master
> > > > > 
> > > > > for you to fetch changes up to
> > > > > 
> > > > > 934d274534275f5f4bbb45697a13c50ae54be346:
> > > > >   odroid: adjust get_dfu_alt_*() functions to new declarations
> > > > >   (2015-02-23 09:48:22 +0100)
> > > > 
> > > > Is this PR still valid please ? Did you change something on your
> > > > DFU/master by any chance , like applying additional patches or
> > > > somesuch please ?
> > > 
> > > I've applied some fastboot related patches on the -dfu branch today.
> > > 
> > > However, the PR is still valid and patches mentioned in it have been
> > > tested and validated.
> > > 
> > > If possible please pull this PR.
> > 
> > Hi!
> > 
> > In case I pull u-boot-dfu/master , I pull 7 patches though, is that
> > right?
> 
> There are 9 patches actually. It's OK to pull them.

Uh oh, next time I'll ask you to send an updated PR.

Applied all 9, thanks!

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

The following changes since commit
6a559bbe2f60ab64e28662f7ab52551a2eaa7163:

  usb_storage: blacklist Enclosure Service Devices (2014-11-08 12:03:35
  +0100)

are available in the git repository at:

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

for you to fetch changes up to 8f605197a872946d0a8a5b34c0eaee38ddd52dbf:

  test: ums: Add sleep before unmount directory (2014-11-14 11:42:09
  +0100)

----------------------------------------------------------------
?ukasz Majewski (2):
      dfu: thor: fix: Modify dfu_get_alt() function to support absolute
paths 
      test: ums: Add sleep before unmount directory

 drivers/dfu/dfu.c           | 29 +++++++++++++++++++++++++++--
 test/ums/ums_gadget_test.sh | 10 +++++++++-
 2 files changed, 36 insertions(+), 3 deletions(-)


Thanks in advance.

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Friday, November 14, 2014 at 05:54:42 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
> The following changes since commit
> 6a559bbe2f60ab64e28662f7ab52551a2eaa7163:
> 
>   usb_storage: blacklist Enclosure Service Devices (2014-11-08 12:03:35
>   +0100)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git

Pulled, thanks!

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

The following changes since commit
68bc8b4f2946c3da06c2b05d4f4c7e594cb1fa48:

  usb: kbd: Allow "usb reset" to continue when an usb kbd is used
  (2014-09-21 12:26:19 +0200)

are available in the git repository at:

  u-boot-denx-usb/master 

for you to fetch changes up to 772ca4fe05bf89f77efea80f5a6ff373d1ee92a9:

  usb: gadget: fastboot: improve download progress bar (2014-09-24
  13:11:35 +0200)

----------------------------------------------------------------
Bo Shen (1):
      usb: gadget: fastboot: improve download progress bar

Eric Nelson (1):
      usb: f_mass_storage: set removable flag in do_inquiry based on LUN

Heiko Schocher (2):
      usb: dfu: add fullspeed support for DFU
      arm: am335x: siemens board use in DFU mode fullspeed only

?ukasz Majewski (1):
      usb: dfu: thor: gadget: Remove dead code

 drivers/usb/gadget/f_dfu.c             | 11 +++--------
 drivers/usb/gadget/f_fastboot.c        | 14 +++++++++-----
 drivers/usb/gadget/f_mass_storage.c    |  1 +
 drivers/usb/gadget/f_thor.c            | 10 ----------
 include/configs/siemens-am33x-common.h |  2 +-
 5 files changed, 14 insertions(+), 24 deletions(-)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Wednesday, September 24, 2014 at 02:24:44 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
> The following changes since commit
> 68bc8b4f2946c3da06c2b05d4f4c7e594cb1fa48:
> 
>   usb: kbd: Allow "usb reset" to continue when an usb kbd is used
>   (2014-09-21 12:26:19 +0200)
> 
> are available in the git repository at:
> 
>   u-boot-denx-usb/master

I know where your repository is, but you should really consider fixing this u-
boot-denx-usb/master string above. Just use the full git://.... path when doing 
git request-pull .

> for you to fetch changes up to 772ca4fe05bf89f77efea80f5a6ff373d1ee92a9:
> 
>   usb: gadget: fastboot: improve download progress bar (2014-09-24
>   13:11:35 +0200)
> 
> ----------------------------------------------------------------
> Bo Shen (1):
>       usb: gadget: fastboot: improve download progress bar
> 
> Eric Nelson (1):
>       usb: f_mass_storage: set removable flag in do_inquiry based on LUN
> 
> Heiko Schocher (2):
>       usb: dfu: add fullspeed support for DFU
>       arm: am335x: siemens board use in DFU mode fullspeed only

Are you sure this is for master ?

> ?ukasz Majewski (1):
>       usb: dfu: thor: gadget: Remove dead code

Pulled it into master though, thanks!

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

> On Wednesday, September 24, 2014 at 02:24:44 PM, Lukasz Majewski
> wrote:
> > Hi Marek,
> > 
> > The following changes since commit
> > 68bc8b4f2946c3da06c2b05d4f4c7e594cb1fa48:
> > 
> >   usb: kbd: Allow "usb reset" to continue when an usb kbd is used
> >   (2014-09-21 12:26:19 +0200)
> > 
> > are available in the git repository at:
> > 
> >   u-boot-denx-usb/master
> 
> I know where your repository is, but you should really consider
> fixing this u- boot-denx-usb/master string above. Just use the full
> git://.... path when doing git request-pull .

Aggr.... I will fix this with next PR.

> 
> > for you to fetch changes up to
> > 772ca4fe05bf89f77efea80f5a6ff373d1ee92a9:
> > 
> >   usb: gadget: fastboot: improve download progress bar (2014-09-24
> >   13:11:35 +0200)
> > 
> > ----------------------------------------------------------------
> > Bo Shen (1):
> >       usb: gadget: fastboot: improve download progress bar
> > 
> > Eric Nelson (1):
> >       usb: f_mass_storage: set removable flag in do_inquiry based
> > on LUN
> > 
> > Heiko Schocher (2):
> >       usb: dfu: add fullspeed support for DFU
> >       arm: am335x: siemens board use in DFU mode fullspeed only
> 
> Are you sure this is for master ?

This is what Heiko need for Siemens boards. As fair as I know they need
this fullspeed to be able to flash their devices on a production line.

> 
> > ?ukasz Majewski (1):
> >       usb: dfu: thor: gadget: Remove dead code
> 
> Pulled it into master though, thanks!

Thanks.

> 
> Best regards,
> Marek Vasut



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Thursday, September 25, 2014 at 09:35:09 AM, Lukasz Majewski wrote:
> Hi Marek,
> 
> > On Wednesday, September 24, 2014 at 02:24:44 PM, Lukasz Majewski
> > 
> > wrote:
> > > Hi Marek,
> > > 
> > > The following changes since commit
> > > 
> > > 68bc8b4f2946c3da06c2b05d4f4c7e594cb1fa48:
> > >   usb: kbd: Allow "usb reset" to continue when an usb kbd is used
> > >   (2014-09-21 12:26:19 +0200)
> > > 
> > > are available in the git repository at:
> > >   u-boot-denx-usb/master
> > 
> > I know where your repository is, but you should really consider
> > fixing this u- boot-denx-usb/master string above. Just use the full
> > git://.... path when doing git request-pull .
> 
> Aggr.... I will fix this with next PR.

Thanks!

> > > for you to fetch changes up to
> > > 
> > > 772ca4fe05bf89f77efea80f5a6ff373d1ee92a9:
> > >   usb: gadget: fastboot: improve download progress bar (2014-09-24
> > >   13:11:35 +0200)
> > > 
> > > ----------------------------------------------------------------
> > > 
> > > Bo Shen (1):
> > >       usb: gadget: fastboot: improve download progress bar
> > > 
> > > Eric Nelson (1):
> > >       usb: f_mass_storage: set removable flag in do_inquiry based
> > > 
> > > on LUN
> > > 
> > > Heiko Schocher (2):
> > >       usb: dfu: add fullspeed support for DFU
> > >       arm: am335x: siemens board use in DFU mode fullspeed only
> > 
> > Are you sure this is for master ?
> 
> This is what Heiko need for Siemens boards. As fair as I know they need
> this fullspeed to be able to flash their devices on a production line.

OK then.

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Dear Marek,

The following changes since commit
74c0d756dea14855bec47f380b6ccb557be5db7c:

  usb: hub: don't check CONNECTION in hub_port_reset() (2014-08-29
  11:27:43 +0200)

are available in the git repository at:

  u-boot-denx-usb/master 

for you to fetch changes up to ea1545301efd42811eded401fcdedc2ffa5a2fd1:

  usb: ci_udc: implement dfu_usb_get_reset (2014-09-01 08:46:59 +0200)

----------------------------------------------------------------
Stephen Warren (1):
      usb: ci_udc: implement dfu_usb_get_reset

?ukasz Majewski (2):
      dfu: Provide means to find difference between dfu-util -e and -R
      udc: dfu: s3c_udc: Provide function to check if USB reset was
asserted

 common/cmd_dfu.c                 | 23 +++++++++++++++++++----
 drivers/dfu/dfu.c                | 31 ++++++++++++++++++++++++++-----
 drivers/usb/gadget/ci_udc.c      |  7 +++++++
 drivers/usb/gadget/f_dfu.c       |  2 +-
 drivers/usb/gadget/s3c_udc_otg.c |  5 +++++
 include/dfu.h                    |  5 ++++-
 6 files changed, 62 insertions(+), 11 deletions(-)


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Tuesday, September 02, 2014 at 09:13:44 AM, Lukasz Majewski wrote:
> Dear Marek,

Pulled, thanks!

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Dear Marek,

The following changes since commit
5ba95541b700d2edecb4d97d4b905f51ed8551b3:

  usb: phy: omap_usb_phy: implement usb_phy_power() for AM437x
  (2014-07-09 22:11:51 +0200)

are available in the git repository at:

  ssh://gu-dfu at git.denx.de/u-boot-dfu/master 

for you to fetch changes up to 24b109300c6e6a35792bc804846d7753baba7a69:

  dfu: fix readback buffer overflow test (2014-07-16 08:47:01 +0200)

----------------------------------------------------------------
Lukasz Majewski (3):
      test:dfu: Add test scripts for testing DFU regression
      test:dfu: README file update
      thor: defer parsing of device string to IO backend

Stephen Warren (11):
      fs: implement size/fatsize/ext4size
      dfu: fix some issues with reads/uploads
      dfu: add write error handling
      dfu: defer parsing of device string to IO backend
      dfu: allow backend to specify a maximum buffer size
      dfu: add free_entity() to struct dfu_entity
      dfu: add SF backend
      test: dfu: script enhancements
      test: dfu: cleanup before execution
      test: dfu: add some more test cases
      dfu: fix readback buffer overflow test

 common/cmd_dfu.c                 |   3 +--
 common/cmd_ext4.c                |  14 ++++++++++++++
 common/cmd_fat.c                 |  13 +++++++++++++
 common/cmd_fs.c                  |  13 +++++++++++++
 common/cmd_thordown.c            |   6 +++---
 drivers/dfu/Makefile             |   1 +
 drivers/dfu/dfu.c                | 105 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------------
 drivers/dfu/dfu_mmc.c            |  80 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++------------------
 drivers/dfu/dfu_nand.c           |   9 +++++++--
 drivers/dfu/dfu_ram.c            |  13 +++++++------
 drivers/dfu/dfu_sf.c             | 139 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/usb/gadget/f_dfu.c       |  20 ++++++++++++++++----
 drivers/usb/gadget/f_thor.c      |   5 +++--
 fs/ext4/ext4fs.c                 |   5 +++++
 fs/fat/fat.c                     |   5 +++++
 fs/fs.c                          |  43 +++++++++++++++++++++++++++++++++++++++++++
 fs/sandbox/sandboxfs.c           |   5 +++++
 include/dfu.h                    |  52 ++++++++++++++++++++++++++++++++++++++++++----------
 include/ext4fs.h                 |   1 +
 include/fat.h                    |   1 +
 include/fs.h                     |   9 +++++++++
 include/sandboxfs.h              |   1 +
 test/dfu/README                  |  37 +++++++++++++++++++++++++++++++++++++ 
 test/dfu/dfu_gadget_test.sh      |  94 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 test/dfu/dfu_gadget_test_init.sh |  31 +++++++++++++++++++++++++++++++
 
25 files changed, 619 insertions(+), 86 deletions(-)
 create mode 100644 drivers/dfu/dfu_sf.c 
 create mode 100644 test/dfu/README 
 create mode 100755 test/dfu/dfu_gadget_test.sh 
 create mode 100755 test/dfu/dfu_gadget_test_init.sh


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Wednesday, July 16, 2014 at 09:18:56 AM, Lukasz Majewski wrote:
> Dear Marek,
> 
> The following changes since commit
> 5ba95541b700d2edecb4d97d4b905f51ed8551b3:
> 
>   usb: phy: omap_usb_phy: implement usb_phy_power() for AM437x
>   (2014-07-09 22:11:51 +0200)
> 
> are available in the git repository at:
> 
>   ssh://gu-dfu at git.denx.de/u-boot-dfu/master
> 
> for you to fetch changes up to 24b109300c6e6a35792bc804846d7753baba7a69:
> 
>   dfu: fix readback buffer overflow test (2014-07-16 08:47:01 +0200)

Fixed up one merge conflict and applied, thanks!

Best regards,
Marek Vasut

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

[Stephen Warren]

On 07/23/2014 12:15 PM, Marek Vasut wrote:
> On Wednesday, July 16, 2014 at 09:18:56 AM, Lukasz Majewski wrote:
>> Dear Marek,
>>
>> The following changes since commit
>> 5ba95541b700d2edecb4d97d4b905f51ed8551b3:
>>
>>    usb: phy: omap_usb_phy: implement usb_phy_power() for AM437x
>>    (2014-07-09 22:11:51 +0200)
>>
>> are available in the git repository at:
>>
>>    ssh://gu-dfu at git.denx.de/u-boot-dfu/master
>>
>> for you to fetch changes up to 24b109300c6e6a35792bc804846d7753baba7a69:
>>
>>    dfu: fix readback buffer overflow test (2014-07-16 08:47:01 +0200)
>
> Fixed up one merge conflict and applied, thanks!

These changes don't seem to have made it into u-boot/master yet. I 
assume they will for the v2014.10 release?

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

[Lukasz Majewski]

On Wed, 06 Aug 2014 09:54:23 -0600
Stephen Warren <swarren@wwwdotorg.org> wrote:

> 
> On 07/23/2014 12:15 PM, Marek Vasut wrote:
> > On Wednesday, July 16, 2014 at 09:18:56 AM, Lukasz Majewski wrote:
> >> Dear Marek,
> >>
> >> The following changes since commit
> >> 5ba95541b700d2edecb4d97d4b905f51ed8551b3:
> >>
> >>    usb: phy: omap_usb_phy: implement usb_phy_power() for AM437x
> >>    (2014-07-09 22:11:51 +0200)
> >>
> >> are available in the git repository at:
> >>
> >>    ssh://gu-dfu at git.denx.de/u-boot-dfu/master
> >>
> >> for you to fetch changes up to
> >> 24b109300c6e6a35792bc804846d7753baba7a69:
> >>
> >>    dfu: fix readback buffer overflow test (2014-07-16 08:47:01
> >> +0200)
> >
> > Fixed up one merge conflict and applied, thanks!
> 
> These changes don't seem to have made it into u-boot/master yet. I 
> assume they will for the v2014.10 release?

Those patches are now in the u-boot-usb Marek's tree.

I hoped that Marek would send PR to Tom and those changes had been
merged in the ongoing merge window.

Marek do you plan to send PR to Tom in a near future?

> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140810/0dda82c0/attachment.pgp>

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

[Marek Vasut]

On Sunday, August 10, 2014 at 06:10:11 PM, Lukasz Majewski wrote:
> On Wed, 06 Aug 2014 09:54:23 -0600
> 
> Stephen Warren <swarren@wwwdotorg.org> wrote:
> > On 07/23/2014 12:15 PM, Marek Vasut wrote:
> > > On Wednesday, July 16, 2014 at 09:18:56 AM, Lukasz Majewski wrote:
> > >> Dear Marek,
> > >> 
> > >> The following changes since commit
> > >> 
> > >> 5ba95541b700d2edecb4d97d4b905f51ed8551b3:
> > >>    usb: phy: omap_usb_phy: implement usb_phy_power() for AM437x
> > >>    (2014-07-09 22:11:51 +0200)
> > >> 
> > >> are available in the git repository at:
> > >>    ssh://gu-dfu at git.denx.de/u-boot-dfu/master
> > >> 
> > >> for you to fetch changes up to
> > >> 
> > >> 24b109300c6e6a35792bc804846d7753baba7a69:
> > >>    dfu: fix readback buffer overflow test (2014-07-16 08:47:01
> > >> 
> > >> +0200)
> > > 
> > > Fixed up one merge conflict and applied, thanks!
> > 
> > These changes don't seem to have made it into u-boot/master yet. I
> > assume they will for the v2014.10 release?
> 
> Those patches are now in the u-boot-usb Marek's tree.
> 
> I hoped that Marek would send PR to Tom and those changes had been
> merged in the ongoing merge window.
> 
> Marek do you plan to send PR to Tom in a near future?

Should be applied now I believe, It might be just that Tom is slowed down by 
some bug which prevents parallel builds.

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek

The following changes since commit
fc25fa27e5f439705e9ca42182014e2d75d9f0ae:

  dfu, nand: add medium specific polltimeout function (2014-05-08
  10:38:30 +0200)

are available in the git repository at:

  ssh://gu-dfu at git.denx.de/u-boot-dfu master

for you to fetch changes up to 03bc75c206e25e8aeb8bbbb070b86424f7860453:

  dfu: Introduction of the "dfu_hash_algo" env variable for checksum
  method setting (2014-05-26 07:31:54 +0200)

----------------------------------------------------------------
Lukasz Majewski (1):
      dfu: Introduction of the "dfu_hash_algo" env variable for
checksum method setting

 drivers/dfu/dfu.c |   47
 ++++++++++++++++++++++++++++++++++++++++++----- 1 file changed, 42
 insertions(+), 5 deletions(-)



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Monday, May 26, 2014 at 07:46:27 AM, Lukasz Majewski wrote:
> Hi Marek
> 
> The following changes since commit
> fc25fa27e5f439705e9ca42182014e2d75d9f0ae:
> 
>   dfu, nand: add medium specific polltimeout function (2014-05-08
>   10:38:30 +0200)
> 
> are available in the git repository at:
> 
>   ssh://gu-dfu at git.denx.de/u-boot-dfu master
> 
> for you to fetch changes up to 03bc75c206e25e8aeb8bbbb070b86424f7860453:
> 
>   dfu: Introduction of the "dfu_hash_algo" env variable for checksum
>   method setting (2014-05-26 07:31:54 +0200)
> 
> ----------------------------------------------------------------
> Lukasz Majewski (1):
>       dfu: Introduction of the "dfu_hash_algo" env variable for
> checksum method setting

I had merge conflict here, so I picked the one patch by hand.

Thanks!

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

> On Monday, May 26, 2014 at 07:46:27 AM, Lukasz Majewski wrote:
> > Hi Marek
> > 
> > The following changes since commit
> > fc25fa27e5f439705e9ca42182014e2d75d9f0ae:
> > 
> >   dfu, nand: add medium specific polltimeout function (2014-05-08
> >   10:38:30 +0200)
> > 
> > are available in the git repository at:
> > 
> >   ssh://gu-dfu at git.denx.de/u-boot-dfu master
> > 
> > for you to fetch changes up to
> > 03bc75c206e25e8aeb8bbbb070b86424f7860453:
> > 
> >   dfu: Introduction of the "dfu_hash_algo" env variable for checksum
> >   method setting (2014-05-26 07:31:54 +0200)
> > 
> > ----------------------------------------------------------------
> > Lukasz Majewski (1):
> >       dfu: Introduction of the "dfu_hash_algo" env variable for
> > checksum method setting
> 
> I had merge conflict here, so I picked the one patch by hand.

I see. Thanks Marek!

> 
> Thanks!
> 
> Best regards,
> Marek Vasut



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Lukasz Majewski]

Hi Marek,

> On Monday, May 26, 2014 at 07:46:27 AM, Lukasz Majewski wrote:
> > Hi Marek
> > 
> > The following changes since commit
> > fc25fa27e5f439705e9ca42182014e2d75d9f0ae:
> > 
> >   dfu, nand: add medium specific polltimeout function (2014-05-08
> >   10:38:30 +0200)
> > 
> > are available in the git repository at:
> > 
> >   ssh://gu-dfu at git.denx.de/u-boot-dfu master
> > 
> > for you to fetch changes up to
> > 03bc75c206e25e8aeb8bbbb070b86424f7860453:
> > 
> >   dfu: Introduction of the "dfu_hash_algo" env variable for checksum
> >   method setting (2014-05-26 07:31:54 +0200)
> > 
> > ----------------------------------------------------------------
> > Lukasz Majewski (1):
> >       dfu: Introduction of the "dfu_hash_algo" env variable for
> > checksum method setting
> 
> I had merge conflict here, so I picked the one patch by hand.

Unfortunately you took the v3 of the patch. As fair as I remember on
the list we have agreed that we will use v4 of it.

You could replace the v3 patch which is still only on the u-boot-usb
tree, or I will prepare a "diff" patch between v3 and v4. 

What do you prefer? 

> 
> Thanks!
> 
> Best regards,
> Marek Vasut



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Lukasz Majewski]

Hi Marek,

> Hi Marek,
> 
> > On Monday, May 26, 2014 at 07:46:27 AM, Lukasz Majewski wrote:
> > > Hi Marek
> > > 
> > > The following changes since commit
> > > fc25fa27e5f439705e9ca42182014e2d75d9f0ae:
> > > 
> > >   dfu, nand: add medium specific polltimeout function (2014-05-08
> > >   10:38:30 +0200)
> > > 
> > > are available in the git repository at:
> > > 
> > >   ssh://gu-dfu at git.denx.de/u-boot-dfu master
> > > 
> > > for you to fetch changes up to
> > > 03bc75c206e25e8aeb8bbbb070b86424f7860453:
> > > 
> > >   dfu: Introduction of the "dfu_hash_algo" env variable for
> > > checksum method setting (2014-05-26 07:31:54 +0200)
> > > 
> > > ----------------------------------------------------------------
> > > Lukasz Majewski (1):
> > >       dfu: Introduction of the "dfu_hash_algo" env variable for
> > > checksum method setting
> > 
> > I had merge conflict here, so I picked the one patch by hand.
> 

I've made a little mistake. The v4 was merged indeed, however on the
list we have agreed to merge v3. Rest of the mail is correct :-).

Sorry for confusion.

> Unfortunately you took the v3 of the patch. As fair as I remember on
> the list we have agreed that we will use v4 of it.
> 
> You could replace the v3 patch which is still only on the u-boot-usb
> tree, or I will prepare a "diff" patch between v3 and v4. 
> 
> What do you prefer? 
> 
> > 
> > Thanks!
> > 
> > Best regards,
> > Marek Vasut
> 
> 
> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Monday, June 09, 2014 at 12:51:43 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
> > Hi Marek,
> > 
> > > On Monday, May 26, 2014 at 07:46:27 AM, Lukasz Majewski wrote:
> > > > Hi Marek
> > > > 
> > > > The following changes since commit
> > > > 
> > > > fc25fa27e5f439705e9ca42182014e2d75d9f0ae:
> > > >   dfu, nand: add medium specific polltimeout function (2014-05-08
> > > >   10:38:30 +0200)
> > > > 
> > > > are available in the git repository at:
> > > >   ssh://gu-dfu at git.denx.de/u-boot-dfu master
> > > > 
> > > > for you to fetch changes up to
> > > > 
> > > > 03bc75c206e25e8aeb8bbbb070b86424f7860453:
> > > >   dfu: Introduction of the "dfu_hash_algo" env variable for
> > > > 
> > > > checksum method setting (2014-05-26 07:31:54 +0200)
> > > > 
> > > > ----------------------------------------------------------------
> > > > 
> > > > Lukasz Majewski (1):
> > > >       dfu: Introduction of the "dfu_hash_algo" env variable for
> > > > 
> > > > checksum method setting
> > > 
> > > I had merge conflict here, so I picked the one patch by hand.
> 
> I've made a little mistake. The v4 was merged indeed, however on the
> list we have agreed to merge v3. Rest of the mail is correct :-).
> 
> Sorry for confusion.

Please send me an incremental patch. I don't want to rewrite history, so I will 
apply it on top of the current stuff.

Best regards,
Marek Vasut

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

[Lukasz Majewski]

Hi Marek,

> On Monday, June 09, 2014 at 12:51:43 PM, Lukasz Majewski wrote:
> > Hi Marek,
> > 
> > > Hi Marek,
> > > 
> > > > On Monday, May 26, 2014 at 07:46:27 AM, Lukasz Majewski wrote:
> > > > > Hi Marek
> > > > > 
> > > > > The following changes since commit
> > > > > 
> > > > > fc25fa27e5f439705e9ca42182014e2d75d9f0ae:
> > > > >   dfu, nand: add medium specific polltimeout function
> > > > > (2014-05-08 10:38:30 +0200)
> > > > > 
> > > > > are available in the git repository at:
> > > > >   ssh://gu-dfu at git.denx.de/u-boot-dfu master
> > > > > 
> > > > > for you to fetch changes up to
> > > > > 
> > > > > 03bc75c206e25e8aeb8bbbb070b86424f7860453:
> > > > >   dfu: Introduction of the "dfu_hash_algo" env variable for
> > > > > 
> > > > > checksum method setting (2014-05-26 07:31:54 +0200)
> > > > > 
> > > > > ----------------------------------------------------------------
> > > > > 
> > > > > Lukasz Majewski (1):
> > > > >       dfu: Introduction of the "dfu_hash_algo" env variable
> > > > > for
> > > > > 
> > > > > checksum method setting
> > > > 
> > > > I had merge conflict here, so I picked the one patch by hand.
> > 
> > I've made a little mistake. The v4 was merged indeed, however on the
> > list we have agreed to merge v3. Rest of the mail is correct :-).
> > 
> > Sorry for confusion.
> 
> Please send me an incremental patch. I don't want to rewrite history,
> so I will apply it on top of the current stuff.

Ok. Great :=)

> 
> Best regards,
> Marek Vasut



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

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

[Marek Vasut]

On Tuesday, June 10, 2014 at 08:10:01 AM, Lukasz Majewski wrote:
> Hi Marek,
> 
> > On Monday, June 09, 2014 at 12:51:43 PM, Lukasz Majewski wrote:
> > > Hi Marek,
> > > 
> > > > Hi Marek,
> > > > 
> > > > > On Monday, May 26, 2014 at 07:46:27 AM, Lukasz Majewski wrote:
> > > > > > Hi Marek
> > > > > > 
> > > > > > The following changes since commit
> > > > > > 
> > > > > > fc25fa27e5f439705e9ca42182014e2d75d9f0ae:
> > > > > >   dfu, nand: add medium specific polltimeout function
> > > > > > 
> > > > > > (2014-05-08 10:38:30 +0200)
> > > > > > 
> > > > > > are available in the git repository at:
> > > > > >   ssh://gu-dfu at git.denx.de/u-boot-dfu master
> > > > > > 
> > > > > > for you to fetch changes up to
> > > > > > 
> > > > > > 03bc75c206e25e8aeb8bbbb070b86424f7860453:
> > > > > >   dfu: Introduction of the "dfu_hash_algo" env variable for
> > > > > > 
> > > > > > checksum method setting (2014-05-26 07:31:54 +0200)
> > > > > > 
> > > > > > ----------------------------------------------------------------
> > > > > > 
> > > > > > Lukasz Majewski (1):
> > > > > >       dfu: Introduction of the "dfu_hash_algo" env variable
> > > > > > 
> > > > > > for
> > > > > > 
> > > > > > checksum method setting
> > > > > 
> > > > > I had merge conflict here, so I picked the one patch by hand.
> > > 
> > > I've made a little mistake. The v4 was merged indeed, however on the
> > > list we have agreed to merge v3. Rest of the mail is correct :-).
> > > 
> > > Sorry for confusion.
> > 
> > Please send me an incremental patch. I don't want to rewrite history,
> > so I will apply it on top of the current stuff.
> 
> Ok. Great :=)

Thanks!

Best regards,
Marek Vasut