Re: [patch v2 1/3] drivers: phy: Make NULL a valid phy reference

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 03, 2014 at 10:06:09PM +0300, Sergei Shtylyov wrote:
> Hello.
> 
> On 02/03/2014 09:00 PM, Andrew Lunn wrote:
> 
> >The common clock framework considers NULL a valid clock
> >reference. This makes handling optional clocks simple, in that if the
> >optional clock is not available, a NULL reference can be used in the
> >place of a real clock, simplifying the clock consumer.
> 
> >Extend this concept to the phy consumer API. A NULL can be passed to
> >the release calls, the phy_init() and phy_exit() calls, and
> >phy_power_on() and phy_power_off() and a NOP is performed.
> 
> >Signed-off-by: Andrew Lunn <andrew@xxxxxxx>
> [...]
> 
> >  When the controller no longer needs the PHY, it has to release the reference
> >diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy-core.c
> >index 645c867c1257..56535253e50c 100644
> >--- a/drivers/phy/phy-core.c
> >+++ b/drivers/phy/phy-core.c
> [...]
> >@@ -308,6 +320,9 @@ err0:
> >   */
> >  void phy_put(struct phy *phy)
> >  {
> >+	if (!phy)
> >+		return;
> >+
> >  	if (IS_ERR(phy))
> >  		return;
> 
>    Why not merge these two?

Hi Sergei

Have you heard Russell King talk about IS_ERR_NULL()?

     Andrew
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux