devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: Ian Campbell <ijc@hellion.org.uk>
Cc: Grant Likely <grant.likely@linaro.org>,
	devicetree@vger.kernel.org, u-boot@lists.denx.de
Subject: Re: [PATCH 5/6] sunxi: video: Add simplefb support
Date: Sun, 16 Nov 2014 18:52:51 +0100	[thread overview]
Message-ID: <5468E473.7050304@redhat.com> (raw)
In-Reply-To: <1416158375.25454.26.camel@hellion.org.uk>

Hi,

On 11/16/2014 06:19 PM, Ian Campbell wrote:
> On Sun, 2014-11-16 at 16:11 +0000, Ian Campbell wrote:
>>
>>> AFAIK Grant agrees with v5
>>
>> AFAIK Grant hasn't actually said that. If he does ack it (or if
>> someone
>> points me to the correct mail) then I have no further objections.
> 
> I finally found
> <CACxGe6ssdeOnQnem1012h_jFK1wWOD_WwosjFgiAcgBoqyDt9A@mail.gmail.com>
> which for some reason isn't in my devicetree@ folder nor in any of the
> archives online I looked at earlier (I could only find it by message-id
> now I know it), but it was in my sunxi folder.
> 
> So with Grant and Rob having accepted them you can consider my concerns
> with the bindings alleviated.

Thanks! I was about to reply to your earlier mail with lets wait for
Grant to get online, and see what he has to say, but this works too.

And thanks for explaining the address and size cells thing better, I
did know that for address and size #cells determines u32 vs u64, I'm
used to gpio-s / clock-s where extra cells are an extra parameter,
hence my confusion.

I'll do a v3 of the simplefb patch which takes the address and size
#cells into account.

Regards,

Hans

  reply	other threads:[~2014-11-16 17:52 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1415984088-6800-1-git-send-email-hdegoede@redhat.com>
     [not found] ` <1415984088-6800-6-git-send-email-hdegoede@redhat.com>
     [not found]   ` <1416138607.25454.13.camel@hellion.org.uk>
     [not found]     ` <5468AC2E.80601@redhat.com>
2014-11-16 14:38       ` [PATCH 5/6] sunxi: video: Add simplefb support Ian Campbell
2014-11-16 15:11         ` Hans de Goede
     [not found]           ` <5468BE9A.8050501-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-11-16 16:11             ` Ian Campbell
     [not found]               ` <1416154297.25454.24.camel-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>
2014-11-16 17:19                 ` [U-Boot] " Ian Campbell
2014-11-16 17:52                   ` Hans de Goede [this message]
2014-11-17  9:58                 ` Grant Likely
2014-11-17 10:10                   ` Hans de Goede
2014-11-17 10:14                   ` Ian Campbell
     [not found]                     ` <1416219254.27385.15.camel-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>
2014-11-17 15:01                       ` [U-Boot] " Grant Likely

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=5468E473.7050304@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=devicetree@vger.kernel.org \
    --cc=grant.likely@linaro.org \
    --cc=ijc@hellion.org.uk \
    --cc=u-boot@lists.denx.de \
    /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).