Thread (46 messages) 46 messages, 5 authors, 2022-02-11

Re: [PATCH v3 4/7] drm/solomon: Add SSD130X OLED displays I2C support

From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: 2022-02-09 15:16:50
Also in: dri-devel, lkml

On Wed, Feb 09, 2022 at 10:03:11AM +0100, Javier Martinez Canillas wrote:
The ssd130x driver only provides the core support for these devices but it
does not have any bus transport logic. Add a driver to interface over I2C.
Thanks!

...
+ * Authors: Javier Martinez Canillas [off-list ref]
s?!

...
+#include <linux/i2c.h>
+#include <linux/module.h>
regmap.h ?
err.h?

...
+	ssd130x = ssd130x_probe(&client->dev, regmap);
+
Redundant blank line.
+	if (IS_ERR(ssd130x))
+		return PTR_ERR(ssd130x);
...
+	{ /* sentinel */ },
No comma for terminator entry.


-- 
With Best Regards,
Andy Shevchenko

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help