michi-jung commented on code in PR #6465:
URL: https://github.com/apache/incubator-nuttx/pull/6465#discussion_r902574720
##########
drivers/lcd/ug-9664hswag01.c:
##########
@@ -440,15 +443,12 @@ static void ug_deselect(FAR struct spi_dev_s *spi)
*
****************************************************************************/
-static int ug_putrun(fb_coord_t row, fb_coord_t col,
+static int ug_putrun(FAR struct lcd_dev_s *dev,
+ fb_coord_t row, fb_coord_t col,
FAR const uint8_t *buffer,
size_t npixels)
{
- /* Because of this line of code, we will only be able to support a single
- * UG device
- */
-
- FAR struct ug_dev_s *priv = &g_ugdev;
Review Comment:
I am afraid I do not fully understand. There is only the declaration of the
global variable `g_ugdev` and a single reference within `ug_initialize()` left.
Do you mean to allocate the ug_dev_s dynamically on the heap?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
