Re: [PATCH 1/3] xc5000: tuner firmware update

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 27, 2014 at 01:57:27PM -0200, Mauro Carvalho Chehab wrote:
> Em Mon, 27 Oct 2014 10:25:48 -0400
> Michael Ira Krufky <mkrufky@xxxxxxxxxxx> escreveu:
> 
> > I like the idea of supporting older firmware revisions if the new one
> > is not present, but, the established president for this sort of thing
> > has always been to replace older firmware with newer firmware without
> > backward compatibility support for older binaries.
> 
> No, we're actually adding backward support. There are some drivers
> already with it. See for example xc4000 (changeset da7bfa2c5df).
> 
> > Although the current driver can work with both old and new firmware
> > versions, this hasn't been the case in the past, and won't always be
> > the case with future firmware revisions.
> 
> Yeah, we did a very crap job breaking backward firmware compat in
> the past. We're not doing it anymore ;)
> 
> > Hauppauge has provided links to the new firmware for both the XC5000
> > and XC5000C chips along with licensing.  Maybe instead, we can just
> > upstream those into the linux-firmware packages for distribution.
> 
> Upstreaming to linux-firmware was done already for the previous firmwares.
> The firmwares at linux-firmware for xc5000 and xc5000c were merged back 
> there for 3.17 a few weeks ago.
> 
> Feel free to submit them a new version.
> 
> > I don't think supporting two different firmware versions is a good
> > idea for the case of the xc5000 driver.
> 
> Why not? It should work as-is with either version. We can always add
> some backward compat code if needed.

FWIW, Linus recently addressed the topic wrt wireless firmware:
http://article.gmane.org/gmane.linux.kernel.wireless.general/126794


HTH,
Johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux