[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20130916112224.GB16984@lee--X1>
Date: Mon, 16 Sep 2013 12:22:24 +0100
From: Lee Jones <lee.jones@...aro.org>
To: wei_wang@...lsil.com.cn
Cc: sameo@...ux.intel.com, devel@...uxdriverproject.org,
linux-kernel@...r.kernel.org, gregkh@...uxfoundation.org,
rogerable@...ltek.com, micky_ching@...lsil.com.cn
Subject: Re: [PATCH v4] mfd: rtsx: Modify rts5249_optimize_phy
On Fri, 13 Sep 2013, wei_wang@...lsil.com.cn wrote:
> From: Wei WANG <wei_wang@...lsil.com.cn>
>
> In some platforms, specially Thinkpad series, rts5249 won't be
> initialized properly. So we need adjust some phy parameters to
> improve the compatibility issue.
>
> It is a little different between simulation and real chip. We have
> no idea about which configuration is better before tape-out. We set
> default settings according to simulation, but need to tune these
> parameters after getting the real chip.
>
> I can't explain every change in detail here. The below information is
> just a rough description:
>
> PHY_REG_REV: Disable internal clkreq_tx, enable rx_pwst
> PHY_BPCR: No change, just turn the magic number to macro definitions
> PHY_PCR: Change OOBS sensitivity, from 60mV to 90mV
> PHY_RCR2: Control charge-pump current automatically
> PHY_FLD4: Use TX cmu reference clock
> PHY_RDR: Change RXDSEL from 30nF to 1.9nF
> PHY_RCR1: Change the duration between adp_st and asserting cp_en from
> 0.32 us to 0.64us
> PHY_FLD3: Adjust internal timers
> PHY_TUNE: Fine tune the regulator12 output voltage
>
> Signed-off-by: Wei WANG <wei_wang@...lsil.com.cn>
> ---
> drivers/mfd/rts5249.c | 48 ++++++++++++++++++++++++++++++++++++--
> include/linux/mfd/rtsx_pci.h | 53 ++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 99 insertions(+), 2 deletions(-)
Much better.
Applied, thanks.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists