| Message ID | 20220615221410.27459-7-laurent.pinchart@ideasonboard.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | dt-bindings: Add macros for video interface bus types | expand |
Hugues, Maxime, Alexandre, could one of you take this patch in your tree for v6.3 ? The two patches that it depends on have both been merged in v6.2. On Thu, Jun 16, 2022 at 01:14:10AM +0300, Laurent Pinchart wrote: > Now that a header exists with macros for the media interface bus-type > values, replace hardcoding numerical constants with the corresponding > macros in the DT sources. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > arch/arm/boot/dts/stm32429i-eval.dts | 3 ++- > arch/arm/boot/dts/stm32mp157c-ev1.dts | 3 ++- > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/stm32429i-eval.dts b/arch/arm/boot/dts/stm32429i-eval.dts > index 0d98aca01736..5fae11e6607b 100644 > --- a/arch/arm/boot/dts/stm32429i-eval.dts > +++ b/arch/arm/boot/dts/stm32429i-eval.dts > @@ -50,6 +50,7 @@ > #include "stm32f429-pinctrl.dtsi" > #include <dt-bindings/input/input.h> > #include <dt-bindings/gpio/gpio.h> > +#include <dt-bindings/media/video-interfaces.h> > > / { > model = "STMicroelectronics STM32429i-EVAL board"; > @@ -186,7 +187,7 @@ &dcmi { > port { > dcmi_0: endpoint { > remote-endpoint = <&ov2640_0>; > - bus-type = <5>; > + bus-type = <MEDIA_BUS_TYPE_PARALLEL>; > bus-width = <8>; > hsync-active = <0>; > vsync-active = <0>; > diff --git a/arch/arm/boot/dts/stm32mp157c-ev1.dts b/arch/arm/boot/dts/stm32mp157c-ev1.dts > index d142dd30e16b..306d41a6138f 100644 > --- a/arch/arm/boot/dts/stm32mp157c-ev1.dts > +++ b/arch/arm/boot/dts/stm32mp157c-ev1.dts > @@ -8,6 +8,7 @@ > #include "stm32mp157c-ed1.dts" > #include <dt-bindings/gpio/gpio.h> > #include <dt-bindings/input/input.h> > +#include <dt-bindings/media/video-interfaces.h> > > / { > model = "STMicroelectronics STM32MP157C eval daughter on eval mother"; > @@ -90,7 +91,7 @@ &dcmi { > port { > dcmi_0: endpoint { > remote-endpoint = <&ov5640_0>; > - bus-type = <5>; > + bus-type = <MEDIA_BUS_TYPE_PARALLEL>; > bus-width = <8>; > hsync-active = <0>; > vsync-active = <0>;
Hi Laurent On 1/14/23 16:36, Laurent Pinchart wrote: > Hugues, Maxime, Alexandre, could one of you take this patch in your tree > for v6.3 ? The two patches that it depends on have both been merged in > v6.2. Yes sure. Cheers Alex > > On Thu, Jun 16, 2022 at 01:14:10AM +0300, Laurent Pinchart wrote: >> Now that a header exists with macros for the media interface bus-type >> values, replace hardcoding numerical constants with the corresponding >> macros in the DT sources. >> >> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> >> --- >> arch/arm/boot/dts/stm32429i-eval.dts | 3 ++- >> arch/arm/boot/dts/stm32mp157c-ev1.dts | 3 ++- >> 2 files changed, 4 insertions(+), 2 deletions(-) >> >> diff --git a/arch/arm/boot/dts/stm32429i-eval.dts b/arch/arm/boot/dts/stm32429i-eval.dts >> index 0d98aca01736..5fae11e6607b 100644 >> --- a/arch/arm/boot/dts/stm32429i-eval.dts >> +++ b/arch/arm/boot/dts/stm32429i-eval.dts >> @@ -50,6 +50,7 @@ >> #include "stm32f429-pinctrl.dtsi" >> #include <dt-bindings/input/input.h> >> #include <dt-bindings/gpio/gpio.h> >> +#include <dt-bindings/media/video-interfaces.h> >> >> / { >> model = "STMicroelectronics STM32429i-EVAL board"; >> @@ -186,7 +187,7 @@ &dcmi { >> port { >> dcmi_0: endpoint { >> remote-endpoint = <&ov2640_0>; >> - bus-type = <5>; >> + bus-type = <MEDIA_BUS_TYPE_PARALLEL>; >> bus-width = <8>; >> hsync-active = <0>; >> vsync-active = <0>; >> diff --git a/arch/arm/boot/dts/stm32mp157c-ev1.dts b/arch/arm/boot/dts/stm32mp157c-ev1.dts >> index d142dd30e16b..306d41a6138f 100644 >> --- a/arch/arm/boot/dts/stm32mp157c-ev1.dts >> +++ b/arch/arm/boot/dts/stm32mp157c-ev1.dts >> @@ -8,6 +8,7 @@ >> #include "stm32mp157c-ed1.dts" >> #include <dt-bindings/gpio/gpio.h> >> #include <dt-bindings/input/input.h> >> +#include <dt-bindings/media/video-interfaces.h> >> >> / { >> model = "STMicroelectronics STM32MP157C eval daughter on eval mother"; >> @@ -90,7 +91,7 @@ &dcmi { >> port { >> dcmi_0: endpoint { >> remote-endpoint = <&ov5640_0>; >> - bus-type = <5>; >> + bus-type = <MEDIA_BUS_TYPE_PARALLEL>; >> bus-width = <8>; >> hsync-active = <0>; >> vsync-active = <0>; >
Hi Laurent On 6/16/22 00:14, Laurent Pinchart wrote: > Now that a header exists with macros for the media interface bus-type > values, replace hardcoding numerical constants with the corresponding > macros in the DT sources. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Applied on stm32-next. Thanks. Alex
diff --git a/arch/arm/boot/dts/stm32429i-eval.dts b/arch/arm/boot/dts/stm32429i-eval.dts index 0d98aca01736..5fae11e6607b 100644 --- a/arch/arm/boot/dts/stm32429i-eval.dts +++ b/arch/arm/boot/dts/stm32429i-eval.dts @@ -50,6 +50,7 @@ #include "stm32f429-pinctrl.dtsi" #include <dt-bindings/input/input.h> #include <dt-bindings/gpio/gpio.h> +#include <dt-bindings/media/video-interfaces.h> / { model = "STMicroelectronics STM32429i-EVAL board"; @@ -186,7 +187,7 @@ &dcmi { port { dcmi_0: endpoint { remote-endpoint = <&ov2640_0>; - bus-type = <5>; + bus-type = <MEDIA_BUS_TYPE_PARALLEL>; bus-width = <8>; hsync-active = <0>; vsync-active = <0>; diff --git a/arch/arm/boot/dts/stm32mp157c-ev1.dts b/arch/arm/boot/dts/stm32mp157c-ev1.dts index d142dd30e16b..306d41a6138f 100644 --- a/arch/arm/boot/dts/stm32mp157c-ev1.dts +++ b/arch/arm/boot/dts/stm32mp157c-ev1.dts @@ -8,6 +8,7 @@ #include "stm32mp157c-ed1.dts" #include <dt-bindings/gpio/gpio.h> #include <dt-bindings/input/input.h> +#include <dt-bindings/media/video-interfaces.h> / { model = "STMicroelectronics STM32MP157C eval daughter on eval mother"; @@ -90,7 +91,7 @@ &dcmi { port { dcmi_0: endpoint { remote-endpoint = <&ov5640_0>; - bus-type = <5>; + bus-type = <MEDIA_BUS_TYPE_PARALLEL>; bus-width = <8>; hsync-active = <0>; vsync-active = <0>;
Now that a header exists with macros for the media interface bus-type values, replace hardcoding numerical constants with the corresponding macros in the DT sources. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- arch/arm/boot/dts/stm32429i-eval.dts | 3 ++- arch/arm/boot/dts/stm32mp157c-ev1.dts | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-)