Hi Andy
Thanks for your question.
On Thu, 5 Aug 2021 at 18:16, Andy Shevchenko andy.shevchenko@gmail.com wrote:
On Sat, Jul 24, 2021 at 6:46 AM dillon.minfei@gmail.com wrote:
From: Dillon Min dillon.minfei@gmail.com
Since the st,sf-tc240t-9370-t dts binding already exist in stm32f429-disco.dts but, the panel driver didn't get accepted from mainline. it's time to submit patch fot it.
This driver can support two different interface by different dts bindings:
- spi+dpi, use spi to configure register, dpi for graphic data. st,sf-tc240t-9370-t
- only spi, just like tiny/ili9341.c (actually, this part is copy from tiny) adafruit,yx240qv29
...
I was submited the first patch last year, you can find it at [1].
submitted
Thanks.
this patch has one major difference from that one, which is replace the low level communication way, from spi_sync() to mipi_dbi_{command, command_stackbuf}() interface, referred from Linus's patch [2].
Can you shed a light on the road map here.
Personally, I'd like to merge tiny/mi0283qt.c, tiny/ili9341.c(already done) into this driver later (keep original author, copyright, dts compatible string). then remove these two drivers under tiny, but it's up to Sam and Laurent agreement.
For long term, just like Peter suggested, let all panel based on ili9xxx with single-dbi or dbi & dpi interface to be supported by single ilitek-ili9xxx.c, something like panel/panel-simple.c (panel/panel-ilitek-ili9322c, tiny/ili9225.c, tiny/ili9486.c, tiny/mi0283qt.c, etc). it's also needs maintainers permission.
I have the SPI panel (tiny) based on the ILI9341 and I'm using actually mi0283qt driver. With yours we will have 3 (three!) drivers for the same chip. I really do not want this. Without road map on the prospective of these all drivers, NAK.
Yes, it will make users confused if there are three different drivers for the same chip. I'll continue to work on this driver.
Thanks again for point this out.
Best Regards Dillon
-- With Best Regards, Andy Shevchenko