#0: Fix noc_async_write_multicast to pass noc when using one packet version

[tt-aho]

Ticket

Link to Github Issue

Problem description

noc_async_write_multicast does not pass user specified noc to underlying one packet api call.

What's changed

Pass noc to one packet api call.

Checklist

• 
• 
• 
• New/Existing tests provide coverage for changes

Model tests

If your changes cover model-related code, you should run tests corresponding to affected models and platforms (Single card, T3K, Galaxy). "Choose your pipeline" workflows facilitate running multiple kinds of tests in a single run. Each offers models-mandatory and models-extended presets.
The former includes a minimal set of tests, to be run always. The latter extends that with additional ones - use your best judgement in deciding which is the most appropriate for your PR.

• 

  • models-mandatory preset (runs: Device perf regressions and Frequent model and ttnn tests)
  • models-extended preset (runs: the mandatory tests, plus Demo and Model perf tests)
  • other selection - specify runs

• 

  • models-mandatory preset (runs: Unit tests)
  • models-extended preset (runs: the mandatory tests, plus Demo and Model perf tests)
  • other selection - specify runs

• 

  • models-mandatory preset (runs: Quick tests)
  • models-extended preset (runs: the mandatory tests, plus Demo and Model perf tests)
  • other selection - specify runs

[Copilot]

Pull request overview

Fixes noc_async_write_multicast so that when the “one packet” path is selected, the user-specified noc argument is correctly forwarded to the underlying noc_async_write_multicast_one_packet call (instead of silently falling back to noc_index).

Changes:

• Forward noc into noc_async_write_multicast_one_packet<false>(...) when max_page_size <= NOC_MAX_BURST_SIZE.

[tt-aho]

/codeowners ping

[tenstorrent-github-bot]

CodeOwners Group Analysis

This PR requires approval from one member of each of the following groups:

Summary: 1 pending groups, 0 approved groups

Group Information:

• ⏳ tt_metal/hw/inc/ (Group) - Members: Almeet Bhullar, Arik Yaacob, Ata Tuzuner, John Bauman, Kevin Stevens, Nathan Sidwell, Rui Zhang, Vuk Vukomanovic | Pending approval

  📁 Files owned by this group (1 files)

  • tt_metal/hw/inc/api/dataflow/dataflow_api.h

Note: At least one approval from each group is sufficient.

[tenstorrent-github-bot]

Hi John Bauman (@jbaumanTT), Rui Zhang (@ruizhangTT), this PR #0: Fix noc_async_write_multicast to pass noc when using one packet version by Austin Ho (@tt-aho) needs your approval/review to merge this.