Re: [PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-30 Thread Stephen Boyd
Quoting Russell King - ARM Linux (2018-11-30 03:04:46) > On Fri, Nov 30, 2018 at 10:25:37AM +, Phil Edworthy wrote: > > On 30 November 2018 09:09 Stephen Boyd wrote: > > > Quoting Phil Edworthy (2018-11-20 06:14:45) > > > > This adds clk_get_optional() and devm_clk_get_optional() functions to

Re: [PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-30 Thread Stephen Boyd
Quoting Russell King - ARM Linux (2018-11-30 03:04:46) > On Fri, Nov 30, 2018 at 10:25:37AM +, Phil Edworthy wrote: > > On 30 November 2018 09:09 Stephen Boyd wrote: > > > Quoting Phil Edworthy (2018-11-20 06:14:45) > > > > This adds clk_get_optional() and devm_clk_get_optional() functions to

Re: [PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-30 Thread Russell King - ARM Linux
On Fri, Nov 30, 2018 at 10:25:37AM +, Phil Edworthy wrote: > Hi Stephen, > > On 30 November 2018 09:09 Stephen Boyd wrote: > > Quoting Phil Edworthy (2018-11-20 06:14:45) > > > This adds clk_get_optional() and devm_clk_get_optional() functions to > > > get optional clocks. > > > They behave

Re: [PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-30 Thread Russell King - ARM Linux
On Fri, Nov 30, 2018 at 10:25:37AM +, Phil Edworthy wrote: > Hi Stephen, > > On 30 November 2018 09:09 Stephen Boyd wrote: > > Quoting Phil Edworthy (2018-11-20 06:14:45) > > > This adds clk_get_optional() and devm_clk_get_optional() functions to > > > get optional clocks. > > > They behave

RE: [PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-30 Thread Phil Edworthy
Hi Stephen, On 30 November 2018 09:09 Stephen Boyd wrote: > Quoting Phil Edworthy (2018-11-20 06:14:45) > > This adds clk_get_optional() and devm_clk_get_optional() functions to > > get optional clocks. > > They behave the same as (devm_)clk_get except where there is no clock > > producer. In

RE: [PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-30 Thread Phil Edworthy
Hi Stephen, On 30 November 2018 09:09 Stephen Boyd wrote: > Quoting Phil Edworthy (2018-11-20 06:14:45) > > This adds clk_get_optional() and devm_clk_get_optional() functions to > > get optional clocks. > > They behave the same as (devm_)clk_get except where there is no clock > > producer. In

Re: [PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-30 Thread Stephen Boyd
Quoting Phil Edworthy (2018-11-20 06:14:45) > This adds clk_get_optional() and devm_clk_get_optional() functions to get > optional clocks. > They behave the same as (devm_)clk_get except where there is no clock > producer. In this case, instead of returning -ENOENT, the function > returns NULL.

Re: [PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-30 Thread Stephen Boyd
Quoting Phil Edworthy (2018-11-20 06:14:45) > This adds clk_get_optional() and devm_clk_get_optional() functions to get > optional clocks. > They behave the same as (devm_)clk_get except where there is no clock > producer. In this case, instead of returning -ENOENT, the function > returns NULL.

[PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-20 Thread Phil Edworthy
This adds clk_get_optional() and devm_clk_get_optional() functions to get optional clocks. They behave the same as (devm_)clk_get except where there is no clock producer. In this case, instead of returning -ENOENT, the function returns NULL. This makes error checking simpler and allows

[PATCH v8] clk: Add (devm_)clk_get_optional() functions

2018-11-20 Thread Phil Edworthy
This adds clk_get_optional() and devm_clk_get_optional() functions to get optional clocks. They behave the same as (devm_)clk_get except where there is no clock producer. In this case, instead of returning -ENOENT, the function returns NULL. This makes error checking simpler and allows