Possible typo in sound/firewire/isight.c

[Jog-Ming]

sound/firewire/isight.c, line 647:
strspy(card->mixername, "iSight");
To my best knowledge there is no function named strspy in standard libraries.
Solution: Change line 647 to
strscpy(card->mixername, "iSight");

[ranj063]

@Jog-Ming Do you want to post the fix upstream? This doesnt seem related to the SOF driver.

[Jog-Ming]

This typo is brought upon by PR #5333, and after looking through the commit history in the corresponding repo bardliao/linux I confirmed that the commit that included the typo, d63fe28, is applied after all the merges of upstream, which means that this change is in fact not an upstream issue but instead an issue in this repo.
I understand that there are more urgent and significant issues than this one, but unfortunately it seems like this repo is the only place where this issue can be fixed.

[ranj063]

@Jog-Ming this is a fork of the linux kernel repo. We do not merge any change unrelated to SOF driver here and even the ones that are SOF-related must be posted upstream after getting reviewed and merged to the SOF repo. Please send your fix upstream.

[Jog-Ming]

Correct me if I am wrong: What I need to do is to send this issue to the bug tracker in kernel.org, even if the file sound/firewire/isight.c in the official Linux kernel does not have any typo in it, and close this issue as unplanned.

[ranj063]

Looks like the typo was introduced by this patch d63fe28

If there was a follow up to fix this upstream, it will be reflected in our repo when we do the next upstream merge.

[bardliao]

It was removed from upstream. https://marc.info/?l=linux-next&m=173934468511436&w=2
#5367 will revert it.

[Jog-Ming]

PR #5367 is merged. I will close the issue.