| Message ID | 20191211140720.10539-1-bparrot@ti.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | ARM: dts: am43x-vpfe/ov2659.patch | expand |
* Benoit Parrot <bparrot@ti.com> [191211 06:04]: > This patch series adds the missing camera endpoint (ov2659) as well as > the required source clocks nodes for the sensor. > > On the am437x-sk-evm the camera sensor is sourced from clkout1 but that > clock nodes/tree was removed as it was unsed at the time, we are > re-adding the needed clock nodes here. Tero, it seems I can already pick this series? Or ou want to queue the changes to am43xx-clocks.dtsi along with all your other clock patches? Regards, Tony > Changes since v2: > - Fixed/added Tony's ack > > Changes since v1: > - Fix clock name to make it generic > - Add non-standard clock node naming to commit message as per Tony's > comment > - Rename all clock nodes to use '-' instead of '_' > > Benoit Parrot (2): > ARM: dts: am437x-sk-evm: Add VPFE and OV2659 entries > ARM: dts: am43x-epos-evm: Add VPFE and OV2659 entries > > Tero Kristo (1): > ARM: dts: am43xx: add support for clkout1 clock > > arch/arm/boot/dts/am437x-sk-evm.dts | 27 +++++++++++++- > arch/arm/boot/dts/am43x-epos-evm.dts | 23 +++++++++++- > arch/arm/boot/dts/am43xx-clocks.dtsi | 54 ++++++++++++++++++++++++++++ > 3 files changed, 102 insertions(+), 2 deletions(-) > > -- > 2.17.1 >
On 12/12/2019 19:41, Tony Lindgren wrote: > * Benoit Parrot <bparrot@ti.com> [191211 06:04]: >> This patch series adds the missing camera endpoint (ov2659) as well as >> the required source clocks nodes for the sensor. >> >> On the am437x-sk-evm the camera sensor is sourced from clkout1 but that >> clock nodes/tree was removed as it was unsed at the time, we are >> re-adding the needed clock nodes here. > > Tero, it seems I can already pick this series? I believe it is ready if you approve the clkout1 clock patch. > Or ou want to queue the changes to am43xx-clocks.dtsi along with all > your other clock patches? Well, I have actually never queued any omap2+ dts patches myself, and I don't think there would be too many of those coming for next merge either. -Tero > > Regards, > > Tony > > >> Changes since v2: >> - Fixed/added Tony's ack >> >> Changes since v1: >> - Fix clock name to make it generic >> - Add non-standard clock node naming to commit message as per Tony's >> comment >> - Rename all clock nodes to use '-' instead of '_' >> >> Benoit Parrot (2): >> ARM: dts: am437x-sk-evm: Add VPFE and OV2659 entries >> ARM: dts: am43x-epos-evm: Add VPFE and OV2659 entries >> >> Tero Kristo (1): >> ARM: dts: am43xx: add support for clkout1 clock >> >> arch/arm/boot/dts/am437x-sk-evm.dts | 27 +++++++++++++- >> arch/arm/boot/dts/am43x-epos-evm.dts | 23 +++++++++++- >> arch/arm/boot/dts/am43xx-clocks.dtsi | 54 ++++++++++++++++++++++++++++ >> 3 files changed, 102 insertions(+), 2 deletions(-) >> >> -- >> 2.17.1 >> -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
* Tero Kristo <t-kristo@ti.com> [191213 07:43]: > On 12/12/2019 19:41, Tony Lindgren wrote: > > * Benoit Parrot <bparrot@ti.com> [191211 06:04]: > > > This patch series adds the missing camera endpoint (ov2659) as well as > > > the required source clocks nodes for the sensor. > > > > > > On the am437x-sk-evm the camera sensor is sourced from clkout1 but that > > > clock nodes/tree was removed as it was unsed at the time, we are > > > re-adding the needed clock nodes here. > > > > Tero, it seems I can already pick this series? > > I believe it is ready if you approve the clkout1 clock patch. OK yeah looks fine. > > Or ou want to queue the changes to am43xx-clocks.dtsi along with all > > your other clock patches? > > Well, I have actually never queued any omap2+ dts patches myself, and I > don't think there would be too many of those coming for next merge either. OK will queue this series then. For the other ones from Benoit looks like we need an immutable clock branch before I can apply anything. Regards, Tony
Tony Lindgren <tony@atomide.com> wrote on Fri [2019-Dec-13 07:29:38 -0800]: > * Tero Kristo <t-kristo@ti.com> [191213 07:43]: > > On 12/12/2019 19:41, Tony Lindgren wrote: > > > * Benoit Parrot <bparrot@ti.com> [191211 06:04]: > > > > This patch series adds the missing camera endpoint (ov2659) as well as > > > > the required source clocks nodes for the sensor. > > > > > > > > On the am437x-sk-evm the camera sensor is sourced from clkout1 but that > > > > clock nodes/tree was removed as it was unsed at the time, we are > > > > re-adding the needed clock nodes here. > > > > > > Tero, it seems I can already pick this series? > > > > I believe it is ready if you approve the clkout1 clock patch. > > OK yeah looks fine. > > > > Or ou want to queue the changes to am43xx-clocks.dtsi along with all > > > your other clock patches? > > > > Well, I have actually never queued any omap2+ dts patches myself, and I > > don't think there would be too many of those coming for next merge either. > > OK will queue this series then. For the other ones from Benoit > looks like we need an immutable clock branch before I can apply > anything. Tony, Tero, Are these merged anyware now? I still don't see any of these on linux-next? And by "these" I mean this one and both ARM: dts: dra7: add vpe nodes ARM: dts: dra7: add cal nodes Regards, Benoit > > Regards, > > Tony
* Benoit Parrot <bparrot@ti.com> [200113 16:51]: > Tony Lindgren <tony@atomide.com> wrote on Fri [2019-Dec-13 07:29:38 -0800]: > > * Tero Kristo <t-kristo@ti.com> [191213 07:43]: > > > On 12/12/2019 19:41, Tony Lindgren wrote: > > > > * Benoit Parrot <bparrot@ti.com> [191211 06:04]: > > > > > This patch series adds the missing camera endpoint (ov2659) as well as > > > > > the required source clocks nodes for the sensor. > > > > > > > > > > On the am437x-sk-evm the camera sensor is sourced from clkout1 but that > > > > > clock nodes/tree was removed as it was unsed at the time, we are > > > > > re-adding the needed clock nodes here. > > > > > > > > Tero, it seems I can already pick this series? > > > > > > I believe it is ready if you approve the clkout1 clock patch. > > > > OK yeah looks fine. > > > > > > Or ou want to queue the changes to am43xx-clocks.dtsi along with all > > > > your other clock patches? > > > > > > Well, I have actually never queued any omap2+ dts patches myself, and I > > > don't think there would be too many of those coming for next merge either. > > > > OK will queue this series then. For the other ones from Benoit > > looks like we need an immutable clock branch before I can apply > > anything. > > Tony, Tero, > > Are these merged anyware now? > I still don't see any of these on linux-next? Tero any news on getting some immutable clock changes branch done? Looks like there are quite a few pending clock patches right now, probably best to set them all into a branch that I can also merge in. > And by "these" I mean this one and both > ARM: dts: dra7: add vpe nodes > ARM: dts: dra7: add cal nodes Yeah looks good to me other than the clock dependency. Regards, Tony
On 14/01/2020 18:04, Tony Lindgren wrote: > * Benoit Parrot <bparrot@ti.com> [200113 16:51]: >> Tony Lindgren <tony@atomide.com> wrote on Fri [2019-Dec-13 07:29:38 -0800]: >>> * Tero Kristo <t-kristo@ti.com> [191213 07:43]: >>>> On 12/12/2019 19:41, Tony Lindgren wrote: >>>>> * Benoit Parrot <bparrot@ti.com> [191211 06:04]: >>>>>> This patch series adds the missing camera endpoint (ov2659) as well as >>>>>> the required source clocks nodes for the sensor. >>>>>> >>>>>> On the am437x-sk-evm the camera sensor is sourced from clkout1 but that >>>>>> clock nodes/tree was removed as it was unsed at the time, we are >>>>>> re-adding the needed clock nodes here. >>>>> >>>>> Tero, it seems I can already pick this series? >>>> >>>> I believe it is ready if you approve the clkout1 clock patch. >>> >>> OK yeah looks fine. >>> >>>>> Or ou want to queue the changes to am43xx-clocks.dtsi along with all >>>>> your other clock patches? >>>> >>>> Well, I have actually never queued any omap2+ dts patches myself, and I >>>> don't think there would be too many of those coming for next merge either. >>> >>> OK will queue this series then. For the other ones from Benoit >>> looks like we need an immutable clock branch before I can apply >>> anything. >> >> Tony, Tero, >> >> Are these merged anyware now? >> I still don't see any of these on linux-next? > > Tero any news on getting some immutable clock changes branch done? > Looks like there are quite a few pending clock patches right now, > probably best to set them all into a branch that I can also merge > in. Working on that now, have been forced to prioritize work lately. I can probably get the branch set-up on Monday and send a pull-req out assuming it is not too late for 5.6 already. -Tero > >> And by "these" I mean this one and both >> ARM: dts: dra7: add vpe nodes >> ARM: dts: dra7: add cal nodes > > Yeah looks good to me other than the clock dependency. > > Regards, > > Tony > -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
* Tero Kristo <t-kristo@ti.com> [200117 14:16]: > On 14/01/2020 18:04, Tony Lindgren wrote: > > * Benoit Parrot <bparrot@ti.com> [200113 16:51]: > > > Tony Lindgren <tony@atomide.com> wrote on Fri [2019-Dec-13 07:29:38 -0800]: > > > > * Tero Kristo <t-kristo@ti.com> [191213 07:43]: > > > > > On 12/12/2019 19:41, Tony Lindgren wrote: > > > > > > * Benoit Parrot <bparrot@ti.com> [191211 06:04]: > > > > > > > This patch series adds the missing camera endpoint (ov2659) as well as > > > > > > > the required source clocks nodes for the sensor. > > > > > > > > > > > > > > On the am437x-sk-evm the camera sensor is sourced from clkout1 but that > > > > > > > clock nodes/tree was removed as it was unsed at the time, we are > > > > > > > re-adding the needed clock nodes here. > > > > > > > > > > > > Tero, it seems I can already pick this series? > > > > > > > > > > I believe it is ready if you approve the clkout1 clock patch. > > > > > > > > OK yeah looks fine. > > > > > > > > > > Or ou want to queue the changes to am43xx-clocks.dtsi along with all > > > > > > your other clock patches? > > > > > > > > > > Well, I have actually never queued any omap2+ dts patches myself, and I > > > > > don't think there would be too many of those coming for next merge either. > > > > > > > > OK will queue this series then. For the other ones from Benoit > > > > looks like we need an immutable clock branch before I can apply > > > > anything. > > > > > > Tony, Tero, > > > > > > Are these merged anyware now? > > > I still don't see any of these on linux-next? > > > > Tero any news on getting some immutable clock changes branch done? > > Looks like there are quite a few pending clock patches right now, > > probably best to set them all into a branch that I can also merge > > in. > > Working on that now, have been forced to prioritize work lately. I can > probably get the branch set-up on Monday and send a pull-req out assuming it > is not too late for 5.6 already. Yes it might be too late, but I'm applying these too into omap-for-v5.6/ti-sysc-dt-cam. Regards, Tony