lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Thu, 17 Jan 2019 11:06:17 +0800
From:   "shenjian (K)" <>
To:     "Wang, Dongsheng" <>,
        Andrew Lunn <>
CC:     Andrew Lunn <andrew@lun>, David Miller <>,
        "" <>,
        "Zhuangyuzeng (Yisen)" <>,
        Salil Mehta <>,
        "lipeng (Y)" <>,
        Yunsheng Lin <>,
        Dengweiwei <>
Subject: Re: Question about default m88e1510 LED configuration of marvell phy

在 2019/1/15 17:08, Wang, Dongsheng 写道:
> On 2019/1/15 14:41, shenjian (K) wrote:
>> 在 2019/1/15 11:39, Andrew Lunn 写道:
>>> On Tue, Jan 15, 2019 at 11:07:17AM +0800, shenjian (K) wrote:
>>>> Hi, all
>>>> 	We encounted a problem when using the marvel 88e151x phy driver, the
>>>> link(LED[0])/ack(LED[1]) leds worked abnormally since we updated our kernel to v4.19.
>>>> The act led kept on and never blink.  It works well when we use genphy driver, aslo
>>>> works well when use marvell phy in kernel v4.18.
>>>> 	I found that the default led configuration had been changed to 0x1177
>>>> by commit commit 077772468ec1(net: phy: marvell: change default m88e1510 LED configuration).
>>> It is a difficult job deciding what the driver should configure. I
>>> would actually prefer to keeps with the reset default, which would be
>>> 0x101e according to the datasheet.
>>> If you are using device tree, you have the option of poking values
>>> into registers, using marvell,reg-init. There are a few examples you
>>> can copy. Doing that guarantees you have the configuration you want.
>>>     Andrew
>>> .
>> Thanks, Andrew.
>> But we are using acpi mode, is there any ohter way to poke values into registers ?
> You can describe property in ACPI _DSD. And parse then in
> marvell_config_init just like of.
> Cheers
> Dongsheng
> .
I will check is this supported by our bios.

Thanks, Dongsheng.

Powered by blists - more mailing lists