On Tue, Jan 21, 2020 at 4:43 AM Hayes Wang <hayeswang@xxxxxxxxxxx> wrote:
>
> Enable DelayPhyPwrChg let the device fail enter the power saving mode
> of P3.
Hayes,
I'm very curious about this commit message: why would one want this to fail?
Did you mean "don't allow the phy to enter P3 power saving mode"?
If P3 power saving mode is broken, what is the symptom?
How long is the delay when this is still enabled? (to help identify
failures when this is still enabled)
BTW, I've reviewed all the patches and don't see any obvious issues
with them - though I don't have the technical documents to verify any
changes in behavior.
I did see two typos in the commit messages that could be corrected if
you need to send out v3:
[PATCH net 5/9] r8152: Disable PLA MCU clock speed down
s/packet lost/packet loss/
[PATCH net 6/9] r8152: disable test IO for RTL8153B
s/casue/cause
cheers,
grant
>
> Signed-off-by: Hayes Wang <hayeswang@xxxxxxxxxxx>
> ---
> drivers/net/usb/r8152.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c
> index 0998b9587943..c999a58ddda9 100644
> --- a/drivers/net/usb/r8152.c
> +++ b/drivers/net/usb/r8152.c
> @@ -31,7 +31,7 @@
> #define NETNEXT_VERSION "11"
>
> /* Information for net */
> -#define NET_VERSION "10"
> +#define NET_VERSION "11"
>
> #define DRIVER_VERSION "v1." NETNEXT_VERSION "." NET_VERSION
> #define DRIVER_AUTHOR "Realtek linux nic maintainers <nic_swsd@xxxxxxxxxxx>"
> @@ -109,6 +109,7 @@
> #define PLA_BP_EN 0xfc38
>
> #define USB_USB2PHY 0xb41e
> +#define USB_SSPHYLINK1 0xb426
> #define USB_SSPHYLINK2 0xb428
> #define USB_U2P3_CTRL 0xb460
> #define USB_CSR_DUMMY1 0xb464
> @@ -384,6 +385,9 @@
> #define USB2PHY_SUSPEND 0x0001
> #define USB2PHY_L1 0x0002
>
> +/* USB_SSPHYLINK1 */
> +#define DELAY_PHY_PWR_CHG BIT(1)
> +
> /* USB_SSPHYLINK2 */
> #define pwd_dn_scale_mask 0x3ffe
> #define pwd_dn_scale(x) ((x) << 1)
> @@ -4993,6 +4997,10 @@ static void rtl8153_up(struct r8152 *tp)
> ocp_data &= ~LANWAKE_PIN;
> ocp_write_byte(tp, MCU_TYPE_PLA, PLA_LWAKE_CTRL_REG, ocp_data);
>
> + ocp_data = ocp_read_word(tp, MCU_TYPE_USB, USB_SSPHYLINK1);
> + ocp_data &= ~DELAY_PHY_PWR_CHG;
> + ocp_write_word(tp, MCU_TYPE_USB, USB_SSPHYLINK1, ocp_data);
> +
> r8153_aldps_en(tp, true);
>
> switch (tp->version) {
> --
> 2.21.0
>
