[PATCH 9/9] media: cedrus: Add H264 decoding support

Chen-Yu Tsai
2018-07-27T14:01:28Z

On Fri, Jul 27, 2018 at 9:56 PM, Paul Kocialkowski
<paul.kocialkowski@bootlin.com> wrote:
> Hi,
>
> On Wed, 2018-06-13 at 16:07 +0200, Maxime Ripard wrote:
>> Introduce some basic H264 decoding support in cedrus. So far, only the
>> baseline profile videos have been tested, and some more advanced features
>> used in higher profiles are not even implemented.
>
> Here are two specific comments about things I noticed when going through
> the h264 code.
>
> [...]
>
>> @@ -88,12 +101,37 @@ struct sunxi_cedrus_ctx {
>> struct work_struct run_work;
>> struct list_head src_list;
>> struct list_head dst_list;
>> +
>> + union {
>> + struct {
>> + void *mv_col_buf;
>> + dma_addr_t mv_col_buf_dma;
>> + ssize_t mv_col_buf_size;
>> + void *neighbor_info_buf;
>> + dma_addr_t neighbor_info_buf_dma;
>
> Should be "neighbour" instead of "neighbor" and the same applies to most
> variables related to this, as well as the register description.

This just means you've been hanging out with people who use American
English. :)

ChenYu