From: Aaro Koskinen Date: Sat, 28 Mar 2015 19:24:34 +0000 (+0200) Subject: staging: octeon-usb: make CVMX_WAIT_FOR_FIELD32 to take condition expression X-Git-Tag: v4.14-rc1~5616^2~41 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=203d77621405345c1f3ffc4c1db3f7ff89571b58;p=platform%2Fkernel%2Flinux-rpi.git staging: octeon-usb: make CVMX_WAIT_FOR_FIELD32 to take condition expression Make CVMX_WAIT_FOR_FIELD32 to take full condition expression. This should make the usage simpler, and the macro more readable. Signed-off-by: Aaro Koskinen Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c index 3e89265..9e5476e 100644 --- a/drivers/staging/octeon-usb/octeon-hcd.c +++ b/drivers/staging/octeon-usb/octeon-hcd.c @@ -393,8 +393,8 @@ struct octeon_hcd { struct cvmx_usb_state usb; }; -/* This macro spins on a field waiting for it to reach a value */ -#define CVMX_WAIT_FOR_FIELD32(address, _union, field, op, value, timeout_usec)\ +/* This macro spins on a register waiting for it to reach a condition. */ +#define CVMX_WAIT_FOR_FIELD32(address, _union, cond, timeout_usec) \ ({int result; \ do { \ uint64_t done = cvmx_get_cycle() + (uint64_t)timeout_usec * \ @@ -403,7 +403,8 @@ struct octeon_hcd { \ while (1) { \ c.u32 = cvmx_usb_read_csr32(usb, address); \ - if (c.s.field op (value)) { \ + \ + if (cond) { \ result = 0; \ break; \ } else if (cvmx_get_cycle() > done) { \ @@ -652,11 +653,11 @@ static void cvmx_fifo_setup(struct cvmx_usb_state *usb) USB_SET_FIELD32(CVMX_USBCX_GRSTCTL(usb->index), cvmx_usbcx_grstctl, txfflsh, 1); CVMX_WAIT_FOR_FIELD32(CVMX_USBCX_GRSTCTL(usb->index), - cvmx_usbcx_grstctl, txfflsh, ==, 0, 100); + cvmx_usbcx_grstctl, c.s.txfflsh == 0, 100); USB_SET_FIELD32(CVMX_USBCX_GRSTCTL(usb->index), cvmx_usbcx_grstctl, rxfflsh, 1); CVMX_WAIT_FOR_FIELD32(CVMX_USBCX_GRSTCTL(usb->index), - cvmx_usbcx_grstctl, rxfflsh, ==, 0, 100); + cvmx_usbcx_grstctl, c.s.rxfflsh == 0, 100); } /**