3 comments:
Patch Set #6, Line 107: struct fb_info;
Shouldn't you #include <framebuffer_info. […]
I don't see why exposing the internals helps here.
The callers doesn't need to know anything but the function prototypes.
This way all users are forced to use the API as it's intended to.
File src/include/framebuffer_info.h:
Patch Set #6, Line 37: bool fb_fill_framebuffer_info(struct fb_info *info,
Does this function really need to be exported separately? I only see it used once with the xgifb stu […]
It's only used in xgifb_modeset. All methods that could be called multiple times use the *_fill_* methods.
It doesn't make much sense to call those methods multiple times, so I'll rework those first.
Patch Set #6, Line 96: info->valid = true;
I'm a bit confused who needs this. […]
yes, if it doesn't pass the pre check:
" if (!x_resolution || !y_resolution || !bytes_per_line || !bits_per_pixel)"
Seems to be a bad design, I'll rework that.
To view, visit change 39002. To unsubscribe, or for help writing mail filters, visit settings.