Re: [PATCHv2 2/2] OMAP: add omap_device_reset()

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

 



On 5/30/2011 4:15 AM, Paul Walmsley wrote:
On Fri, 27 May 2011, Tomi Valkeinen wrote:

However, VENC doesn't use the standard syscontrol mechanism, so that
cannot be done via omap_device interface anyway.

struct omap_hwmod_class has a .reset function pointer for this purpose.
It's already been necessary to deal with the I2C IP block:

    https://patchwork.kernel.org/patch/804072/

If all that's needed are the steps in 4430 TRM Rev. T Section 10.6.5.1.2.3
"Subsequence--Video Encoder Software Reset", then it should be possible to
handle this internal reset with hwmod code.

Mmm, but in that case, if this is purely handled inside the IP, it is up to the driver to manage that.

hwmod should not manage more than the SYSCONFIG ocpreset because this is part of the PRCM/OCP wrapper and not really the IP.

In the case of the VENC, the reset is part of the regular init sequence, so the driver will take care of it.

Don't you think so?

Regards,
Benoit
--
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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux