pcm_chn_create failed with 2004-08-12 kernel

[Don Lewis]

On 12 Aug, Michael W. Oliver wrote:
> Ok, this world was cvsup'd on 2004.08.12 at 09:59:03 EDT.  I removed the
> 'sound' and 'snd_maestro' device lines from my kernel config and decided
> to load them as modules (after I got the same output shown below with
> sound and snd_maestro compiled into the kernel).  When I kldload
> 'sound', I get no output.  After that, when I kldload 'snd_maestro', I
> get the following output:
> 
> 
> pcm0: <ESS Technology Maestro-2E> port 0x1400-0x14ff irq 10 at device 8.0 on pci0
> pcm0: <SigmaTel STAC9721/23 AC97 Codec>
> pcm0: [GIANT-LOCKED]
> pcm0: offset 0xfffa1000 exceeds limit. pcm0: chn_init(pcm0:play:0) failed: err = 19
> pcm0: pcm_chn_create(aggch, 1, 0xc223e700) failed
> pcm0: offset 0xfffa1000 exceeds limit. pcm0: chn_init(pcm0:play:0) failed: err = 19
> pcm0: pcm_chn_create(aggch, 1, 0xc223e700) failed
> pcm0: offset 0xfffa1000 exceeds limit. pcm0: chn_init(pcm0:play:0) failed: err = 19
> pcm0: pcm_chn_create(aggch, 1, 0xc223e700) failed
> pcm0: offset 0xfffa1000 exceeds limit. pcm0: chn_init(pcm0:play:0) failed: err = 19
> pcm0: pcm_chn_create(aggch, 1, 0xc223e700) failed

It sure looks to me like physaddr is getting set to zero in
aggch_init().  It also looks like the device_printf() format string is
missing a newline.

Try adding a device_printf() to aggch_init() to print out both physaddr
and ess->baseaddr so that we can figure out where the calculation
	 ch->offset = physaddr - ess->baseaddr;
is going wrong.  The source file is /usr/src/sys/dev/sound/pci/maestro.c