linux-kernel - Re: [PATCH 1/2] clk: vc5: Add support for IDT VersaClock 5P49V5925

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0f450e21-545e-b2e7-ed7c-1df4655cd7e3@gmail.com>
Date:   Sun, 9 Jul 2017 15:29:40 +0200
From:   Marek Vasut <marek.vasut@...il.com>
To:     Vladimir Barinov <vladimir.barinov@...entembedded.com>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...eaurora.org>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>
Cc:     linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org,
        devicetree@...r.kernel.org
Subject: Re: [PATCH 1/2] clk: vc5: Add support for IDT VersaClock 5P49V5925

On 07/07/2017 03:57 AM, Vladimir Barinov wrote:
> From: Vladimir Barinov <vladimir.barinov+renesas@...entembedded.com>
> 
> 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 <vladimir.barinov+renesas@...entembedded.com>

Reviewed-by: Marek Vasut <marek.vasut@...il.com>

> ---
>  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