Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1552438

Re: [PATCH 2/2] phy: rockchip-emmc: try to get drive impedance from DT

From Doug Anderson <dianders@chromium.org>
Newsgroups linux.kernel
Subject Re: [PATCH 2/2] phy: rockchip-emmc: try to get drive impedance from DT
Date 2017-01-06 02:00 +0100
Message-ID <sWr2W-2iO-7@gated-at.bofh.it> (permalink)
References <sWcGB-15H-11@gated-at.bofh.it> <sWcGB-15H-19@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi,

On Thu, Jan 5, 2017 at 1:31 AM, Shawn Lin <shawn.lin@rock-chips.com> wrote:
> Try to get drive impedance from DT and use it, otherwise
> use 50ohm by default in order not to break the existing boards
> as 50ohm works fine for them already.
>
> Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
> ---
>
>  drivers/phy/phy-rockchip-emmc.c | 29 ++++++++++++++++++++++++++++-
>  1 file changed, 28 insertions(+), 1 deletion(-)

I could have sworn that you somehow needed to make sure that the eMMC
part itself needed to also be updated to be told the matching drive
impedance so you don't get a mismatch.  How do you make that work?
...or am I just confused.  I meant to try to dig more, but ran out of
time today.  :(

-Doug

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 1/2] Documentation: phy: introduce new optional property to specify drive impedance Shawn Lin <shawn.lin@rock-chips.com> - 2017-01-05 10:40 +0100
  [PATCH 2/2] phy: rockchip-emmc: try to get drive impedance from DT Shawn Lin <shawn.lin@rock-chips.com> - 2017-01-05 10:40 +0100
    Re: [PATCH 2/2] phy: rockchip-emmc: try to get drive impedance from DT Doug Anderson <dianders@chromium.org> - 2017-01-06 02:00 +0100
      Re: [PATCH 2/2] phy: rockchip-emmc: try to get drive impedance from  DT Shawn Lin <shawn.lin@rock-chips.com> - 2017-01-06 02:20 +0100
        Re: [PATCH 2/2] phy: rockchip-emmc: try to get drive impedance from DT Doug Anderson <dianders@chromium.org> - 2017-01-06 19:30 +0100
  Re: [PATCH 1/2] Documentation: phy: introduce new optional property to specify drive impedance Heiko Stübner <heiko@sntech.de> - 2017-01-05 13:00 +0100
    Re: [PATCH 1/2] Documentation: phy: introduce new optional property  to specify drive impedance Shawn Lin <shawn.lin@rock-chips.com> - 2017-01-06 01:50 +0100
  Re: [PATCH 1/2] Documentation: phy: introduce new optional property  to specify drive impedance Rob Herring <robh@kernel.org> - 2017-01-09 19:10 +0100

csiph-web