From: Marcel Holtmann on
Hi Luis,

> >> >> > > > That doesn't make much sense anyway. If the firmware filename is
> >> >> > > > foo-$APIVER-$CODEVER every code change would need a corresponding
> >> >> > > driver
> >> >> > > > change. If it is just foo-$APIVER then the $CODEVER can be embedded
> >> >> > > in
> >> >> > > > the firmware file and printed so you know which code you're using,
> >> >> > > but
> >> >> > > > if it doesn't influence the API I don't see why it should be part of
> >> >> > > the
> >> >> > > > filename?
> >> >> > >
> >> >> > > The idea is that just like with shared libraries, you have a symlink
> >> >> > > from the 'soname' foo-3.fw to the actual file foo-3-1.4.1.fw.
> >> >> >
> >> >> > Ah ok. I indeed do that manually with iwlwifi firmware :)
> >> >> >
> >> >> > > For shared libraries, it's easy to create those symlinks automatically
> >> >> > > using ldconfig. For firmware that doesn't really work though -- since
> >> >> > > the soname isn't encoded in the file like it is in ELF libraries.
> >> >> >
> >> >> > Right. Though I guess we could come up with a unified firmware wrapper
> >> >> > format that the firmware loader can unwrap.
> >> >>
> >> >> I suppose we could, but this seems like overkill to me.
> >> >
> >> > I have to agree. This looks like total overkill to me.
> >> >
> >> > Just use the $APIVER in the firmware filename.
> >>
> >> OK -- so what goes into linux-firmware is just the latest
> >>
> >> foo-$(API)
> >>
> >> > And if someone wants to
> >> > keep track of more details then they can manually symlink them.
> >>
> >> Well do we want the older foo-$(API)-$(VAR) files in linux-firmware
> >> too for those companies/developers wishing to do this?
> >
> > personally I would say no, but for some projects it might make sense. I
> > would just leave this up to the project/driver itself.
>
> OK
>
> >> What about deprecating APIs of the firmware based on kernel releases.
> >> I see it reasonable to deprecate a firmware API completely for a
> >> future kernel release provided you maintain all features and
> >> functionality in par. Does that sound reasonable?
> >
> > Realistically you have to deprecate firmware versions at some point
> > anyway. So yes, that makes sense.
>
> OK
>
> > However I would leave this again up to
> > the driver itself and how far the maintainers wanna go in supported
> > older API versions.
>
> I was looking for general rules and guidelines to document. That
> includes things to do and things to not do. It sounds we are flexible
> with what people can do with their firmware and their driver firmware
> API. The things to not do are not so clear though still. I am
> wondering specifically about deprecation of older firmware APIs, or
> are we flexible enough to allow any firmware API rewrite so long as
> its properly supported?

this is more a driver problem. If the firmware API changes, then the
driver must support some of the older versions. Obviously supporting all
API versions becomes a major pain at some point. And then you have to
deprecate a firmware version.

How many firmware API you support should be really up to the driver
maintainer, but at minimum it should be the previous one. That is at
least my take on this.

> >> > Unless we have full control over the source code of every firmware used
> >> > in the kernel, why bother. It is up to the companies providing them
> >> > anyway to make sure everything works as expected and the community can't
> >> > fix it.
> >>
> >> Well that's exactly it -- we do have access to the code for ar9170 for
> >> example, so these details will become more relevant in the future.
> >
> > That is one of the few exception. However even for ar9170 there has to
> > be release engineering team that creates the binary and uploads it into
> > linux-firmware.git
>
> Release engineering team? Hah.

Let me translate this a bit. Whoever does create the source tarball
release should also build a binary release. So both packages have proper
version information, but inside the binary release, the filename is only
firmware-$(API).

> > since I don't see the majority of people compiling it
> > from source.
>
> I am curious how distributions will want to handle this. Distributions
> with sane people might just ship a gcc built open firmware binary but
> utopian distributions like Debian might require to build the whole
> thing for redistribution. I don't know.

I would welcome distros going that far and building the firmware if it
is possible, but I am not sure if most people would bother.

> > And at that point the release engineering needs to test the
> > API compatibility.
>
> Release engineering.. heh...

As I said, whoever releases the source tarball should also do basic
checks with the binary firmware that results from it. Someone has to do
it. Otherwise there is no point here anyway.

Regards

Marcel


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/