Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c8c39796 authored by Frank Schaefer's avatar Frank Schaefer Committed by Mauro Carvalho Chehab
Browse files

[media] em28xx: make em28xx_set_outfmt() working with EM25xx family bridges 



Streaming doesn't work with the EM2765 if bit 5 of the output format register
0x27 is set.
It's actually not clear if really has to be set for the other chips, but for
now let's keep it to avoid regressions and add a comment to the code.

Signed-off-by: default avatarFrank Schäfer <fschaefer.oss@googlemail.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent e507e0e5

drivers/media/usb/em28xx/em28xx-core.c

+16 −5
Original line number Diff line number Diff line
@@ -697,10 +697,21 @@ int em28xx_vbi_supported(struct em28xx *dev) 
int em28xx_set_outfmt(struct em28xx *dev)

{

	int ret;

	u8 vinctrl;

	u8 fmt, vinctrl;



	ret = em28xx_write_reg_bits(dev, EM28XX_R27_OUTFMT,

				dev->format->reg | 0x20, 0xff);

	fmt = dev->format->reg;

	if (!dev->is_em25xx)

		fmt |= 0x20;

	/*

	 * NOTE: it's not clear if this is really needed !

	 * The datasheets say bit 5 is a reserved bit and devices seem to work

	 * fine without it. But the Windows driver sets it for em2710/50+em28xx

	 * devices and we've always been setting it, too.

	 *

	 * em2765 (em25xx, em276x/7x/8x) devices do NOT work with this bit set,

	 * it's likely used for an additional (compressed ?) format there.

	 */

	ret = em28xx_write_reg(dev, EM28XX_R27_OUTFMT, fmt);

	if (ret < 0)

		return ret;