Re: [PATCH v5 2/2 RESEND] drm: panel: Add novatek nt35596s panel driver

[Linus Walleij]

On Tue, Jul 25, 2023 at 4:16 PM Arnaud Ferraris
<arnaud.ferraris@xxxxxxxxxxxxx> wrote:

> Their magic number sequences are quite different, however all the other
> bits (power/reset sequences and timings) fit in, which isn't the case of
> the other novatek panel drivers. Moreover, they have similar (although
> not identical) resolutions.

Hm OK I guess.

> I found an old (2012) datasheet for the NT35596 (without the final "S")
> which is marked "Draft", so I'm really unsure this will be enough to
> make sense of those numbers.

I would try to add the most obvious cases. (Such as "command FF", I
am pretty sure it is clear what that does.)

As I pointed out in some other review, long term we want to support
things such as gamma correction for panels, and then this kind of
information will help greatly, even small pieces.

Yours,
Linus Walleij