Skip to content

ubs: tegra: two fixes and cleanup

Stefan Agner u-boot 3 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v5 Latest3 patches 0 repliesNew
  2. v43 patches 0 repliesSuperseded
  3. v33 patches 0 repliesSuperseded

Changes in v5 from the cover letter

- Merged Tegra124 USB header file as well

Patches in v5 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3usb: tegra: fix USB2 powerdown for Tegra30 and laterdrivers/usb/host/ehci-tegra.c+3 -01 reply
2/3usb: tegra: fix PHY configurationarch/arm/include/asm/arch-tegra/usb.h arch/arm/include/asm/arch-tegra20/usb.h drivers/usb/host/ehci-tegra.c+27 -90 replies
3/3usb: tegra: combine header filearch/arm/include/asm/arch-tegra/usb.h arch/arm/include/asm/arch-tegra114/usb.h arch/arm/include/asm/arch-tegra124/usb.h +4 more+225 -7550 replies

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchStephen WarrenJim Lin
SeriesA-b
1/3A-b T-b
2/3R-b
3/3R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v5 thread

Cover letter [U-Boot] [PATCH v5 0/3] ubs: tegra: two fixes and cleanup

Only patch 1 is an actual issue while the second patch is something I stumbled upon, and the third is more about housekeeping.

I descieded to ifdef the driver, I think that this is easier to read and does not mix register access style. The function ehci_get_port_speed however don't get a pointer to the usb_ctlr struct and thouse has to work around that. I don't changed the register access style there because of this reason.

The third patch is optional and implements Stephen's suggestion to merge the USB headers. I'm not sure wheater Tegra124 would also profit from this merge, if yes, I think then its definitely worth doing it.

---
Changes since v4:
- Merged Tegra124 USB header file as well

Changes since v3:
- Minor cleanup in USB header file

Changes since v2:
- Split into multiple patches
- Using ifdef to account for differences between Tegra20/Tegra30
- Merging of the header files

Changes since v1:
- Take differences between USBD and other USB ports into account port_sc1

---
Stefan Agner (3):
usb: tegra: fix USB2 powerdown for Tegra30 and later
usb: tegra: fix PHY configuration
usb: tegra: combine header file

 arch/arm/include/asm/arch-tegra/usb.h    | 223 ++++++++++++++++++++++++-
 arch/arm/include/asm/arch-tegra114/usb.h | 156 ------------------
 arch/arm/include/asm/arch-tegra124/usb.h | 268 -------------------------------
 arch/arm/include/asm/arch-tegra20/usb.h  | 155 ------------------
 arch/arm/include/asm/arch-tegra30/usb.h  | 168 -------------------
 board/nvidia/common/board.c              |   1 -
 drivers/usb/host/ehci-tegra.c            |  28 +++-
 7 files changed, 245 insertions(+), 754 deletions(-)
 delete mode 100644 arch/arm/include/asm/arch-tegra114/usb.h
 delete mode 100644 arch/arm/include/asm/arch-tegra124/usb.h
 delete mode 100644 arch/arm/include/asm/arch-tegra20/usb.h
 delete mode 100644 arch/arm/include/asm/arch-tegra30/usb.h
Signature · 1 line
1.9.0