2021-11-24 18:10:48

by Ivan Bornyakov

Subject: [PATCH v2 0/2] bus: imx-weim: optionally enable continuous burst clock

Introduce option to enable continuous burst clock, if burst clock itself
is enabled.

Changelog:
v1 -> v2:
* documentation about this option added to
Documentation/devicetree/bindings/bus/imx-weim.txt

Ivan Bornyakov (2):
bus: imx-weim: optionally enable continuous burst clock
dt-bindings: bus: imx-weim: add words about continuous bclk

.../devicetree/bindings/bus/imx-weim.txt | 5 +++++
drivers/bus/imx-weim.c | 18 ++++++++++++++++--
2 files changed, 21 insertions(+), 2 deletions(-)

--
2.32.0

2021-11-24 18:10:55

by Ivan Bornyakov

Subject: [PATCH v2 2/2] dt-bindings: bus: imx-weim: add words about continuous bclk

Document continuous Burst Clock option. With this option Burst Clock, if
enabled, will output continuous clock, otherwise Burst Clock will output
clock only when necessary.
---
Documentation/devicetree/bindings/bus/imx-weim.txt | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/bus/imx-weim.txt b/Documentation/devicetree/bindings/bus/imx-weim.txt
index 1b1d1c5c21ea..e7f502070d77 100644
--- a/Documentation/devicetree/bindings/bus/imx-weim.txt
+++ b/Documentation/devicetree/bindings/bus/imx-weim.txt
@@ -48,6 +48,11 @@ Optional properties:
devices, the presence of this property indicates that
the weim bus should operate in Burst Clock Mode.

+ - fsl,continuous-burst-clk Make Burst Clock to output continuous clock.
+ Without this option Burst Clock will output clock
+ only when necessary. This takes effect only if
+ "fsl,burst-clk-enable" is set.
+
Timing property for child nodes. It is mandatory, not optional.

- fsl,weim-cs-timing: The timing array, contains timing values for the
--
2.32.0

2021-12-01 10:25:51

by Ivan Bornyakov

Subject: Re: [PATCH v2 0/2] bus: imx-weim: optionally enable continuous burst clock

On Wed, Nov 24, 2021 at 08:55:40PM +0300, Ivan Bornyakov wrote:
> Introduce option to enable continuous burst clock, if burst clock itself
> is enabled.
>
> Changelog:
> v1 -> v2:
> * documentation about this option added to
> Documentation/devicetree/bindings/bus/imx-weim.txt
>
> Ivan Bornyakov (2):
> bus: imx-weim: optionally enable continuous burst clock
> dt-bindings: bus: imx-weim: add words about continuous bclk
>
> .../devicetree/bindings/bus/imx-weim.txt | 5 +++++
> drivers/bus/imx-weim.c | 18 ++++++++++++++++--
> 2 files changed, 21 insertions(+), 2 deletions(-)
>
> --
> 2.32.0
>

Friendly ping.
Any feedback?

2021-12-01 22:18:42

by Rob Herring (Arm)

Subject: Re: [PATCH v2 2/2] dt-bindings: bus: imx-weim: add words about continuous bclk

On Wed, Nov 24, 2021 at 08:55:42PM +0300, Ivan Bornyakov wrote:
> Document continuous Burst Clock option. With this option Burst Clock, if
> enabled, will output continuous clock, otherwise Burst Clock will output
> clock only when necessary.

Missing S-o-b. checkpatch.pl will tell you this.

> ---
> Documentation/devicetree/bindings/bus/imx-weim.txt | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/bus/imx-weim.txt b/Documentation/devicetree/bindings/bus/imx-weim.txt
> index 1b1d1c5c21ea..e7f502070d77 100644
> --- a/Documentation/devicetree/bindings/bus/imx-weim.txt
> +++ b/Documentation/devicetree/bindings/bus/imx-weim.txt
> @@ -48,6 +48,11 @@ Optional properties:
> devices, the presence of this property indicates that
> the weim bus should operate in Burst Clock Mode.
>
> + - fsl,continuous-burst-clk Make Burst Clock to output continuous clock.
> + Without this option Burst Clock will output clock
> + only when necessary. This takes effect only if
> + "fsl,burst-clk-enable" is set.
> +
> Timing property for child nodes. It is mandatory, not optional.
>
> - fsl,weim-cs-timing: The timing array, contains timing values for the
> --
> 2.32.0
>
>
>