Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753120AbbBYPXf (ORCPT ); Wed, 25 Feb 2015 10:23:35 -0500 Received: from mail-we0-f174.google.com ([74.125.82.174]:45400 "EHLO mail-we0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752120AbbBYPXe (ORCPT ); Wed, 25 Feb 2015 10:23:34 -0500 Date: Wed, 25 Feb 2015 15:23:22 +0000 From: Lee Jones To: micky_ching@realsil.com.cn Cc: sameo@linux.intel.com, devel@linuxdriverproject.org, linux-kernel@vger.kernel.org, gregkh@linuxfoundation.org, rogerable@realtek.com, wei_wang@realsil.com.cn Subject: Re: [PATCH v3 5/9] mfd: rtsx: update phy register Message-ID: <20150225152322.GL5132@x1> References: <941b86cc1c6c651106faa86732a6e8756451f5fd.1424842997.git.micky_ching@realsil.com.cn> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <941b86cc1c6c651106faa86732a6e8756451f5fd.1424842997.git.micky_ching@realsil.com.cn> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 8457 Lines: 251 On Wed, 25 Feb 2015, micky_ching@realsil.com.cn wrote: > From: Micky Ching > > Update some phy register name and value for rts5249, > the updated value makes chip more stable on some platform. > > Signed-off-by: Micky Ching > --- > drivers/mfd/rts5249.c | 29 +++++++----- > include/linux/mfd/rtsx_pci.h | 109 ++++++++++++++++++++++--------------------- > 2 files changed, 72 insertions(+), 66 deletions(-) Acked-by: Lee Jones > diff --git a/drivers/mfd/rts5249.c b/drivers/mfd/rts5249.c > index 2fe2854..8de8220 100644 > --- a/drivers/mfd/rts5249.c > +++ b/drivers/mfd/rts5249.c > @@ -132,11 +132,12 @@ static int rts5249_optimize_phy(struct rtsx_pcr *pcr) > if (err < 0) > return err; > > - err = rtsx_pci_write_phy_register(pcr, PHY_REG_REV, > - PHY_REG_REV_RESV | PHY_REG_REV_RXIDLE_LATCHED | > - PHY_REG_REV_P1_EN | PHY_REG_REV_RXIDLE_EN | > - PHY_REG_REV_RX_PWST | PHY_REG_REV_CLKREQ_DLY_TIMER_1_0 | > - PHY_REG_REV_STOP_CLKRD | PHY_REG_REV_STOP_CLKWR); > + err = rtsx_pci_write_phy_register(pcr, PHY_REV, > + PHY_REV_RESV | PHY_REV_RXIDLE_LATCHED | > + PHY_REV_P1_EN | PHY_REV_RXIDLE_EN | > + PHY_REV_CLKREQ_TX_EN | PHY_REV_RX_PWST | > + PHY_REV_CLKREQ_DT_1_0 | PHY_REV_STOP_CLKRD | > + PHY_REV_STOP_CLKWR); > if (err < 0) > return err; > > @@ -147,19 +148,21 @@ static int rts5249_optimize_phy(struct rtsx_pcr *pcr) > PHY_BPCR_IB_FILTER | PHY_BPCR_CMIRROR_EN); > if (err < 0) > return err; > + > err = rtsx_pci_write_phy_register(pcr, PHY_PCR, > PHY_PCR_FORCE_CODE | PHY_PCR_OOBS_CALI_50 | > PHY_PCR_OOBS_VCM_08 | PHY_PCR_OOBS_SEN_90 | > - PHY_PCR_RSSI_EN); > + PHY_PCR_RSSI_EN | PHY_PCR_RX10K); > if (err < 0) > return err; > + > err = rtsx_pci_write_phy_register(pcr, PHY_RCR2, > PHY_RCR2_EMPHASE_EN | PHY_RCR2_NADJR | > - PHY_RCR2_CDR_CP_10 | PHY_RCR2_CDR_SR_2 | > - PHY_RCR2_FREQSEL_12 | PHY_RCR2_CPADJEN | > - PHY_RCR2_CDR_SC_8 | PHY_RCR2_CALIB_LATE); > + PHY_RCR2_CDR_SR_2 | PHY_RCR2_FREQSEL_12 | > + PHY_RCR2_CDR_SC_12P | PHY_RCR2_CALIB_LATE); > if (err < 0) > return err; > + > err = rtsx_pci_write_phy_register(pcr, PHY_FLD4, > PHY_FLD4_FLDEN_SEL | PHY_FLD4_REQ_REF | > PHY_FLD4_RXAMP_OFF | PHY_FLD4_REQ_ADDA | > @@ -167,11 +170,12 @@ static int rts5249_optimize_phy(struct rtsx_pcr *pcr) > PHY_FLD4_BER_CHK_EN); > if (err < 0) > return err; > - err = rtsx_pci_write_phy_register(pcr, PHY_RDR, PHY_RDR_RXDSEL_1_9); > + err = rtsx_pci_write_phy_register(pcr, PHY_RDR, > + PHY_RDR_RXDSEL_1_9 | PHY_SSC_AUTO_PWD); > if (err < 0) > return err; > err = rtsx_pci_write_phy_register(pcr, PHY_RCR1, > - PHY_RCR1_ADP_TIME | PHY_RCR1_VCO_COARSE); > + PHY_RCR1_ADP_TIME_4 | PHY_RCR1_VCO_COARSE); > if (err < 0) > return err; > err = rtsx_pci_write_phy_register(pcr, PHY_FLD3, > @@ -179,10 +183,11 @@ static int rts5249_optimize_phy(struct rtsx_pcr *pcr) > PHY_FLD3_RXDELINK); > if (err < 0) > return err; > + > return rtsx_pci_write_phy_register(pcr, PHY_TUNE, > PHY_TUNE_TUNEREF_1_0 | PHY_TUNE_VBGSEL_1252 | > PHY_TUNE_SDBUS_33 | PHY_TUNE_TUNED18 | > - PHY_TUNE_TUNED12); > + PHY_TUNE_TUNED12 | PHY_TUNE_TUNEA12); > } > > static int rts5249_turn_on_led(struct rtsx_pcr *pcr) > diff --git a/include/linux/mfd/rtsx_pci.h b/include/linux/mfd/rtsx_pci.h > index 87cff60..0103210 100644 > --- a/include/linux/mfd/rtsx_pci.h > +++ b/include/linux/mfd/rtsx_pci.h > @@ -630,16 +630,47 @@ > > /* Phy register */ > #define PHY_PCR 0x00 > +#define PHY_PCR_FORCE_CODE 0xB000 > +#define PHY_PCR_OOBS_CALI_50 0x0800 > +#define PHY_PCR_OOBS_VCM_08 0x0200 > +#define PHY_PCR_OOBS_SEN_90 0x0040 > +#define PHY_PCR_RSSI_EN 0x0002 > +#define PHY_PCR_RX10K 0x0001 > + > #define PHY_RCR0 0x01 > #define PHY_RCR1 0x02 > +#define PHY_RCR1_ADP_TIME_4 0x0400 > +#define PHY_RCR1_VCO_COARSE 0x001F > + > #define PHY_RCR2 0x03 > +#define PHY_RCR2_EMPHASE_EN 0x8000 > +#define PHY_RCR2_NADJR 0x4000 > +#define PHY_RCR2_CDR_SR_2 0x0100 > +#define PHY_RCR2_FREQSEL_12 0x0040 > +#define PHY_RCR2_CDR_SC_12P 0x0010 > +#define PHY_RCR2_CALIB_LATE 0x0002 > + > #define PHY_RTCR 0x04 > #define PHY_RDR 0x05 > +#define PHY_RDR_RXDSEL_1_9 0x4000 > +#define PHY_SSC_AUTO_PWD 0x0600 > #define PHY_TCR0 0x06 > #define PHY_TCR1 0x07 > #define PHY_TUNE 0x08 > +#define PHY_TUNE_TUNEREF_1_0 0x4000 > +#define PHY_TUNE_VBGSEL_1252 0x0C00 > +#define PHY_TUNE_SDBUS_33 0x0200 > +#define PHY_TUNE_TUNED18 0x01C0 > +#define PHY_TUNE_TUNED12 0X0020 > +#define PHY_TUNE_TUNEA12 0x0004 > + > #define PHY_IMR 0x09 > #define PHY_BPCR 0x0A > +#define PHY_BPCR_IBRXSEL 0x0400 > +#define PHY_BPCR_IBTXSEL 0x0100 > +#define PHY_BPCR_IB_FILTER 0x0080 > +#define PHY_BPCR_CMIRROR_EN 0x0040 > + > #define PHY_BIST 0x0B > #define PHY_RAW_L 0x0C > #define PHY_RAW_H 0x0D > @@ -654,12 +685,35 @@ > #define PHY_BPNR 0x16 > #define PHY_BRNR2 0x17 > #define PHY_BENR 0x18 > -#define PHY_REG_REV 0x19 > +#define PHY_REV 0x19 > +#define PHY_REV_RESV 0xE000 > +#define PHY_REV_RXIDLE_LATCHED 0x1000 > +#define PHY_REV_P1_EN 0x0800 > +#define PHY_REV_RXIDLE_EN 0x0400 > +#define PHY_REV_CLKREQ_TX_EN 0x0200 > +#define PHY_REV_CLKREQ_RX_EN 0x0100 > +#define PHY_REV_CLKREQ_DT_1_0 0x0040 > +#define PHY_REV_STOP_CLKRD 0x0020 > +#define PHY_REV_RX_PWST 0x0008 > +#define PHY_REV_STOP_CLKWR 0x0004 > + > #define PHY_FLD0 0x1A > #define PHY_FLD1 0x1B > #define PHY_FLD2 0x1C > #define PHY_FLD3 0x1D > +#define PHY_FLD3_TIMER_4 0x0800 > +#define PHY_FLD3_TIMER_6 0x0020 > +#define PHY_FLD3_RXDELINK 0x0004 > + > #define PHY_FLD4 0x1E > +#define PHY_FLD4_FLDEN_SEL 0x4000 > +#define PHY_FLD4_REQ_REF 0x2000 > +#define PHY_FLD4_RXAMP_OFF 0x1000 > +#define PHY_FLD4_REQ_ADDA 0x0800 > +#define PHY_FLD4_BER_COUNT 0x00E0 > +#define PHY_FLD4_BER_TIMER 0x000A > +#define PHY_FLD4_BER_CHK_EN 0x0001 > + > #define PHY_DUM_REG 0x1F > > #define LCTLR 0x80 > @@ -675,59 +729,6 @@ > #define PCR_SETTING_REG2 0x814 > #define PCR_SETTING_REG3 0x747 > > -/* Phy bits */ > -#define PHY_PCR_FORCE_CODE 0xB000 > -#define PHY_PCR_OOBS_CALI_50 0x0800 > -#define PHY_PCR_OOBS_VCM_08 0x0200 > -#define PHY_PCR_OOBS_SEN_90 0x0040 > -#define PHY_PCR_RSSI_EN 0x0002 > - > -#define PHY_RCR1_ADP_TIME 0x0100 > -#define PHY_RCR1_VCO_COARSE 0x001F > - > -#define PHY_RCR2_EMPHASE_EN 0x8000 > -#define PHY_RCR2_NADJR 0x4000 > -#define PHY_RCR2_CDR_CP_10 0x0400 > -#define PHY_RCR2_CDR_SR_2 0x0100 > -#define PHY_RCR2_FREQSEL_12 0x0040 > -#define PHY_RCR2_CPADJEN 0x0020 > -#define PHY_RCR2_CDR_SC_8 0x0008 > -#define PHY_RCR2_CALIB_LATE 0x0002 > - > -#define PHY_RDR_RXDSEL_1_9 0x4000 > - > -#define PHY_TUNE_TUNEREF_1_0 0x4000 > -#define PHY_TUNE_VBGSEL_1252 0x0C00 > -#define PHY_TUNE_SDBUS_33 0x0200 > -#define PHY_TUNE_TUNED18 0x01C0 > -#define PHY_TUNE_TUNED12 0X0020 > - > -#define PHY_BPCR_IBRXSEL 0x0400 > -#define PHY_BPCR_IBTXSEL 0x0100 > -#define PHY_BPCR_IB_FILTER 0x0080 > -#define PHY_BPCR_CMIRROR_EN 0x0040 > - > -#define PHY_REG_REV_RESV 0xE000 > -#define PHY_REG_REV_RXIDLE_LATCHED 0x1000 > -#define PHY_REG_REV_P1_EN 0x0800 > -#define PHY_REG_REV_RXIDLE_EN 0x0400 > -#define PHY_REG_REV_CLKREQ_DLY_TIMER_1_0 0x0040 > -#define PHY_REG_REV_STOP_CLKRD 0x0020 > -#define PHY_REG_REV_RX_PWST 0x0008 > -#define PHY_REG_REV_STOP_CLKWR 0x0004 > - > -#define PHY_FLD3_TIMER_4 0x7800 > -#define PHY_FLD3_TIMER_6 0x00E0 > -#define PHY_FLD3_RXDELINK 0x0004 > - > -#define PHY_FLD4_FLDEN_SEL 0x4000 > -#define PHY_FLD4_REQ_REF 0x2000 > -#define PHY_FLD4_RXAMP_OFF 0x1000 > -#define PHY_FLD4_REQ_ADDA 0x0800 > -#define PHY_FLD4_BER_COUNT 0x00E0 > -#define PHY_FLD4_BER_TIMER 0x000A > -#define PHY_FLD4_BER_CHK_EN 0x0001 > - > #define rtsx_pci_init_cmd(pcr) ((pcr)->ci = 0) > > struct rtsx_pcr; -- 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/