On 12/12/2013 11:25 PM, Laxman Dewangan wrote: > On Friday 13 December 2013 03:42 AM, Stephen Warren wrote: >> From: Stephen Warren <swarren@xxxxxxxxxx> >> >> Other boards use PULL_NONE for their debug UART pins, and without this >> change, the board doesn't accept any serial input. >> >> Don't set the I2S port pins to tristate mode, or no audio signal will >> be sent out. >> >> Fixes: 605ae5804385 ("ARM: tegra: add default pinctrl nodes for Venice2") >> Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx> >> --- >> v2: Added fix for I2S port. >> >> Laxman, can you please go through the whole pinctrl configuration for >> Venice2 and make sure it's correct. Perhaps the best approach would be >> to compare it against our downstream kernels for this board. > > The patch is generated based on downstream ref platforms setting, > Let me again check with very similar board in dowsntream. Oh, you mean this isn't a pinctrl configuration for Venice2 then? As you know, we do have a separate downstream kernel with full Venice2 support; you should use that as a reference. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html