On 06-Dec-18 10:20, Gal Pressman wrote: > On 05-Dec-18 22:41, Hefty, Sean wrote: >>> +static u32 efa_com_reg_read32(struct efa_com_dev *edev, u16 offset) { >> >> I noticed several places in this file where the open and close braces are on the wrong line. > > Missed that, will fix. > >> Revisiting this, I think your mail client did something wrong - It looks fine in my patch/code.