On Mon, Sep 06, 2021 at 12:15:57AM +0530, Saurav Girepunje wrote: > Move the common execution for read_macreg_hdl, write_macreg_hdl, > write_bbreg_hdl and write_rfreg_hdl in to a new function > common_read_write_hdl. You said _what_ you did here, but not _why_ you did this. I can't easily see why you did this, please explain it better in the next version of this patch. thanks, greg k-h