Skip to content

docs: 通しのレビューの軽微な直し (ハブの電源の余裕・部品の数・C34/C36 の並び) - #53

Merged
5ym merged 2 commits into
mainfrom
docs/hub-review-nits
Nov 14, 2026
Merged

5ym merged 2 commits into
mainfrom
docs/hub-review-nits

Conversation

@5ym

@5ym 5ym commented Nov 14, 2026

Copy link
Copy Markdown
Contributor

発注前の通しのレビュー (#44〜#51 を含む) で出た「低」の指摘のうち、docs で直せるものを直す。回路・基板・製造ファイルは変わらない。

  • bringup.md: 最初に測ることの 14 に、ハブ (U8) の電源の電圧を足した。カードが動いている状態で、VDD33 (C35 の 1 番) の電圧とカードの負荷が変わるときの落ち込みを測る。CH334P の低電圧のリセットのしきい値は最大 3.2V で、+3V3 (公称 3.307V、カードの 3A と同じ線) との余裕は 0.1V ほどしかない。「確かめていないこと」にも足した
  • parts.md: 部品の数を 125 から 121 に直した。125 は rev D の KiCad の 115 個 (取り付け穴 H1〜H4 を含む) から数えていた。載せる部品は 108 個、載せない部品は 13 個
  • design.md: 「近い方を大きい容量にした」を書き直した。実際は +3V3 が C36 (10µF) → C34 (100nF) → C35 (1µF) → U8 の順に通るので、U8 の近くにあるのは小さい方。通る順と足からの距離を書いた

確認: bun run lint・bun test (281 件 pass) をコンテナで実行した。

🤖 Generated with Claude Code

https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU

- bringup.md: 最初に測ることの 14 に、カードが動いている状態での U8 の VDD33 の電圧と落ち込みを足す
  (低電圧のリセットのしきい値は最大 3.2V で、+3V3 3.307V との余裕は 0.1V ほど)。確かめていないことにも足す
- parts.md: 部品の数を 121 個に (125 は rev D の取り付け穴 H1〜H4 を含めた数だった)。載せるのは 108 個
- design.md: C34・C36 の「近い方を大きい容量にした」を、+3V3 が通る順と足からの距離に書き直す

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU
@claude

claude Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 16s —— View job


レビュー結果

docs だけの変更で、回路・基板・製造ファイルには触れていません。コードの不具合やセキュリティ上の問題はありません。テストの追加が要る変更でもありません。

  • bringup.md: 項目 14 と「確かめていないこと」の追記は、互いに参照が合っています。CH334P のしきい値 (最大 3.2V) と +3V3 の公称 3.307V から出した余裕 0.1V ほどという計算も合っています。
  • design.md: 通る順 (C36 → C34 → C35 → U8) と、足からの距離の書き方は、PR 本文と一致しています。「近い方を大きい容量に」という誤りが直っています。
  • parts.md: 数の整合は取れています。
    • 111 − 1 (ヘッダ) + 10 + 1 (F2) = 121
    • 108 + 13 = 121
    • 115 − 4 (H1〜H4) = 111

要確認 (軽微)

  • docs/parts.md: 「取り付け穴 H1〜H6 は数えない」と書いてありますが、PR 本文と KiCad の記述にあるのは H1〜H4 です。rev D の穴が本当に H1〜H6 の 6 個なら問題ありません。4 個なら、書き間違いです。私は基板のファイルまでは確かめていません。
  • docs/bringup.md の項目 14: 落ち込んだときの対策に、「C36 を大きくする」とあります。C36 は +3V3 の線で U8 から一番遠く、足から約 11mm あります。落ち込みを抑える効果が C35 側より小さい可能性があります。測ったうえで判断する前提なら、このままでも構いません。

問題はほぼありません。上の H1〜H6 の表記だけ確認してください。

@claude

claude Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 13s —— View job


レビュー結果

docs だけの変更です。回路・基板・製造ファイルには触れていないので、不具合、セキュリティ、テストの面で指摘はありません。

前回の指摘のうち、bringup.md 項目 14 の対策は直っています。「C36 を大きくする」が「U8 のそばの C35 を大きくする」になりました。design.md に書いた距離 (C35 約 1.5mm、C34 約 7.5mm、C36 約 11mm) とも合っています。

要確認 (軽微)

  • docs/parts.md:8 に「取り付け穴 H1〜H6 は数えない」とありますが、同じファイルの 35 行目の表と、同じ行の後ろの「H1〜H4 を含む」は H1〜H4 です。穴が 4 個なら、H1〜H4 に直してください。6 個なら、表と後ろの文のほうを直してください。基板のファイルまでは確かめていません。
  • 部品の数は合っています。115 − 4 = 111、111 − 1 + 10 + 1 = 121。載せない 13 個は J5、JP1〜JP7 の 7 個、TP1〜TP5 の 5 個で、108 + 13 = 121 になります。

それ以外は問題ありません。

@5ym
5ym merged commit 1b71d0b into main Nov 14, 2026
7 checks passed
@5ym
5ym deleted the docs/hub-review-nits branch November 14, 2026 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant