WinUSB 版 px4_drv に ISDB2056 / ISDB2056N のサポートを追加

[tsukumijima]

極めて有益な変更と思われることから、tsukumijima/px4_drv にもマージしたいです。

@hendecarows 勝手なメンションで申し訳ないのですが、よければ feature/isdb2056 ブランチでの変更意図についてご説明いただけないでしょうか。

• b3ec905 : 私の手元にある ISDB2056N では tc90522_s0 の初期化処理を入れずとも動作しているが、なぜ追加する必要があるのか
  • 私は isdb2056n.sys のデコンパイル結果とのにらめっこしかやれてないのですが、usbpcap で傍受した Windows + 公式ドライバでの USB 通信内容を模倣した、とかでしょうか？
• 8af1ed9 : コード自体は申し分ないが、どの機種でどの程度動作確認されているのか
  • PX-M1UR / ISDB2056 / ISDB2056N に対応されているようですが、m1ur_device.c と s1ur_device.c の差分からも分かる通り PX-S1UR は PX-M1UR から BS 用チューナーを取り除いた以外は PX-M1UR と共通設計なので、せっかくなら PX-S1UR にも対応させたいところではある…
    • 私も PX-S1UR の実機は持っていませんが、対応だけでもしておけば誰かが動作確認してくれるはず
  • ちなみに Linux 版 px4_drv で PX-M1UR / PX-S1UR のドライバ実装が ISDB2056/ISDB2056N と分かれている理由は、マージ元の https://github.com/techmadot/px4_drv でそうなっていたからに過ぎません
    • 特に m1ur_device.c はデバイス名以外は isdb2056_device.c とほとんど共通なため DRY の観点からは共通化したい
    • ただ私が C 言語に詳しくないこと、動作確認できている以上既存のコードを壊したくないことから、そのままになってしまっている

[tsukumijima]

通知がいっていないのかお返事いただけていませんが、有用な変更には間違いありませんので、一旦マージします。

[hendecarows]

返事が遅れてすいません。
普段githubにはログインしておらず、emailもほとんど使っていないものを設定していたため気づきませんでした。

• b3ec905 : 私の手元にある ISDB2056N では tc90522_s0 の初期化処理を入れずとも動作しているが、なぜ追加する必要があるのか

確かにLinux版では初期化処理をしなくとも動いていますが、Windows版作成時に初期化処理をしないとクラッシュすることに気づいたため、Linux版も初期化処理をする方が安全だろうと考えて追加しました。

• 8af1ed9 : コード自体は申し分ないが、どの機種でどの程度動作確認されているのか

PX-M1UR, DTV02-1T1S-U (Aなしの古い機種), DTV02A-1T1S-U 2309 の3機種に対してTVTestでチャンネルを変更しつつ数分程度映像が映るか確認したという感じです。長時間の視聴や録画テストはしていません。

• PX-M1UR / ISDB2056 / ISDB2056N に対応されているようですが、m1ur_device.c と s1ur_device.c の差分からも分かる通り PX-S1UR は PX-M1UR から BS 用チューナーを取り除いた以外は PX-M1UR と共通設計なので、せっかくなら PX-S1UR にも対応させたいところではある…
  • 私も PX-S1UR の実機は持っていませんが、対応だけでもしておけば誰かが動作確認してくれるはず

自分でPX-S1URを買った時にでも対応させるつもりでしたが、「対応だけでもしておけば誰かが動作確認してくれるはず 」はなるほど良い考え方だなと思いました。PX-S1UR対応を考えてみます。

[tsukumijima]

@hendecarows

返信いただきありがとうございます！

Windows版作成時に初期化処理をしないとクラッシュすることに気づいたため、Linux版も初期化処理をする方が安全だろうと考えて追加しました。

そうだったんですね…。仰る通り初期化しておくに越したことはないと思います。

TVTestでチャンネルを変更しつつ数分程度映像が映るか確認したという感じです。長時間の視聴や録画テストはしていません

了解です！こちらでも一通り動作テストしておきます。

「対応だけでもしておけば誰かが動作確認してくれるはず 」はなるほど良い考え方だなと思いました。PX-S1UR対応を考えてみます

ありがとうございます！私の方で実装しようとも思っていましたが、hendecarows 氏に実装していただけるならそれを待とうと思います。