On Fri, 2015-11-20 at 07:04PM +0530, Nava kishore Manne wrote: > Breaking the single big ISR that has both Rx and Tx > in a single function into smaller ones > > Signed-off-by: Nava kishore Manne <[email protected]> > --- > Changes for v2: > --Splits up the ISR without any functional changes as suggested > by Peter Hurley > > drivers/tty/serial/xilinx_uartps.c | 247 > ++++++++++++++++++++----------------- > 1 file changed, 135 insertions(+), 112 deletions(-) > > diff --git a/drivers/tty/serial/xilinx_uartps.c > b/drivers/tty/serial/xilinx_uartps.c > index 009e0db..2e1b0a8 100644 > --- a/drivers/tty/serial/xilinx_uartps.c > +++ b/drivers/tty/serial/xilinx_uartps.c > @@ -172,6 +172,9 @@ struct cdns_uart { > #define to_cdns_uart(_nb) container_of(_nb, struct cdns_uart, \ > clk_rate_change_nb); > > +static void cdns_uart_handle_tx(void *dev_id); > +static void cdns_uart_handle_rx(void *dev_id, unsigned int isrstatus);
Can't you just move these functions here instead of adding prototypes here? Also, this is likely to conflict with https://lkml.org/lkml/2015/11/19/622 Sören -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
