On Thu, Apr 17, 2014 at 05:53:10PM +0200, Andrew Lunn wrote:

> + - compatible:       "realtek,alc5623"

You've not added an ID table to the driver for this.

> +  - add-ctrl:        Default register value for Reg-40h, Additional Control 
> Register.
> +             If absent, the default is 0.

> +  - jack-det-ctrl:   Default register value for Reg-5Ah, Jack Detect
> +                     Control Register. If absent, the default is 0.

I would expect the default for these to be to leave the hardware
defaults untouched - why is it different, what does setting to zero mean?

Attachment: signature.asc
Description: Digital signature

Reply via email to