Move some Device methods to private section

[ayerofieiev-tt]

Ticket

None

Problem description

There are plenty of Device methods which are only used by device itself but they are in a public section

This PR depends on #16256

What's changed

This PR does not add any new functionality, nor changes existing. It shuffles things around.

• Moved near every method thats only used by Device to the private section (19)
• Re-groupped methods logically: methods that proxy to allocator, to cluster
• Moved a couple implementations to cpp
• Reviewed usage of some methods, left comments where its desirable to review usage with owners
• Removed template where it was not used

Whats next:

• Discuss and align what we can/want to do about the two outlined groups of methods
• Review notes with owners, align on next steps

Checklist

• Post commit CI

[cfjchu]

🔥 Great, thank you! Couple requests:

1. Remove the debug comments before merge.
2. It would also be helpful to have a list of the methods removed and methods switched from public->private in the PR description since it's hard to see in the diff.

[cfjchu]

There are a number of these kind of methods that just reach into tt::Cluster::instance() to do the lookup. We should be able to strip these methods from Device.

[ayerofieiev-tt]

Yes.
There are also 32 allocator related methods in device.

But before acting on this, must align with stakeholders.

[tt-aho]

Looks good to me as long as the review comments are removed prior to merge. Review/discussions of the api choice/design can happen later if we're just interested in getting this part of the cleanup merged first.

[dmakoviichuk-tt]

I think you can add a couple nodiscards to some apis. in this pr.

[ayerofieiev-tt]

I think you can add a couple nodiscards to some apis. in this pr.

Not doing this in this PR, keeping the scope down.