Re: [PATCHv2 2/5] sh_eth: enable wake-on-lan for Gen2 devices
On 12/14/2016 04:37 PM, Sergei Shtylyov wrote: Tested on Gen2 r8a7791/Koelsch. Signed-off-by: Niklas Söderlund --- drivers/net/ethernet/renesas/sh_eth.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c index 87640b9..348ed22 100644 --- a/drivers/net/ethernet/renesas/sh_eth.c +++ b/drivers/net/ethernet/renesas/sh_eth.c @@ -624,8 +624,9 @@ static struct sh_eth_cpu_data r8a779x_data = { .register_type= SH_ETH_REG_FAST_RCAR, -.ecsr_value= ECSR_PSRTO | ECSR_LCHNG | ECSR_ICD, -.ecsipr_value= ECSIPR_PSRTOIP | ECSIPR_LCHNGIP | ECSIPR_ICDIP, +.ecsr_value= ECSR_PSRTO | ECSR_LCHNG | ECSR_ICD | ECSR_MPD, +.ecsipr_value= ECSIPR_PSRTOIP | ECSIPR_LCHNGIP | ECSIPR_ICDIP | + ECSIPR_MPDIP, These expressions seem to have been sorted by the bit # before your patch, now they aren't... care to fix? :-) After looking at the SH7743/64 code, nevermind this request. Sorry. :-) MBR, Sergei
Re: [PATCHv2 2/5] sh_eth: enable wake-on-lan for Gen2 devices
Hello! You forgot "R-Car" before "Gen2" in the subject. On 12/12/2016 07:09 PM, Niklas Söderlund wrote: Tested on Gen2 r8a7791/Koelsch. Signed-off-by: Niklas Söderlund --- drivers/net/ethernet/renesas/sh_eth.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c index 87640b9..348ed22 100644 --- a/drivers/net/ethernet/renesas/sh_eth.c +++ b/drivers/net/ethernet/renesas/sh_eth.c @@ -624,8 +624,9 @@ static struct sh_eth_cpu_data r8a779x_data = { .register_type = SH_ETH_REG_FAST_RCAR, - .ecsr_value = ECSR_PSRTO | ECSR_LCHNG | ECSR_ICD, - .ecsipr_value = ECSIPR_PSRTOIP | ECSIPR_LCHNGIP | ECSIPR_ICDIP, + .ecsr_value = ECSR_PSRTO | ECSR_LCHNG | ECSR_ICD | ECSR_MPD, + .ecsipr_value = ECSIPR_PSRTOIP | ECSIPR_LCHNGIP | ECSIPR_ICDIP | + ECSIPR_MPDIP, These expressions seem to have been sorted by the bit # before your patch, now they aren't... care to fix? :-) [...] MBR, Sergei
Re: [PATCHv2 2/5] sh_eth: enable wake-on-lan for Gen2 devices
On Mon, Dec 12, 2016 at 5:09 PM, Niklas Söderlund wrote: > Tested on Gen2 r8a7791/Koelsch. > > Signed-off-by: Niklas Söderlund Tested-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
[PATCHv2 2/5] sh_eth: enable wake-on-lan for Gen2 devices
Tested on Gen2 r8a7791/Koelsch. Signed-off-by: Niklas Söderlund --- drivers/net/ethernet/renesas/sh_eth.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c index 87640b9..348ed22 100644 --- a/drivers/net/ethernet/renesas/sh_eth.c +++ b/drivers/net/ethernet/renesas/sh_eth.c @@ -624,8 +624,9 @@ static struct sh_eth_cpu_data r8a779x_data = { .register_type = SH_ETH_REG_FAST_RCAR, - .ecsr_value = ECSR_PSRTO | ECSR_LCHNG | ECSR_ICD, - .ecsipr_value = ECSIPR_PSRTOIP | ECSIPR_LCHNGIP | ECSIPR_ICDIP, + .ecsr_value = ECSR_PSRTO | ECSR_LCHNG | ECSR_ICD | ECSR_MPD, + .ecsipr_value = ECSIPR_PSRTOIP | ECSIPR_LCHNGIP | ECSIPR_ICDIP | + ECSIPR_MPDIP, .eesipr_value = 0x01ff009f, .tx_check = EESR_FTC | EESR_CND | EESR_DLC | EESR_CD | EESR_RTO, @@ -641,6 +642,7 @@ static struct sh_eth_cpu_data r8a779x_data = { .tpauser= 1, .hw_swap= 1, .rmiimode = 1, + .magic = 1, }; #endif /* CONFIG_OF */ -- 2.10.2