public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 0/7] Add support for Colibri Vybrid Modules
Date: Thu, 23 Apr 2015 23:13:47 +0200	[thread overview]
Message-ID: <201504232313.47448.marex@denx.de> (raw)
In-Reply-To: <20150423131351.GZ16702@bill-the-cat>

On Thursday, April 23, 2015 at 03:13:51 PM, Tom Rini wrote:
> On Thu, Apr 23, 2015 at 06:08:43PM +0530, maitysanchayan at gmail.com wrote:
> > Hello,
> > 
> > On 15-04-15 16:24:21, Sanchayan Maity wrote:
> > > Hello,
> > > 
> > > This is the third version of the patchset which adds support for the
> > > Toradex Colibri Vybrid VF50 and VF61 modules. Boot up has been tested
> > > using the serial loader over UART. Compile tested for
> > > vf610twr_defconfig and vf610twr_nand_defconfig as well.
> > > 
> > > First patch in the series refactors the DDR related code for use by
> > > both the tower board and colibri modules. It also introduces a DDR3
> > > based JEDEC timing structure.
> > > 
> > > Second third and fourth patch in this series are improvement patches
> > > related to RTC, SoC/CPU detection and caches.
> > > 
> > > Fifth patch introduces USB support for Vybrid modules. Much of the code
> > > is similar to the ehci-mx6 driver. Both host and client modes are
> > > working and DFU has also been tested with client. Currently, we
> > > restrict the ports to be in one of host and client mode.
> > > 
> > > Sixth patch adds the actual support for the Colibri modules.
> > > 
> > > Comments and feedback are most welcome. Thanks for the feedback till
> > > now.
> > > 
> > > The patchset is based and tested on the latest master branch as of
> > > this writing.
> > > 
> > > Discussion on the version 2 of the patchset can be found at the below
> > > link:
> > > https://www.mail-archive.com/u-boot at lists.denx.de/msg168727.html
> > > 
> > > Discussion on the version 1 of the patchset can be found at the below
> > > link:
> > > https://www.mail-archive.com/u-boot at lists.denx.de/msg168136.html
> > > 
> > > Changes since v2:
> > > - Rework the USB driver to use register + offset method in light of
> > > discussion which Fabio Estevam pointed out instead of the regular
> > > struct{} method which v2 used. The discussion is at the below link:
> > > https://www.marc.info/?l=u-boot&m=142609602127309&w=2
> > > 
> > > - Reorder the patchset, putting the USB support in the end and add an
> > > additional patch for adding USB support to Colibri modules. By chance
> > > if more discussions happen on the USB support, this allows picking up
> > > of atleast the first patches on which no issues have been reported so
> > > far.
> > > 
> > > - The register definitions have been moved under arch/arm/include/asm/
> > > imx-common in the regs-usbphy.h file. This was agreed on after
> > > discussion with Marek and some input from Peter Chen. Since it is not
> > > clear if SoC's other than Freescale's use the Sigmatel Phy's which seem
> > > to be use in iMX/VF/MXS, put the USH PHY register definitions in
> > > imx-common rather than include/usb in a chipidea specific file.
> > > 
> > > - Remove setting of a PLL divisor select which was added for USB but is
> > > actually not required considering default value. It also seems to break
> > > USB after my latest rebase. The file in question concerning the change
> > > is colibri_vf.c. PLL divisor selects the PLL Multiplication factor
> > > which by default is 0, setting Fout = Fref * 20 giving 480MHz. The
> > > earlier patch set this to 1 giving Fout = Fref * 22 where Fref =
> > > 24MHz.
> > > 
> > > - Rebased on the latest master branch.
> > > 
> > > Changes since v1:
> > > - Rework the USB driver to use register offsets using the regular
> > > struct {} method
> > > 
> > > - Some cleanups and fixes in the sixth patch for the colibri_vf.h file
> > > which takes care of environment variables in uboot
> > > 
> > > - Purge some useless defines in the fifth and sixth patch which were
> > > related to USB.
> > 
> > Ping!?
> > 
> > Anything preventing this patch from getting applied?
> 
> I'll pick this up soon, thanks!

This should go through u-boot-imx though ;-)

  reply	other threads:[~2015-04-23 21:13 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-15 10:54 [U-Boot] [PATCH v3 0/7] Add support for Colibri Vybrid Modules Sanchayan Maity
2015-04-15 10:54 ` [U-Boot] [PATCH v3 1/7] ARM: vf610: Move DDR3 initialization to imx-common Sanchayan Maity
2015-04-23 22:03   ` [U-Boot] [U-Boot, v3, " Tom Rini
2015-04-15 10:54 ` [U-Boot] [PATCH v3 2/7] ARM: vf610: Enable external 32KHz oscillator Sanchayan Maity
2015-04-23 22:03   ` [U-Boot] [U-Boot, v3, " Tom Rini
2015-04-15 10:54 ` [U-Boot] [PATCH v3 3/7] ARM: vf610: Add SoC and CPU type detection Sanchayan Maity
2015-04-23 22:04   ` [U-Boot] [U-Boot, v3, " Tom Rini
2015-04-15 10:54 ` [U-Boot] [PATCH v3 4/7] ARM: vf610: Enable caches Sanchayan Maity
2015-04-23 22:04   ` [U-Boot] [U-Boot,v3,4/7] " Tom Rini
2015-04-15 10:54 ` [U-Boot] [PATCH v3 5/7] ARM: vf610: Initial integration for Colibri VF50/VF61 Sanchayan Maity
2015-04-16  6:36   ` Stefan Agner
2015-04-23 22:04   ` [U-Boot] [U-Boot, v3, " Tom Rini
2015-04-15 10:54 ` [U-Boot] [PATCH v3 6/7] usb: host: Add ehci-vf USB driver for ARM Vybrid SoC's Sanchayan Maity
2015-04-23 22:04   ` [U-Boot] [U-Boot, v3, " Tom Rini
2015-04-15 10:54 ` [U-Boot] [PATCH v3 7/7] colibri_vf: Enable USB support for Colibri Vybrid Sanchayan Maity
2015-04-16 20:21   ` Marek Vasut
2015-04-17  9:21     ` maitysanchayan at gmail.com
2015-04-17  9:30       ` Marek Vasut
2015-04-17 13:26         ` [U-Boot] [PATCH v4 " Sanchayan Maity
2015-04-17 20:16           ` Marek Vasut
2015-04-23 22:04           ` [U-Boot] [U-Boot, v4, " Tom Rini
2015-04-23 12:38 ` [U-Boot] [PATCH v3 0/7] Add support for Colibri Vybrid Modules maitysanchayan at gmail.com
2015-04-23 13:13   ` Tom Rini
2015-04-23 21:13     ` Marek Vasut [this message]
2015-04-27 17:54       ` Tom Rini
2015-04-27 18:18         ` Marek Vasut
2015-04-27 20:00           ` Otavio Salvador
2015-04-27 21:39             ` Marek Vasut
2015-04-27 20:02         ` Stefano Babic

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201504232313.47448.marex@denx.de \
    --to=marex@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox