On 3/6/23 5:34 PM, Dan Scally wrote:
> Good morning Bingbu
>
> On 03/03/2023 18:15, bingbu.cao@xxxxxxxxx wrote:
>> From: Bingbu Cao <bingbu.cao@xxxxxxxxx>
>>
>> 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@xxxxxxxxx>
>
> Thanks; this looks ok to me now:
>
>
> Reviewed-by: Daniel Scally <dan.scally@xxxxxxxxxxxxxxxx>
>
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
