All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Nelson <eric.nelson@boundarydevices.com>
To: Pushpalatha <pushpalatha.sg@mistralsolutions.com>
Cc: "meta-freescale@yoctoproject.org" <meta-freescale@yoctoproject.org>
Subject: Re: [meta-fsl-arm][PATCH] Added support for i.MX6 series Nitrogen6w board Signed-off-by: Pushpalatha <pushpalatha.sg@mistralsolutions.com>
Date: Mon, 17 Dec 2012 08:12:25 -0700	[thread overview]
Message-ID: <50CF3659.1000906@boundarydevices.com> (raw)

Hi Pushpalatha,

> ---
>  conf/machine/imx6qnitrogen6w.conf                  |    18 +
>  .../support_nitrogen6x_config.patch                | 27324 +++++++++++++++++++
>  recipes-bsp/u-boot/u-boot-fslc_2012.10.bb          |     2 +
>  .../linux-imx-3.0.35/imx6qnitrogen6w/defconfig     |  3020 ++
>  .../support_for_nitrogen6_configuration.patch      | 11469 ++++++++
>  recipes-kernel/linux/linux-imx_3.0.35.bb           |     3 +
>  6 files changed, 41836 insertions(+)
>  create mode 100644 conf/machine/imx6qnitrogen6w.conf
>  create mode 100644 recipes-bsp/u-boot/u-boot-fslc-v2012.10/imx6qnitrogen6w/support_nitrogen6x_config.patch
>  create mode 100644 recipes-kernel/linux/linux-imx-3.0.35/imx6qnitrogen6w/defconfig
>  create mode 100644 recipes-kernel/linux/linux-imx-3.0.35/imx6qnitrogen6w/support_for_nitrogen6_configuration.patch
>
> diff --git a/conf/machine/imx6qnitrogen6w.conf b/conf/machine/imx6qnitrogen6w.conf
> new file mode 100644
> index 0000000..474ab26
> --- /dev/null
> +++ b/conf/machine/imx6qnitrogen6w.conf
> @@ -0,0 +1,18 @@
> +#@TYPE: Machine
> +#@NAME: i.MX6Q Nitrogen6w
> +#@DESCRIPTION: Machine configuration for Freescale i.MX6Q Nitrogen6W
> +
> +include conf/machine/include/imx-base.inc
> +include conf/machine/include/tune-cortexa9.inc
> +
> +SOC_FAMILY = "mx6q:mx6"
> +
> +KERNEL_DEVICETREE = "${S}/arch/arm/boot/dts/imx6q-nitrogen6w.dts"
> +
> +UBOOT_MACHINE = "nitrogen6x_config"
> +
> +SERIAL_CONSOLE = "115200 ttymxc1"
> +
> +MACHINE_FEATURES += " pci wifi bluetooth"
> +
> +

Thanks for pushing this, but I have some comments and questions.

To begin with, the name 'Nitrogen6W' is deprecated. We used it
on our first i.MX6 Nitrogen boards and were planning to build
Nitrogen6 (no Wireless and no W) separately. Our intent was that
Nitrogen6 with no W would be 100% compatible with SABRE Lite.

Our plans have changed, and we've decided to use the same design
with and without a wireless module using the same design. We'll
also be adding a Nitrogen6S (Solo) and perhaps Nitrogen6DL (Dual-Lite)
option using the same design with slightly different BOMs.

I see some e-mails from Otavio and Daiane that ask further questions
about Nitrogen6X and SABRE Lite. I'll respond to those separately
and elaborate on the product differences.

Now on to the questions...

Otavio has already pushed support for Nitrogen6X into meta-freescale.
How does your patch differ from that? Is it

We should definitely re-fresh Otavio's patch. In particular,
there are a few recent patches on our 'boundary-L3.0.35_12.09.01_GA'
branch that dramatically improve performance under high (>400Mbps)
ethernet load.

	https://github.com/boundarydevices/linux-imx6/commits/boundary-L3.0.35_12.09.01_GA

I believe that this one also is the trick to getting camera input
working with our Nit6x_5MP parallel camera module:
	https://github.com/boundarydevices/linux-imx6/commit/1c382d81eef3c4ab60a392b260f3d54871e38f2b

Are there any other updates in this patch set? Since it's one
big un-differentiated patch, it's difficult to tell whether
there are other updates.

Please advise,


Eric


             reply	other threads:[~2012-12-17 15:12 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-17 15:12 Eric Nelson [this message]
  -- strict thread matches above, loose matches on Subject: below --
2012-12-17 12:09 [meta-fsl-arm][PATCH] Added support for i.MX6 series Nitrogen6w board Signed-off-by: Pushpalatha <pushpalatha.sg@mistralsolutions.com> Pushpalatha
2012-12-17 12:53 ` Daiane Angolini
2012-12-17 12:59   ` Otavio Salvador
2012-12-17 15:49     ` Eric Nelson
2012-12-17 15:42   ` Eric Nelson
2012-12-17 16:07     ` Otavio Salvador

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=50CF3659.1000906@boundarydevices.com \
    --to=eric.nelson@boundarydevices.com \
    --cc=meta-freescale@yoctoproject.org \
    --cc=pushpalatha.sg@mistralsolutions.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.