> -----Original Message----- > From: kvalo=codeaurora.org@xxxxxxxxxxxxxxxxx [mailto:kvalo=codeaurora.org@xxxxxxxxxxxxxxxxx] On > Behalf Of Kalle Valo > Sent: Monday, April 12, 2021 7:48 PM > To: Pkshih > Cc: tony0620emma@xxxxxxxxx; linux-wireless@xxxxxxxxxxxxxxx; DeanKu; Bernie Huang; Shaofu; Steven Ting; > Kevin Yang > Subject: Re: [PATCH 6/7] rtw88: 8822c: add CFO tracking > > Pkshih <pkshih@xxxxxxxxxxx> writes: > > >> -----Original Message----- > >> From: kvalo=codeaurora.org@xxxxxxxxxxxxxxxxx > >> [mailto:kvalo=codeaurora.org@xxxxxxxxxxxxxxxxx] On > >> Behalf Of Kalle Valo > >> Sent: Sunday, April 11, 2021 5:20 PM > >> To: Pkshih > >> Cc: tony0620emma@xxxxxxxxx; linux-wireless@xxxxxxxxxxxxxxx; DeanKu; > >> Bernie Huang; Shaofu; Steven Ting; > >> Kevin Yang > >> Subject: Re: [PATCH 6/7] rtw88: 8822c: add CFO tracking > >> > >> Ping-Ke Shih <pkshih@xxxxxxxxxxx> wrote: > >> > >> > From: Po-Hao Huang <phhuang@xxxxxxxxxxx> > >> > > >> > Add CFO tracking mechanism to mitigate the effect of oscillator > >> > frequency discrepancy. > >> > > >> > Signed-off-by: Po-Hao Huang <phhuang@xxxxxxxxxxx> > >> > Signed-off-by: Ping-Ke Shih <pkshih@xxxxxxxxxxx> > >> > >> What's CFO? I need to understand the commit log, so please write it so that any > >> engineer working with wireless understands it. > >> > > > > CFO tracking, which stands for central frequency offset tracking, is used to > > adjust oscillator to align central frequency of connected AP. Then, we can > > have better performance. > > > > Should I resend this patch to explain CFO? > > Yes, please send a new version with an updated commit log. > A single patch is sent: https://lore.kernel.org/linux-wireless/20210415045913.16097-1-pkshih@xxxxxxxxxxx/T/#u -- Ping-Ke