DynamicTablesPkg: Fix always true condition

[vbpandya]

In "if (RpInfo[Index].Sun != MAX_UINT64)", "RpInfo[Index].Sun" is UINT32 type.
Which means it is never going to be equal to MAX_UINT64, so use "RpInfo[Index].Sun != MAX_UINT32" instead.

Description

• Breaking change?
  • Breaking change - Does this PR cause a break in build or boot behavior?
  • Examples: Does it add a new library class or move a module to a different repo.
• Impacts security?
  • Security - Does this PR have a direct security impact?
  • Examples: Crypto algorithm change or buffer overflow fix.
• Includes tests?
  • Tests - Does this PR include any explicit test code?
  • Examples: Unit tests or integration tests.

How This Was Tested

This issue causes an error while building with clang, with this commit there is no error.

Integration Instructions

N/A

[tianocore-pr-automation]

Pull Request Formatting Issues

⚠️ Remove the following template lines from your PR description:
<_Include a description of the change and why this change was made._>
<_For each item, place an "x" in between [and]if true. Example:[x] (you can also check items in GitHub UI)_>
<_Create the PR as a Draft PR if it is only created to run CI checks._>
<_Delete lines in \<\> tags before creating the PR._>

Address these issues and the validation will automatically re-run when you update your pull request.

[pierregondois]

Hello Varshit,
The changes seems correct, but would it be possible to remove the 'Merge' commit ?
I.e. just rebasing on the latest master

[vbpandya]

Hello Varshit, The changes seems correct, but would it be possible to remove the 'Merge' commit ? I.e. just rebasing on the latest master

Apologies, didn't mean to do that, I guess I don't understand the rebase from GUI yet :D

[pierregondois]

All good for me. Just need to wait next week for the end of the feature freeze:
https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning