Hi Sakari,

Thank you for the patch.

On Monday 16 March 2015 02:01:20 Sakari Ailus wrote:
> Add support for the primary camera of the Nokia N950 and N9.
> 
> Signed-off-by: Sakari Ailus <sakari.ai...@iki.fi>
> ---
>  arch/arm/boot/dts/omap3-n9.dts   |   37 +++++++++++++++++++++++++++++++++++
>  arch/arm/boot/dts/omap3-n950.dts |   37 +++++++++++++++++++++++++++++++++++
>  2 files changed, 74 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/omap3-n9.dts b/arch/arm/boot/dts/omap3-n9.dts
> index 9938b5d..7711df1 100644
> --- a/arch/arm/boot/dts/omap3-n9.dts
> +++ b/arch/arm/boot/dts/omap3-n9.dts
> @@ -16,3 +16,40 @@
>       model = "Nokia N9";
>       compatible = "nokia,omap3-n9", "ti,omap36xx", "ti,omap3";
>  };
> +
> +&i2c2 {
> +     smia_1: camera@10 {
> +             compatible = "nokia,smia";
> +             reg = <0x10>;
> +             /* No reset gpio */
> +             vana-supply = <&vaux3>;
> +             clocks = <&isp 0>;
> +             clock-frequency = <9600000>;
> +             nokia,nvm-size = <(16 * 64)>;
> +             port {
> +                     smia_1_1: endpoint {
> +                             link-frequencies = /bits/ 64 <199200000 
> 210000000 499200000>;
> +                             clock-lanes = <0>;
> +                             data-lanes = <1 2>;
> +                             remote-endpoint = <&csi2a_ep>;
> +                     };
> +             };
> +     };
> +};
> +
> +&isp {
> +     vdd-csiphy1-supply = <&vaux2>;
> +     vdd-csiphy2-supply = <&vaux2>;
> +     ports {
> +             port@2 {
> +                     reg = <2>;
> +                     csi2a_ep: endpoint {
> +                             remote-endpoint = <&smia_1_1>;
> +                             clock-lanes = <2>;
> +                             data-lanes = <1 3>;
> +                             crc = <1>;
> +                             lane-polarity = <1 1 1>;
> +                     };
> +             };
> +     };
> +};

Wouldn't it make sense to move the common parts to arch/arm/boot/dts/omap3-
n950-n9.dtsi ?

> diff --git a/arch/arm/boot/dts/omap3-n950.dts
> b/arch/arm/boot/dts/omap3-n950.dts index 261c558..761f275 100644
> --- a/arch/arm/boot/dts/omap3-n950.dts
> +++ b/arch/arm/boot/dts/omap3-n950.dts
> @@ -16,3 +16,40 @@
>       model = "Nokia N950";
>       compatible = "nokia,omap3-n950", "ti,omap36xx", "ti,omap3";
>  };
> +
> +&i2c2 {
> +     smia_1: camera@10 {
> +             compatible = "nokia,smia";
> +             reg = <0x10>;
> +             /* No reset gpio */
> +             vana-supply = <&vaux3>;
> +             clocks = <&isp 0>;
> +             clock-frequency = <9600000>;
> +             nokia,nvm-size = <(16 * 64)>;
> +             port {
> +                     smia_1_1: endpoint {
> +                             link-frequencies = /bits/ 64 <210000000 
> 333600000 398400000>;
> +                             clock-lanes = <0>;
> +                             data-lanes = <1 2>;
> +                             remote-endpoint = <&csi2a_ep>;
> +                     };
> +             };
> +     };
> +};
> +
> +&isp {
> +     vdd-csiphy1-supply = <&vaux2>;
> +     vdd-csiphy2-supply = <&vaux2>;
> +     ports {
> +             port@2 {
> +                     reg = <2>;
> +                     csi2a_ep: endpoint {
> +                             remote-endpoint = <&smia_1_1>;
> +                             clock-lanes = <2>;
> +                             data-lanes = <3 1>;
> +                             crc = <1>;
> +                             lane-polarity = <1 1 1>;
> +                     };
> +             };
> +     };
> +};

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to