Hello Andi, On 02/09/2017 11:22 PM, Andi Shyti wrote: > Add the device tree node for the ir-spi driver which enable the > ir led for remote controlling. > > This patch sets first the GPR3[3] gpio line as a regulator-fixed > for enabling an external regulator which powers the IR LED. > > Removes also the default assignment of GPG3[7] related to the > MOSI line of the SPI3 bus. > > Signed-off-by: Andi Shyti <andi.shyti@xxxxxxxxxxx> > --- > Changelog v1 -> v2 > ================== > v1: https://marc.info/?l=linux-kernel&m=148645523229894&w=2 > > - used 'GPIO_ACTIVE_HIGH' instead of '0' > - added back the gpg3-5 gpio initialization related to the SPI3 > CS line that was erroneously removed > > .../boot/dts/exynos/exynos5433-tm2-common.dtsi | 26 +++++++++++++++++++++- > 1 file changed, 25 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi > index 098ad557fee3..4df6b57a0a68 100644 > --- a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi > +++ b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi > @@ -106,6 +106,13 @@ > }; > }; > > + irda_regulator: irda-regulator { > + compatible = "regulator-fixed"; > + enable-active-high; > + gpio = <&gpr3 3 GPIO_ACTIVE_HIGH>; > + regulator-name = "irda_regulator"; How is this regulator named in the board schematics? My understanding is that regulator-name should match this. I don't have access to this so it may be "irda_regulator" although I was expecting something more like "VDD_IRDA". Patch looks good to me though: Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |