 
            
            
            
            
                9 Feb
                
                    2011
                
            
            
                9 Feb
                
                '11
                
            
            
            
        
    
                1:11 p.m.
            
        On Wed, Feb 09, 2011 at 02:42:25PM +0530, Koul, Vinod wrote:
+void disable_jack_btn(struct snd_soc_codec *codec) +{
- snd_soc_write(codec, SN95031_BTNCTRL2, 0x00);
+} +EXPORT_SYMBOL_GPL(disable_jack_btn);
This needs to be namespaced. It's also surprising that the matching enable function isn't exposed...
The code itself looks OK.