Comment 12 for bug 1927808

Just wondering (maybe I am way off ...).
In kernel code I often see segments like
  lock(thing)
  do stuff
  unlock(thing)
but in drivers/net/wireless/realtek/rtw88/main.c there is:

void rtw_core_stop(struct rtw_dev *rtwdev)
{
 ...
 mutex_unlock(&rtwdev->mutex);
 ...
 mutex_lock(&rtwdev->mutex);
 ...
}

and that seems back-to-front. Is that as should be?