Re: [PATCH 1/4] venus: firmware: check fw size against DT memory region size

2019-01-23 Thread Stanimir Varbanov
Hi Alex, Thanks for the review! On 1/23/19 8:10 AM, Alexandre Courbot wrote: > Sorry for the delayed review! >_< > > On Wed, Jan 9, 2019 at 5:46 PM Stanimir Varbanov > wrote: >> >> By historical reasons we defined firmware memory size to be 6MB even >> that the firmware size for all supported

Re: [PATCH 1/4] venus: firmware: check fw size against DT memory region size

2019-01-22 Thread Alexandre Courbot
Sorry for the delayed review! >_< On Wed, Jan 9, 2019 at 5:46 PM Stanimir Varbanov wrote: > > By historical reasons we defined firmware memory size to be 6MB even > that the firmware size for all supported Venus versions is 5MBs. Correct > that by compare the required firmware size returned from

Re: [PATCH 1/4] venus: firmware: check fw size against DT memory region size

2019-01-18 Thread Stanimir Varbanov
Hi Alex, If you have time please review this patch, I'd like to send a pull request for this series. On 1/9/19 10:46 AM, Stanimir Varbanov wrote: > By historical reasons we defined firmware memory size to be 6MB even > that the firmware size for all supported Venus versions is 5MBs. Correct >

[PATCH 1/4] venus: firmware: check fw size against DT memory region size

2019-01-09 Thread Stanimir Varbanov
By historical reasons we defined firmware memory size to be 6MB even that the firmware size for all supported Venus versions is 5MBs. Correct that by compare the required firmware size returned from mdt loader and the one provided by DT reserved memory region. We proceed further if the required