arm64: dts: exynos: Fix i2s0 pins pull control for Exynos5433 SoCs

Message ID 1484834310-21173-1-git-send-email-m.szyprowski@samsung.com
State New
Headers show

Commit Message

Marek Szyprowski Jan. 19, 2017, 1:58 p.m.
Common definition for I2S0 bus should not define any pull control for
the individual I2S pins. Correct this by changing samsung,pin-pud property
to EXYNOS_PIN_PULL_NONE like it is used for other Exynos SoCs.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

---
 arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Javier Martinez Canillas Jan. 19, 2017, 3:18 p.m. | #1
Hello Marek,

On 01/19/2017 10:58 AM, Marek Szyprowski wrote:
> Common definition for I2S0 bus should not define any pull control for

> the individual I2S pins. Correct this by changing samsung,pin-pud property

> to EXYNOS_PIN_PULL_NONE like it is used for other Exynos SoCs.

> 

> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

> ---


Looks good to me.

Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>


Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Krzysztof Kozlowski Jan. 19, 2017, 8:35 p.m. | #2
On Thu, Jan 19, 2017 at 02:58:30PM +0100, Marek Szyprowski wrote:
> Common definition for I2S0 bus should not define any pull control for

> the individual I2S pins. Correct this by changing samsung,pin-pud property

> to EXYNOS_PIN_PULL_NONE like it is used for other Exynos SoCs.

> 

> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

> ---

>  arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 


Makes sense. Two questions here:
1. The same for pcm0_bus?
2. This will affect the specific boards: TM2s. Do they need pull down or
   in fact you wanted to fix them as well? Usage of words "common
   definition" suggests the board might be different and in fact you are
   changing the board now.

Best regards,
Krzysztof

> diff --git a/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi

> index d49879bd34bb..2171ff965212 100644

> --- a/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi

> +++ b/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi

> @@ -135,7 +135,7 @@

>  		samsung,pins = "gpz0-0", "gpz0-1", "gpz0-2", "gpz0-3",

>  				"gpz0-4", "gpz0-5", "gpz0-6";

>  		samsung,pin-function = <EXYNOS_PIN_FUNC_2>;

> -		samsung,pin-pud = <EXYNOS_PIN_PULL_DOWN>;

> +		samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>;

>  		samsung,pin-drv = <EXYNOS5433_PIN_DRV_FAST_SR1>;

>  	};

>  

> -- 

> 1.9.1

> 

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Marek Szyprowski Jan. 20, 2017, 6:51 a.m. | #3
Hi Krzysztof,

On 2017-01-19 21:35, Krzysztof Kozlowski wrote:
> On Thu, Jan 19, 2017 at 02:58:30PM +0100, Marek Szyprowski wrote:

>> Common definition for I2S0 bus should not define any pull control for

>> the individual I2S pins. Correct this by changing samsung,pin-pud property

>> to EXYNOS_PIN_PULL_NONE like it is used for other Exynos SoCs.

>>

>> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

>> ---

>>   arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi | 2 +-

>>   1 file changed, 1 insertion(+), 1 deletion(-)

>>

> Makes sense. Two questions here:

> 1. The same for pcm0_bus?


Right, and for I2S1, PMC1 and SPDIF. I will send a new patch.

> 2. This will affect the specific boards: TM2s. Do they need pull down or

>     in fact you wanted to fix them as well? Usage of words "common

>     definition" suggests the board might be different and in fact you are

>     changing the board now.


I've check, audio on TM2 works fine without this pull down. I wrote 'common'
because all other Exynos SoCs define audio pins without pull down. If pull
down would be really needed for some reasons on TM2, I would override pull
control in tm2.dts

>

> Best regards,

> Krzysztof

>

>> diff --git a/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi

>> index d49879bd34bb..2171ff965212 100644

>> --- a/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi

>> +++ b/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi

>> @@ -135,7 +135,7 @@

>>   		samsung,pins = "gpz0-0", "gpz0-1", "gpz0-2", "gpz0-3",

>>   				"gpz0-4", "gpz0-5", "gpz0-6";

>>   		samsung,pin-function = <EXYNOS_PIN_FUNC_2>;

>> -		samsung,pin-pud = <EXYNOS_PIN_PULL_DOWN>;

>> +		samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>;

>>   		samsung,pin-drv = <EXYNOS5433_PIN_DRV_FAST_SR1>;

>>   	};

>>   

>> -- 

>> 1.9.1

>>

>

>


Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Krzysztof Kozlowski Jan. 20, 2017, 3 p.m. | #4
On Fri, Jan 20, 2017 at 07:51:43AM +0100, Marek Szyprowski wrote:
> Hi Krzysztof,

> 

> On 2017-01-19 21:35, Krzysztof Kozlowski wrote:

> > On Thu, Jan 19, 2017 at 02:58:30PM +0100, Marek Szyprowski wrote:

> > > Common definition for I2S0 bus should not define any pull control for

> > > the individual I2S pins. Correct this by changing samsung,pin-pud property

> > > to EXYNOS_PIN_PULL_NONE like it is used for other Exynos SoCs.

> > > 

> > > Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

> > > ---

> > >   arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi | 2 +-

> > >   1 file changed, 1 insertion(+), 1 deletion(-)

> > > 

> > Makes sense. Two questions here:

> > 1. The same for pcm0_bus?

> 

> Right, and for I2S1, PMC1 and SPDIF. I will send a new patch.

> 

> > 2. This will affect the specific boards: TM2s. Do they need pull down or

> >     in fact you wanted to fix them as well? Usage of words "common

> >     definition" suggests the board might be different and in fact you are

> >     changing the board now.

> 

> I've check, audio on TM2 works fine without this pull down. I wrote 'common'

> because all other Exynos SoCs define audio pins without pull down. If pull

> down would be really needed for some reasons on TM2, I would override pull

> control in tm2.dts


OK, thanks for explanation.

Best regards,
Krzysztof
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch hide | download patch | download mbox

diff --git a/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi
index d49879bd34bb..2171ff965212 100644
--- a/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi
@@ -135,7 +135,7 @@ 
 		samsung,pins = "gpz0-0", "gpz0-1", "gpz0-2", "gpz0-3",
 				"gpz0-4", "gpz0-5", "gpz0-6";
 		samsung,pin-function = <EXYNOS_PIN_FUNC_2>;
-		samsung,pin-pud = <EXYNOS_PIN_PULL_DOWN>;
+		samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>;
 		samsung,pin-drv = <EXYNOS5433_PIN_DRV_FAST_SR1>;
 	};