Messages in this thread | | | Date | Tue, 10 Sep 2013 08:29:25 +0100 | From | Lee Jones <> | Subject | Re: [PATCH v2] mfd: rtsx: Modify rts5249_optimize_phy |
| |
On Tue, 10 Sep 2013, wwang wrote:
> 于 2013年09月09日 21:02, Lee Jones 写道: > >> #define PHY_FLD4 0x1E > >>>+#define FLDEN_SEL 0x4000 > >>>+#define REQ_REF 0x2000 > >>>+#define RXAMP_OFF 0x1000 > >>>+#define REQ_ADDA 0x0800 > >>>+#define BER_COUNT 0x00E0 > >>>+#define BER_TIMER 0x000A > >>>+#define BER_CHK_EN 0x0001 > >>> #define PHY_DUM_REG 0x1F > >>> > #define LCTLR 0x80 > >This doesn't look right. > > > >We had a nicely structured, ordered list and now you've seemingly > >randomly shoved a truck load of un-prefixed defines between them. > > > >Am I missing something? Is there method to the madness? > > Are you suggesting that I should define the macros using the same > prefix like below? > > #define PHY_FLD4 0x1E > #define FLD4_FLDEN_SEL 0x4000 > #define FLD4_REQ_REF 0x2000 > #define FLD4_RXAMP_OFF 0x1000
Ah good, so there was at least some kind of method there. :)
Yes I am, but make them seperate, so perhaps:
/* Phy registers */ #define PHY_PCR 0x00 #define PHY_RCR0 0x01 #define PHY_RCR1 0x02 #define PHY_RCR2 0x03
<snip>
+/* 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
-- 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/
| |