[alsa-devel] [PATCH 2/3] lx6464es: dump MAC in standard form

Takashi Iwai tiwai at suse.de
Wed May 29 13:03:23 CEST 2013


At Wed, 29 May 2013 14:00:57 +0300,
Andy Shevchenko wrote:
> 
> On Wed, 2013-05-29 at 12:15 +0200, Takashi Iwai wrote: 
> > At Wed, 29 May 2013 13:03:21 +0300,
> > Andy Shevchenko wrote:
> > > 
> > > Let's use %pM to print MAC wherever we need it.
> > > 
> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> > > ---
> > >  sound/pci/lx6464es/lx6464es.c | 8 ++------
> > >  1 file changed, 2 insertions(+), 6 deletions(-)
> > > 
> > > diff --git a/sound/pci/lx6464es/lx6464es.c b/sound/pci/lx6464es/lx6464es.c
> > > index 3230e57..33eb301 100644
> > > --- a/sound/pci/lx6464es/lx6464es.c
> > > +++ b/sound/pci/lx6464es/lx6464es.c
> > > @@ -798,9 +798,7 @@ static int lx_init_dsp(struct lx6464es *chip)
> > >  
> > >  mac_ready:
> > >  	snd_printd(LXP "mac address ready read after: %dms\n", i);
> > > -	snd_printk(LXP "mac address: %02X.%02X.%02X.%02X.%02X.%02X\n",
> > > -		   chip->mac_address[0], chip->mac_address[1], chip->mac_address[2],
> > > -		   chip->mac_address[3], chip->mac_address[4], chip->mac_address[5]);
> > > +	snd_printk(LXP "mac address: %pM\n", chip->mac_address);
> > >  
> > >  	err = lx_init_get_version_features(chip);
> > >  	if (err)
> > > @@ -1113,9 +1111,7 @@ static int snd_lx6464es_probe(struct pci_dev *pci,
> > >  	sprintf(card->id, "LX6464ES_%02X%02X%02X",
> > >  		chip->mac_address[3], chip->mac_address[4], chip->mac_address[5]);
> > >  
> > > -	sprintf(card->shortname, "LX6464ES %02X.%02X.%02X.%02X.%02X.%02X",
> > > -		chip->mac_address[0], chip->mac_address[1], chip->mac_address[2],
> > > -		chip->mac_address[3], chip->mac_address[4], chip->mac_address[5]);
> > > +	sprintf(card->shortname, "LX6464ES %pM", chip->mac_address);
> > 
> > This will change from the upper to the lower case, thus it'll lead to
> > an incompatible name string, unfortunately.
> 
> Who is the user of this string?

It's exposed to the user-space via ioctl and it can be used as an id
string.  So this will break user-space compatibility.


Takashi


More information about the Alsa-devel mailing list