Re: [PATCH] firmware: Allow release-specific firmware dir
From: Marcel Holtmann
Date: Thu Sep 11 2008 - 12:39:11 EST
Hi Greg,
> > > > This isn't compatible with the version of udev that people actually have
> > > > installed, today. And it's also the _wrong_ thing to do.
> > > >
> > > > Firmware really _isn't_ version-specific.
> > >
> > > Tell that to every Debian and Debian derived system on the planet.
> > >
> > > To my knowledge, it is only fedora and possibly one or two other dists
> > > that put the firmware files in a unary /lib/firmware location, rather
> > > than a versioned /lib/firmware/$KERNELRELASE one.
> >
> > We (Mandriva Linux) do too.
> > Why other distro cannot package firmware separatly for all their kernel
> > flavors?
> >
> > Firmware are not version-specific.
>
> Yes it is, when it is bundled with the kernel source tree itself.
>
> Why do people not realize this?
>
> This is a real problem for distros, and for anyone who wants to have
> multiple kernel versions on a single machine. Jeff's patch fixes this.
once you move the firmware into its own package built from a separate
source tree (and not the kernel) this is no longer a problem.
Nobody ever made the claim that the firmware depends on the kernel
version. It might depend on a specific version of the firmware, but that
is different from the kernel version. If you break the firmware or if a
driver requires a specific firmware version it should indicate this.
And yes, the in-kernel firmware versions should follow here at some
point, since the external firmware versions had to do it already for
quite some time, because otherwise more people would have complained
loudly when installing their own kernels.
What is wrong with moving the firmware into its own package with its own
version number?
Regards
Marcel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/