Re: [PATCH 1/2] drm/radeon: Using unsigned long instead of unsigned to fix possible overflow

[Hanjun Guo <guohanjun@xxxxxxxxxx>]

On 2022/11/5 2:31, Alex Deucher wrote:
> On Fri, Nov 4, 2022 at 6:05 AM Hanjun Guo <guohanjun@xxxxxxxxxx> wrote:
> > VBIOSImageOffset in struct UEFI_ACPI_VFCT is ULONG (unsigned long),
> > but it will be assigned to "unsigned offset", so use unsigned long
> > instead of unsigned for the offset, to avoid possible overflow.
>
> ULONG in atombios is 32 bits so an int should be fine.

I saw the typedef in the atombios.h is unsigned long but I missed
that the real one is uint32_t in atom-types.h, please drop this patch
and take a look at the PATCH 2/2.

Thanks for the prompt reply.

Thanks
Hanjun