[alsa-devel] [PATCH 18/31] HDA patch_via.c: Add 48K sample rate limit for S/PDIF streams.

[Harald Welte]

Hi all,

I agree with the previous posts in this thread.

The use of > 48kHz sample rate is common use today, so if the hardware supports
it, the driver should not disable it.

Since there are some users that successfully use SPDIF with > 48kHz, it cannot
be a general hardware problem.  If there's a general hardware problem, then
it would never work for anyone.

If there are some codec chips/variants/versions, or even certain boards that
have bad SPDIf signal performance, then we should black-list those bad
chips/versions/boards, rather than disable the feature for everyone.

Also, as far as I understand, the worst thing that can happen when a "too high"
sample rate is used:  It will not work for some users on some hardware.  That,
I think, is acceptable (if no workaround / black-list can be created).

This would be different if there are other effects:  If the high sample rate
would cause some hardware-lock-up or similar, then it definitely needs to be
disabled.

Regards,
-- 
- Harald Welte <laforge at gnumonks.org>           http://laforge.gnumonks.org/
============================================================================
"Privacy in residential applications is a desirable marketing option."
                                                  (ETSI EN 300 175-7 Ch. A6)