[alsa-devel] [PATCH 1/1] ALSA: hdspm - Fix wordclock status on AES32

Takashi Iwai tiwai at suse.de
Wed Jan 9 17:00:53 CET 2013


At Wed, 09 Jan 2013 16:45:27 +0100,
Adrian Knoth wrote:
> 
> On 01/09/2013 04:22 PM, Takashi Iwai wrote:
> 
> >> +	wcLock = status & HDSPM_AES32_wcLock;
> >> +	wcSync = wcLock && (status & HDSPM_AES32_wcSync);
> >> +
> >>  	snd_iprintf(buffer, "Word: %s  Frequency: %d\n",
> >> -		    (status & HDSPM_AES32_wcLock) ? "Sync   " : "No Lock",
> >> +		    (wcLock) ? (wcSync ? "Sync   " : "Lock") : "No Lock",
> > 
> > I guess you wanted to align the string size by extra spaces after
> > "Sync", but forgot after "Lock"?
> 
> Wow, good catch. Though it's purely cosmetic, feel free to add the
> missing three spaces to please the viewer's eye. ;)

OK, applied with aligned spaces.


thanks,

Takashi


More information about the Alsa-devel mailing list