From mboxrd@z Thu Jan 1 00:00:00 1970 From: Markus Niebel Date: Tue, 17 Jun 2014 19:40:53 +0200 Subject: [U-Boot] [Patch v2 0/7] Add support for TQMa6 modules In-Reply-To: <53A064FC.8030905@denx.de> References: <1402937486-13397-1-git-send-email-list-09_u-boot@tqsc.de> <53A064FC.8030905@denx.de> Message-ID: <53A07DA5.10502@tqsc.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am 17.06.2014 17:55, wrote Stefano Babic: > Hi Markus, > > On 16/06/2014 18:51, Markus Niebel wrote: >> From: Markus Niebel >> >> This series add support for the TQMa6 boards from TQ Systems. >> The first Patch adds basic support for the modiules and the >> starterkit baseboard while the other patches add additional features >> >> Markus Niebel (7): >> Add TQ Systems TQMa6 board support >> TQMa6: mba6: add RGMII enet >> TQMa6: add SPI support >> TQMa6: add SPI NOR boot support >> TQMa6: MBa6: add usb support >> TQMa6: add I2C support >> TQMa6: add pmic support >> >> board/tqc/tqma6/Makefile | 9 + >> board/tqc/tqma6/README | 35 ++++ >> board/tqc/tqma6/clocks.cfg | 24 +++ >> board/tqc/tqma6/tqma6.c | 258 +++++++++++++++++++++++ >> board/tqc/tqma6/tqma6_bb.h | 30 +++ >> board/tqc/tqma6/tqma6_mba6.c | 358 +++++++++++++++++++++++++++++++ >> board/tqc/tqma6/tqma6q.cfg | 125 +++++++++++ >> board/tqc/tqma6/tqma6s.cfg | 125 +++++++++++ >> boards.cfg | 4 + >> include/configs/tqma6.h | 477 ++++++++++++++++++++++++++++++++++++++++++ > > I understand that your patchset follow your internal development. > However, it is very uncommon that patches introducing new files are > followed by patches changing the same files again. > > I think you should squash your patches. This makes easy to review them. > I thought to have at first a simple and restricted config / board support would be easier to read, but no problem - will squash it. > Do you take a look at the last patches to have a single image running on > different variations of i.MX6 ? Dual and Quad are supported. Some work > for the Solo is required. > > Is it not an attractive alternative for you ? Instead of having several > entries in boards.cfg for each variation of your board, you could have > maybe only one or a couple. Sure it is - main difference will be boot devices. The env on different boot media in one image is AFAIK not possible - so at the moment four configs are needed: SP vs MMC and i.MX6Q/D vs i.MX6S (But this is slightly OT) > > Best regards, > Stefano Babic > Best regards, Markus Niebel