linux-kernel - Re: [PATCH v2 2/2] phy: add a driver for the Rockchip SoC internal eMMC PHY

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] [day] [month] [year] [list]

Message-ID: <56B1AB27.3020906@rock-chips.com>
Date:	Wed, 3 Feb 2016 15:24:23 +0800
From:	Shawn Lin <shawn.lin@...k-chips.com>
To:	Kishon Vijay Abraham I <kishon@...com>
Cc:	shawn.lin@...k-chips.com, linux-kernel@...r.kernel.org,
	linux-rockchip@...ts.infradead.org
Subject: Re: [PATCH v2 2/2] phy: add a driver for the Rockchip SoC internal
 eMMC PHY

Hi Kishon,

On 2016/2/3 14:39, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Tuesday 05 January 2016 06:38 AM, Shawn Lin wrote:
>> This patch to add a generic PHY driver for ROCKCHIP eMMC PHY.
>> Access the PHY via registers provided by GRF (general register

[...]

>> + *
>> + * Copyright (C) 2015 Shawn Lin <shawn.lin@...k-chips.com>
>> + * Copyright (C) 2015 ROCKCHIP, Inc.
>
> 2016?

:) I will update it.

>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License as published by

[...]

>> +	}
>> +
>> +	/*
>> +	 * Enable analog DLL circuits, and we need extra 10.2us
>> +	 * for dll to be ready for work.
>> +	 */
>> +	regmap_write(rk_phy->reg_base,
>> +		     rk_phy->reg_offset + GRF_EMMCPHY_CON6,
>> +		     HIWORD_UPDATE(PHYCTRL_ENDLL_ENABLE,
>> +				   PHYCTRL_ENDLL_MASK,
>> +				   PHYCTRL_ENDLL_SHIFT));
>> +	udelay(11);
>
> add a comment for this delay too.

Got it.

>> +	regmap_read(rk_phy->reg_base,
>> +		    rk_phy->reg_offset + GRF_EMMCPHY_STATUS,
>> +		    &dllrdy);
>> +	dllrdy = (dllrdy >> PHYCTRL_DLLRDY_SHIFT) & PHYCTRL_DLLRDY_MASK;
>> +	if (dllrdy != PHYCTRL_DLLRDY_DONE) {
>> +		pr_err("rockchip_emmc_phy_power: dllrdy timeout.\n");
>> +		return -ETIMEDOUT;
>> +	}
>> +
>> +	return 0;
>> +}
>> +

[...]

>> +static int rockchip_emmc_phy_init(struct phy *phy)
>> +{
>> +	return 0;
>> +}
>> +
>> +static int rockchip_emmc_phy_exit(struct phy *phy)
>> +{
>> +	return 0;
>> +}
>
> empty init/exit functions are not required.
>

yep.

> Thanks
> Kishon
>
>
>


-- 
Best Regards
Shawn Lin