Re: [PATCH v2 60/63] fs/ocfs2: Use the enum req_op and blk_opf_t types

[Bart Van Assche <bvanassche@xxxxxxx>]

On 6/30/22 18:47, Joseph Qi wrote:
> On 6/30/22 7:31 AM, Bart Van Assche wrote:
> > -	bio = o2hb_setup_one_bio(reg, write_wc, &slot, slot+1, REQ_OP_WRITE,
> > +	bio = o2hb_setup_one_bio(reg, write_wc, &slot, slot+1, REQ_OP_WRITE |
> >   				 REQ_SYNC);
>
> Better to let 'REQ_OP_WRITE | REQ_SYNC' at the same line.
> Other looks fine.
> Reviewed-by: Joseph Qi <joseph.qi@xxxxxxxxxxxxxxxxx>

Hi Joseph,

I will make sure that REQ_OP_WRITE | REQ_SYNC appears on the same line. 
Thanks for the review.

Bart.