Re: [PATCH 1/2] clk: Fix error handling in fixed clock hardware type register fn
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: "Turquette, Mike" <mturquette@xxxxxx>
- Subject: Re: [PATCH 1/2] clk: Fix error handling in fixed clock hardware type register fn
- From: "Saravana Kannan" <skannan@xxxxxxxxxxxxxx>
- Date: Tue, 20 Mar 2012 23:33:05 -0700 (PDT)
- Cc: "Saravana Kannan" <skannan@xxxxxxxxxxxxxx>, "Sascha Hauer" <s.hauer@xxxxxxxxxxxxxx>, "Andrew Lunn" <andrew@xxxxxxx>, "Grant Likely" <grant.likely@xxxxxxxxxxxx>, "Jamie Iles" <jamie@xxxxxxxxxxxxx>, "Jeremy Kerr" <jeremy.kerr@xxxxxxxxxxxxx>, "Magnus Damm" <magnus.damm@xxxxxxxxx>, "Deepak Saxena" <dsaxena@xxxxxxxxxx>, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, "Arnd Bergman" <arnd.bergmann@xxxxxxxxxx>, linux-arm-msm@xxxxxxxxxxxxxxx, "Rob Herring" <rob.herring@xxxxxxxxxxx>, "Russell King" <linux@xxxxxxxxxxxxxxxx>, "Thomas Gleixner" <tglx@xxxxxxxxxxxxx>, "Richard Zhao" <richard.zhao@xxxxxxxxxx>, "Shawn Guo" <shawn.guo@xxxxxxxxxxxxx>, "Paul Walmsley" <paul@xxxxxxxxx>, "Linus Walleij" <linus.walleij@xxxxxxxxxxxxxx>, "Mark Brown" <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>, "Stephen Boyd" <sboyd@xxxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, "Amit Kucheria" <amit.kucheria@xxxxxxxxxx>
- Importance: Normal
- In-reply-to: <CAJOA=zNteC+24oWpqkQJRnL8oqAJe2NcsjHNLbJb1MGN--VEsw@mail.gmail.com>
- User-agent: SquirrelMail/1.4.17
On Tue, March 20, 2012 10:45 pm, Turquette, Mike wrote:
> IIRC, that discussion was with respect to the .parent member of struct
> clk. It was decided that having .parent = NULL does not imply that a
> clock is a root clock, but instead we rely on the CLK_IS_ROOT flag. I
> can't think of any other instance where a NULL clk when returned from
> a registration function would be useful.
I wasn't referring to a discussion with regards to this patch series. I'm
referring to what I believe was a discussion with Russell King on whether
a NULL from clk_get() should be considered a clock and whether it should
be usable as a "clock" with the clock APIs. That's the reason clk_get()
returns -ENOENT instead of NULL.
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Linux ARM Kernel]
[Linux for Sparc]
[Asterisk Internet PBX]