Hi,
On Wed, 21 Jun 2017 09:59:05 +0200 Lothar Waßmann wrote:
> Hi,
>
> On Tue, 20 Jun 2017 12:26:29 -0600 Simon Glass wrote:
> > Hi Lothar,
> >
> > On 20 June 2017 at 04:25, Lothar Waßmann wrote:
> > > LCD_MAX_WIDTH, LCD_MAX_HEIGHT and LCD_MAX_LSBPP are not alternative
> > > values for one spec
Hello,
Am 23.06.2017 um 01:49 schrieb Tom Rini:
On Thu, Jun 22, 2017 at 10:05:48AM -0600, Simon Glass wrote:
Hi Bin,
On 20 June 2017 at 17:19, Bin Meng wrote:
+Simon
On Wed, Jun 21, 2017 at 4:08 AM, Otavio Salvador
wrote:
On Tue, Jun 20, 2017 at 2:42 PM, Tom Rini wrote:
On Mon, Jun 19,
On 2017/6/23 1:29, Heiko Stuebner wrote:
Hi,
Am Mittwoch, 21. Juni 2017, 11:58:24 CEST schrieb Meng Dongyang:
Use dev_read_bool() instead of fdt_getprop() to get the property
from DTS. And add a comment for "hnp-srp-disable" property to
fully describe its effect.
Signed-off-by: Meng Dongyang
On Thu, Jun 22, 2017 at 10:05:48AM -0600, Simon Glass wrote:
> Hi Bin,
>
> On 20 June 2017 at 17:19, Bin Meng wrote:
> >
> > +Simon
> >
> > On Wed, Jun 21, 2017 at 4:08 AM, Otavio Salvador
> > wrote:
> > > On Tue, Jun 20, 2017 at 2:42 PM, Tom Rini wrote:
> > >> On Mon, Jun 19, 2017 at 11:56:04A
Hi York,
There are about 258 mpc85xx boards. In converting things to Kconfig quite a
bit of the work comes in this area. I am just wondering if all these boards
are still needed, or it would be possible to prune them a bit? I see you
have already done a bit of this.
Regards,
Simon
___
On Thu, Jun 22, 2017 at 2:49 AM, Masahiro Yamada
wrote:
> This is only used in cmd/net.c
>
> Signed-off-by: Masahiro Yamada
Acked-by: Joe Hershberger
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
Hi,
Am Mittwoch, 21. Juni 2017, 11:58:24 CEST schrieb Meng Dongyang:
> Use dev_read_bool() instead of fdt_getprop() to get the property
> from DTS. And add a comment for "hnp-srp-disable" property to
> fully describe its effect.
>
> Signed-off-by: Meng Dongyang
> ---
>
> Changes in v6:
> - New
Guillaume,
> On 22 Jun 2017, at 10:11, Guillaume GARDET wrote:
>
> Revert commit 253c60a557d6740f15169a1f15772d7e64928d9b as it breaks the
> return value of 'mkimage -T rkimage' and print the following error:
> './tools/mkimage: Can't print header for Rockchip Boot Image support: Success’
If
Hi Bin,
On 20 June 2017 at 17:19, Bin Meng wrote:
>
> +Simon
>
> On Wed, Jun 21, 2017 at 4:08 AM, Otavio Salvador
> wrote:
> > On Tue, Jun 20, 2017 at 2:42 PM, Tom Rini wrote:
> >> On Mon, Jun 19, 2017 at 11:56:04AM -0300, Otavio Salvador wrote:
> >>
> >>> To allow the loading of environment fr
The correct name is 'Celsius', so fix it accordingly.
Signed-off-by: Fabio Estevam
---
arch/arm/cpu/armv7/mx6/soc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/cpu/armv7/mx6/soc.c b/arch/arm/cpu/armv7/mx6/soc.c
index dd94797..2bedbdb 100644
--- a/arch/arm/cpu/arm
Hello Christophe,
Am 22.06.2017 um 13:20 schrieb Christophe LEROY:
Le 22/06/2017 à 11:59, Wolfgang Denk a écrit :
Dear Christophe,
In message <0784ad6e-86ab-9c1d-1b81-a5cacaf2b...@c-s.fr> you wrote:
Please see my previous comments to Tom's message. At least, please
clean up the directory
On Thu, 22 Jun 2017 17:42:38 +0530
Vignesh R wrote:
>
>
> On Wednesday 21 June 2017 01:39 PM, Lukasz Majewski wrote:
> > Hi Vignesh,
> >
> >> Hi,
> >>
> >> On Tuesday 20 June 2017 07:14 PM, Lukasz Majewski wrote:
> >>> Hi Marek, Vignesh,
> >> [...]
> >
> > All gadget drivers like ether
On Wednesday 21 June 2017 01:39 PM, Lukasz Majewski wrote:
> Hi Vignesh,
>
>> Hi,
>>
>> On Tuesday 20 June 2017 07:14 PM, Lukasz Majewski wrote:
>>> Hi Marek, Vignesh,
>> [...]
>
> All gadget drivers like ether.c or f_mass_storage.c call
> usb_gadget_handle_interrupts() just passing
Le 22/06/2017 à 11:59, Wolfgang Denk a écrit :
Dear Christophe,
In message <0784ad6e-86ab-9c1d-1b81-a5cacaf2b...@c-s.fr> you wrote:
Please see my previous comments to Tom's message. At least, please
clean up the directory structure and get rid of unrelated (4xx) and
untested/broken code (b
Remove the error which causes compilation failure when
dcache is off for builds otherthan SPL. There may be
cases where user wants to disable dcache completely
eventhough it is not SPL.
Signed-off-by: Siva Durga Prasad Paladugu
---
arch/arm/cpu/armv8/cache_v8.c | 9 -
1 file changed, 9 d
Dear Christophe,
In message <0784ad6e-86ab-9c1d-1b81-a5cacaf2b...@c-s.fr> you wrote:
>
> > Please see my previous comments to Tom's message. At least, please
> > clean up the directory structure and get rid of unrelated (4xx) and
> > untested/broken code (bedbug, probably kgdb, pcmcia, usb).
>
From: Kunihiko Hayashi
Add initial device tree support for LD11 Global board.
Signed-off-by: Kunihiko Hayashi
Signed-off-by: Masahiro Yamada
---
arch/arm/dts/Makefile | 1 +
arch/arm/dts/uniphier-ld11-global.dts | 79 +++
2 files changed, 80 i
From: Kunihiko Hayashi
Add initial device tree support for LD20 Global board.
Signed-off-by: Kunihiko Hayashi
Signed-off-by: Masahiro Yamada
---
arch/arm/dts/Makefile | 1 +
arch/arm/dts/uniphier-ld20-global.dts | 61 +++
2 files changed, 62 i
On 06/22/2017 09:35 AM, Masahiro Yamada wrote:
> Sparse reports "... was not declared. Should it be static?"
>
> Signed-off-by: Masahiro Yamada
Applied, thanks.
> ---
>
> drivers/usb/host/ehci-hcd.c | 2 +-
> drivers/usb/host/usb-uclass.c | 18 +-
> drivers/usb/host/xhci-dw
On 22.06.2017 11:13, m...@marvell.com wrote:
From: Ken Ma
In armada_37xx_gpiochip_register, the return value of fdtdec_get_bool
should be true when gpio-controller is found; current codes makes a
wrong inverse return value judgement, this patch fixes it.
Signed-off-by: Ken Ma
Cc: Stefan Roese
Hi Ken,
On 22.06.2017 11:13, m...@marvell.com wrote:
> From: Ken Ma
>
> If north bridge selection register bit1 is clear, pins [10:8] are for
> SDIO0 Resetn, Wakeup, and PDN while if bit1 is set, pins [10:8]are for
> GPIO; when bit1 is clear, pin 9 and pin 10 can be used for uart2 RTSn
> and CTS
Hello Wolfgang,
On 2017/6/22 15:07, Wolfgang Denk wrote:
Dear Wenyou,
In message
you wrote:
After supporting the driver model and device tree, the size of SPL is beyond 4K.
SPL is not suitable in 4K internal SRAM
If you are aware of this situation, you should not post patches that
break a
From: Ken Ma
If north bridge selection register bit1 is clear, pins [10:8] are for
SDIO0 Resetn, Wakeup, and PDN while if bit1 is set, pins [10:8]are for
GPIO; when bit1 is clear, pin 9 and pin 10 can be used for uart2 RTSn
and CTSn, so bit1 should be added to uart2 group and it must be set
for b
From: Ken Ma
In armada_37xx_gpiochip_register, the return value of fdtdec_get_bool
should be true when gpio-controller is found; current codes makes a
wrong inverse return value judgement, this patch fixes it.
Signed-off-by: Ken Ma
Cc: Stefan Roese
Cc: Kostya Porotchkin
Cc: Gregory CLEMENT
C
From: Ken Ma
*** BLURB HERE ***
Fix 2 armada-37xx pinctrl issues.
Ken Ma (2):
pinctrl: a3700: Fix uart2 group selection register mask
pinctrl: a3700: Fix the issue that gpio controller is registered with
wrong node id
drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 7 ---
1 file chan
On Thu, Jun 22, 2017 at 4:49 PM, Masahiro Yamada
wrote:
> These are locally used in lib/efi_loader/efi_boottime.c
>
> Signed-off-by: Masahiro Yamada
> ---
>
> lib/efi_loader/efi_boottime.c | 20 +++-
> 1 file changed, 11 insertions(+), 9 deletions(-)
>
Reviewed-by: Bin Meng
___
DTB is encoded in big endian. When we retrieve property values,
we need to use fdt32_to_cpu (aka be32_to_cpu) for endian conversion.
This is a bit error-prone, but sparse is useful to detect endian
mismatch.
We need to use (fdt32_t *) instead of (u32 *) for a pointer of a
property value. Otherwi
DTB is encoded in big endian. When we retrieve property values,
we need to use fdt32_to_cpu (aka be32_to_cpu) for endian conversion.
This is a bit error-prone, but sparse is useful to detect endian
mismatch.
We need to use (fdt32_t *) instead of (u32 *) for a pointer of a
property value. Otherwi
Le 21/06/2017 à 23:43, Tom Rini a écrit :
On Wed, Jun 21, 2017 at 11:41:02PM +0200, Christophe LEROY wrote:
Le 21/06/2017 à 17:26, Tom Rini a écrit :
From: Christophe Leroy
Please fill me in better with v3. I squahed the previous 1-10 into a
single commit that brings back the core of mpc8
These are locally used in lib/efi_loader/efi_boottime.c
Signed-off-by: Masahiro Yamada
---
lib/efi_loader/efi_boottime.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c
index 51080cbeed2f..ba
On Thu, Jun 22, 2017 at 4:12 PM, Masahiro Yamada
wrote:
>> dm: include from driver/core/dump.c
>
> A typo in the subject.
> I mean "drivers/core/dump.c"
>
Reviewed-by: Bin Meng
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinf
Le 22/06/2017 à 09:20, Wolfgang Denk a écrit :
Dear Tom,
In message <20170621152200.GJ27196@bill-the-cat> you wrote:
With what I'm going to send out as a v2 of the rest of the series, I'm
OK with not including an empty directory for now and bringing back
mpc8xx, thanks again!
Sorry, but I
Le 22/06/2017 à 09:35, Wolfgang Denk a écrit :
Dear Christophe Leroy,
In message <20170621213804.c5aca67...@pc13941vm.idsi0.si.c-s.fr> you wrote:
Signed-off-by: Christophe Leroy
---
v3: Fixed build error in arch/powerpc/include/asm/ppc.h ; removed commproc.h
from 4xx
v2: Tom squashed p
On Thu, Jun 22, 2017 at 4:22 PM, Masahiro Yamada
wrote:
> 2017-06-22 16:52 GMT+09:00 Masahiro Yamada :
>> This function is only used in common/spl/spl_mmc.c[
>
> The '[' at the end of line is a typo.
>
> Please drop it when applied.
>
Reviewed-by: Bin Meng
___
On Thu, Jun 22, 2017 at 3:51 PM, Masahiro Yamada
wrote:
> These are only used in drivers/mis/i2c_eeprom.c
>
> Signed-off-by: Masahiro Yamada
> ---
>
> drivers/misc/i2c_eeprom.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
Reviewed-by: Bin Meng
_
2017-06-22 16:52 GMT+09:00 Masahiro Yamada :
> This function is only used in common/spl/spl_mmc.c[
The '[' at the end of line is a typo.
Please drop it when applied.
--
Best Regards
Masahiro Yamada
___
U-Boot mailing list
U-Boot@lists.denx.de
https
On Thu, Jun 22, 2017 at 3:50 PM, Masahiro Yamada
wrote:
> Signed-off-by: Masahiro Yamada
> ---
>
> drivers/gpio/gpio-uclass.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
Reviewed-by: Bin Meng
___
U-Boot mailing list
U-Boot@lists.den
On Thu, Jun 22, 2017 at 3:50 PM, Masahiro Yamada
wrote:
> Fix sparse warnings "... was not declared. Should it be static?"
>
> Also, fix redefinition of dm_warn/dm_dbg.
>
> Signed-off-by: Masahiro Yamada
> ---
>
> drivers/core/util.c | 5 +
> 1 file changed, 5 insertions(+)
>
Reviewed-by: B
On Thu, Jun 22, 2017 at 3:49 PM, Masahiro Yamada
wrote:
> This is only used in cmd/net.c
>
> Signed-off-by: Masahiro Yamada
> ---
>
> cmd/net.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Reviewed-by: Bin Meng
___
U-Boot mailing list
U-
On Thu, Jun 22, 2017 at 3:48 PM, Masahiro Yamada
wrote:
> Add missing static to serial_stub_puts().
>
> Unexport serial_stub_{getc,tstc} because they are used locally.
>
> Signed-off-by: Masahiro Yamada
> ---
>
> drivers/serial/serial-uclass.c | 11 +++
> drivers/serial/serial.c|
On Thu, Jun 22, 2017 at 3:35 PM, Masahiro Yamada
wrote:
> Sparse reports "... was not declared. Should it be static?"
>
> Signed-off-by: Masahiro Yamada
> ---
>
> drivers/usb/host/ehci-hcd.c | 2 +-
> drivers/usb/host/usb-uclass.c | 18 +-
> drivers/usb/host/xhci-dwc3.c | 2
> dm: include from driver/core/dump.c
A typo in the subject.
I mean "drivers/core/dump.c"
2017-06-22 17:10 GMT+09:00 Masahiro Yamada :
> Include to fix sparse warnings:
> symbol 'dm_dump_all' was not declared. Should it be static?
> symbol 'dm_dump_uclass' was not declared. Should it be stati
Revert commit 253c60a557d6740f15169a1f15772d7e64928d9b as it breaks the
return value of 'mkimage -T rkimage' and print the following error:
'./tools/mkimage: Can't print header for Rockchip Boot Image support: Success'
Signed-off-by: Guillaume GARDET
Cc: Philipp Tomsich
Cc: Simon Glass
Cc:
Include to fix sparse warnings:
symbol 'dm_dump_all' was not declared. Should it be static?
symbol 'dm_dump_uclass' was not declared. Should it be static?
Signed-off-by: Masahiro Yamada
---
drivers/core/dump.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/core/dump.c b/drivers/co
This function returns the pointer to the value of a node property.
The current name ofnode_read_prop() is confusing. Follow the naming
of_get_property() from Linux.
The return type (const u32 *) is wrong. DT property values can be
strings as well as integers. This is why of_get_property/fdt_get
The previous commit renamed ofnode_read_prop() to ofnode_get_propery()
and fixed its return type. Likewise, rename dev_read_prop() and fix
its return type.
Signed-off-by: Masahiro Yamada
---
drivers/core/read.c| 3 ++-
drivers/input/key_matrix.c | 4 ++--
include/dm/read.h | 9
Reuse ofnode_get_property() to simplify the implementation.
Signed-off-by: Masahiro Yamada
---
drivers/core/ofnode.c | 14 +-
1 file changed, 5 insertions(+), 9 deletions(-)
diff --git a/drivers/core/ofnode.c b/drivers/core/ofnode.c
index 87e731dc2b77..5a3e1d452fbb 100644
--- a/dri
The code inside the if-block is the same as of_get_property().
Signed-off-by: Masahiro Yamada
---
drivers/core/ofnode.c | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/drivers/core/ofnode.c b/drivers/core/ofnode.c
index b8b16bf5e275..08b3e03b1e8e 100644
--- a/dr
This will clarify the code.
Signed-off-by: Masahiro Yamada
---
drivers/core/ofnode.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/core/ofnode.c b/drivers/core/ofnode.c
index ac312d654654..b8b16bf5e275 100644
--- a/drivers/core/ofnode.c
+++ b/drivers/cor
This function is only used in common/spl/spl_mmc.c[
Signed-off-by: Masahiro Yamada
---
common/spl/spl_mmc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c
index 18c1b59b22cc..bb48cac1efac 100644
--- a/common/spl/spl_mmc.c
+++ b/co
These are only used in drivers/mis/i2c_eeprom.c
Signed-off-by: Masahiro Yamada
---
drivers/misc/i2c_eeprom.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/misc/i2c_eeprom.c b/drivers/misc/i2c_eeprom.c
index da6e2b05f730..a14e83225bcc 100644
--- a/drivers/misc/i
Signed-off-by: Masahiro Yamada
---
drivers/gpio/gpio-uclass.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpio/gpio-uclass.c b/drivers/gpio/gpio-uclass.c
index f611996f1729..9faf3357af8b 100644
--- a/drivers/gpio/gpio-uclass.c
+++ b/drivers/gpio/gpio-uclass.c
Fix sparse warnings "... was not declared. Should it be static?"
Also, fix redefinition of dm_warn/dm_dbg.
Signed-off-by: Masahiro Yamada
---
drivers/core/util.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/core/util.c b/drivers/core/util.c
index 5ceac8bbb15b..2e232d57a14f
Add missing static to serial_stub_puts().
Unexport serial_stub_{getc,tstc} because they are used locally.
Signed-off-by: Masahiro Yamada
---
drivers/serial/serial-uclass.c | 11 +++
drivers/serial/serial.c| 4 ++--
include/common.h | 5 -
3 files changed, 9
This is only used in cmd/net.c
Signed-off-by: Masahiro Yamada
---
cmd/net.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cmd/net.c b/cmd/net.c
index df8b6c9b53f0..5e91d3ab8a19 100644
--- a/cmd/net.c
+++ b/cmd/net.c
@@ -42,7 +42,7 @@ U_BOOT_CMD(
);
#ifdef CONFIG_CMD_TF
Signed-off-by: Masahiro Yamada
---
arch/arm/dts/uniphier-ld11-ref.dts | 2 +-
arch/arm/dts/uniphier-ld11.dtsi | 40 ++---
arch/arm/dts/uniphier-ld20-ref.dts | 2 +-
arch/arm/dts/uniphier-ld20.dtsi | 40 ++---
arc
Fix warnings reported by sparse:
- ... was not declared. Should it be static?"
- cast to restricted __be32
While fixing those, the type conflict of cci500_init() was found.
Signed-off-by: Masahiro Yamada
---
arch/arm/mach-uniphier/arm64/arm-cci500.c| 2 ++
arch/arm/mach-uniphier/arm64/s
Hi Tom,
The following changes since commit 235c5b8315c6a9eb566fd3d99a098cc6db869fc5:
Merge branch 'master' of git://git.denx.de/u-boot-samsung
(2017-06-21 08:01:07 -0400)
are available in the git repository at:
git://git.denx.de/u-boot-x86.git
for you to fetch changes up to c8f258d8a8452b9
Sparse reports "... was not declared. Should it be static?"
Signed-off-by: Masahiro Yamada
---
drivers/usb/host/ehci-hcd.c | 2 +-
drivers/usb/host/usb-uclass.c | 18 +-
drivers/usb/host/xhci-dwc3.c | 2 +-
drivers/usb/host/xhci.c | 4 ++--
4 files changed, 13 insert
Dear Christophe Leroy,
In message <20170621213804.c5aca67...@pc13941vm.idsi0.si.c-s.fr> you wrote:
> Signed-off-by: Christophe Leroy
> ---
> v3: Fixed build error in arch/powerpc/include/asm/ppc.h ; removed commproc.h
> from 4xx
>
> v2: Tom squashed patches 1-10 of the serie
Please see my pr
Dear Tom,
In message <1498058762-1909-1-git-send-email-tr...@konsulko.com> you wrote:
>
> Please fill me in better with v3. I squahed the previous 1-10 into a
> single commit that brings back the core of mpc8xx. Please add an entry
> to MAINTAINERS for yourself for mpc8xx and do a travis-ci bui
Dear Tom,
In message <20170621152200.GJ27196@bill-the-cat> you wrote:
>
> With what I'm going to send out as a v2 of the rest of the series, I'm
> OK with not including an empty directory for now and bringing back
> mpc8xx, thanks again!
Sorry, but I disagree here.
This revert series is a mess.
A valid input slot context for a 'configure endpoint' command requires
the 'Context Entries' field to be initialized to the index of the last
valid endpoint context that is defined by the target configuration. We
set up the 'Context Entries' field, but we forget to include the input
slot context in
Now that xHCD does not use CONFIG_SYS_USB_XHCI_MAX_ROOT_PORTS,
remove it in all boards' config files.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2: None
include/configs/am43xx_evm.h | 1 -
include/configs/am57xx
USB 3.0 hubs have a slightly different hub descriptor than USB 2.0
hubs, with a fixed (rather than variable length) size. Change the
host controller drivers that access those last two fields
(DeviceRemovable and PortPowerCtrlMask) to use the union.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
USB 3.0 defines bMaxPacketSize0 field in the device descriptor as
the exponent of 2, so let's print the calculated actual size.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2:
- Change to use uint and also 'normal' variabl
Now that EHCD does not use CONFIG_SYS_USB_EHCI_MAX_ROOT_PORTS,
remove it in all boards' config files.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2: None
arch/arm/include/asm/ehci-omap.h| 4
include/configs/MPC8
Add CONFIG_USB_XHCI_PCI as a Kconfig option.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2:
- Update description to use "PCI-based"
- Remove CONFIG_USB_XHCI_PCI from config_whitelist.txt
drivers/usb/host/Kconfig | 6
Occasionally it was observed that on Intel MinnowMax board, with a
USB 2.0 device connected to the bottom port, when doing 'usb start'
on the xHCI controller:
scanning bus 0 for devices... cannot reset port 3!?
But neither of the two USB ports is routed to xHCI root port 3.
Adding some debug in
BayTrail SoC supports both EHCI and xHCI controllers. However only
one host controller (either EHCI or xHCI) can be used. To enable
HSIC and SS ports, xHCI must be used. This turns on xHCI support on
Intel MinnowMax board.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roes
HCSPARAMS1:MaxPorts field specifies the maximum port number value,
and its valid values are in the range of 1 to 255.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2: None
drivers/usb/host/xhci.h | 6 +++---
1 file changed
xHC reports supported maximum number of ports in the HCSPARAMS1
register, so it's unnecessary to use a hardcoded config option
CONFIG_SYS_USB_XHCI_MAX_ROOT_PORTS.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2: None
drive
Testing a USB 3.0 hub by connecting it to the xHCI port on Intel
MinnowMax, when issuing 'get hub descriptor' to the hub, xHCI
reports a transfer event TRB with a completion code 6 which means
'Stall Error'.
In fact super speed USB hub descriptor type is 0x2a, not 0x29.
Sending correct SETUP packe
USB 3.0 hub port status has different bit position regarding to
port power, port speed, etc. But others are the same as 2.0 hubs.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2: None
include/usb_defs.h | 7 ++-
1 file
EHC reports supported maximum number of ports in the HCSPARAMS
register, so it's unnecessary to use a hardcoded config option
CONFIG_SYS_USB_EHCI_MAX_ROOT_PORTS.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2: None
driver
It was observed that on Intel MinnowMax board, when xHCI is enabled
in the BayTrail SoC, with a USB 3.0 device connected to the bottom
USB 3.0 port (mapped to xHCI root port #7), its PORTSC register is
always 0x201203 (CCS = 1, CSC = 0). The root cause of such behavior
is unknown yet. Connect statu
For accuracy, we should use 'sizeof(struct usb_port_status)' as the
wLength for 'get port status' request, although it happens to be
equal to 'sizeof(struct usb_hub_status)'.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2:
There is no member called 'dma' in struct xhci_container_ctx. Remove
the comments that mentions it.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2: None
drivers/usb/host/xhci.h | 3 +--
1 file changed, 1 insertion(+), 2 d
The scratchpad buffer array is used to define the locations of
statically allocated memory pages that are available for the
private use of the xHC. The xHCI spec explicitly mentions that
system software shall allocate the scratchpad buffers before
placing the xHC in to Run mode (Run/Stop (R/S) = ‘1
In xhci_queue_command(), when the command is not 'reset endpoint',
'stop endpoint' or 'set TR dequeue pointer', endpoint ID should not
be encoded in the TRB.
Signed-off-by: Bin Meng
Reviewed-by: Simon Glass
Reviewed-by: Stefan Roese
Tested-by: Stefan Roese
---
Changes in v2: None
drivers/us
Currently U-Boot xHCI driver does not work on Intel BayTrail SoC based
boards that have USB 3.0 ports. Trying to do a 'usb start' simply hangs
the board. This series fixes a bunch of xHCI driver issues to make it
work on Intel's platform.
Issues identified in the U-Boot xHCI driver are really spec
Dear Wenyou,
In message
you wrote:
>
> After supporting the driver model and device tree, the size of SPL is beyond
> 4K.
> SPL is not suitable in 4K internal SRAM
If you are aware of this situation, you should not post patches that
break a number of existing boards.
Instead, we should search
82 matches
Mail list logo
