Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-05-05 Thread Stanimir Varbanov
Hi Bjorn On 05/02/2017 09:52 PM, Bjorn Andersson wrote: > On Tue 02 May 01:52 PDT 2017, Stanimir Varbanov wrote: > >> Hei Sakari, >> >> On 04/30/2017 01:21 AM, Sakari Ailus wrote: >>> Hi, Stan!! >>> >>> On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: >>> ... +int helper_ge

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-05-03 Thread Jordan Crouse
On Tue, May 02, 2017 at 12:17:20PM +0300, Stanimir Varbanov wrote: > Hi, > > On 04/29/2017 11:22 PM, Bjorn Andersson wrote: > > On Fri 28 Apr 15:02 PDT 2017, Jordan Crouse wrote: > > > >> On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: > >>> +int venus_boot(struct device *paren

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-05-02 Thread Bjorn Andersson
On Tue 02 May 01:52 PDT 2017, Stanimir Varbanov wrote: > Hei Sakari, > > On 04/30/2017 01:21 AM, Sakari Ailus wrote: > > Hi, Stan!! > > > > On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: > > ... > >> +int helper_get_bufreq(struct venus_inst *inst, u32 type, > >> +

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-05-02 Thread Stanimir Varbanov
Hi, On 04/29/2017 11:22 PM, Bjorn Andersson wrote: > On Fri 28 Apr 15:02 PDT 2017, Jordan Crouse wrote: > >> On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: >>> +int venus_boot(struct device *parent, struct device *fw_dev) >>> +{ >>> + const struct firmware *mdt; >>> + phys

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-05-02 Thread Stanimir Varbanov
Hei Sakari, On 04/30/2017 01:21 AM, Sakari Ailus wrote: > Hi, Stan!! > > On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: > ... >> +int helper_get_bufreq(struct venus_inst *inst, u32 type, >> + struct hfi_buffer_requirements *req) >> +{ >> +u32 ptype = HFI_P

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-04-29 Thread Sakari Ailus
Hi, Stan!! On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: ... > +int helper_get_bufreq(struct venus_inst *inst, u32 type, > + struct hfi_buffer_requirements *req) > +{ > + u32 ptype = HFI_PROPERTY_CONFIG_BUFFER_REQUIREMENTS; > + union hfi_get_property

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-04-29 Thread Bjorn Andersson
On Fri 28 Apr 15:02 PDT 2017, Jordan Crouse wrote: > On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: > > +int venus_boot(struct device *parent, struct device *fw_dev) > > +{ > > + const struct firmware *mdt; > > + phys_addr_t mem_phys; > > + ssize_t fw_size; > > + size_t

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-04-28 Thread Jordan Crouse
On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: > +int venus_boot(struct device *parent, struct device *fw_dev) > +{ > + const struct firmware *mdt; > + phys_addr_t mem_phys; > + ssize_t fw_size; > + size_t mem_size; > + void *mem_va; > + int ret; > + > +

[PATCH v8 05/10] media: venus: adding core part and helper functions

2017-04-28 Thread Stanimir Varbanov
* core.c has implemented the platform driver methods, file operations and v4l2 registration. * helpers.c has implemented common helper functions for: - buffer management - vb2_ops and functions for format propagation, - functions for allocating and freeing buffers for internal usag