[U-Boot] [ PATCH v3] net: move Broadcom SF2 driver to Kconfig

2017-07-10 Thread Steve Rae
adcom.com> Tested-by: Suji Velupillai <suji.velupil...@broadcom.com> Reviewed-by: JD Zheng <jiandong.zh...@broadcom.com> Reviewed-by: Scott Branden <scott.bran...@broadcom.com> Signed-off-by: Steve Rae <steve@raedomain.com> --- Changes in v3: - use Kconfig "im

[U-Boot] [ PATCH v2] net: move Broadcom SF2 driver to Kconfig

2017-04-12 Thread Steve Rae
adcom.com> Tested-by: Suji Velupillai <suji.velupil...@broadcom.com> Reviewed-by: JD Zheng <jiandong.zh...@broadcom.com> Reviewed-by: Scott Branden <scott.bran...@broadcom.com> Signed-off-by: Steve Rae <steve@raedomain.com> --- Changes in v

[U-Boot] [ PATCH] ARM: cache: runtime flag to suppress the "cache misaligned" message

2017-03-06 Thread Steve Rae
The "chunks" in the "fastboot sparse image" are not aligned, resulting in many "cached misaligned" messages from check_cache_range(). Implement a runtime flag to suppress this message, and use this flag when processing the "fastboot sparse image". Signed-of

Re: [U-Boot] arm: cache: misaligned operation with fastboot

2017-03-06 Thread Steve Rae
On Fri, Mar 3, 2017 at 9:27 AM, Gary Bisson <gary.bis...@boundarydevices.com > wrote: > Hi Steve, > > On Fri, Mar 3, 2017 at 12:03 AM, Steve Rae <steve@raedomain.com> > wrote: > > > > Hi Gary, > > > > On Thu, Mar 2, 2017 at 3:12 AM, Luk

Re: [U-Boot] [ PATCH 1/2] net: fix cache misaligned issue in Broadcom SF2 driver

2017-03-03 Thread Steve Rae
On Fri, Mar 3, 2017 at 5:06 PM, Steve Rae <steve@raedomain.com> wrote: > From: Suji Velupillai <suji.velupil...@broadcom.com> > > Fixed cache misaligned issue in the net driver. The issue shows-up when > a call to flush_dcache_range is made with unaligned memory. The

[U-Boot] [ PATCH 2/2] net: move Broadcom SF2 driver to Kconfig

2017-03-03 Thread Steve Rae
i Velupillai <suji.velupil...@broadcom.com> Reviewed-by: JD Zheng <jiandong.zh...@broadcom.com> Reviewed-by: Scott Branden <scott.bran...@broadcom.com> Signed-off-by: Steve Rae <steve@raedomain.com> --- arch/arm/include/asm/arch-bcmcygnus/configs.h | 6 +- configs/bcm281

[U-Boot] [ PATCH 1/2] net: fix cache misaligned issue in Broadcom SF2 driver

2017-03-03 Thread Steve Rae
adcom.com> Tested-by: Suji Velupillai <suji.velupil...@broadcom.com> Reviewed-by: Arun Parameswaran <arun.parameswa...@broadcom.com> Reviewed-by: JD Zheng <jiandong.zh...@broadcom.com> Reviewed-by: Shamez Kurji <shamez.ku...@broadcom.com> Signed-off-by: Steve Rae <

Re: [U-Boot] [PATCH] fastboot: avoid printing invalid data

2017-03-02 Thread Steve Rae
mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > This change looks OK -- but since it is the first time that "%.*s" is used with "error()" in U-Boot; I created a test build in order to hit this error condition and ensured that

Re: [U-Boot] arm: cache: misaligned operation with fastboot

2017-03-02 Thread Steve Rae
Hi Gary, On Thu, Mar 2, 2017 at 3:12 AM, Lukasz Majewski wrote: > Hi, > > > Hi Fabio, Lukasz, > > > > On Wed, Feb 15, 2017 at 02:24:40PM -0200, Fabio Estevam wrote: > > > On Wed, Feb 15, 2017 at 2:04 PM, Gary Bisson > > > wrote: > > > > Hi, > > >

Re: [U-Boot] [PATCH] Fix fastboot boot address

2016-12-30 Thread Steve Rae
au> > > Reviewed-by: Tom Rini <tr...@konsulko.com> > > -- > Tom > > ___ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot Acked-by: Steve Rae <steve@raedomain.com> Thanks, Steve _

Re: [U-Boot] [PATCH v3 10/26] ARM: boot0 hook: remove macro, include whole header file

2016-12-30 Thread Steve Rae
b reset; \ > + b reset > .space 1532 > - > -#endif /* __BOOT0_H */ > diff --git a/arch/arm/lib/vectors.S b/arch/arm/lib/vectors.S > index 5cc132b..9fe7415 100644 > --- a/arch/arm/lib/vectors.S > +++ b/arch/arm/lib/vectors.S > @@ -6

Re: [U-Boot] [PATCH v2 09/23] ARM: boot0 hook: remove macro, include whole header file

2016-12-30 Thread Steve Rae
b reset > .space 1532 > - > -#endif /* __BOOT0_H */ > diff --git a/arch/arm/lib/vectors.S b/arch/arm/lib/vectors.S > index 5cc132b..9fe7415 100644 > --- a/arch/arm/lib/vectors.S > +++ b/arch/arm/lib/vectors.S > @@ -67,7 +67,6 @@ _start: >

Re: [U-Boot] [PATCH 2/2] bcm: fastboot: implement 'reboot-bootloader'

2016-09-24 Thread Steve Rae
On Aug 23, 2016 16:39, "Steve Rae" <steve@raedomain.com> wrote: > > on bcm235xx and bcm281xx boards > > Signed-off-by: Steve Rae <steve@raedomain.com> > --- > > board/broadcom/bcm23550_w1d/bcm23550_w1d.c | 30 ++

Re: [U-Boot] [PATCH 1/2] fastboot: more support for reboot-bootloader command

2016-09-24 Thread Steve Rae
On Aug 25, 2016 01:30, "Paul Kocialkowski" <cont...@paulk.fr> wrote: > > Le mercredi 24 août 2016 à 16:52 -0700, Steve Rae a écrit : > > So, I wanted to: > > (1) simplify this to not depend on any env variable, and not depend on > > the CONFIG_BOOTCOM

Re: [U-Boot] [PATCH] Kconfig: update FASTBOOT_FLASH_MMC_DEV

2016-09-24 Thread Steve Rae
On Aug 27, 2016 15:16, "Steve Rae" <steve@raedomain.com> wrote: > > handle FASTBOOT_FLASH_MMC_DEV default properly > > Signed-off-by: Steve Rae <steve@raedomain.com> > --- > I was hoping that the FASTBOOT_FLASH_MMC_DEV Kconfig option could be > an

Re: [U-Boot] [PATCH v2 0/4] Fastboot MBR and generic partition support

2016-09-12 Thread Steve Rae
On Fri, Sep 9, 2016 at 1:27 AM, Petr Kulhavy wrote: > This set extends the Fastboot implementation from GPT-only to any partition > support. Further it adds a special target "mbr" (configurable) to write the > DOS MBR. > > Version 2: > Add a fourth patch into the set to move

Re: [U-Boot] [PATCH v2 4/4] fastboot: move FASTBOOT_FLASH options into Kconfig

2016-09-12 Thread Steve Rae
NAME). >> Remove the now redundant GPT_ENTRY_NAME. >> >> Signed-off-by: Petr Kulhavy <br...@jikos.cz> > > Reviewed-by: Tom Rini <tr...@konsulko.com> > > -- > Tom > > ___ > U-Boot mailing list > U-Boot@li

Re: [U-Boot] [PATCH v2 3/4] disk: part: refactor generic name creation for DOS and ISO

2016-09-12 Thread Steve Rae
t.h > @@ -165,6 +165,20 @@ int blk_get_device_part_str(const char *ifname, const > char *dev_part_str, > int part_get_info_by_name(struct blk_desc *dev_desc, > const char *name, disk_partition_t *info); > > +/** > + * part_set_generic_na

Re: [U-Boot] [PATCH v2 2/4] fastboot: add support for writing MBR

2016-09-12 Thread Steve Rae
> + > In Action > = > Enter into fastboot by executing the fastboot command in u-boot and you > diff --git a/include/part.h b/include/part.h > index bd8fd49..b17c219 100644 > --- a/include/part.h > +++ b/include/part.h > @@ -351,4 +351,27 @@ int gpt_verify_partitions

Re: [U-Boot] [PATCH v2 1/4] disk: part: implement generic function part_get_info_by_name()

2016-09-12 Thread Steve Rae
er { > #include > /* disk/part_efi.c */ > /** > - * part_get_info_efi_by_name() - Find the specified GPT partition table entry > - * > - * @param dev_desc - block device descriptor > - * @param gpt_name - the specified table entry name > - * @param info - returns the disk partition info > - * > - * @return - '0' on match, '-1' on no match, otherwise error > - */ > -int part_get_info_efi_by_name(struct blk_desc *dev_desc, > - const char *name, disk_partition_t *info); > - > -/** > * write_gpt_table() - Write the GUID Partition Table to disk > * > * @param dev_desc - block device descriptor > -- > 2.7.4 > > ___ > U-Boot mailing list > u-b...@lists.denx.deenteries > http://lists.denx.de/mailman/listinfo/u-boot Acked-by: Steve Rae <steve@raedomain.com> ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 4/4] config: rk3399: add support for dwc3 gadget

2016-08-27 Thread Steve Rae
Hi Kever, On Aug 23, 2016 20:48, "Kever Yang" wrote: > > To support fastboot, we need to enable the controller first. > rk3399 is using dwc3 as usb device controller, this patch enable > the configs for dwc3 gadget. > > Signed-off-by: Kever Yang

[U-Boot] [PATCH] Kconfig: update FASTBOOT_FLASH_MMC_DEV

2016-08-27 Thread Steve Rae
handle FASTBOOT_FLASH_MMC_DEV default properly Signed-off-by: Steve Rae <steve@raedomain.com> --- I was hoping that the FASTBOOT_FLASH_MMC_DEV Kconfig option could be an integer (eg. 0, 1, or 2 etc.) or undefined (to signify that it is not being used). However, it seems that (Kconfig e

Re: [U-Boot] [PATCH 1/2] fastboot: more support for reboot-bootloader command

2016-08-24 Thread Steve Rae
elp? Thanks, Steve On Wed, Aug 24, 2016 at 3:07 AM, Paul Kocialkowski <cont...@paulk.fr> wrote: > Hi, > > Le mardi 23 août 2016 à 16:38 -0700, Steve Rae a écrit : >> The "fastboot reboot-bootloader" command is defined to >> re-enter into fastboot mode after

[U-Boot] [PATCH 2/2] bcm: fastboot: implement 'reboot-bootloader'

2016-08-23 Thread Steve Rae
on bcm235xx and bcm281xx boards Signed-off-by: Steve Rae <steve@raedomain.com> --- board/broadcom/bcm23550_w1d/bcm23550_w1d.c | 30 ++ board/broadcom/bcm28155_ap/bcm28155_ap.c | 30 ++ 2 files changed, 60 insertions(+) diff

[U-Boot] [PATCH 1/2] fastboot: more support for reboot-bootloader command

2016-08-23 Thread Steve Rae
uld launch fastboot during the boot sequence via this __weak fb_handle_reboot_flag() function. The actual handling this reset flag should be implemented by board/SoC specific code. Signed-off-by: Steve Rae <steve@raedomain.com> cc: Alexey Firago <alexey_fir...@mentor.com> cc: Paul Kocialko

[U-Boot] [PATCH 2/2] fastboot: move to Kconfig

2016-08-15 Thread Steve Rae
- move bcm23550_w1d to Kconfig - move bcm28155_ap to Kconfig Signed-off-by: Steve Rae <steve@raedomain.com> --- configs/bcm23550_w1d_defconfig | 8 configs/bcm28155_ap_defconfig | 8 include/configs/bcm23550_w1d.h | 8 include/configs/bcm28155_ap.

[U-Boot] [PATCH 1/2] fastboot: implement Kconfig

2016-08-15 Thread Steve Rae
implement Kconfig for the 'fastboot' feature set Signed-off-by: Steve Rae <steve@raedomain.com> --- It did not seem feasible to use the "moveconfig.py" tool -- mostly because some of these configurations use defines rather than actual values... In addition, implemented

Re: [U-Boot] [PATCH 1/2] mmc: sdhci: set to INT_DATA_END when there are data

2016-07-26 Thread Steve Rae
HI Lukasz, On Tue, Jul 26, 2016 at 2:13 AM, Lukasz Majewski wrote: > Hi Steve, > >> with this change, I can also set the following back to 100: >> >> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c >> index de8d8ea..d593dc6 100644 >> --- a/drivers/mmc/sdhci.c >>

Re: [U-Boot] [PATCH 1/2] mmc: sdhci: set to INT_DATA_END when there are data

2016-07-25 Thread Steve Rae
On Mon, Jul 25, 2016 at 4:23 PM, Steve Rae <steve@raedomain.com> wrote: > Hi Jaehoon, > > On Mon, Jul 25, 2016 at 3:21 AM, Lukasz Majewski <l.majew...@samsung.com> > wrote: >> Hi Jaehoon, >> >>> Hi All, >>> >>>

Re: [U-Boot] [PATCH 1/2] mmc: sdhci: set to INT_DATA_END when there are data

2016-07-25 Thread Steve Rae
nt >> > update journal finished >> > 32505856 bytes written in 4884 ms (6.3 MiB/s) >> > >> > >> > > > > -- > Best regards, > > Lukasz Majewski > > Samsung R Institute Poland (SRPOL) | Linux Platform Group with this change, I can also set th

Re: [U-Boot] [PATCH 3/4] rockchip: rk3288: add fastboot support

2016-06-30 Thread Steve Rae
Hi Ziyuan, On Thu, Jun 30, 2016 at 8:23 AM, Simon Glass wrote: > Hi Ziyuan, > > On 30 June 2016 at 00:22, Ziyuan Xu wrote: >> Enable fastboot feature on rk3288. >> >> This path doesn't support the fastboot flash function command entirely. >> We will hit

[U-Boot] [PATCH v4] mtd: fix compiler warnings

2016-06-29 Thread Steve Rae
- add missing declaration - update debug output format specifiers Signed-off-by: Steve Rae <steve@raedomain.com> --- the checkpatch warning: warning: cmd/mtdparts.c,1494: quoted string split across lines is for the existing code; it is not introduced with this change... Changes

Re: [U-Boot] [RFC PATCH v3 2/4] mtd: fix compiler warnings

2016-06-29 Thread Steve Rae
Hi Scott, On Tue, Jun 28, 2016 at 6:17 PM, Scott Wood <o...@buserror.net> wrote: > On Sat, 2016-03-12 at 14:48 -0800, Steve Rae wrote: >> - add missing declaration >> - update debug output format specifiers >> >> Signed-off-by: Steve Rae <s...@broadcom.c

[U-Boot] [PATCH v2] mmc: increase MMC SDHCI read status timeout

2016-06-29 Thread Steve Rae
is controlled by the default value: CONFIG_SDHCI_CMD_DEFAULT_TIMEOUT It only increases the retries when reading: SDHCI_INT_STATUS to avoid the timeout error. Signed-off-by: Steve Rae <steve@raedomain.com> Reviewed-by: Stefan Roese <s...@denx.de> Tested-by: Masahiro Yamada &

Re: [U-Boot] [PATCH] arm: mmc: increase MMC SDHCI read status timeout

2016-06-29 Thread Steve Rae
Hi Masahiro, On Wed, Jun 29, 2016 at 4:52 AM, Masahiro Yamada wrote: > Hi Steve, > > >> @@ -127,6 +127,7 @@ static int sdhci_transfer_data(struct sdhci_host *host, >> struct mmc_data *data, >> #define CONFIG_SDHCI_CMD_MAX_TIMEOUT 3200 >> #endif >>

Re: [U-Boot] [PATCH] arm: mmc: increase MMC SDHCI read status timeout

2016-06-28 Thread Steve Rae
On Tue, Jun 28, 2016 at 3:21 PM, Tom Rini <tr...@konsulko.com> wrote: > On Tue, Jun 28, 2016 at 01:53:52PM -0700, Steve Rae wrote: >> Hi Tom, >> >> On Tue, Jun 28, 2016 at 1:34 PM, Tom Rini <tr...@konsulko.com> wrote: >> > On Tue, Jun 28, 2016 at 01:30:09

Re: [U-Boot] [PATCH] arm: mmc: increase MMC SDHCI read status timeout

2016-06-28 Thread Steve Rae
Hi Tom, On Tue, Jun 28, 2016 at 1:34 PM, Tom Rini <tr...@konsulko.com> wrote: > On Tue, Jun 28, 2016 at 01:30:09PM -0700, Steve Rae wrote: >> Hi Stefan, >> >> On Tue, Jun 28, 2016 at 8:00 AM, Stefan Roese <s...@denx.de> wrote: >> > Hi Steve, >

Re: [U-Boot] [PATCH] arm: mmc: increase MMC SDHCI read status timeout

2016-06-28 Thread Steve Rae
Hi Stefan, On Tue, Jun 28, 2016 at 8:00 AM, Stefan Roese <s...@denx.de> wrote: > Hi Steve, > > On 27.06.2016 23:43, Steve Rae wrote: >> >> Otherwise, ocassionally see errors like this: >>Flashing sparse image at offset 2078720 >>Flashing Sparse

[U-Boot] [PATCH] maintainers: new email address

2016-06-27 Thread Steve Rae
Update the email address for the boards that I maintain. Signed-off-by: Steve Rae <steve@raedomain.com> --- - motivated to do it now because of [1] which closes next Monday - Tim Kryger has been gone for ~2 years [1] http://www.cypress.com/news/cypress-acquire-broadcom-s-wireless-in

[U-Boot] [PATCH] arm: mmc: increase MMC SDHCI read status timeout

2016-06-27 Thread Steve Rae
is controlled by the default value: CONFIG_SDHCI_CMD_DEFAULT_TIMEOUT It only increases the retries when reading: SDHCI_INT_STATUS to avoid the timeout error. Signed-off-by: Steve Rae <steve@raedomain.com> --- as per the discussion in: http://lists.denx.de/pipermail/u-boot/2016-June/

Re: [U-Boot] error: sdhci_send_command: Timeout for status update!

2016-06-24 Thread Steve Rae
Hi Stefan, Masahiro On Thu, Jun 23, 2016 at 9:56 PM, Stefan Roese <s...@denx.de> wrote: > Steve, > > > On 24.06.2016 00:06, Steve Rae wrote: >> >> I am asking you about this code, because of this: >>commit 29905a4 - mmc: sdhci: Use timer based timeou

[U-Boot] error: sdhci_send_command: Timeout for status update!

2016-06-23 Thread Steve Rae
Stefan, I am asking you about this code, because of this: commit 29905a4 - mmc: sdhci: Use timer based timeout detection in sdhci_send_command() Occasionally, I am seeing failures when writing to flash on my device: Flashing sparse image at offset 2078720 Flashing Sparse Image

[U-Boot] [PATCH 3/4] arm: bcm235xx: fix kps ccu

2016-06-21 Thread Steve Rae
From: Chris Brand <chris.br...@broadcom.com> The Kona Peripheral Slave CCU has 4 policy mask registers, not 8. Signed-off-by: Chris Brand <chris.br...@broadcom.com> Signed-off-by: Steve Rae <s...@broadcom.com> --- arch/arm/cpu/armv7/bcm235xx/clk-bcm235xx.c | 6 +-

[U-Boot] [PATCH 4/4] arm: bcm235xx: update clock framework

2016-06-21 Thread Steve Rae
The handling of the "usage counter" is incorrect, and the clock should only be disabled when transitioning from 1 to 0. Reported-by: Chris Brand <chris.br...@broadcom.com> Signed-off-by: Steve Rae <s...@broadcom.com> --- arch/arm/cpu/armv7/bcm235xx/clk-core.c | 10 +--

[U-Boot] [PATCH 1/4] arm: bcm235xx: choose 8-bit phy bus width

2016-06-21 Thread Steve Rae
The Kona PHY supports an 8-bit wide UTMI interface, therefore, choose this Kconfig setting. Signed-off-by: Steve Rae <s...@broadcom.com> --- configs/bcm23550_w1d_defconfig | 1 + include/configs/bcm23550_w1d.h | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/c

[U-Boot] [PATCH 2/4] arm: bcm235xx: implement the boot0 hook code

2016-06-21 Thread Steve Rae
Choose the Kconfig boot0 hook option and implement the required code. Signed-off-by: Steve Rae <s...@broadcom.com> --- arch/arm/include/asm/arch-bcm235xx/boot0.h | 15 +++ configs/bcm23550_w1d_defconfig | 1 + 2 files changed, 16 insertions(+) create mode 10064

[U-Boot] [PATCH 0/4] This series contains minor updates for the bcm23550:

2016-06-21 Thread Steve Rae
(1) implement Kconfig options which have been recently applied, and (2) fix two outstanding issues. Chris Brand (1): arm: bcm235xx: fix kps ccu Steve Rae (3): arm: bcm235xx: choose 8-bit phy bus width arm: bcm235xx: implement the boot0 hook code arm: bcm235xx: update clock framework

Re: [U-Boot] [PATCH v2 5/5] fastboot: sparse: improve CHUNK_TYPE_FILL write performance

2016-06-16 Thread Steve Rae
On Thu, Jun 16, 2016 at 10:34 AM, Steve Rae <s...@broadcom.com> wrote: > On Wed, Jun 15, 2016 at 1:36 AM, Maxime Ripard > <maxime.rip...@free-electrons.com> wrote: >> On Tue, Jun 07, 2016 at 11:19:39AM -0700, Steve Rae wrote: >>> - increase the size of the fill buf

Re: [U-Boot] [PATCH v2 5/5] fastboot: sparse: improve CHUNK_TYPE_FILL write performance

2016-06-16 Thread Steve Rae
On Wed, Jun 15, 2016 at 1:36 AM, Maxime Ripard <maxime.rip...@free-electrons.com> wrote: > On Tue, Jun 07, 2016 at 11:19:39AM -0700, Steve Rae wrote: >> - increase the size of the fill buffer >> - testing has shown a 10x improvement when the sparse image >> has l

Re: [U-Boot] [PATCH v2 2/5] fastboot: sparse: resync common/image-sparse.c (part 1)

2016-06-16 Thread Steve Rae
On Wed, Jun 15, 2016 at 1:18 AM, Maxime Ripard <maxime.rip...@free-electrons.com> wrote: > > On Tue, Jun 07, 2016 at 11:19:36AM -0700, Steve Rae wrote: > > This file originally came from upstream code. > > > > While retaining the storage abstraction feat

[U-Boot] [PATCH v5 2/2] arm: bcm281xx: choose 8-bit phy bus width

2016-06-07 Thread Steve Rae
The Kona PHY supports an 8-bit wide UTMI interface, therefore, choose this Kconfig setting. Signed-off-by: Steve Rae <s...@broadcom.com> --- from [1] (line ~124) /* The Kona PHY supports an 8-bit wide UTMI interface */ [...snip...] module_platform_driver(bcm_kona_usb2_

[U-Boot] [PATCH v5 1/2] usb: dwc2_udc_otg: support 8-bit interface

2016-06-07 Thread Steve Rae
Define CONFIG_USB_GADGET_DWC2_OTG_PHY_BUS_WIDTH_8 to allow the physical interface to be 8-bit (rather than 16-bit). Signed-off-by: Steve Rae <s...@broadcom.com> --- from [1] (line ~124) /* The Kona PHY supports an 8-bit wide UTMI interface */ [...snip...] module_platform_

Re: [U-Boot] [PATCH v4 2/2] USB: DWC2: choose 8-bit phy bus width

2016-06-07 Thread Steve Rae
On Tue, Jun 7, 2016 at 3:24 PM, Marek Vasut <ma...@denx.de> wrote: > On 06/07/2016 05:57 PM, Steve Rae wrote: >> The Kona PHY supports an 8-bit wide UTMI interface, >> therefore, choose this Kconfig setting. >> >> Signed-off-by: Steve Rae <s...@broadcom.com&g

Re: [U-Boot] [PATCH v2 0/5] Update fastboot sparse image handling

2016-06-07 Thread Steve Rae
Tom, On Tue, Jun 7, 2016 at 11:19 AM, Steve Rae <s...@broadcom.com> wrote: > While retaining the storage abstraction feature implemented in U-Boot, > this series updates the fastboot sparse image handling by > (1) fixing broken code, > (2) resync'ing with the upstream code, a

Re: [U-Boot] [PATCH v2 0/5] Update fastboot sparse image handling

2016-06-07 Thread Steve Rae
On Tue, Jun 7, 2016 at 11:36 AM, Sergey Kubushyn <k...@koi8.net> wrote: > On Tue, 7 Jun 2016, Steve Rae wrote: > > Quick question before diving in -- does anybody work on making > fastboot able to flash multiple devices? > > There are some braindead designs (e.g. Varisc

[U-Boot] [PATCH v2 5/5] fastboot: sparse: improve CHUNK_TYPE_FILL write performance

2016-06-07 Thread Steve Rae
- increase the size of the fill buffer - testing has shown a 10x improvement when the sparse image has large CHUNK_TYPE_FILL chunks Signed-off-by: Steve Rae <s...@broadcom.com> --- Changes in v2: None common/image-sparse.c | 37 +++-- 1 file chang

[U-Boot] [PATCH v2 4/5] fastboot: sparse: implement reserve()

2016-06-07 Thread Steve Rae
In order to process the CHUNK_TYPE_DONT_CARE properly, there is a requirement to be able to 'reserve' a specified number of blocks in the storage media. Because of the special handling of "bad blocks" in NAND devices, this is implemented in a storage abstraction function. Signed-off-by:

[U-Boot] [PATCH v2 3/5] fastboot: sparse: resync common/image-sparse.c (part 2)

2016-06-07 Thread Steve Rae
/quic/la/kernel/lk/plain/app/aboot/aboot.c?h=LE.BR.1.2.1 Signed-off-by: Steve Rae <s...@broadcom.com> --- Changes in v2: None common/fb_mmc.c | 40 common/fb_nand.c| 38 +++--- common

[U-Boot] [PATCH v2 2/5] fastboot: sparse: resync common/image-sparse.c (part 1)

2016-06-07 Thread Steve Rae
=LE.BR.1.2.1 Signed-off-by: Steve Rae <s...@broadcom.com> --- Changes in v2: None common/fb_mmc.c| 32 ++-- common/fb_nand.c | 58 --- common/image-sparse.c | 449 + include/image-sparse.h | 25 +-- 4 files changed, 211 inse

[U-Boot] [PATCH v2 1/5] fastboot: sparse: remove session-id logic

2016-06-07 Thread Steve Rae
This "session-id" alogrithm is not required, and currently corrupts the stored image whenever more the one "session" is required. Signed-off-by: Steve Rae <s...@broadcom.com> --- for more information, see the thread starting at [1] [1] http://lists.denx.de/pipermail/u-b

[U-Boot] [PATCH v2 0/5] Update fastboot sparse image handling

2016-06-07 Thread Steve Rae
-rc1 Steve Rae (5): fastboot: sparse: remove session-id logic fastboot: sparse: resync common/image-sparse.c (part 1) fastboot: sparse: resync common/image-sparse.c (part 2) fastboot: sparse: implement reserve() fastboot: sparse: improve CHUNK_TYPE_FILL write performance common/fb_mmc.c

[U-Boot] [PATCH v4 2/2] USB: DWC2: choose 8-bit phy bus width

2016-06-07 Thread Steve Rae
The Kona PHY supports an 8-bit wide UTMI interface, therefore, choose this Kconfig setting. Signed-off-by: Steve Rae <s...@broadcom.com> --- from [1] (line ~124) /* The Kona PHY supports an 8-bit wide UTMI interface */ [...snip...] module_platform_driver(bcm_kona_usb2_

[U-Boot] [PATCH v4 1/2] usb: dwc2_udc_otg: support 8-bit interface

2016-06-07 Thread Steve Rae
Define CONFIG_USB_GADGET_DWC2_OTG_PHY_BUS_WIDTH_8 to allow the physical interface to be 8-bit (rather than 16-bit). Signed-off-by: Steve Rae <s...@broadcom.com> --- from [1] (line ~124) /* The Kona PHY supports an 8-bit wide UTMI interface */ [...snip...] module_platform_

[U-Boot] [PATCH v3] usb: dwc2_udc_otg: support 8-bit interface

2016-06-06 Thread Steve Rae
Define CONFIG_USB_GADGET_DWC2_OTG_PHY_BUS_WIDTH_8 to allow the physical interface to be 8-bit (rather than 16-bit). Signed-off-by: Steve Rae <s...@broadcom.com> --- from [1] (line ~124) /* The Kona PHY supports an 8-bit wide UTMI interface */ [...snip...] module_platform_

Re: [U-Boot] [PATCH v2] usb: dwc2_udc_otg: support 8-bit interface

2016-06-06 Thread Steve Rae
On Mon, Jun 6, 2016 at 5:38 PM, Marek Vasut <ma...@denx.de> wrote: > On 06/07/2016 01:58 AM, Steve Rae wrote: >> Define CONFIG_USB_GADGET_DWC2_OTG_PHY_BUS_WIDTH_8 to allow the >> physical interface to be 8-bit (rather than 16-bit). >> >> Signed-off-by: Steve Rae

[U-Boot] [PATCH v2] usb: dwc2_udc_otg: support 8-bit interface

2016-06-06 Thread Steve Rae
Define CONFIG_USB_GADGET_DWC2_OTG_PHY_BUS_WIDTH_8 to allow the physical interface to be 8-bit (rather than 16-bit). Signed-off-by: Steve Rae <s...@broadcom.com> --- from [1] (line ~124) /* The Kona PHY supports an 8-bit wide UTMI interface */ [...snip...] module_platform_

Re: [U-Boot] [PATCH] usb: dwc2_udc_otg: support 8-bit interface

2016-06-06 Thread Steve Rae
On Mon, Jun 6, 2016 at 11:58 AM, Steve Rae <steve@broadcom.com> wrote: > On Mon, Jun 6, 2016 at 10:03 AM, Marek Vasut <ma...@denx.de> wrote: >> >> On 06/06/2016 06:50 PM, Steve Rae wrote: >> > >> > >> > On Mon, Jun 6, 2016 at 9:30 AM,

Re: [U-Boot] [PATCH] usb: dwc2_udc_otg: support 8-bit interface

2016-06-06 Thread Steve Rae
On Mon, Jun 6, 2016 at 10:03 AM, Marek Vasut <ma...@denx.de> wrote: > > On 06/06/2016 06:50 PM, Steve Rae wrote: > > > > > > On Mon, Jun 6, 2016 at 9:30 AM, Marek Vasut <ma...@denx.de > > <mailto:ma...@denx.de>> wrote: > > > > On 0

Re: [U-Boot] [PATCH] usb: dwc2_udc_otg: support 8-bit interface

2016-06-06 Thread Steve Rae
On Mon, Jun 6, 2016 at 9:30 AM, Marek Vasut <ma...@denx.de> wrote: > On 06/06/2016 06:15 PM, Steve Rae wrote: > > On Mon, Jun 6, 2016 at 8:59 AM, Marek Vasut <ma...@denx.de> wrote: > >> > >> On 06/06/2016 05:57 PM, Steve Rae wrote: > >>&g

Re: [U-Boot] [PATCH] usb: dwc2_udc_otg: support 8-bit interface

2016-06-06 Thread Steve Rae
On Mon, Jun 6, 2016 at 8:59 AM, Marek Vasut <ma...@denx.de> wrote: > > On 06/06/2016 05:57 PM, Steve Rae wrote: > > Define CONFIG_USB_GADGET_DWC2_PHY_8_BIT to allow the > > physical interface to be 8-bit (rather than 16-bit). > > > > Signed-off-by: Steve Rae <

[U-Boot] [PATCH] usb: dwc2_udc_otg: support 8-bit interface

2016-06-06 Thread Steve Rae
Define CONFIG_USB_GADGET_DWC2_PHY_8_BIT to allow the physical interface to be 8-bit (rather than 16-bit). Signed-off-by: Steve Rae <s...@broadcom.com> --- drivers/usb/gadget/dwc2_udc_otg.c | 4 include/configs/bcm28155_ap.h | 1 + 2 files changed, 5 insertions(+) diff --git a/d

[U-Boot] [PATCH] board: arm:: Add support for Broadcom BCM23550

2016-06-02 Thread Steve Rae
Add support for the Broadcom BCM23550 board. Signed-off-by: Steve Rae <s...@broadcom.com> --- arch/arm/Kconfig| 5 + arch/arm/cpu/armv7/Makefile | 1 + arch/arm/cpu/armv7/bcm235xx/Makefile| 12 + arch/arm/cpu/armv7/bcm235xx/clk-bcm2

Re: [U-Boot] [PATCH 0/5] Update fastboot sparse image handling

2016-05-31 Thread Steve Rae
On Fri, May 20, 2016 at 6:05 PM, Steve Rae <s...@broadcom.com> wrote: > While retaining the storage abstraction feature implemented in U-Boot, > this series updates the fastboot sparse image handling by > (1) fixing broken code, > (2) resync'ing with the upstream code, a

[U-Boot] [PATCH v4] arm/arm64: implement a boot header capability

2016-05-31 Thread Steve Rae
ed-off-by: Andre Przywara <andre.przyw...@arm.com> Signed-off-by: Steve Rae <s...@broadcom.com> Commit Notes: Please note that the current code: start.S (arm64) and vectors.S (arm) already jumps over some portion of data already, so this option basically just increases the s

Re: [U-Boot] [PATCH v3] arm64: arm: implement a boot header capability

2016-05-30 Thread Steve Rae
Hi Tom, On Mon, May 30, 2016 at 11:14 AM, Tom Rini <tr...@konsulko.com> wrote: > > On Mon, May 30, 2016 at 09:51:22AM -0700, Steve Rae wrote: > > > From: Andre Przywara <andre.przyw...@arm.com> > > > > Some SPL loaders (like Allwinner's boot0, and Broadc

[U-Boot] [PATCH v3] arm64: arm: implement a boot header capability

2016-05-30 Thread Steve Rae
header to allow booting A64 based boards. For the Pine64 we need a 1536 byte header (including the branch instruction) at the moment, so we add this to the defconfig. Signed-off-by: Andre Przywara <andre.przyw...@arm.com> Signed-off-by: Steve Rae <s...@broadcom.com> [1] https://github.com/ap

Re: [U-Boot] [PATCH v2] arm64: allow space reservation for boot header

2016-05-25 Thread Steve Rae
On Wed, May 25, 2016 at 9:50 AM, Andre Przywara <andre.przyw...@arm.com> wrote: > Hi, > > On 25/05/16 17:15, Steve Rae wrote: > > Hi, > > > > On Wed, May 25, 2016 at 1:48 AM, Hans de Goede <hdego...@redhat.com > > <mailto:hdego...@redhat.com>> w

Re: [U-Boot] [PATCH v2] arm64: allow space reservation for boot header

2016-05-25 Thread Steve Rae
Hi, On Wed, May 25, 2016 at 1:48 AM, Hans de Goede wrote: > Hi, > > On 25-05-16 10:40, Andre Przywara wrote: > >> Some SPL loaders (like Allwinner's boot0) require a header before the >> actual U-Boot binary to both check its validity and to find other data >> to load. >>

[U-Boot] [PATCH 0/5] Update fastboot sparse image handling

2016-05-20 Thread Steve Rae
While retaining the storage abstraction feature implemented in U-Boot, this series updates the fastboot sparse image handling by (1) fixing broken code, (2) resync'ing with the upstream code, and (3) improving performance when writing CHUNK_TYPE_FILL Steve Rae (5): fastboot: sparse: remove

[U-Boot] [PATCH 4/5] fastboot: sparse: implement reserve()

2016-05-20 Thread Steve Rae
In order to process the CHUNK_TYPE_DONT_CARE properly, there is a requirement to be able to 'reserve' a specified number of blocks in the storage media. Because of the special handling of "bad blocks" in NAND devices, this is implemented in a storage abstraction function. Signed-off-by:

[U-Boot] [PATCH 5/5] fastboot: sparse: improve CHUNK_TYPE_FILL write performance

2016-05-20 Thread Steve Rae
- increase the size of the fill buffer - testing has shown a 10x improvement when the sparse image has large CHUNK_TYPE_FILL chunks Signed-off-by: Steve Rae <s...@broadcom.com> --- common/image-sparse.c | 37 +++-- 1 file changed, 27 insertions(+), 10 del

[U-Boot] [PATCH 2/5] fastboot: sparse: resync common/image-sparse.c (part 1)

2016-05-20 Thread Steve Rae
=LE.BR.1.2.1 Signed-off-by: Steve Rae <s...@broadcom.com> --- common/fb_mmc.c| 32 ++-- common/fb_nand.c | 52 +++--- common/image-sparse.c | 449 + include/image-sparse.h | 25 +-- 4 files changed, 208 insertions(+), 350 del

[U-Boot] [PATCH 1/5] fastboot: sparse: remove session-id logic

2016-05-20 Thread Steve Rae
This "session-id" alogrithm is not required, and currently corrupts the stored image whenever more the one "session" is required. Signed-off-by: Steve Rae <s...@broadcom.com> --- for more information, see the thread starting at [1] [1] http://lists.denx.de/pipermail/u-b

[U-Boot] [PATCH 3/5] fastboot: sparse: resync common/image-sparse.c (part 2)

2016-05-20 Thread Steve Rae
/quic/la/kernel/lk/plain/app/aboot/aboot.c?h=LE.BR.1.2.1 Signed-off-by: Steve Rae <s...@broadcom.com> --- common/fb_mmc.c | 40 common/fb_nand.c| 38 +++--- common/image-sparse.c

Re: [U-Boot] [RFC] fastboot: sparse image handling and sessionId

2016-05-13 Thread Steve Rae
On Mon, May 9, 2016 at 8:00 AM, Maxime Ripard < maxime.rip...@free-electrons.com> wrote: > Hi Steve, > > Sorry for the late reply. > > On Fri, Apr 15, 2016 at 09:14:41PM -0700, Steve Rae wrote: > > Maxime, > > > > I suspect that the issue fixed > >

Re: [U-Boot] [PATCH 3/6] arm64: sunxi: reserve space for boot0 header

2016-05-06 Thread Steve Rae
On Fri, May 6, 2016 at 2:27 PM, André Przywara <andre.przyw...@arm.com> wrote: > On 06/05/16 22:15, Steve Rae wrote: > > Hi Andre, > > > > On Wed, May 4, 2016 at 2:15 PM, Andre Przywara <andre.przyw...@arm.com > > <mailto:andre.przyw...@arm.com>> wr

Re: [U-Boot] [PATCH 3/6] arm64: sunxi: reserve space for boot0 header

2016-05-06 Thread Steve Rae
Hi Andre, On Wed, May 4, 2016 at 2:15 PM, Andre Przywara wrote: > The Allwinner provided boot0 boot loader requires a header before the > U-Boot binary to both check its validity and to find other blobs to > load. There is a tool called boot0img which fills the header >

Re: [U-Boot] [RFC] fastboot: sparse image handling and sessionId

2016-05-01 Thread Steve Rae
On Apr 15, 2016 21:14, "Steve Rae" <steve@broadcom.com> wrote: > > Maxime, > > I suspect that the issue fixed > in commit c7529dbfastboot: sparse: fix block addressing for > don't care chunk type > > complicated the assumptions which led

Re: [U-Boot] [PATCH 3/3] usb: s3c-otg: Fix remaining bytes in debug messages

2016-04-21 Thread Steve Rae
On Thu, Apr 21, 2016 at 7:49 PM, Steve Rae <steve@broadcom.com> wrote: > > > On Thu, Apr 21, 2016 at 3:05 AM, Lukasz Majewski <l.majew...@samsung.com> > wrote: > >> Hi Roger, >> >> > Remaining bytes means bytes that are not yet transfer

Re: [U-Boot] [PATCH 1/3] fastboot: Clean up bulk-out logic

2016-04-21 Thread Steve Rae
On Thu, Apr 21, 2016 at 3:03 AM, Lukasz Majewski <l.majew...@samsung.com> wrote: > Hi Roger, > > > Just use ep->maxpacket to get the maxpacket size > > and simplify the bulk-out maxpacket alignment. > > > > Signed-off-by: Roger Quadros <rog...@ti.com>

Re: [U-Boot] [PATCH 3/3] usb: s3c-otg: Fix remaining bytes in debug messages

2016-04-21 Thread Steve Rae
plete_tx(struct > > dwc2_udc *dev, u8 ep_num) "%s: TX DMA done : ep = %d, tx bytes = > > %d/%d, " "is_short = %d, DIEPTSIZ = 0x%x, remained bytes = %d\n", > > __func__, ep_num, req->req.actual, req->req.length, > > - is_short, ep_t

Re: [U-Boot] [PATCH] fastboot: Fix OUT transaction length alignment

2016-04-18 Thread Steve Rae
On Mon, Apr 18, 2016 at 6:55 AM, Roger Quadros <rog...@ti.com> wrote: > On 18/04/16 16:47, Lukasz Majewski wrote: >> Hi Roger, Steve, >> >>> +Lukazs >>> >>> On 18/04/16 10:56, Roger Quadros wrote: >>>> On 15/04/16 22:44, Steve Rae wro

Re: [U-Boot] couple questions about tail end of common/Makefile

2016-04-16 Thread Steve Rae
On Sat, Apr 16, 2016 at 2:25 AM, Robert P. J. Day wrote: > > first, this related to CONFIG_FASTBOOT_FLASH: > > # This option is not just y/n - it can have a numeric value > ifdef CONFIG_FASTBOOT_FLASH > obj-y += image-sparse.o > ifdef CONFIG_FASTBOOT_FLASH_MMC_DEV

[U-Boot] [RFC] fastboot: sparse image handling and sessionId

2016-04-15 Thread Steve Rae
Maxime, I suspect that the issue fixed in commit c7529dbfastboot: sparse: fix block addressing for don't care chunk type complicated the assumptions which led to the implementation of the "sessionId" in commit 1f8690asparse: Implement several chunks flashing The fastboot client

Re: [U-Boot] [PATCH v2] fastboot: OUT transaction length must be aligned to wMaxPacketSize

2016-04-15 Thread Steve Rae
On Thu, Apr 14, 2016 at 4:15 AM, Roger Quadros <rog...@ti.com> wrote: > Steve, > > On 13/04/16 04:55, Steve Rae wrote: > > On Tue, Apr 12, 2016 at 6:50 AM, Roger Quadros <rog...@ti.com> wrote: > >> Lukasz, > >> > >> On 12/04/16 16:37,

Re: [U-Boot] [PATCH] fastboot: Fix OUT transaction length alignment

2016-04-15 Thread Steve Rae
On Thu, Apr 14, 2016 at 3:18 AM, Roger Quadros wrote: > Hi, > > On 13/04/16 19:56, Sam Protsenko wrote: > > On Wed, Apr 13, 2016 at 3:32 PM, Roger Quadros wrote: > >> Hi, > >> > >> On 13/04/16 15:01, Semen Protsenko wrote: > >>> From: Sam Protsenko

Re: [U-Boot] [PATCH] fastboot: Fix OUT transaction length alignment

2016-04-13 Thread Steve Rae
On Wed, Apr 13, 2016 at 9:56 AM, Sam Protsenko wrote: > On Wed, Apr 13, 2016 at 3:32 PM, Roger Quadros wrote: > > Hi, > > > > On 13/04/16 15:01, Semen Protsenko wrote: > >> From: Sam Protsenko > >> > >> Some UDC controllers

Re: [U-Boot] [PATCH 1/2] fastboot: Fix wMaxPacketSize for High-Speed IN endpoint

2016-04-12 Thread Steve Rae
oops -- my last reply was interpreted as a new patch: http://patchwork.ozlabs.org/patch/609843/ so I'll try again: Tested-by: Steve Rae <s...@broadcom.com> [Test HW: bcm235xx board] On Tue, Apr 12, 2016 at 5:51 AM, Roger Quadros <rog...@ti.com> wrote: > wMaxPacketSize for IN e

Re: [U-Boot] [PATCH v2] fastboot: OUT transaction length must be aligned to wMaxPacketSize

2016-04-12 Thread Steve Rae
On Tue, Apr 12, 2016 at 6:50 AM, Roger Quadros wrote: > Lukasz, > > On 12/04/16 16:37, Lukasz Majewski wrote: >> Hi Roger, >> >>> Hi, >>> >>> On 12/04/16 14:19, Lukasz Majewski wrote: Hi Tom, Mugunthan > On Mon, Apr 11, 2016 at 05:04:56PM +0530, Mugunthan V N wrote:

Re: [U-Boot] [PATCH 2/2] fastboot: Enable the respective speed endpoints at runtime

2016-04-12 Thread Steve Rae
quot;); >> return ret; >> @@ -280,7 +290,8 @@ static int fastboot_set_alt(struct usb_function >> *f, } >> f_fb->out_req->complete = rx_handler_command; >> >> - ret = usb_ep_enable(f_fb->in_ep, _ep_in); >> + d = fb_ep_

  1   2   3   4   >