public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Soeren Moch <smoch@web.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/10 v2] i.MX6: move duplicated options to mx6_common to standardise mx6 config
Date: Mon, 11 May 2015 20:43:26 +0200	[thread overview]
Message-ID: <5550F84E.4000603@web.de> (raw)
In-Reply-To: <1431361332-24997-1-git-send-email-pbrobinson@gmail.com>

[V2: sorry forgot to Cc: the list]

 > There's a lot of common options in the i.MX6 boards that are repeated across a
 > lot of the devices. There's a mx6_common.h which is little used but makes sense
 > to be the central location for all the options we want across all mx6 boards to
 > ensure a consistent set of features. This is a first pass at moving those options
 > and unifying the common options to a standard default.
 >
 > Changes since v1:
 > * Move CONFIG_SYS_NO_FLASH changes from patch 7 to patch 2 and reorder includes
 > so we don't need to undef CONFIG_CMD_FLASH / CONFIG_CMD_IMLS
 > * Add CONFIG_CMD_GPIO to mx6_common.f (patch 5)
 > * Use the default for all CONFIG_SYS_PROMPT_HUSH_PS2 (patch 7)
 > * Drop LZO change (patch 8)
 >
 > I think I got all the review points
 >
 > Peter Robinson (10):
 >       novena: standardise mx6_common.h include
 >       imx6: move all standard includes to mx6_common.h
 >       imx6: move generic imx6 options to mx6_common.h
 >       imx6: move standard ATAG configs to mx6_common.h
 >       imx6: move MXC_GPIO define to mx6_common.h
 >       imx6: centralise common boot options in mx6_common.h
 >       imx6: move generic miscellaneous and overwrite options
 >       imx6: standardise filesystem and boot options
 >       imx6: generic MMC config options to mx6_common
 >       mx6: standardise CONFIG_CMD_CACHE
 >
 > Reviewed-by: Eric Nelson <eric.nelson@boundarydevices.com>

Besides my comments on patch 8/10,
for the whole series (especially the tbs2910 part):

Acked-by: Soeren Moch <smoch@web.de>

Thanks for this great cleanup!
  Soeren

  parent reply	other threads:[~2015-05-11 18:43 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-11 16:22 [U-Boot] [PATCH 0/10 v2] i.MX6: move duplicated options to mx6_common to standardise mx6 config Peter Robinson
2015-05-11 16:22 ` [U-Boot] [PATCH 01/10] novena: standardise mx6_common.h include Peter Robinson
2015-05-11 16:22 ` [U-Boot] [PATCH 02/10] imx6: move all standard includes to mx6_common.h Peter Robinson
2015-05-11 16:22 ` [U-Boot] [PATCH 03/10] imx6: move generic imx6 options " Peter Robinson
2015-05-12  8:40   ` Markus Niebel
2015-05-12 10:51     ` Tom Rini
2015-05-12 11:12       ` Markus Niebel
2015-05-15 11:30       ` Stefano Babic
2015-05-11 16:22 ` [U-Boot] [PATCH 04/10] imx6: move standard ATAG configs " Peter Robinson
2015-05-11 16:22 ` [U-Boot] [PATCH 05/10] imx6: move MXC_GPIO define " Peter Robinson
2015-05-11 16:22 ` [U-Boot] [PATCH 06/10] imx6: centralise common boot options in mx6_common.h Peter Robinson
2015-05-11 16:22 ` [U-Boot] [PATCH 07/10] [PATCH 07/10] imx6: move generic miscellaneous and overwrite options Peter Robinson
2015-05-11 16:22 ` [U-Boot] [PATCH 08/10] imx6: standardise filesystem and boot options Peter Robinson
2015-05-11 17:21   ` Soeren Moch
2015-05-11 17:55     ` Soeren Moch
2015-05-11 16:22 ` [U-Boot] [PATCH 09/10] imx6: generic MMC config options to mx6_common Peter Robinson
2015-05-11 16:22 ` [U-Boot] [PATCH 10/10] mx6: standardise CONFIG_CMD_CACHE Peter Robinson
2015-05-11 18:43 ` Soeren Moch [this message]
2015-05-15  9:35   ` [U-Boot] [PATCH 0/10 v2] i.MX6: move duplicated options to mx6_common to standardise mx6 config Stefano Babic
2015-05-15 13:11     ` Soeren Moch
2015-05-19 12:53 ` Stefano Babic
2015-05-21 23:15   ` Peter Robinson
2015-05-22  7:44     ` 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=5550F84E.4000603@web.de \
    --to=smoch@web.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