Re: [PATCH v2 1/2] media: i2c: Add ON Semiconductor AP1302 ISP driver

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

 



Hi Anil,

On Tue, Sep 07, 2021 at 03:16:19PM +0000, Anil Kumar Mamidala wrote:
> Hi Sakari,
> 
> Can you please review the patches and provide your valuable feedback.
> 
> Please find the change list w.r.to v1 patch below.
> 1.  Updated code with the review comments from you for patch v1.
> 	- Add check for firmware size
> 	- Remove hardcoding of MIPI lanes
> 	- Use fwnode_graph_get_endpoint_by_id() API instead of fwnode_graph_get_next_endpoint
> 2.  Add streaming flag to avoid multiple times calling of stream on/off functionality if already in that state. 
> 3.  Add v4l2_ctrls for setting brightness, saturation, AE metering mode, contrast, gain ctrls.

The patches seem fine to me. I was waiting for Rob Herring's ack, but I see
he's not cc'd nor is the devicetree list. Could you resend the set, with
both cc'd?

Thanks.

-- 
Kind regards,

Sakari Ailus



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux