Re: [PATCH 3/8] media: v4l2-async: simplify v4l2_async_subdev structure

2017-12-18 Thread Alexandre Belloni
On 18/12/2017 at 17:53:57 -0200, Mauro Carvalho Chehab wrote:
> The V4L2_ASYNC_MATCH_FWNODE match criteria requires just one
> struct to be filled (struct fwnode_handle). The V4L2_ASYNC_MATCH_DEVNAME
> match criteria requires just a device name.
> 
> So, it doesn't make sense to enclose those into structs,
> as the criteria can go directly into the union.
> 
> That makes easier to document it, as we don't need to document
> weird senseless structs.
> 
> At drivers, this makes even clearer about the match criteria.
> 

For atmel:
Acked-by: Alexandre Belloni <alexandre.bell...@free-electrons.com>

> Acked-by: Sylwester Nawrocki <s.nawro...@samsung.com>
> Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>
> ---
>  drivers/media/platform/am437x/am437x-vpfe.c|  6 +++---
>  drivers/media/platform/atmel/atmel-isc.c   |  2 +-
>  drivers/media/platform/atmel/atmel-isi.c   |  2 +-
>  drivers/media/platform/davinci/vpif_capture.c  |  4 ++--
>  drivers/media/platform/exynos4-is/media-dev.c  |  4 ++--
>  drivers/media/platform/pxa_camera.c|  2 +-
>  drivers/media/platform/qcom/camss-8x16/camss.c |  2 +-
>  drivers/media/platform/rcar-vin/rcar-core.c|  2 +-
>  drivers/media/platform/rcar_drif.c |  4 ++--
>  drivers/media/platform/soc_camera/soc_camera.c |  2 +-
>  drivers/media/platform/stm32/stm32-dcmi.c  |  2 +-
>  drivers/media/platform/ti-vpe/cal.c|  2 +-
>  drivers/media/platform/xilinx/xilinx-vipp.c|  2 +-
>  drivers/media/v4l2-core/v4l2-async.c   | 16 
>  drivers/media/v4l2-core/v4l2-fwnode.c  | 10 +-
>  drivers/staging/media/imx/imx-media-dev.c  |  4 ++--
>  include/media/v4l2-async.h |  8 ++--
>  17 files changed, 35 insertions(+), 39 deletions(-)
> 
> diff --git a/drivers/media/platform/am437x/am437x-vpfe.c 
> b/drivers/media/platform/am437x/am437x-vpfe.c
> index 0997c640191d..601ae6487617 100644
> --- a/drivers/media/platform/am437x/am437x-vpfe.c
> +++ b/drivers/media/platform/am437x/am437x-vpfe.c
> @@ -2304,8 +2304,8 @@ vpfe_async_bound(struct v4l2_async_notifier *notifier,
>   vpfe_dbg(1, vpfe, "vpfe_async_bound\n");
>  
>   for (i = 0; i < ARRAY_SIZE(vpfe->cfg->asd); i++) {
> - if (vpfe->cfg->asd[i]->match.fwnode.fwnode ==
> - asd[i].match.fwnode.fwnode) {
> + if (vpfe->cfg->asd[i]->match.fwnode ==
> + asd[i].match.fwnode) {
>   sdinfo = >cfg->sub_devs[i];
>   vpfe->sd[i] = subdev;
>   vpfe->sd[i]->grp_id = sdinfo->grp_id;
> @@ -2510,7 +2510,7 @@ vpfe_get_pdata(struct platform_device *pdev)
>   }
>  
>   pdata->asd[i]->match_type = V4L2_ASYNC_MATCH_FWNODE;
> - pdata->asd[i]->match.fwnode.fwnode = of_fwnode_handle(rem);
> + pdata->asd[i]->match.fwnode = of_fwnode_handle(rem);
>   of_node_put(rem);
>   }
>  
> diff --git a/drivers/media/platform/atmel/atmel-isc.c 
> b/drivers/media/platform/atmel/atmel-isc.c
> index 0c2635647f69..34676409ca08 100644
> --- a/drivers/media/platform/atmel/atmel-isc.c
> +++ b/drivers/media/platform/atmel/atmel-isc.c
> @@ -2088,7 +2088,7 @@ static int isc_parse_dt(struct device *dev, struct 
> isc_device *isc)
>   subdev_entity->pfe_cfg0 |= ISC_PFE_CFG0_PPOL_LOW;
>  
>   subdev_entity->asd->match_type = V4L2_ASYNC_MATCH_FWNODE;
> - subdev_entity->asd->match.fwnode.fwnode =
> + subdev_entity->asd->match.fwnode =
>   of_fwnode_handle(rem);
>   list_add_tail(_entity->list, >subdev_entities);
>   }
> diff --git a/drivers/media/platform/atmel/atmel-isi.c 
> b/drivers/media/platform/atmel/atmel-isi.c
> index e900995143a3..9958918e2449 100644
> --- a/drivers/media/platform/atmel/atmel-isi.c
> +++ b/drivers/media/platform/atmel/atmel-isi.c
> @@ -1128,7 +1128,7 @@ static int isi_graph_parse(struct atmel_isi *isi, 
> struct device_node *node)
>   /* Remote node to connect */
>   isi->entity.node = remote;
>   isi->entity.asd.match_type = V4L2_ASYNC_MATCH_FWNODE;
> - isi->entity.asd.match.fwnode.fwnode = of_fwnode_handle(remote);
> + isi->entity.asd.match.fwnode = of_fwnode_handle(remote);
>   return 0;
>   }
>  }
> diff --git a/drivers/media/platform/davinci/vpif_capture.c 
> b/drivers/media/platform/davinci/vpif_capture.c
> index e45916f69def..e1c273c8b9a6 100644
> --- a/drivers/media/platform/da

Re: [trivial PATCH] treewide: Align function definition open/close braces

2017-12-18 Thread Alexandre Belloni
On 17/12/2017 at 16:28:44 -0800, Joe Perches wrote:
> Some functions definitions have either the initial open brace and/or
> the closing brace outside of column 1.
> 
> Move those braces to column 1.
> 
> This allows various function analyzers like gnu complexity to work
> properly for these modified functions.
> 
> Miscellanea:
> 
> o Remove extra trailing ; and blank line from xfs_agf_verify
> 
> Signed-off-by: Joe Perches <j...@perches.com>

For RTC:

Acked-by: Alexandre Belloni <alexandre.bell...@free-electrons.com>


-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


Re: [PATCH v2 0/8] ARM: at91: dts: sama5d3: add dt support for atmel isi and ov2640 sensor

2015-01-14 Thread Alexandre Belloni
Hi Nicolas,

BTW, you can add my ack on the remaining patches (5-7).

On 13/01/2015 at 16:05:57 +0100, Nicolas Ferre wrote :
 Le 04/01/2015 10:02, Josh Wu a écrit :
  This patch series add ISI and ov2640 support on dts files.
  
  As the ov2640 driver dt is still in review. The patch is in: 
  https://patchwork.linuxtv.org/patch/27554/
  So I want to send this dt patch early for a review.
  
  v1 - v2:
1. add one more patch to change the pin name of ISI_MCK
2. rewrite the commit [4/8] ARM: at91: dts: sama5d3: change name of 
  pinctrl_isi_{power,reset}.
3. move the common chip parts of ISI node to sama5d3.dtsi.
  
  Bo Shen (3):
ARM: at91: dts: sama5d3: split isi pinctrl
ARM: at91: dts: sama5d3: add missing pins of isi
ARM: at91: dts: sama5d3: move the isi mck pin to mb
  
  Josh Wu (5):
ARM: at91: dts: sama5d3: add isi clock
ARM: at91: dts: sama5d3: change name of pinctrl_isi_{power,reset}
ARM: at91: dts: sama5d3: change name of pinctrl of ISI_MCK
ARM: at91: dts: sama5d3: add ov2640 camera sensor support
ARM: at91: sama5: enable atmel-isi and ov2640 in defconfig
 
 Josh,
 
 It seems that this patch doesn't show up in the series: I only received
 up to 6/8 patches (2 missing?). Can you please send it(them?)?
 
 Bye,
 
   arch/arm/boot/dts/sama5d3.dtsi| 24 ++-
   arch/arm/boot/dts/sama5d3xmb.dtsi | 40 
  +++
   arch/arm/configs/sama5_defconfig  |  6 ++
   3 files changed, 61 insertions(+), 9 deletions(-)
  
 
 
 -- 
 Nicolas Ferre

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 6/7] ARM: at91: dts: sama5d3: add ov2640 camera sensor support

2014-12-22 Thread Alexandre Belloni
Hi Josh,

On 22/12/2014 at 15:06:07 +0800, Josh Wu wrote :
 I've acked your previous patch but maybe it should be named
 pinctrl_isi_pck1_as_mck to be clearer (you used the handle to pck1
 below).
 It's a good idea. Maybe I prefer to use the name: pinctrl_pck1_as_isi_mck ?
 If you are ok with this name, in next version, I will add one more patch in
 the series to do this.
 And I will keep your acked-by in my previous patch.
 

Sounds good to me!

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/7] ARM: at91: dts: sama5d3: add isi clock

2014-12-19 Thread Alexandre Belloni
On 18/12/2014 at 16:51:01 +0800, Josh Wu wrote :
 Add ISI peripheral clock in sama5d3.dtsi.
 
 Signed-off-by: Josh Wu josh...@atmel.com
Acked-by: Alexandre Belloni alexandre.bell...@free-electrons.com

 ---
  arch/arm/boot/dts/sama5d3.dtsi | 2 ++
  1 file changed, 2 insertions(+)
 
 diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
 index 5f4144d..61746ef 100644
 --- a/arch/arm/boot/dts/sama5d3.dtsi
 +++ b/arch/arm/boot/dts/sama5d3.dtsi
 @@ -214,6 +214,8 @@
   compatible = atmel,at91sam9g45-isi;
   reg = 0xf0034000 0x4000;
   interrupts = 37 IRQ_TYPE_LEVEL_HIGH 5;
 + clocks = isi_clk;
 + clock-names = isi_clk;
   status = disabled;
   };
  
 -- 
 1.9.1
 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 5/7] ARM: at91: dts: sama5d3: change name of pinctrl_isi_{power,reset}

2014-12-19 Thread Alexandre Belloni
Hi Josh,

On 18/12/2014 at 16:51:05 +0800, Josh Wu wrote :
 For sama5d3xmb board, the pins: pinctrl_isi_{power,reset} is used to
 power-down or reset camera sensor.
 
 So we should let camera sensor instead of ISI to configure the pins.
 This patch will change pinctrl name from pinctrl_isi_{power,reset} to
 pinctrl_sensor_{power,reset}.
 
 Signed-off-by: Josh Wu josh...@atmel.com
 ---
  arch/arm/boot/dts/sama5d3.dtsi| 2 ++
  arch/arm/boot/dts/sama5d3xmb.dtsi | 6 ++
  2 files changed, 4 insertions(+), 4 deletions(-)
 
 diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
 index ed734e9..ff0fa3a 100644
 --- a/arch/arm/boot/dts/sama5d3.dtsi
 +++ b/arch/arm/boot/dts/sama5d3.dtsi
 @@ -214,6 +214,8 @@
   compatible = atmel,at91sam9g45-isi;
   reg = 0xf0034000 0x4000;
   interrupts = 37 IRQ_TYPE_LEVEL_HIGH 5;
 + pinctrl-names = default;
 + pinctrl-0 = pinctrl_isi_data_0_7;

This should probably not be is that patch.

   clocks = isi_clk;
   clock-names = isi_clk;
   status = disabled;
 diff --git a/arch/arm/boot/dts/sama5d3xmb.dtsi 
 b/arch/arm/boot/dts/sama5d3xmb.dtsi
 index 6af1cba..0aaebc6 100644
 --- a/arch/arm/boot/dts/sama5d3xmb.dtsi
 +++ b/arch/arm/boot/dts/sama5d3xmb.dtsi
 @@ -60,8 +60,6 @@
   };
  
   isi: isi@f0034000 {
 - pinctrl-names = default;
 - pinctrl-0 = pinctrl_isi_data_0_7 
 pinctrl_isi_pck_as_mck pinctrl_isi_power pinctrl_isi_reset;
   };
  
   mmc1: mmc@f800 {
 @@ -122,12 +120,12 @@
   AT91_PIOD 31 
 AT91_PERIPH_B AT91_PINCTRL_NONE; /* PD31 periph B ISI_MCK */
   };
  
 - pinctrl_isi_reset: isi_reset-0 {
 + pinctrl_sensor_reset: sensor_reset-0 {
   atmel,pins =
   AT91_PIOE 24 
 AT91_PERIPH_GPIO AT91_PINCTRL_NONE;   /* PE24 gpio */
   };
  
 - pinctrl_isi_power: isi_power-0 {
 + pinctrl_sensor_power: sensor_power-0 {
   atmel,pins =
   AT91_PIOE 29 
 AT91_PERIPH_GPIO AT91_PINCTRL_NONE; /* PE29 gpio */
   };
 -- 
 1.9.1
 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/7] ARM: at91: dts: sama5d3: split isi pinctrl

2014-12-19 Thread Alexandre Belloni
On 18/12/2014 at 16:51:02 +0800, Josh Wu wrote :
 From: Bo Shen voice.s...@atmel.com
 
 As the ISI has 12 data lines, however we only use 8 data lines with
 sensor module. So, split the data line into two groups which make
 it can be choosed depends on the hardware design.
 
 Signed-off-by: Bo Shen voice.s...@atmel.com
 Acked-by: Nicolas Ferre nicolas.fe...@atmel.com
Acked-by: Alexandre Belloni alexandre.bell...@free-electrons.com
 Signed-off-by: Josh Wu josh...@atmel.com
 ---
  arch/arm/boot/dts/sama5d3.dtsi| 11 ---
  arch/arm/boot/dts/sama5d3xmb.dtsi |  2 +-
  2 files changed, 9 insertions(+), 4 deletions(-)
 
 diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
 index 61746ef..595609f 100644
 --- a/arch/arm/boot/dts/sama5d3.dtsi
 +++ b/arch/arm/boot/dts/sama5d3.dtsi
 @@ -547,7 +547,7 @@
   };
  
   isi {
 - pinctrl_isi: isi-0 {
 + pinctrl_isi_data_0_7: isi-0-data-0-7 {
   atmel,pins =
   AT91_PIOA 16 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PA16 periph C ISI_D0, conflicts with 
 LCDDAT16 */
AT91_PIOA 17 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PA17 periph C ISI_D1, conflicts with 
 LCDDAT17 */
 @@ -559,10 +559,15 @@
AT91_PIOA 23 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PA23 periph C ISI_D7, conflicts with 
 LCDDAT23, PWML1 */
AT91_PIOC 30 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PC30 periph C ISI_PCK, conflicts with 
 UTXD0 */
AT91_PIOA 31 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PA31 periph C ISI_HSYNC, conflicts with 
 TWCK0, UTXD1 */
 -  AT91_PIOA 30 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PA30 periph C ISI_VSYNC, conflicts with 
 TWD0, URXD1 */
 -  AT91_PIOC 29 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PC29 periph C ISI_PD8, conflicts with 
 URXD0, PWMFI2 */
 +  AT91_PIOA 30 
 AT91_PERIPH_C AT91_PINCTRL_NONE; /* PA30 periph C ISI_VSYNC, conflicts with 
 TWD0, URXD1 */
 + };
 +
 + pinctrl_isi_data_8_9: isi-0-data-8-9 {
 + atmel,pins =
 + AT91_PIOC 29 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PC29 periph C ISI_PD8, conflicts with 
 URXD0, PWMFI2 */
AT91_PIOC 28 
 AT91_PERIPH_C AT91_PINCTRL_NONE; /* PC28 periph C ISI_PD9, conflicts with 
 SPI1_NPCS3, PWMFI0 */
   };
 +
   pinctrl_isi_pck_as_mck: 
 isi_pck_as_mck-0 {
   atmel,pins =
   AT91_PIOD 31 
 AT91_PERIPH_B AT91_PINCTRL_NONE; /* PD31 periph B ISI_MCK */
 diff --git a/arch/arm/boot/dts/sama5d3xmb.dtsi 
 b/arch/arm/boot/dts/sama5d3xmb.dtsi
 index 49c10d3..2530541 100644
 --- a/arch/arm/boot/dts/sama5d3xmb.dtsi
 +++ b/arch/arm/boot/dts/sama5d3xmb.dtsi
 @@ -61,7 +61,7 @@
  
   isi: isi@f0034000 {
   pinctrl-names = default;
 - pinctrl-0 = pinctrl_isi 
 pinctrl_isi_pck_as_mck pinctrl_isi_power pinctrl_isi_reset;
 + pinctrl-0 = pinctrl_isi_data_0_7 
 pinctrl_isi_pck_as_mck pinctrl_isi_power pinctrl_isi_reset;
   };
  
   mmc1: mmc@f800 {
 -- 
 1.9.1
 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 3/7] ARM: at91: dts: sama5d3: add missing pins of isi

2014-12-19 Thread Alexandre Belloni
On 18/12/2014 at 16:51:03 +0800, Josh Wu wrote :
 From: Bo Shen voice.s...@atmel.com
 
 The ISI has 12 data lines, add the missing two data lines.
 
 Signed-off-by: Bo Shen voice.s...@atmel.com
 Acked-by: Nicolas Ferre nicolas.fe...@atmel.com
Acked-by: Alexandre Belloni alexandre.bell...@free-electrons.com

 ---
  arch/arm/boot/dts/sama5d3.dtsi | 6 ++
  1 file changed, 6 insertions(+)
 
 diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
 index 595609f..b3ac156 100644
 --- a/arch/arm/boot/dts/sama5d3.dtsi
 +++ b/arch/arm/boot/dts/sama5d3.dtsi
 @@ -568,6 +568,12 @@
AT91_PIOC 28 
 AT91_PERIPH_C AT91_PINCTRL_NONE; /* PC28 periph C ISI_PD9, conflicts with 
 SPI1_NPCS3, PWMFI0 */
   };
  
 + pinctrl_isi_data_10_11: 
 isi-0-data-10-11 {
 + atmel,pins =
 + AT91_PIOC 27 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PC27 periph C ISI_PD10, conflicts with 
 SPI1_NPCS2, TWCK1 */
 +  AT91_PIOC 26 
 AT91_PERIPH_C AT91_PINCTRL_NONE; /* PC26 periph C ISI_PD11, conflicts with 
 SPI1_NPCS1, TWD1 */
 + };
 +
   pinctrl_isi_pck_as_mck: 
 isi_pck_as_mck-0 {
   atmel,pins =
   AT91_PIOD 31 
 AT91_PERIPH_B AT91_PINCTRL_NONE; /* PD31 periph B ISI_MCK */
 -- 
 1.9.1
 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 4/7] ARM: at91: dts: sama5d3: move the isi mck pin to mb

2014-12-19 Thread Alexandre Belloni
On 18/12/2014 at 16:51:04 +0800, Josh Wu wrote :
 From: Bo Shen voice.s...@atmel.com
 
 The mck is decided by the board design, move it to mb related
 dtsi file.
 
 Signed-off-by: Bo Shen voice.s...@atmel.com
 Acked-by: Nicolas Ferre nicolas.fe...@atmel.com
Acked-by: Alexandre Belloni alexandre.bell...@free-electrons.com

 Signed-off-by: Josh Wu josh...@atmel.com
 ---
  arch/arm/boot/dts/sama5d3.dtsi| 5 -
  arch/arm/boot/dts/sama5d3xmb.dtsi | 5 +
  2 files changed, 5 insertions(+), 5 deletions(-)
 
 diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
 index b3ac156..ed734e9 100644
 --- a/arch/arm/boot/dts/sama5d3.dtsi
 +++ b/arch/arm/boot/dts/sama5d3.dtsi
 @@ -573,11 +573,6 @@
   AT91_PIOC 27 
 AT91_PERIPH_C AT91_PINCTRL_NONE   /* PC27 periph C ISI_PD10, conflicts with 
 SPI1_NPCS2, TWCK1 */
AT91_PIOC 26 
 AT91_PERIPH_C AT91_PINCTRL_NONE; /* PC26 periph C ISI_PD11, conflicts with 
 SPI1_NPCS1, TWD1 */
   };
 -
 - pinctrl_isi_pck_as_mck: 
 isi_pck_as_mck-0 {
 - atmel,pins =
 - AT91_PIOD 31 
 AT91_PERIPH_B AT91_PINCTRL_NONE; /* PD31 periph B ISI_MCK */
 - };
   };
  
   mmc0 {
 diff --git a/arch/arm/boot/dts/sama5d3xmb.dtsi 
 b/arch/arm/boot/dts/sama5d3xmb.dtsi
 index 2530541..6af1cba 100644
 --- a/arch/arm/boot/dts/sama5d3xmb.dtsi
 +++ b/arch/arm/boot/dts/sama5d3xmb.dtsi
 @@ -117,6 +117,11 @@
   AT91_PIOD 30 
 AT91_PERIPH_B AT91_PINCTRL_NONE; /* PD30 periph B */
   };
  
 + pinctrl_isi_pck_as_mck: 
 isi_pck_as_mck-0 {
 + atmel,pins =
 + AT91_PIOD 31 
 AT91_PERIPH_B AT91_PINCTRL_NONE; /* PD31 periph B ISI_MCK */
 + };
 +
   pinctrl_isi_reset: isi_reset-0 {
   atmel,pins =
   AT91_PIOE 24 
 AT91_PERIPH_GPIO AT91_PINCTRL_NONE;   /* PE24 gpio */
 -- 
 1.9.1
 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 6/7] ARM: at91: dts: sama5d3: add ov2640 camera sensor support

2014-12-19 Thread Alexandre Belloni
On 18/12/2014 at 16:51:06 +0800, Josh Wu wrote :
 According to v4l2 dt document, we add:
   a camera host: ISI port.
   a i2c camera sensor: ov2640 port.
 to sama5d3xmb.dtsi.
 
 In the ov2640 node, it defines the pinctrls, clocks and isi port.
 In the ISI node, it also reference to a ov2640 port.
 
 Signed-off-by: Josh Wu josh...@atmel.com
 ---
  arch/arm/boot/dts/sama5d3xmb.dtsi | 32 
  1 file changed, 32 insertions(+)
 
 diff --git a/arch/arm/boot/dts/sama5d3xmb.dtsi 
 b/arch/arm/boot/dts/sama5d3xmb.dtsi
 index 0aaebc6..958a528 100644
 --- a/arch/arm/boot/dts/sama5d3xmb.dtsi
 +++ b/arch/arm/boot/dts/sama5d3xmb.dtsi
 @@ -52,6 +52,29 @@
   };
   };
  
 + i2c1: i2c@f0018000 {
 + ov2640: camera@0x30 {
 + compatible = ovti,ov2640;
 + reg = 0x30;
 + pinctrl-names = default;
 + pinctrl-0 = pinctrl_isi_pck_as_mck 
 pinctrl_sensor_power pinctrl_sensor_reset;

I've acked your previous patch but maybe it should be named
pinctrl_isi_pck1_as_mck to be clearer (you used the handle to pck1
below).

 + resetb-gpios = pioE 24 
 GPIO_ACTIVE_LOW;
 + pwdn-gpios = pioE 29 
 GPIO_ACTIVE_HIGH;
 + /* use pck1 for the master clock of 
 ov2640 */
 + clocks = pck1;
 + clock-names = xvclk;
 + assigned-clocks = pck1;
 + assigned-clock-rates = 2500;
 +
 + port {
 + ov2640_0: endpoint {
 + remote-endpoint = 
 isi_0;
 + bus-width = 8;
 + };
 + };
 + };
 + };
 +
   usart1: serial@f002 {
   dmas = 0, 0;/*  Do not use DMA for 
 usart1 */
   pinctrl-names = default;
 @@ -60,6 +83,15 @@
   };
  
   isi: isi@f0034000 {
 + port {
 + #address-cells = 1;
 + #size-cells = 0;
 +
 + isi_0: endpoint {
 + remote-endpoint = ov2640_0;
 + bus-width = 8;
 + };
 + };
   };
  
   mmc1: mmc@f800 {
 -- 
 1.9.1
 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 7/7] ARM: at91: sama5: enable atmel-isi and ov2640 in defconfig

2014-12-19 Thread Alexandre Belloni
On 18/12/2014 at 16:51:07 +0800, Josh Wu wrote :
 Signed-off-by: Josh Wu josh...@atmel.com
Acked-by: Alexandre Belloni alexandre.bell...@free-electrons.com
 ---
  arch/arm/configs/sama5_defconfig | 6 ++
  1 file changed, 6 insertions(+)
 
 diff --git a/arch/arm/configs/sama5_defconfig 
 b/arch/arm/configs/sama5_defconfig
 index b58fb32..92f1d71 100644
 --- a/arch/arm/configs/sama5_defconfig
 +++ b/arch/arm/configs/sama5_defconfig
 @@ -139,6 +139,12 @@ CONFIG_POWER_RESET=y
  CONFIG_SSB=m
  CONFIG_REGULATOR=y
  CONFIG_REGULATOR_ACT8865=y
 +CONFIG_MEDIA_SUPPORT=y
 +CONFIG_MEDIA_CAMERA_SUPPORT=y
 +CONFIG_V4L_PLATFORM_DRIVERS=y
 +CONFIG_SOC_CAMERA=y
 +CONFIG_SOC_CAMERA_OV2640=y
 +CONFIG_VIDEO_ATMEL_ISI=y
  CONFIG_FB=y
  CONFIG_BACKLIGHT_LCD_SUPPORT=y
  # CONFIG_LCD_CLASS_DEVICE is not set
 -- 
 1.9.1
 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html