Hi Xiaojun, Comments inline.
Best Regards, Xiao > -----Original Message----- > From: Xiaojun Liu <[email protected]> > Sent: Friday, February 28, 2020 4:38 PM > To: Wang, Xiao W <[email protected]>; Zhang, Qi Z > <[email protected]>; Kwan, Ngai-mint <[email protected]>; Keller, > Jacob E <[email protected]> > Cc: [email protected]; Xiaojun Liu <[email protected]> > Subject: [PATCH v1 2/5] net/fm10k: add epl serdes and port control functions > > Add epl serdes include loading spico, > controlling pcsl, dma, dfe, ical. Add spico code. > Add state machine for epl lane and port, it creates > a pthread to handle the state changing event. > Add external port management, which will use > state machine to handle the event from lane and port. > The lane state will change between DOWN, WAIT_PLL_LOCK, > WAIT_SIGNAL_OK, WAIT_DFE_ICAL, WAIT_DFE_PCAL, UP. > The port state will change between DOWN, WAIT_LANE_UP, UP. > > To enable the switch management, you need add > CONFIG_RTE_FM10K_MANAGEMENT=y in > config/common_linux when building. No need to add above section in every patch. I suggest to put it into doc/guides/nics/fm10k.rst, and you can have a more detailed description about this big feature there. Please also check the empty lines and alignment issues.
