On 10/23/22 21:13, Greg KH wrote: >>> @@ -1007,7 +986,6 @@ static int p80211_rx_typedrop(struct wlandevice *wlandev, u16 fc) >>> wlandev->rx.cfack_cfpoll++; >>> break; >>> default: >>> - /* printk("unknown"); */ >>> wlandev->rx.data_unknown++; >>> break; >>> } >> >> Shouldn't these printks be guarded under CONFIG_DEBUG_KERNEL instead? > > No, they should just be removed as was done here. > What if in case of debugging without these printks? -- An old man doll... just what I always wanted! - Clara