On 07/11/2020 17:08, Richard Cochran wrote:
On Fri, Nov 06, 2020 at 03:56:45PM +0800, Wang Qing wrote:
We always have to update the value of ret, otherwise the
error value may be the previous one.
Signed-off-by: Wang Qing
Acked-by: Richard Cochran
Following Richard's comments:
Revi
On Fri, Nov 06, 2020 at 03:56:45PM +0800, Wang Qing wrote:
> We always have to update the value of ret, otherwise the
> error value may be the previous one.
>
> Signed-off-by: Wang Qing
Acked-by: Richard Cochran
> ---
> drivers/net/ethernet/ti/am65-cpts.c | 3 +--
> 1 file changed, 1 insert
On Fri, Nov 06, 2020 at 01:34:04PM +0200, Grygorii Strashko wrote:
> And ptp_clock_register() can return NULL only if PTP support is disabled.
Not true in general ...
> In which case, we should not even get here.
only because the Kconfig uses "depends on" instead of "implies"
PTP_1588_CLOCK.
>
On 06/11/2020 14:58, Kurt Kanzenbach wrote:
On Fri Nov 06 2020, Arnd Bergmann wrote:
On Fri, Nov 6, 2020 at 12:35 PM Grygorii Strashko
wrote:
On 06/11/2020 09:56, Wang Qing wrote:
+++ b/drivers/net/ethernet/ti/am65-cpts.c
@@ -1001,8 +1001,7 @@ struct am65_cpts *am65_cpts_create(struct de
On Fri Nov 06 2020, Arnd Bergmann wrote:
> On Fri, Nov 6, 2020 at 12:35 PM Grygorii Strashko
> wrote:
>> On 06/11/2020 09:56, Wang Qing wrote:
>
>> > +++ b/drivers/net/ethernet/ti/am65-cpts.c
>> > @@ -1001,8 +1001,7 @@ struct am65_cpts *am65_cpts_create(struct device
>> > *dev, void __iomem *regs
On Fri, Nov 6, 2020 at 12:35 PM Grygorii Strashko
wrote:
> On 06/11/2020 09:56, Wang Qing wrote:
> > +++ b/drivers/net/ethernet/ti/am65-cpts.c
> > @@ -1001,8 +1001,7 @@ struct am65_cpts *am65_cpts_create(struct device
> > *dev, void __iomem *regs,
>
> there is
> cpts->ptp_clock = ptp_clo
On 06/11/2020 09:56, Wang Qing wrote:
We always have to update the value of ret, otherwise the
error value may be the previous one.
Signed-off-by: Wang Qing
---
drivers/net/ethernet/ti/am65-cpts.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/ethernet
