On 6/7/2016 5:40 PM, Jerin Jacob wrote:
> Adds hardware specific API for ThunderX nicvf inbuilt NIC device under
> drivers/net/thunderx/nicvf/base directory.
> 
> Signed-off-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>
> Signed-off-by: Maciej Czekaj <maciej.czekaj at caviumnetworks.com>
> Signed-off-by: Kamil Rytarowski <Kamil.Rytarowski at caviumnetworks.com>
> Signed-off-by: Zyta Szpak <zyta.szpak at semihalf.com>
> Signed-off-by: Slawomir Rosek <slawomir.rosek at semihalf.com>
> Signed-off-by: Radoslaw Biernacki <rad at semihalf.com>
> ---

...

> +
> +struct pf_rq_cfg { union { struct {
> +#if __BYTE_ORDER__ == __ORDER_BIG_ENDIAN__
> +     uint64_t reserverd1:1;
doesn't really matter but, as a detail, s/reserved/reserverd ? A few
more occurrence below.

> +     uint64_t reserverd0:34;
> +     uint64_t strip_pre_l2:1;
> +     uint64_t caching:2;
> +     uint64_t cq_qs:7;
> +     uint64_t cq_idx:3;
> +     uint64_t rbdr_cont_qs:7;
> +     uint64_t rbdr_cont_idx:1;
> +     uint64_t rbdr_strt_qs:7;
> +     uint64_t rbdr_strt_idx:1;
> +#else
> +     uint64_t rbdr_strt_idx:1;
> +     uint64_t rbdr_strt_qs:7;
> +     uint64_t rbdr_cont_idx:1;
> +     uint64_t rbdr_cont_qs:7;
> +     uint64_t cq_idx:3;
> +     uint64_t cq_qs:7;
> +     uint64_t caching:2;
> +     uint64_t strip_pre_l2:1;
> +     uint64_t reserverd0:34;
> +     uint64_t reserverd1:1;
> +#endif
> +     };
> +     uint64_t value;
> +}; };
> +

...

Reply via email to