From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1764620AbcINTqE (ORCPT ); Wed, 14 Sep 2016 15:46:04 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:42222 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753427AbcINTqA (ORCPT ); Wed, 14 Sep 2016 15:46:00 -0400 Subject: Re: [PATCH 4/9] net: ethernet: ti: cpts: move dt props parsing to cpts driver To: Richard Cochran References: <20160914130231.3035-1-grygorii.strashko@ti.com> <20160914130231.3035-5-grygorii.strashko@ti.com> <20160914135535.GB28592@localhost.localdomain> CC: "David S. Miller" , , Mugunthan V N , Sekhar Nori , , , WingMan Kwok From: Grygorii Strashko Message-ID: <179724dc-dcc2-6052-84c6-eef9807d867c@ti.com> Date: Wed, 14 Sep 2016 22:45:54 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20160914135535.GB28592@localhost.localdomain> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/14/2016 04:55 PM, Richard Cochran wrote: > On Wed, Sep 14, 2016 at 04:02:26PM +0300, Grygorii Strashko wrote: >> Move DT properties parsing into CPTS driver to simplify consumer's >> code and CPTS driver porting on other SoC in the future >> (like Keystone 2). > > And just who is the consumer? Now CPSW, and Keystone 2 netcp in the future. > >> Signed-off-by: Grygorii Strashko >> --- >> drivers/net/ethernet/ti/cpsw.c | 16 +--------------- >> drivers/net/ethernet/ti/cpsw.h | 2 -- >> drivers/net/ethernet/ti/cpts.c | 29 ++++++++++++++++++++++++++--- >> drivers/net/ethernet/ti/cpts.h | 5 +++-- >> 4 files changed, 30 insertions(+), 22 deletions(-) > > You have more (+) than (-). I wouldn't call that a simplification. With this change It will not be required to add the same DT parsing code in Keystone 2 netcp driver, so overall number of lines will be reduced. As I mentioned in cover letter - this is preparation step and this series indented help and simplify CPTS integration with few TI net drivers (now cpsw, soon KS2 netcp), because similar CPTS IP is integrated in two different TI's SoC families (and might be in more the future). I've not posted KS2 patches here, because of two reasons: - they are still under internal review - I hate big patch series -- regards, -grygorii