> > It looks meaningless that the (*sc_vendor_workaround)() hook function > > is inside of MD if (sc->sc_type == COM_TYPE_ARMADAXP) statement. > > > > Isn't it simpler to prepare a MD hook function that returns > > (possible pre-processed) IIR register value that can be used > > to check if the interrupt for the device is pending or not? > > I performed operation to a com register into com.c. > I think that it is not so brief to return the value of both IIR(u_char) > and an error moreover.
Sorry I don't understand what you mean. Do you think the name of "sc_vendor_workaround" is really appropriate for the function that is called only in COM_TYPE_ARMADAXP case? The requirement in the MI interrupt handler is just "whether interrupts are pending or not," isn't it? --- Izumi Tsutsui
