From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH/RFC net-next 2/5] ravb: correct ptp does failure after suspend and resume Date: Sat, 21 Apr 2018 23:33:48 +0300 Message-ID: <51e8277c-5ac3-adc9-9daa-855bf4243468@cogentembedded.com> References: <20180417085030.32650-1-horms+renesas@verge.net.au> <20180417085030.32650-3-horms+renesas@verge.net.au> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Magnus Damm , netdev@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Wolfram Sang , Kazuya Mizuguchi To: Simon Horman Return-path: Received: from mail-lf0-f45.google.com ([209.85.215.45]:43095 "EHLO mail-lf0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753355AbeDUUdw (ORCPT ); Sat, 21 Apr 2018 16:33:52 -0400 Received: by mail-lf0-f45.google.com with SMTP id v207-v6so9550412lfa.10 for ; Sat, 21 Apr 2018 13:33:52 -0700 (PDT) In-Reply-To: <20180417085030.32650-3-horms+renesas@verge.net.au> Content-Language: en-MW Sender: netdev-owner@vger.kernel.org List-ID: Hello! s/failure/fail/ in the subject. On 04/17/2018 11:50 AM, Simon Horman wrote: > From: Kazuya Mizuguchi > > This patch fixes the problem that ptp4l command does not work after > suspend and resume. > Add the initial setting in ravb_suspend() and ravb_resume(), > because ptp does not work. > > Fixes: a0d2f20650e8 ("Renesas Ethernet AVB PTP clock driver") > Signed-off-by: Kazuya Mizuguchi > Signed-off-by: Simon Horman [...] Other than the earlier comments, this seems good: Reviewed-by: Sergei Shtylyov MBR, Sergei