[AD9361] Issues with FIR parsing

[Tey]

The second and third examples of the documentation suggest it is possible to load a filter only for Rx or Tx paths. However, using the third example (Tx only) to feed filter_fir_config produces an error because ad9361_parse_fir() expects that both tx and rx variable are valid (code between lines 4963 and 4985). There's also a typo at line 4899, and a possible buffer overflow if the number of coefficients given in the file is bigger than 128.

I don't know whether this is a bug in the driver code or a documentation error, but in case of the former, a patch like this one should be enough to fix it.

[mhennerich]

Hi,

Your right - patch looks fine and applies without issues.
Can you give me your signed-off-by?

-Michael

[Tey]

If nicknames are okay, then you can use that one:
Signed-off-by: Teyut <teyut@free.fr>

Otherwise, this is a problem as I don't want this nickname to be linked to my real name :( If it's just a matter of copyrights, I give away any copyrights I might have on that patch.

[mhennerich]

Applied with this commit:

7d47a4d

Thanks for reporting and patch.

-Michael