[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v4 02/39] ARM: OMAP2+: gpmc: Adapt to HWMOD
- Subject: RE: [PATCH v4 02/39] ARM: OMAP2+: gpmc: Adapt to HWMOD
- From: "Mohammed, Afzal" <afzal@xxxxxx>
- Date: Thu, 3 May 2012 08:37:05 +0000
- Accept-language: en-US
- Cc: "tony@xxxxxxxxxxx" <tony@xxxxxxxxxxx>, "linux@xxxxxxxxxxxxxxxx" <linux@xxxxxxxxxxxxxxxx>, "Hilman, Kevin" <khilman@xxxxxx>, "Balbi, Felipe" <balbi@xxxxxx>, "dwmw2@xxxxxxxxxxxxx" <dwmw2@xxxxxxxxxxxxx>, "kyungmin.park@xxxxxxxxxxx" <kyungmin.park@xxxxxxxxxxx>, "gregkh@xxxxxxxxxxxxxxxxxxx" <gregkh@xxxxxxxxxxxxxxxxxxx>, "Menon, Nishanth" <nm@xxxxxx>, "grinberg@xxxxxxxxxxxxxx" <grinberg@xxxxxxxxxxxxxx>, "notasas@xxxxxxxxx" <notasas@xxxxxxxxx>, "artem.bityutskiy@xxxxxxxxxxxxxxx" <artem.bityutskiy@xxxxxxxxxxxxxxx>, "vimal.newwork@xxxxxxxxx" <vimal.newwork@xxxxxxxxx>, "dbaryshkov@xxxxxxxxx" <dbaryshkov@xxxxxxxxx>, "linux-omap@xxxxxxxxxxxxxxx" <linux-omap@xxxxxxxxxxxxxxx>, "linux-arm-kernel@xxxxxxxxxxxxxxxxxxx" <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>, "linux-usb@xxxxxxxxxxxxxxx" <linux-usb@xxxxxxxxxxxxxxx>, "linux-mtd@xxxxxxxxxxxxxxxxxxx" <linux-mtd@xxxxxxxxxxxxxxxxxxx>
- In-reply-to: <4FA04A8C.email@example.com>
- Thread-topic: [PATCH v4 02/39] ARM: OMAP2+: gpmc: Adapt to HWMOD
On Wed, May 02, 2012 at 02:11:48, Hunter, Jon wrote:
> > +
> > + pdata->clk_prd = gpmc_get_fclk_period();
> Does this need to be done here? May be this should be done in the probe
> function. You could store the handle to the main clk in the pdata.
This is done so that migration of gpmc driver to the drivers folder
would be smooth, remember that this function will still live here.
> > + pr_err("error: clk_get on %s\n", oh->main_clk);
> > + return -EINVAL;
> > }
> > clk_enable(gpmc_l3_clk);
> I would have thought we should be able to remove the gpmc_init function
> completely by now. Most of the code should be moved to the probe function.
> Also now with hwmod in place, we should be able to remove the
> clk_enable/disable functions and use the pm_runtime APIs instead.
There was no plan to add rpm in this series, but now that you have
brought it up, I will adapt the driver to rpm.
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Linux Arm (vger)]
[Linux USB Devel]
[Video for Linux]
[Linux Audio Users]
[Free Online Dating]