On Sat, Aug 06 2022, Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> wrote: $SUBJECT: s/comment/message/ > L and S are swapped/ > s/VFIO_FLS_MC/VFIO_FSL_MC/ > > Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> > --- > All the dev_ logging functions in the file have the "VFIO_FSL_MC: " > prefix. > As they are dev_ function, the driver should already be displayed. > > So, does it make sense or could they be all removed? >From a quick glance, there seem to be messages for when the device is _not_ bound to the fsl-mc driver (e.g. in vfio_fsl_mc_bus_notifier()); I'd just fix the typo for now. > --- > drivers/vfio/fsl-mc/vfio_fsl_mc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Acked-by: Cornelia Huck <cohuck@xxxxxxxxxx>