| Date | Tue, 23 Jul 2013 14:09:54 -0500 | Subject | Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data | From | Bin Liu <> |
| |
(re-send, due to last delivery failure to the mailing list...)
On Tue, Jul 23, 2013 at 1:23 PM, Bin Liu <binmlist@gmail.com> wrote: > Hi Sebastian, > > On Mon, Jul 22, 2013 at 1:09 PM, Sebastian Andrzej Siewior > <bigeasy@linutronix.de> wrote: >> >> This patch renames the type struct from ti81xx_driver_data to >> am33xx_driver_data since it is not used for ti81xx anymore. The EOI >> member is also removed since the am33xx SoC does not have such register. >> The interrupt is acknowledged by writting into the stat register. > > I guess the EOI register is removed from the TRM because AM33xx does not use > it, there is no need to write to it to acknowledge. It does not hurt to > write to it though since the register still exists, it just does nothing, I > guess. > > But I am not sure if it is a good idea to remove eoi from the musb_dsps > driver. If the intension is to merge the support for other SoC, such as > AM35xx, AM18xx, then EOI handling might be still needed. I just don't know > how those devices use EOI. > > Regards, > -Bin.
|