mbox series

[v2,0/4] drm/tinydrm: new dirver for ILI9341 displays

Message ID 20180525193623.15533-1-david@lechnology.com
Headers show
Series drm/tinydrm: new dirver for ILI9341 displays | expand

Message

David Lechner May 25, 2018, 7:36 p.m. UTC
This series adds a new tinydrm driver for the Ilitek ILI9341 controller and
a 2.4" display panel that uses this controller.

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

Comments

Robert P. J. Day May 25, 2018, 7:36 p.m. UTC | #1
"dirver"?
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Noralf Trønnes June 3, 2018, 10 p.m. UTC | #2
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?

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
>

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
David Lechner June 4, 2018, 1:21 a.m. UTC | #3
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

> 
> 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
>>
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Noralf Trønnes June 5, 2018, 3:49 p.m. UTC | #4
(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?

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

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Noralf Trønnes June 19, 2018, 3:36 p.m. UTC | #5
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.

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
>

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Daniel Vetter June 20, 2018, 9:07 a.m. UTC | #6
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
David Lechner June 26, 2018, 6:16 p.m. UTC | #7
On 06/20/2018 04:07 AM, Daniel Vetter wrote:

> 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

The account was supposedly setup, but I am not able to use it for some
reason. Are there any particular nicks I should ping on IRC?

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Daniel Vetter June 27, 2018, 6:58 a.m. UTC | #8
On Tue, Jun 26, 2018 at 8:16 PM, David Lechner <david@lechnology.com> wrote:
> On 06/20/2018 04:07 AM, Daniel Vetter wrote:
>
>> 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
>
>
> The account was supposedly setup, but I am not able to use it for some
> reason. Are there any particular nicks I should ping on IRC?

#freedesktop on freenode is the channel where fd.o admins hang out.
-Daniel
David Lechner June 27, 2018, 6:08 p.m. UTC | #9
On 06/27/2018 01:58 AM, Daniel Vetter wrote:
> On Tue, Jun 26, 2018 at 8:16 PM, David Lechner <david@lechnology.com> wrote:
>> On 06/20/2018 04:07 AM, Daniel Vetter wrote:
>>
>>> 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
>>
>>
>> The account was supposedly setup, but I am not able to use it for some
>> reason. Are there any particular nicks I should ping on IRC?
> 
> #freedesktop on freenode is the channel where fd.o admins hang out.

Got that sorted out and pushed this series.

Hope I didn't break anything. :-o
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html