Hyperion.NG, Amlogic improvements

MunkeyBalls · 3 December 2019 18:00

Managed to compile your repository and test it on my box but the problems I needed to fix remained

Live video feed doesn’t work (not a problem as long as the LED view works)
Reds and blues were still swapped on the grabber

But with your help I forked your repository and made the change as mentioned in topic I linked, changed the code and pointed the coreelec package.mk for the addon to my new repository, compiled it and now it the grabber has the correct colors

I noticed that they changed the code a bit since last time someone looked at it and it might be that something isn’t enabled on my system.

This is the line of code I had to change from BGR32…

github.com

MunkeyBalls/hyperion.ng/blob/5eeaa05e4a8fa2f3a31cf9d194615278c161d884/libsrc/grabber/framebuffer/FramebufferFrameGrabber.cpp#L52


	bytesPerPixel = vinfo.bits_per_pixel / 8;
	capSize = vinfo.xres * vinfo.yres * bytesPerPixel;

	switch (vinfo.bits_per_pixel)
	{
		case 16: pixelFormat = PIXELFORMAT_BGR16; break;
		case 24: pixelFormat = PIXELFORMAT_BGR24; break;
#ifdef ENABLE_AMLOGIC
		case 32: pixelFormat = PIXELFORMAT_RGB32; break;
#else
		case 32: pixelFormat = PIXELFORMAT_RGB32; break;
#endif
		default:
			Error(_log, "Unknown pixel format: %d bits per pixel", vinfo.bits_per_pixel);
			close(_fbfd);
			return -1;
	}

	/* map the device to memory */
	_fbp = (unsigned char*)mmap(0, capSize, PROT_READ, MAP_PRIVATE | MAP_NORESERVE, _fbfd, 0);

But if ENABLE_AMLOGIC would’ve been true it would’ve done that automagically, so seems like a bug or something in my config?

Anyway, here’s a link to the compiled addon in case anyone needs it: https://github.com/MunkeyBalls/hyperion.ng/releases/download/0.1/service.hyperion.ng-9.2.102_RGBfix.zip