LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Daniel Vetter <daniel@ffwll.ch>
To: "Noralf Trønnes" <noralf@tronnes.org>
Cc: David Lechner <david@lechnology.com>,
	dri-devel <dri-devel@lists.freedesktop.org>,
	devicetree@vger.kernel.org, Mark Rutland <mark.rutland@arm.com>,
	Nitin Patil <nitin@mindsensors.com>,
	limor@ladyada.net,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>
Subject: Re: [PATCH v2 0/4] drm/tinydrm: new dirver for ILI9341 displays
Date: Wed, 20 Jun 2018 11:07:53 +0200	[thread overview]
Message-ID: <CAKMK7uF-hBjqnRG28fX7NXOJDc5YEWhq+G6s0M8usA+TXM1BKw@mail.gmail.com> (raw)
In-Reply-To: <5a2bc4e7-c6cd-b270-61d3-61bfa5d9e90d@tronnes.org>

On Tue, Jun 19, 2018 at 5:36 PM, Noralf Trønnes <noralf@tronnes.org> wrote:
>
> Den 05.06.2018 17.49, skrev Noralf Trønnes:
>>
>> (cc: drm-misc maintainers)
>>
>>
>> Den 04.06.2018 03.21, skrev David Lechner:
>>>
>>>
>>>
>>> On 6/3/18 5:00 PM, Noralf Trønnes wrote:
>>>>
>>>>
>>>> Den 25.05.2018 21.36, skrev David Lechner:
>>>>>
>>>>> This series adds a new tinydrm driver for the Ilitek ILI9341 controller
>>>>> and
>>>>> a 2.4" display panel that uses this controller.
>>>>
>>>>
>>>> David, do you have commit rights now?
>>>
>>>
>>> No. Opened a bug a while back to request access, but I guess the
>>> right person didn't see it.
>>>
>>> https://bugs.freedesktop.org/show_bug.cgi?id=105166
>>>
>>
>> Could someone please look at this?
>>
>
> David, let's not stall this, if you resend with fixed up commit message and
> acks/rbs, I'll apply it.

I acked and forwarded the account request, sorry for the delay. Next
time around, poking maintainers on irc helps if this stuff is stuck.
-Daniel

>
> Noralf.
>
>
>> Noralf.
>>
>>>>
>>>> Noralf.
>>>>
>>>>> A few things to note here:
>>>>> * The datasheet for this display[1] doesn't have a vendor mentioned on
>>>>> it
>>>>>    anywhere, so I have used "adafruit" as the vendor prefix. If someone
>>>>> has a
>>>>>    better suggestion, please speak up.
>>>>> * The MAINTAINERS patch for ili9225 is included so we don't end up with
>>>>> a merge
>>>>>    conflict later on.
>>>>>
>>>>> v2 changes:
>>>>> * change vendor prefix from "noname" to "adafruit"
>>>>> * new patch for "adafruit" vendor prefix
>>>>> * minor style changes
>>>>> * drop regulator from driver (instead of adding to DT bindings)
>>>>>
>>>>> [1]:
>>>>> https://cdn-learn.adafruit.com/assets/assets/000/046/879/original/SPEC-YX240QV29-T_Rev.A__1_.pdf
>>>>>
>>>>>
>>>>> David Lechner (4):
>>>>>    MAINTAINERS: fix path to ilitek,ili9225 device tree bindings
>>>>>    dt-bindings: Add vendor prefix for Adafruit
>>>>>    dt-bindings: new binding for Ilitek ILI9341 display panels
>>>>>    drm/tinydrm: new driver for ILI9341 display panels
>>>>>
>>>>>   .../bindings/display/ilitek,ili9341.txt       |  27 ++
>>>>>   .../devicetree/bindings/vendor-prefixes.txt   |   1 +
>>>>>   MAINTAINERS                                   |   8 +-
>>>>>   drivers/gpu/drm/tinydrm/Kconfig               |  10 +
>>>>>   drivers/gpu/drm/tinydrm/Makefile              |   1 +
>>>>>   drivers/gpu/drm/tinydrm/ili9341.c             | 233
>>>>> ++++++++++++++++++
>>>>>   6 files changed, 279 insertions(+), 1 deletion(-)
>>>>>   create mode 100644
>>>>> Documentation/devicetree/bindings/display/ilitek,ili9341.txt
>>>>>   create mode 100644 drivers/gpu/drm/tinydrm/ili9341.c
>>>>>
>>>>
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

  reply	other threads:[~2018-06-20  9:08 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-25 19:36 David Lechner
2018-05-25 19:36 ` [PATCH v2 1/4] MAINTAINERS: fix path to ilitek,ili9225 device tree bindings David Lechner
2018-05-25 19:36 ` [PATCH v2 2/4] dt-bindings: Add vendor prefix for Adafruit David Lechner
2018-05-31  3:38   ` Rob Herring
2018-05-25 19:36 ` [PATCH v2 3/4] dt-bindings: new binding for Ilitek ILI9341 display panels David Lechner
2018-05-25 20:14   ` David Lechner
2018-05-31  3:39     ` Rob Herring
2018-05-25 19:36 ` [PATCH v2 4/4] drm/tinydrm: new driver for " David Lechner
2018-05-27 14:04   ` kbuild test robot
2018-05-27 14:04   ` kbuild test robot
2018-05-27 14:50   ` kbuild test robot
2018-05-25 19:36 ` [PATCH v2 0/4] drm/tinydrm: new dirver for ILI9341 displays Robert P. J. Day
2018-06-03 22:00 ` Noralf Trønnes
2018-06-04  1:21   ` David Lechner
2018-06-05 15:49     ` Noralf Trønnes
2018-06-19 15:36       ` Noralf Trønnes
2018-06-20  9:07         ` Daniel Vetter [this message]
2018-06-26 18:16           ` David Lechner
2018-06-27  6:58             ` Daniel Vetter
2018-06-27 18:08               ` David Lechner

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=CAKMK7uF-hBjqnRG28fX7NXOJDc5YEWhq+G6s0M8usA+TXM1BKw@mail.gmail.com \
    --to=daniel@ffwll.ch \
    --cc=david@lechnology.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=limor@ladyada.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=nitin@mindsensors.com \
    --cc=noralf@tronnes.org \
    --cc=robh+dt@kernel.org \
    --subject='Re: [PATCH v2 0/4] drm/tinydrm: new dirver for ILI9341 displays' \
    /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

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