Message-ID: <CAKMK7uF-hBjqnRG28fX7NXOJDc5YEWhq+G6s0M8usA+TXM1BKw@mail.gmail.com>
Date:   Wed, 20 Jun 2018 11:07:53 +0200
From:   Daniel Vetter <daniel@...ll.ch>
To:     Noralf Trønnes <noralf@...nnes.org>
Cc:     David Lechner <david@...hnology.com>,
        dri-devel <dri-devel@...ts.freedesktop.org>,
        devicetree@...r.kernel.org, Mark Rutland <mark.rutland@....com>,
        Nitin Patil <nitin@...dsensors.com>, limor@...yada.net,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Rob Herring <robh+dt@...nel.org>
Subject: Re: [PATCH v2 0/4] drm/tinydrm: new dirver for ILI9341 displays

On Tue, Jun 19, 2018 at 5:36 PM, Noralf Trønnes <noralf@...nnes.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@...ts.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@...ts.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

Powered by blists - more mailing lists