From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kishon Vijay Abraham I Date: Wed, 26 Jun 2013 12:49:42 +0000 Subject: Re: [PATCH v2 1/5] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs Message-Id: <51CAE337.3010508@ti.com> List-Id: References: <1372170110-12993-1-git-send-email-s.nawrocki@samsung.com> <20130625150649.GA21334@arwen.pp.htv.fi> <51C9D714.4000703@samsung.com> <20130625205452.GC9748@arwen.pp.htv.fi> <51CACEBE.9000505@ti.com> <51CAD89E.3060800@samsung.com> <20130626122238.GD12640@arwen.pp.htv.fi> In-Reply-To: <20130626122238.GD12640@arwen.pp.htv.fi> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On Wednesday 26 June 2013 05:52 PM, Felipe Balbi wrote: > On Wed, Jun 26, 2013 at 02:03:42PM +0200, Sylwester Nawrocki wrote: >> On 06/26/2013 01:21 PM, Kishon Vijay Abraham I wrote: >>>>>>> +static int exynos_video_phy_probe(struct platform_device *pdev) >>>>>>>>>>> +{ >>>>>>>>>>> + struct exynos_video_phy *state; >>>>>>>>>>> + struct device *dev = &pdev->dev; >>>>>>>>>>> + struct resource *res; >>>>>>>>>>> + struct phy_provider *phy_provider; >>>>>>>>>>> + int i; >>>>>>>>>>> + >>>>>>>>>>> + state = devm_kzalloc(dev, sizeof(*state), GFP_KERNEL); >>>>>>>>>>> + if (!state) >>>>>>>>>>> + return -ENOMEM; >>>>>>>>>>> + >>>>>>>>>>> + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); >>>>>>>>>>> + >>>>>>>>>>> + state->regs = devm_ioremap_resource(dev, res); >>>>>>>>>>> + if (IS_ERR(state->regs)) >>>>>>>>>>> + return PTR_ERR(state->regs); >>>>>>>>>>> + >>>>>>>>>>> + dev_set_drvdata(dev, state); >>>>>>>>> >>>>>>>>> you can use platform_set_drvdata(pdev, state); >>>>>>> >>>>>>> I had it in the previous version, but changed for symmetry with >>>>>>> dev_set_drvdata(). I guess those could be replaced with >>>>>>> phy_{get, set}_drvdata as you suggested. >>> >>> right. currently I was setting dev_set_drvdata of phy (core) device >>> in phy-core.c and the corresponding dev_get_drvdata in phy provider driver >>> which is little confusing. >>> So I'll add phy_set_drvdata and phy_get_drvdata in phy.h (as suggested by >>> Felipe) to be used by phy provider drivers. So after creating the PHY, the >>> phy provider should use phy_set_drvdata and in phy_ops, it can use >>> phy_get_drvdata. (I'll remove the dev_set_drvdata in phy_create). >>> >>> This also means _void *priv_ in phy_create is useless. So I'll be removing >>> _priv_ from phy_create. >> >> Yeah, sounds good. Then in the phy ops phy_get_drvdata(&phy->dev) would > > phy_get_drvdata(phy); > > accessing the dev pointer will be done inside the helper :-) right :-) -Kishon