Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752354AbdGINiQ (ORCPT ); Sun, 9 Jul 2017 09:38:16 -0400 Received: from mail-wr0-f194.google.com ([209.85.128.194]:32856 "EHLO mail-wr0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752225AbdGINiO (ORCPT ); Sun, 9 Jul 2017 09:38:14 -0400 Subject: Re: [PATCH 1/2] clk: vc5: Add support for IDT VersaClock 5P49V5925 To: Vladimir Barinov , Michael Turquette , Stephen Boyd , Rob Herring , Mark Rutland References: <1499392623-9011-1-git-send-email-vladimir.barinov@cogentembedded.com> <1499392668-9062-1-git-send-email-vladimir.barinov@cogentembedded.com> Cc: linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org From: Marek Vasut Message-ID: <0f450e21-545e-b2e7-ed7c-1df4655cd7e3@gmail.com> Date: Sun, 9 Jul 2017 15:29:40 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <1499392668-9062-1-git-send-email-vladimir.barinov@cogentembedded.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2380 Lines: 72 On 07/07/2017 03:57 AM, Vladimir Barinov wrote: > From: Vladimir Barinov > > Update IDT VersaClock 5 driver to support 5P49V5925. This chip has only > external clock input, four fractional dividers (FODs) and five clock > outputs (four universal clock outputs and one reference clock output at > OUT0_SELB_I2C). > > Signed-off-by: Vladimir Barinov Reviewed-by: Marek Vasut > --- > drivers/clk/clk-versaclock5.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/drivers/clk/clk-versaclock5.c b/drivers/clk/clk-versaclock5.c > index ea7d552..88b749d 100644 > --- a/drivers/clk/clk-versaclock5.c > +++ b/drivers/clk/clk-versaclock5.c > @@ -126,6 +126,7 @@ > /* Supported IDT VC5 models. */ > enum vc5_model { > IDT_VC5_5P49V5923, > + IDT_VC5_5P49V5925, > IDT_VC5_5P49V5933, > IDT_VC5_5P49V5935, > }; > @@ -595,6 +596,7 @@ static int vc5_map_index_to_output(const enum vc5_model model, > case IDT_VC5_5P49V5933: > return (n == 0) ? 0 : 3; > case IDT_VC5_5P49V5923: > + case IDT_VC5_5P49V5925: > case IDT_VC5_5P49V5935: > default: > return n; > @@ -785,6 +787,13 @@ static int vc5_remove(struct i2c_client *client) > .flags = 0, > }; > > +static const struct vc5_chip_info idt_5p49v5925_info = { > + .model = IDT_VC5_5P49V5925, > + .clk_fod_cnt = 4, > + .clk_out_cnt = 5, > + .flags = 0, > +}; > + > static const struct vc5_chip_info idt_5p49v5933_info = { > .model = IDT_VC5_5P49V5933, > .clk_fod_cnt = 2, > @@ -801,6 +810,7 @@ static int vc5_remove(struct i2c_client *client) > > static const struct i2c_device_id vc5_id[] = { > { "5p49v5923", .driver_data = IDT_VC5_5P49V5923 }, > + { "5p49v5925", .driver_data = IDT_VC5_5P49V5925 }, > { "5p49v5933", .driver_data = IDT_VC5_5P49V5933 }, > { "5p49v5935", .driver_data = IDT_VC5_5P49V5935 }, > { } > @@ -809,6 +819,7 @@ static int vc5_remove(struct i2c_client *client) > > static const struct of_device_id clk_vc5_of_match[] = { > { .compatible = "idt,5p49v5923", .data = &idt_5p49v5923_info }, > + { .compatible = "idt,5p49v5925", .data = &idt_5p49v5925_info }, > { .compatible = "idt,5p49v5933", .data = &idt_5p49v5933_info }, > { .compatible = "idt,5p49v5935", .data = &idt_5p49v5935_info }, > { }, > -- Best regards, Marek Vasut