[MPlayer-dev-eng] [Patch 1/2] dvd_set_speed fixes/addons
Sebastian Kemper
sebastian_ml at gmx.net
Sat Nov 17 13:06:33 CET 2007
Hello Reimar!
On Sat, Nov 17, 2007 at 01:46:42AM +0100, Reimar Döffinger wrote:
> > - unsigned speed to int speed (because restoring the initial speed means
> > to set speed=-1)
>
> Any reason to do the except out of principle? Because then the less
> complex fix is to replace the -1 by ~0 (which is somewhat pointless as well).
I don't think it's _that_ complex ;) And 0 already has a meaning for
dvd_set_speed() (do nothing and exit right away). (Maybe I'm misreading
your ~0, though -- you did mean 0, right?)
> > - use the correct ATAPI command length for SET STREAMING (12 bytes); at
> > least libata (for now anyways) won't let commands through when the
> > command length is wrong ("old" ATA is more forgiving and cuts the last
> > 4 bytes)
BTW of course this was the main reason to write the patch at all. See
http://marc.info/?l=linux-kernel&m=119497215212990&w=2
> > - add a check to make sure the device really is an SG device (taken from
> > sg_simple2.c)
>
> Why?
You're right. Scrap that.
Regards
Sebastian
More information about the MPlayer-dev-eng
mailing list