From:  Martti Kuparinen <martti.kuparinen@iki.fi>
	Date:  Wed, 01 Oct 2003 12:22:13 +0300

> > @@ -474,7 +476,7 @@
> >  	}
> >  
> >  	SIMPLEQ_INIT(&amr->amr_ccb_queue);
> > -	kthread_create(amr_thread_create, amr);
> > +	/*kthread_create(amr_thread_create, amr);*/
> >  }
> >  
> >  /*
> 
> > @@ -1186,7 +1188,8 @@
> >  
> >  	ac->ac_start_time = (time_t)mono_time.tv_sec;
> >  	ac->ac_flags |= AC_ACTIVE;
> > -	amr_outl(amr, AMR_QREG_IDB, amr->amr_mbox_paddr | AMR_QIDB_SUBMIT);
> > +	amr_outl(amr, AMR_QREG_IDB,
> > +	    (amr->amr_mbox_paddr + 16) | AMR_QIDB_SUBMIT);
> >  	return (0);
> >  }
> >  
> 
> How does these affect the driver and/or controller?

I don't know.  I make no claims to understand the driver (or even to have 
studied the code in any detail) which is one reason why I've been happy to 
leave committing the patch to someone else. All I can say is that it fixed the 
problem I was seeing without it and since it came from the author of the 
driver...

cheers
mark