For this series, maybe squashing is not neccessary. It is pretty goodWould be nice will make them inline and squash it in the current+void rpcif_enable_rpm(struct rpcif *rpc)Should these go to header as static inlines?
Okay, that's some rather evil code. It should really beThird parameter in regmap_read_poll_timeout() is the variable in which value is read and the fourth parameter condition has to be tied with the third parameter (there are similar instance in the kernel).+static int wait_msg_xfer_end(struct rpcif *rpc)This can't be right. sts is used uninitialized here.
REGMAP_READ_POLL_TIMEOUT, because it is macro with weird semantics.
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany