Re: [PATCH v2 00/25] media: venus: Enable 6xx support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 23/03/2021 12:20, Stanimir Varbanov wrote:
Hi Bryan,

Thanks for your work!

On 3/12/21 7:30 PM, Bryan O'Donoghue wrote:
V2:
- Adds Acked-by as indicated - Stan
- Fixes typo in patch #1 22000000 -> 220000000 - Stan
- Fixes setting of clk_set_rate in core_clks_enable
   unbreaks regression for 1xx/db410c - Stan
- "Add 6xx AXI halt logic"
   * Polled read removed - Stan
   * Redundant comments removed - Stan
   * Delay assocaited with LPI write removed entirely
     experimentation shows a delay is not required - Stan/Bryan
- Unifies intbuf_types_6xx_enc and intbuf_types_6xx_dec into
   intbuf_types_6xx
   Looking at the code the separate arrays was a NOP anyway - Stan/Bryan
- Ensures venus_helper_set_format_constraints() runs for 6xx only
- Differentiates stop address between 6xx and >= 4xx
   0xdeadb000 >= 4xx
   0x00000000 == 6xx - Stan

With the fixes in place for db410c I've verified this code now on
sm8250/rb5 sdm845/rb3 and msm8916/db410c

You have my ack for all patches

Acked-by: Stanimir Varbanov <stanimir.varbanov@xxxxxxxxxx>

except 21/25 and 24/25 who have comments.


yaml: pending - acked waiting application
https://www.spinics.net/lists/devicetree/msg406892.html

I'll take this through media-tree once driver patchset is ready.


Thanks Stan.

I'll get something re-posted this week after Connect.

---
bod



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux