On Tue, Jun 26, 2018 at 9:29 PM, Hans de Goede [off-list ref] wrote:

quoted

quoted

+       for (i = 0; i < FB_MAX; i++) {
+               if (registered_fb[i])
+                       fbcon_fb_registered(registered_fb[i]);
+       }

quoted

Simple git grep shows that we perhaps do

#define for_each_registered_fbcon(i) ...

(As an example see for_each_pci_bridge() how it's done there)

This is probably a worthwhile cleanup for all the fbdev related
code / drivers. But outside of the scope of this patchset.

True.

Yisheng, in case you are interested in doing such clean up...

-- 
With Best Regards,
Andy Shevchenko