Fixed build WARNINGs

Fixed following WARNINGs:
for thesofproject#355, removed this unnecessary log print.
for thesofproject#364, added a variable to "struct es8316_priv", and tried to let
          function es8316_enable_jack_detect() align to mainline code.
for thesofproject#452, removed it since we did not verified the patch on other platform.

	WARNING: Unnecessary ftrace-like logging - prefer using ftrace
	thesofproject#355: FILE: sound/soc/codecs/es8316.c:751:
	+    dev_dbg(comp->dev, "Exit %s\n", __func__);

	ERROR: do not initialise statics to 0
	thesofproject#364: FILE: sound/soc/codecs/es8316.c:760:
	+    static int initial = 0;

	WARNING: DT compatible string "everest,es8336" appears un-documented -- check ./Documentation/devicetree/bindings/
	thesofproject#452: FILE: sound/soc/codecs/es8316.c:932:
	+    { .compatible = "everest,es8336", },

Signed-off-by: Huajun Li <[email protected]>

sound/soc/codecs/es8316.c

@@ -43,6 +43,7 @@ struct es8316_priv {
 	unsigned int allowed_rates[NR_SUPPORTED_MCLK_LRCK_RATIOS];
 	struct snd_pcm_hw_constraint_list sysclk_constraints;
 	bool jd_inverted;
+	bool jd_initialized;
 };
 
 /*
@@ -67,11 +68,6 @@ static const SNDRV_CTL_TLVD_DECLARE_DB_RANGE(adc_pga_gain_tlv,
 	8, 10, TLV_DB_SCALE_ITEM(1800, 300, 0),
 );
 
-static const SNDRV_CTL_TLVD_DECLARE_DB_RANGE(hpout_vol_tlv,
-	0, 0, TLV_DB_SCALE_ITEM(-4800, 0, 0),
-	1, 3, TLV_DB_SCALE_ITEM(-2400, 1200, 0),
-);
-
 static const char * const ng_type_txt[] =
 	{ "Constant PGA Gain", "Mute ADC Output" };
 static const struct soc_enum ng_type =
@@ -655,8 +651,6 @@ static irqreturn_t es8316_irq(int irq, void *data)
 	struct snd_soc_component *comp = es8316->component;
 	unsigned int flags;
 
-	dev_dbg(comp->dev, "Enter into %s\n", __func__);
-
 	mutex_lock(&es8316->lock);
 
 	snd_soc_component_write(comp, ES8316_GPIO_DEBOUNCE, 0xf0);
@@ -748,7 +742,6 @@ static irqreturn_t es8316_irq(int irq, void *data)
 	snd_soc_component_write(comp, ES8316_GPIO_DEBOUNCE, 0xf2);
 
 	mutex_unlock(&es8316->lock);
-	dev_dbg(comp->dev, "Exit %s\n", __func__);
 
 	return IRQ_HANDLED;
 }
@@ -757,10 +750,17 @@ static void es8316_enable_jack_detect(struct snd_soc_component *component,
 				      struct snd_soc_jack *jack)
 {
 	struct es8316_priv *es8316 = snd_soc_component_get_drvdata(component);
-	static int initial = 0;
 
-	if (!initial) {
-		es8316->jd_inverted = false;
+	/*
+	 * Init es8316->jd_inverted here and not in the probe, as we cannot
+	 * guarantee that the bytchr-es8316 driver, which might set this
+	 * property, will probe before us.
+	 */
+	es8316->jd_inverted = device_property_read_bool(component->dev,
+							"everest,jack-detect-inverted");
+
+	if (!es8316->jd_initialized) {
+		es8316->jd_initialized = true;
 
 		mutex_lock(&es8316->lock);
 
@@ -773,11 +773,10 @@ static void es8316_enable_jack_detect(struct snd_soc_component *component,
 					      ES8316_GPIO_ENABLE_INTERRUPT);
 
 		mutex_unlock(&es8316->lock);
+
 		/* Enable irq and sync initial jack state */
 		enable_irq(es8316->irq);
 		es8316_irq(es8316->irq, es8316);
-
-		initial = 1;
 	}
 }
 
@@ -897,6 +896,7 @@ static int es8316_i2c_probe(struct i2c_client *i2c_client,
 
 	i2c_set_clientdata(i2c_client, es8316);
 
+	es8316->jd_initialized = false;
 	es8316->regmap = devm_regmap_init_i2c(i2c_client, &es8316_regmap);
 	if (IS_ERR(es8316->regmap))
 		return PTR_ERR(es8316->regmap);
@@ -929,7 +929,6 @@ MODULE_DEVICE_TABLE(i2c, es8316_i2c_id);
 
 #ifdef CONFIG_OF
 static const struct of_device_id es8316_of_match[] = {
-	{ .compatible = "everest,es8336", },
 	{ .compatible = "everest,es8316", },
 	{},
 };