Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

From: Lad, Prabhakar
Date: Wed Feb 04 2015 - 10:02:20 EST


Hi Rob,

Thanks for the review.

On Wed, Feb 4, 2015 at 2:55 PM, Rob Herring <robherring2@xxxxxxxxx> wrote:
> On Thu, Jan 15, 2015 at 5:39 PM, Lad, Prabhakar
> <prabhakar.csengg@xxxxxxxxx> wrote:
>> From: Benoit Parrot <bparrot@xxxxxx>
>>
>> this patch adds support for omnivision's ov2659
>> sensor.
>>
>> Signed-off-by: Benoit Parrot <bparrot@xxxxxx>
>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx>
>> ---
>
> [...]
>
>> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
>> index b1df0ad..153cd92 100644
>> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
>> @@ -118,6 +118,7 @@ nvidia NVIDIA
>> nxp NXP Semiconductors
>> onnn ON Semiconductor Corp.
>> opencores OpenCores.org
>> +ovt OmniVision Technologies
>
> I'm surprised there are not already compatible strings with
> OmniVision. There are some examples using "omnivision", but no dts
> files and examples don't count.
>
> The stock ticker is ovti, so please use that.
>
OK will update in the v2 version, I went with website which was ovt [1].

[1] http://www.ovt.com/

Regards,
--Prabhakar Lad
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/