> +     if (eni_stats_needed) {
> +             ena_update_hw_stats(adapter);
> +             for (i = 0; i < ENA_STATS_ARRAY_ENI(adapter); i++) {
> +                     ena_stats = &ena_stats_eni_strings[i];
> +
> +                     ptr = (u64 *)((unsigned long)&adapter->eni_stats +
> +                             ena_stats->stat_offset);

Yet more ugly casts. Please fix this. If done correctly, you should
not need any casts at all.

    Andrew

Reply via email to