All of lore.kernel.org
 help / color / mirror / Atom feed
From: vianney.leclement@essensium.com (Vianney le Clément)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: imx6: Add Embest MarS Board
Date: Mon, 27 Apr 2015 11:42:38 +0200	[thread overview]
Message-ID: <1430127758.1475.2@mail.essensium.com> (raw)
In-Reply-To: <20150427083917.GD9231@dragon>

Hi Shawn,

Thanks for taking time reviewing this.  I will send a follow-up patch.

On Mon, Apr 27, 2015 at 10:39 , Shawn Guo <shawn.guo@linaro.org> wrote:
> If eMMC doesn't work yet, you should just not add it.

Thanks to Lucas and Catalin it does work now :).  Will update it in v2.


> For new dts file, we suggest to use GPL/X11 dual-license for non-Linux
> users. Find example below.
> 
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/imx6q-hummingboard.dts

No problem for dual-licensing on my part, but this dts is heavily based 
on imx6q-riotboard.dts (as mentioned in the header), which is only 
licensed under the GPL.


>>  +		pinctrl_usbotg: usbotggrp {
>>  +			fsl,pins = <
>>  +				MX6QDL_PAD_ENET_RX_ER__USB_OTG_ID	0x17059
>>  +				MX6QDL_PAD_EIM_D22__GPIO3_IO22		0x000b0	/* 
>> MX6QDL_PAD_EIM_D22__USB_OTG_PWR */
> 
> What does the comment mean?

It is a leftover from imx6q-riotboard.dts.  I will change it.

Regards,
Vianney

Vianney le Cl?ment de Saint-Marcq
Embedded Software Developper ? +32 485 84 55 05
Essensium, Mind division ? www.mind.be
GPG: 727B CFE9 355F 1B02 0820 EE1C 9042 3E07 A804 0316

WARNING: multiple messages have this Message-ID (diff)
From: "Vianney le Clément" <vianney.leclement-buIOx9BAs4sybS5Ee8rs3A@public.gmane.org>
To: Shawn Guo <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH] ARM: dts: imx6: Add Embest MarS Board
Date: Mon, 27 Apr 2015 11:42:38 +0200	[thread overview]
Message-ID: <1430127758.1475.2@mail.essensium.com> (raw)
In-Reply-To: <20150427083917.GD9231@dragon>

Hi Shawn,

Thanks for taking time reviewing this.  I will send a follow-up patch.

On Mon, Apr 27, 2015 at 10:39 , Shawn Guo <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote:
> If eMMC doesn't work yet, you should just not add it.

Thanks to Lucas and Catalin it does work now :).  Will update it in v2.


> For new dts file, we suggest to use GPL/X11 dual-license for non-Linux
> users. Find example below.
> 
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/imx6q-hummingboard.dts

No problem for dual-licensing on my part, but this dts is heavily based 
on imx6q-riotboard.dts (as mentioned in the header), which is only 
licensed under the GPL.


>>  +		pinctrl_usbotg: usbotggrp {
>>  +			fsl,pins = <
>>  +				MX6QDL_PAD_ENET_RX_ER__USB_OTG_ID	0x17059
>>  +				MX6QDL_PAD_EIM_D22__GPIO3_IO22		0x000b0	/* 
>> MX6QDL_PAD_EIM_D22__USB_OTG_PWR */
> 
> What does the comment mean?

It is a leftover from imx6q-riotboard.dts.  I will change it.

Regards,
Vianney

Vianney le Clément de Saint-Marcq
Embedded Software Developper · +32 485 84 55 05
Essensium, Mind division · www.mind.be
GPG: 727B CFE9 355F 1B02 0820 EE1C 9042 3E07 A804 0316

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2015-04-27  9:42 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-01  8:10 [PATCH] ARM: dts: imx6: Add Embest MarS Board Vianney le Clément de Saint-Marcq
2015-04-01  8:10 ` Vianney le Clément de Saint-Marcq
2015-04-01  8:15 ` Lucas Stach
2015-04-01  8:15   ` Lucas Stach
2015-04-01  8:32   ` Vianney le Clément
2015-04-01  8:32     ` Vianney le Clément
2015-04-24 12:22     ` Catalin Crenguta
2015-04-24 12:22       ` Catalin Crenguta
2015-04-24 12:28       ` Lucas Stach
2015-04-24 12:28         ` Lucas Stach
2015-04-27  6:47         ` Catalin Crenguta
2015-04-27  6:47           ` Catalin Crenguta
2015-04-27  7:07           ` Vianney le Clément
2015-04-27  7:07             ` Vianney le Clément
2015-04-27  9:19             ` Catalin Crenguta
2015-04-27  9:19               ` Catalin Crenguta
2015-04-01 11:39 ` Fabio Estevam
2015-04-01 11:39   ` Fabio Estevam
2015-04-01 13:25   ` Vianney le Clément
2015-04-01 13:25     ` Vianney le Clément
2015-04-27  8:39 ` Shawn Guo
2015-04-27  8:39   ` Shawn Guo
2015-04-27  9:42   ` Vianney le Clément [this message]
2015-04-27  9:42     ` Vianney le Clément

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=1430127758.1475.2@mail.essensium.com \
    --to=vianney.leclement@essensium.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.