Re: [PATCH 3/9] net: ethernet: ti: cpts: rework initialization/deinitialization

On Wed, Sep 14, 2016 at 11:10:48PM +0300, Grygorii Strashko wrote:

On 09/14/2016 04:52 PM, Richard Cochran wrote:

quoted

On Wed, Sep 14, 2016 at 04:02:25PM +0300, Grygorii Strashko wrote:

quoted

quoted

-	if (!cpts->rx_enable)
+	if (!cpts || !cpts->rx_enable)
 		return;

Ok. I can't say I'd like all this checks, but there are internal requirement 
to allow CPTS to be disabled though DT on KS2 (even if built in).
I'd try to clarify and return back here.

But It'll be good to know your position - acceptable/can be discussed/completely unacceptable?

Look at that code snippet.  We already test for @cpts->rx_enable.  If
you want to disable cpts at run time, just avoid setting that field.
There is no need for any new tests or return codes.

Thanks,
Richard