Re: [PATCH v2] media: ipu3-cio2: support more camera sensors in cio2-bridge

[Bingbu Cao <bingbu.cao@linux.intel.com>]

On 3/6/23 5:34 PM, Dan Scally wrote:
> Good morning Bingbu
> 
> On 03/03/2023 18:15, bingbu.cao@intel.com wrote:
>> From: Bingbu Cao <bingbu.cao@intel.com>
>>
>> Add more camera sensors into the supported camera sensors list
>> to make cio2-bridge to support more camera sensors.
>>
>> Signed-off-by: Bingbu Cao <bingbu.cao@intel.com>
> 
> Thanks; this looks ok to me now:
> 
> 
> Reviewed-by: Daniel Scally <dan.scally@ideasonboard.com>
> 
Sakari, 

Will you pick this change?

>> ---
>> Changes from v1:
>>   - remove some sensors without available drivers
>>   - add .link_frequencies for each sensor
>>
>> ---
>>   drivers/media/pci/intel/ipu3/cio2-bridge.c | 8 ++++++++
>>   1 file changed, 8 insertions(+)
>>
>> diff --git a/drivers/media/pci/intel/ipu3/cio2-bridge.c b/drivers/media/pci/intel/ipu3/cio2-bridge.c
>> index dfefe0d8aa95..d59bec8a94e2 100644
>> --- a/drivers/media/pci/intel/ipu3/cio2-bridge.c
>> +++ b/drivers/media/pci/intel/ipu3/cio2-bridge.c
>> @@ -29,6 +29,14 @@ static const struct cio2_sensor_config cio2_supported_sensors[] = {
>>       CIO2_SENSOR_CONFIG("INT347E", 1, 319200000),
>>       /* Omnivision OV2680 */
>>       CIO2_SENSOR_CONFIG("OVTI2680", 0),
>> +    /* Omnivision ov8856 */
>> +    CIO2_SENSOR_CONFIG("OVTI8856", 3, 180000000, 360000000, 720000000),
>> +    /* Omnivision ov2740 */
>> +    CIO2_SENSOR_CONFIG("INT3474", 1, 360000000),
>> +    /* Hynix hi556 */
>> +    CIO2_SENSOR_CONFIG("INT3537", 1, 437000000),
>> +    /* Omnivision ov13b10 */
>> +    CIO2_SENSOR_CONFIG("OVTIDB10", 1, 560000000),
>>   };
>>     static const struct cio2_property_names prop_names = {

-- 
Best regards,
Bingbu Cao