[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <e0e7444c-60ef-4f7e-a7e6-18ac7580b402@shenghaoyang.info>
Date: Sat, 5 Oct 2024 19:17:47 +0800
From: Shenghao Yang <me@...nghaoyang.info>
To: Andrew Lunn <andrew@...n.ch>
Cc: netdev@...r.kernel.org, f.fainelli@...il.com, olteanv@...il.com,
pavana.sharma@...i.com, ashkan.boldaji@...i.com, kabel@...nel.org
Subject: Re: [PATCH net 2/3] net: dsa: mv88e6xxx: read cycle counter period
from hardware
On 30/9/24 23:31, Andrew Lunn wrote:
> The mv88e6xxx_tai_read() MV88E6XXX_TAI_CLOCK_PERIOD is not going to
> fail, except for the hardware is dead. There is nothing you can do
> about that, so return the error code and let the probe fail.
> What you are more worried about is if the value you get back is not
> what you expect. It is not 8000 or 10000. I would do a dev_err() and
> return -ENODEV, and let the probe fail.
Hi Andrew,
Thanks! I'll sort those and the naming in v2.
Shenghao
Powered by blists - more mailing lists