qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 04/35] atapi: fill in AUDIO_CTL page correctly


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH 04/35] atapi: fill in AUDIO_CTL page correctly
Date: Mon, 17 Oct 2011 16:04:24 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:7.0) Gecko/20110927 Thunderbird/7.0

On 10/17/2011 04:05 PM, Kevin Wolf wrote:
>  diff --git a/hw/ide/atapi.c b/hw/ide/atapi.c
>  index 347c38d..10f161f 100644
>  --- a/hw/ide/atapi.c
>  +++ b/hw/ide/atapi.c
>  @@ -731,6 +731,8 @@ static void cmd_mode_sense(IDEState *s, uint8_t *buf)
>                buf[7] = 0;
>
>                /* Fill with CDROM audio volume */
>  +            buf[8] = 0x0e;

0x0e would be more readable if it was written as MODE_PAGE_AUDIO_CTL.

>  +            buf[9] = 0x0e;

And this is really the length. But I'm not sure if writing it as 24 - 10
would help a lot.

Yes, the coincidence is ugly.

Of course the same is true for the other pages, so it could be done for
all of them on top. Or maybe you actually have included it later in the
series?

Only for SCSI.  I can queue it as a cleanup for ATAPI.

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]