Re: [PATCH 2/2] arm64: dts: renesas: salvator: Switch eMMC bus to 1V8

2018-11-04 Thread Marek Vasut
On 10/31/2018 02:14 PM, Wolfram Sang wrote: > >> But can we discuss this in the context of describing the hardware? > > The SDHI node needs two kinds of pinmux settings, one for normal speeds > and one for highspeeds. They might differ in supplied voltage, i.e. 3v3 > and 1v8. This eMMC always

Re: [PATCH 2/2] arm64: dts: renesas: salvator: Switch eMMC bus to 1V8

2018-10-31 Thread Wolfram Sang
> But can we discuss this in the context of describing the hardware? The SDHI node needs two kinds of pinmux settings, one for normal speeds and one for highspeeds. They might differ in supplied voltage, i.e. 3v3 and 1v8. This eMMC always works with 1v8, so both settings needed by the SDHI node

Re: [PATCH 2/2] arm64: dts: renesas: salvator: Switch eMMC bus to 1V8

2018-10-31 Thread Marek Vasut
On 10/31/2018 01:46 PM, Simon Horman wrote: > On Mon, Oct 29, 2018 at 08:57:21AM +, Wolfram Sang wrote: >> <_pins>;". So, basically the same phandles for both pinctrls. We can re-add the second one when we need it. >>> >>> I wonder if removing the sdhi2_pins_uhs is what we want to

Re: [PATCH 2/2] arm64: dts: renesas: salvator: Switch eMMC bus to 1V8

2018-10-31 Thread Simon Horman
On Mon, Oct 29, 2018 at 08:57:21AM +, Wolfram Sang wrote: > > > > <_pins>;". So, basically the same phandles for both pinctrls. We > > > can re-add the second one when we need it. > > > > I wonder if removing the sdhi2_pins_uhs is what we want to do, given > > that we might need to adjust

Re: [PATCH 2/2] arm64: dts: renesas: salvator: Switch eMMC bus to 1V8

2018-10-29 Thread Wolfram Sang
> > <_pins>;". So, basically the same phandles for both pinctrls. We > > can re-add the second one when we need it. > > I wonder if removing the sdhi2_pins_uhs is what we want to do, given > that we might need to adjust TDSEL or pull resistor configurations for > the HS200/HS400 modes in the

Re: [PATCH 2/2] arm64: dts: renesas: salvator: Switch eMMC bus to 1V8

2018-10-29 Thread Marek Vasut
On 10/28/2018 10:34 PM, Wolfram Sang wrote: > Hi Marek, Hi, > On Sat, Oct 27, 2018 at 06:34:10PM +0200, Marek Vasut wrote: >> The eMMC card has two supplies, VCC and VCCQ. The VCC supplies the NAND >> array and the VCCQ supplies the bus. On this particular board, the VCC is >> connected to 3.3V

Re: [PATCH 2/2] arm64: dts: renesas: salvator: Switch eMMC bus to 1V8

2018-10-28 Thread Wolfram Sang
Hi Marek, On Sat, Oct 27, 2018 at 06:34:10PM +0200, Marek Vasut wrote: > The eMMC card has two supplies, VCC and VCCQ. The VCC supplies the NAND > array and the VCCQ supplies the bus. On this particular board, the VCC is > connected to 3.3V rail, while the VCCQ is connected to 1.8V rail. Adjust >

[PATCH 2/2] arm64: dts: renesas: salvator: Switch eMMC bus to 1V8

2018-10-27 Thread Marek Vasut
The eMMC card has two supplies, VCC and VCCQ. The VCC supplies the NAND array and the VCCQ supplies the bus. On this particular board, the VCC is connected to 3.3V rail, while the VCCQ is connected to 1.8V rail. Adjust the pinmux to match the bus, which is always operating in 1.8V mode.