linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: hdegoede@redhat.com (Hans de Goede)
To: linux-arm-kernel@lists.infradead.org
Subject: [linux-sunxi] Re: [PATCH] ARM: dts: sun6i: Add a dts file for the Mele A1000G quad top set box
Date: Wed, 03 Jun 2015 09:22:35 +0200	[thread overview]
Message-ID: <556EAB3B.1050708@redhat.com> (raw)
In-Reply-To: <20150602214946.GF23777@lukather>

Hi,

On 02-06-15 23:49, Maxime Ripard wrote:
> On Tue, Jun 02, 2015 at 08:32:59PM +0200, Hans de Goede wrote:
>> Hi,
>>
>> On 06/02/2015 05:58 PM, Maxime Ripard wrote:
>>> On Mon, Jun 01, 2015 at 05:02:59PM +0200, Hans de Goede wrote:
>>>> The Mele A1000G-quad and the Mele M9 have the same PCB, sofar we've been using
>>>> the same dts for both models. Unfortunately this does not work for the otg
>>>> controller, on the M9 this is routed to a micro-usb connector on the outside,
>>>> while as on the A1000G-quad it is connected to an usb to sata bridge
>>>> (which is not populated on the M9 pcb).
>>>>
>>>> This commit adds a new dts for the Mele-A1000G-quad to allow using
>>>> different otg controller settings on the 2 boards.
>>>>
>>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>>>> ---
>>>>   arch/arm/boot/dts/Makefile                                        | 1 +
>>>>   arch/arm/boot/dts/{sun6i-a31-m9.dts => sun6i-a31-a1000g-quad.dts} | 4 ++--
>>>
>>> Didn't we say that we were trying to set the vendor name in the DTs
>>> now, at least for the new ones?
>>
>> I think we said that Paul would write a proposal to standardize
>> naming for future boards, but that never materialized.
>>
>>> I can fix this and apply it if it's ok for you.
>>
>> I'm fine with throwing -mele- in there, or not, what ever you
>> prefer.
>
> Ok, applied and fixed the name.
>
> This is probably going to be the last patch pulled for 4.2, unless
> something important shows up in the next few days.

That is fine with me, the only thing which I've pending on the dts
side is a hole lot of musb related patches which belong in 4.3 anyways.

What about the "ARM: dts: sun8i: Add Ippo-q8h v1.2 with A33 and 1024x600 lcd support"
commit? You had a remark on doing that differently to which I replied,
so that one is still in limbo a bit...

Regards,

Hans

      reply	other threads:[~2015-06-03  7:22 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-01 15:02 [PATCH] ARM: dts: sun6i: Add a dts file for the Mele A1000G quad top set box Hans de Goede
2015-06-01 18:07 ` Ian Campbell
2015-06-01 18:12   ` Hans de Goede
2015-06-02 15:58 ` Maxime Ripard
2015-06-02 18:32   ` Hans de Goede
2015-06-02 21:49     ` Maxime Ripard
2015-06-03  7:22       ` Hans de Goede [this message]

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=556EAB3B.1050708@redhat.com \
    --to=hdegoede@redhat.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 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).