[PATCH] drm/tiny: add driver for newhaven,1.8-128160EF

Thomas Zimmermann tzimmermann at suse.de
Fri Feb 12 09:56:52 UTC 2021


Hi

Am 12.02.21 um 10:47 schrieb Daniel Mack:
> Hi,
> 
> Thanks for the review!
> 
> On 2/12/21 8:13 AM, Thomas Zimmermann wrote:
>> thanks for the driver. I only have some minor comments
>>
>> Am 11.02.21 um 10:50 schrieb Daniel Mack:
>>> +#define ILI9163_MADCTL_BGR    BIT(3)
>>> +#define ILI9163_MADCTL_MV    BIT(5)
>>> +#define ILI9163_MADCTL_MX    BIT(6)
>>> +#define ILI9163_MADCTL_MY    BIT(7)
>>
>> The indention is inconsistent. Or maybe it's just my email client.
> 
> Yes, the latter :)
> 
> Addressed the rest of your remarks. Will send a v2.

You should also add yourself to MAINTAINERS for this driver.

Best regards
Thomas

> 
> 
> Thanks,
> Daniel
> 

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Felix Imendörffer

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20210212/c4159716/attachment.sig>


More information about the dri-devel mailing list