Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote: > From: Yan-Hsuan Chuang <yhchuang@xxxxxxxxxxx> > > This patch updates the btcoexist to a newer version with some callbacks. > > Specific issues fixed are > pre_load_firmware() > fix the unstable issues before FW is ready > power_on_seting() > fix the unstable issues before HW is ready > Some notify finctions, check hardware type and call corresponding > functions to specific chip > > As noted by the kbuild test robot <lkp@xxxxxxxxx>, there was a missing > global. This problem has been fixed. > > Signed-off-by: Yan-Hsuan Chuang <yhchuang@xxxxxxxxxxx> > Signed-off-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx> > Cc: Ping-Ke Shih <pkshih@xxxxxxxxxxx> > Cc: Birming Chiu <birming@xxxxxxxxxxx> > Cc: Shaofu <shaofu@xxxxxxxxxxx> > Cc: Steven Ting <steventing@xxxxxxxxxxx> > Cc: lkp@xxxxxxxxx There are two red flags which immediately caught my attention: "update to a newer version" and listing multiple changes in the commit log. A patch should have one logical change but this one seems to have a lot of changes which makes it difficult to track what has changed. This definitely should split into smaller patches. I'll still try to commit patches 2-7, hopefully they don't depend on anything in this patch. Patch set to Changes Requested. -- https://patchwork.kernel.org/patch/9750977/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches