Re: [dpdk-dev] [RFC PATCH 5/6] bus/mlx5_pci: register a PCI driver

Thu, 18 Jun 2020 02:48:22 -0700 


> From: Thomas Monjalon <[email protected]>
> Sent: Wednesday, June 17, 2020 1:49 PM
> To: Parav Pandit <[email protected]>; Gaëtan Rivet <[email protected]>
> Cc: [email protected]; [email protected]; Ori Kam <[email protected]>;
> Matan Azrad <[email protected]>
> Subject: Re: [dpdk-dev] [RFC PATCH 5/6] bus/mlx5_pci: register a PCI driver
> 
> 15/06/2020 23:46, Gaëtan Rivet:
> > On 10/06/20 17:17 +0000, Parav Pandit wrote:
> > > +         DRV_LOG(ERR, "Invalid mlx5 classes %s. Maybe typo in
> device"
> > > +                 " class argument setting?", value);
> >
> > Error message should not be cut in half, it makes it difficult to grep.
> > If you differentiate between typo in name and invalid combo you could
> > directly warn the user about the proper error.
> >
> > (You can ignore the warning from checkpatch.sh about the long lines on
> > a string if there is one.)
> 
> The best is to cut the message after variable placeholder.
> Here you can cut at the end of the first sentence "%s. "
> 
o.k.























					Reply via email to