hal/ia32: cast operands to correct sizes

[badochov]

Copy "in" related function from efi.
Newer binutils versions check closely if asm instructions get operands of correct size.

JIRA: RTOS-912

Description

Motivation and Context

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Chore (refactoring, style fixes, git/CI config, submodule management, no code logic changes)

How Has This Been Tested?

• Already covered by automatic testing.
• New test added: (add PR link here).
• Tested by hand on: (list targets here).

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing linter checks and tests passed.
• My changes generate no new compilation warnings for any of the targets.

Special treatment

• This PR needs additional PRs to work (list the PRs, preferably in merge-order).
• I will merge this PR by myself when appropriate.

[badochov]

	__asm__ volatile(
		"inw %1, %0; "
	: "=a" (val)
	: "d" ((uint16_t)addr));

generates
inw dx, ax;

whereas

	__asm__ volatile(
		"inw %1, %0; "
	: "=a" (val)
	: "d" (addr));

generares
inw edx,ax;

[badochov]

BTW, why is this code copied 3 times in plo?

[github-actions]

Unit Test Results

7 700 tests  ±0   6 985 ✅ ±0   39m 16s ⏱️ -6s
  436 suites ±0     715 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 41d9dfb. ± Comparison against base commit cfd2a03.

♻️ This comment has been updated with latest results.

[agkaminski]

IMHO double cast here is somewhat better (like you did in the kernel). Perhaps move API void * -> u16, if easily done

[agkaminski]

Thanks!