linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
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

  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).