Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-09-05 Thread Paulo
Hi, > The latest 5.15.y is 5.15.120 , can you re-test on that version ? > unfortunately it seems that I won't be able to do testing with 5.15.120. I won't have access to hardware any more. > > [ 20.255330] Hardware name: XXX i.MX8XX board:XXX (DT) > > Is this some NXP downstream kernel fork

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-07-12 Thread Marek Vasut
On 7/12/23 17:10, Paulo Pavacic wrote: Hi, [...] Or whether it makes sense to outright have a separate driver. The later would introduce duplication, but maybe that much duplication is OK. I would like to create new driver because panel-st7701 seems to be outdated and is using non-standard

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-07-12 Thread Paulo Pavacic
Hi Marek, sri, 12. srp 2023. u 15:42 Marek Vasut napisao je: > > On 7/12/23 14:07, Paulo Pavacic wrote: > > Hello all, > > > > sub, 8. srp 2023. u 14:53 Marek Vasut napisao je: > >> > >> On 7/7/23 17:26, Paulo Pavacic wrote: > >>> Hello Marek, > >> > >> Hi, > >> > >>> čet, 6. srp 2023. u 17:26

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-07-12 Thread Marek Vasut
On 7/12/23 14:07, Paulo Pavacic wrote: Hello all, sub, 8. srp 2023. u 14:53 Marek Vasut napisao je: On 7/7/23 17:26, Paulo Pavacic wrote: Hello Marek, Hi, čet, 6. srp 2023. u 17:26 Marek Vasut napisao je: On 7/6/23 17:18, Paulo Pavacic wrote: Hello Linus, čet, 22. lip 2023. u 10:22

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-07-12 Thread Paulo Pavacic
Hello all, sub, 8. srp 2023. u 14:53 Marek Vasut napisao je: > > On 7/7/23 17:26, Paulo Pavacic wrote: > > Hello Marek, > > Hi, > > > čet, 6. srp 2023. u 17:26 Marek Vasut napisao je: > >> > >> On 7/6/23 17:18, Paulo Pavacic wrote: > >>> Hello Linus, > >>> > >>> čet, 22. lip 2023. u 10:22 Linus

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-07-08 Thread Marek Vasut
On 7/7/23 17:26, Paulo Pavacic wrote: Hello Marek, Hi, čet, 6. srp 2023. u 17:26 Marek Vasut napisao je: On 7/6/23 17:18, Paulo Pavacic wrote: Hello Linus, čet, 22. lip 2023. u 10:22 Linus Walleij napisao je: On Wed, Jun 21, 2023 at 5:09 PM Paulo Pavacic wrote: A lot of

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-07-08 Thread Jagan Teki
On Thu, 22 Jun 2023 at 13:52, Linus Walleij wrote: > > On Wed, Jun 21, 2023 at 5:09 PM Paulo Pavacic wrote: > > > A lot of modifications to st7701 are required. I believe it would > > result in a driver that doesn't look or work the same. e.g compare > > delays between initialization sequences

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-07-08 Thread Paulo Pavačić
Hello Marek, Jul 6, 2023 5:26:15 PM Marek Vasut : > On 7/6/23 17:18, Paulo Pavacic wrote: >> Hello Linus, >> čet, 22. lip 2023. u 10:22 Linus Walleij napisao >> je: >>> >>> On Wed, Jun 21, 2023 at 5:09 PM Paulo Pavacic wrote: >>> A lot of modifications to st7701 are required. I believe

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-07-06 Thread Marek Vasut
On 7/6/23 17:18, Paulo Pavacic wrote: Hello Linus, čet, 22. lip 2023. u 10:22 Linus Walleij napisao je: On Wed, Jun 21, 2023 at 5:09 PM Paulo Pavacic wrote: A lot of modifications to st7701 are required. I believe it would result in a driver that doesn't look or work the same. e.g compare

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-07-06 Thread Paulo Pavacic
Hello Linus, čet, 22. lip 2023. u 10:22 Linus Walleij napisao je: > > On Wed, Jun 21, 2023 at 5:09 PM Paulo Pavacic wrote: > > > A lot of modifications to st7701 are required. I believe it would > > result in a driver that doesn't look or work the same. e.g compare > > delays between

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-22 Thread Linus Walleij
On Wed, Jun 21, 2023 at 5:09 PM Paulo Pavacic wrote: > A lot of modifications to st7701 are required. I believe it would > result in a driver that doesn't look or work the same. e.g compare > delays between initialization sequences of panel-fannal-c3004 and > panel-st7701. I think it would be

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-21 Thread Paulo Pavacic
pet, 16. lip 2023. u 14:53 Linus Walleij napisao je: > > On Fri, Jun 16, 2023 at 2:31 PM Paulo Pavacic wrote: > > pet, 16. lip 2023. u 13:44 Linus Walleij napisao > > je: > > > > > > On Fri, Jun 16, 2023 at 11:57 AM Paulo Pavacic > > > wrote: > > > > čet, 15. lip 2023. u 21:55 Linus Walleij

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-16 Thread Linus Walleij
On Fri, Jun 16, 2023 at 2:31 PM Paulo Pavacic wrote: > pet, 16. lip 2023. u 13:44 Linus Walleij napisao > je: > > > > On Fri, Jun 16, 2023 at 11:57 AM Paulo Pavacic wrote: > > > čet, 15. lip 2023. u 21:55 Linus Walleij > > > napisao je: > > > > > > > > I doubt that the display controller is

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-16 Thread Paulo Pavacic
pet, 16. lip 2023. u 13:44 Linus Walleij napisao je: > > On Fri, Jun 16, 2023 at 11:57 AM Paulo Pavacic wrote: > > čet, 15. lip 2023. u 21:55 Linus Walleij napisao > > je: > > > > > > I doubt that the display controller is actually by Fannal, but I guess > > > you tried to find out? We usually

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-16 Thread Linus Walleij
On Fri, Jun 16, 2023 at 11:57 AM Paulo Pavacic wrote: > čet, 15. lip 2023. u 21:55 Linus Walleij napisao > je: > > > > I doubt that the display controller is actually by Fannal, but I guess > > you tried to find out? We usually try to identify the underlying display > > controller so the driver

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-16 Thread Paulo Pavacic
Hello Linus, thank you for the comments. čet, 15. lip 2023. u 21:55 Linus Walleij napisao je: > > Hi Paulo, > > thanks for your patch! > > Overall this looks very good. > > I doubt that the display controller is actually by Fannal, but I guess > you tried to find out? We usually try to identify

Re: [PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-15 Thread Linus Walleij
Hi Paulo, thanks for your patch! Overall this looks very good. I doubt that the display controller is actually by Fannal, but I guess you tried to find out? We usually try to identify the underlying display controller so the driver can be named after it and reused for more display panels. Some

[PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-07 Thread Paulo Pavacic
Fannal C3004 is a 480x800 display made by fannal that requires DCS initialization sequences. Signed-off-by: Paulo Pavacic --- v4 changelog: - removal of blank lines, code wrapping changes - reset pin initialization handling changes v3 changelog: - formatting and style changes - remove