On Mon, Nov 30, 2020 at 09:03:49PM +0900, Bongsu Jeon wrote: > From: Bongsu Jeon <bongsu.j...@samsung.com> > > Extract the common phy blocks to reuse it. > The UART module will use the common blocks. > > Signed-off-by: Bongsu Jeon <bongsu.j...@samsung.com> > --- > Changes in v3: > - move the phy_common object to s3fwrn.ko to avoid duplication. > - include the header files to include everything which is used inside. > - wrap the lines. > > Changes in v2: > - remove the common function's definition in common header file. > - make the common phy_common.c file to define the common function. > - wrap the lines. > - change the Header guard. > - remove the unused common function. > > drivers/nfc/s3fwrn5/Makefile | 2 +- > drivers/nfc/s3fwrn5/i2c.c | 117 > +++++++++++++-------------------------- > drivers/nfc/s3fwrn5/phy_common.c | 63 +++++++++++++++++++++ > drivers/nfc/s3fwrn5/phy_common.h | 36 ++++++++++++ > 4 files changed, 139 insertions(+), 79 deletions(-) > create mode 100644 drivers/nfc/s3fwrn5/phy_common.c > create mode 100644 drivers/nfc/s3fwrn5/phy_common.h
I am sorry, but I am not going to review this. Please send properly a next version - v4 of entire patchset - while fixing issues pointed out in my other email (so use proper workflow). Best regards, Krzysztof