From: marex@denx.de (Marek Vasut)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] ARM: dts: imx6q: add Novena board
Date: Thu, 19 Nov 2015 17:11:54 +0100 [thread overview]
Message-ID: <201511191711.54699.marex@denx.de> (raw)
In-Reply-To: <CAOMZO5C1oYge+TUKPYsgBRV75HVDwj42oAo27aaK-XqtHw8-7g@mail.gmail.com>
On Wednesday, November 18, 2015 at 01:00:50 PM, Fabio Estevam wrote:
> Hi Marek,
Hi!
> On Wed, Nov 18, 2015 at 7:48 AM, Marek Vasut <marex@denx.de> wrote:
> > +++ b/arch/arm/boot/dts/imx6q-novena.dts
> > @@ -0,0 +1,825 @@
> > +/*
> > + * Copyright 2014 Sutajio Ko-Usagi PTE LTD
> > + * Copyright 2012 Freescale Semiconductor, Inc.
> > + * Copyright 2011 Linaro Ltd.
> > + *
> > + * The code contained herein is licensed under the GNU General Public
> > + * License. You may obtain a copy of the GNU General Public License
> > + * Version 2 or later at the following locations:
> > + *
> > + * http://www.opensource.org/licenses/gpl-license.html
> > + * http://www.gnu.org/copyleft/gpl.html
>
> Could you make this GPL/X11 dual-license?
I'll talk to Sean about this, but I hope this is not a problem.
> Take a look at arch/arm/boot/dts/imx6qdl-cubox-i.dtsi for a reference.
>
> > +&iomuxc {
> > + audmux {
> > + pinctrl_audmux_novena: audmuxgrp-novena {
> > + fsl,pins = <
> > + MX6QDL_PAD_CSI0_DAT7__AUD3_RXD
> > 0x80000000 +
> > MX6QDL_PAD_CSI0_DAT4__AUD3_TXC 0x80000000 +
> > MX6QDL_PAD_CSI0_DAT5__AUD3_TXD 0x80000000 +
> > MX6QDL_PAD_CSI0_DAT6__AUD3_TXFS 0x80000000
>
> Please avoid these 0x80000000 throughout this file.
>
> We should better not rely on the IOMUX settings done in the bootloader.
Fixed, thanks!
Best regards,
Marek Vasut
next prev parent reply other threads:[~2015-11-19 16:11 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-18 9:48 [PATCH 1/2] devicetree: bindings: Add vendor prefix for Kosagi Marek Vasut
2015-11-18 9:48 ` [PATCH 2/2] ARM: dts: imx6q: add Novena board Marek Vasut
2015-11-18 10:10 ` Lucas Stach
2015-11-18 10:35 ` Marek Vasut
2015-11-18 11:25 ` Lucas Stach
2015-11-18 11:32 ` Marek Vasut
2015-11-18 11:36 ` Lucas Stach
2015-11-19 16:03 ` Marek Vasut
2015-11-18 12:14 ` Fabio Estevam
2015-11-18 12:00 ` Fabio Estevam
2015-11-19 16:11 ` Marek Vasut [this message]
2015-11-20 2:49 ` Sean Cross
2015-11-20 9:07 ` Marek Vasut
2015-11-20 20:46 ` [PATCH V2 " Marek Vasut
2015-11-20 20:58 ` Fabio Estevam
2015-11-20 21:05 ` Marek Vasut
2015-11-23 9:15 ` Lucas Stach
2015-11-24 11:04 ` Marek Vasut
2015-11-20 21:19 ` [PATCH 1/2] devicetree: bindings: Add vendor prefix for Kosagi Arnd Bergmann
2015-11-20 22:30 ` Marek Vasut
2015-11-20 22:46 ` Arnd Bergmann
2015-11-21 0:08 ` Marek Vasut
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=201511191711.54699.marex@denx.de \
--to=marex@denx.de \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).