| Message ID | 1597406966-13740-17-git-send-email-abel.vesa@nxp.com (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | Add BLK_CTRL support for i.MX8MP | expand |
On Fri, Aug 14, 2020 at 8:12 PM Abel Vesa <abel.vesa@nxp.com> wrote: > > Some of the features of the media_ctrl will be used by some > different drivers in a way those drivers will know best, so adding the > syscon compatible we allow those to do just that. Only the resets > and the clocks are registered bit the clk-blk-ctrl driver. > > Signed-off-by: Abel Vesa <abel.vesa@nxp.com> > --- > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > index dede0ae..2d6d213 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > @@ -736,6 +736,22 @@ > }; > }; > > + aips4: bus@32c00000 { > + compatible = "simple-bus"; > + reg = <0x32c00000 0x400000>; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges; > + > + media_blk_ctrl: clock-controller@32ec0000 { For this combo device, maybe we can directly name it as blk-ctrl@32ec0000. Rob, do you think if we can do that? > + compatible = "fsl,imx8mp-media-blk-ctrl", "syscon"; > + reg = <0x32ec0000 0x10000>; > + Remove unnecessary blank line Otherwise: Reviewed-by: Dong Aisheng <aisheng.dong@nxp.com> Regards Aisheng > + #clock-cells = <1>; > + #reset-cells = <1>; > + }; > + }; > + > aips5: bus@30c00000 { > compatible = "fsl,aips-bus", "simple-bus"; > reg = <0x30c00000 0x400000>; > -- > 2.7.4 >
On 20-08-18 19:34:14, Dong Aisheng wrote: > On Fri, Aug 14, 2020 at 8:12 PM Abel Vesa <abel.vesa@nxp.com> wrote: > > > > Some of the features of the media_ctrl will be used by some > > different drivers in a way those drivers will know best, so adding the > > syscon compatible we allow those to do just that. Only the resets > > and the clocks are registered bit the clk-blk-ctrl driver. > > > > Signed-off-by: Abel Vesa <abel.vesa@nxp.com> > > --- > > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++++++++++ > > 1 file changed, 16 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > index dede0ae..2d6d213 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > @@ -736,6 +736,22 @@ > > }; > > }; > > > > + aips4: bus@32c00000 { > > + compatible = "simple-bus"; > > + reg = <0x32c00000 0x400000>; > > + #address-cells = <1>; > > + #size-cells = <1>; > > + ranges; > > + > > + media_blk_ctrl: clock-controller@32ec0000 { > > For this combo device, maybe we can directly name it as blk-ctrl@32ec0000. > Rob, do you think if we can do that? > I think it was Stephen who suggested we change it to clock-controller in the last's version thread. TBH, I agree with you here, since it makes more sense to be called blk-ctrl provided that this is not really just a clock controller. > > + compatible = "fsl,imx8mp-media-blk-ctrl", "syscon"; > > + reg = <0x32ec0000 0x10000>; > > + > > Remove unnecessary blank line > Will do. > Otherwise: > Reviewed-by: Dong Aisheng <aisheng.dong@nxp.com> > > Regards > Aisheng > > > + #clock-cells = <1>; > > + #reset-cells = <1>; > > + }; > > + }; > > + > > aips5: bus@30c00000 { > > compatible = "fsl,aips-bus", "simple-bus"; > > reg = <0x30c00000 0x400000>; > > -- > > 2.7.4 > >
Hi Rob, Stephen, On Thu, Aug 20, 2020 at 4:37 AM Abel Vesa <abel.vesa@nxp.com> wrote: > > On 20-08-18 19:34:14, Dong Aisheng wrote: > > On Fri, Aug 14, 2020 at 8:12 PM Abel Vesa <abel.vesa@nxp.com> wrote: > > > > > > Some of the features of the media_ctrl will be used by some > > > different drivers in a way those drivers will know best, so adding the > > > syscon compatible we allow those to do just that. Only the resets > > > and the clocks are registered bit the clk-blk-ctrl driver. > > > > > > Signed-off-by: Abel Vesa <abel.vesa@nxp.com> > > > --- > > > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++++++++++ > > > 1 file changed, 16 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > > index dede0ae..2d6d213 100644 > > > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > > @@ -736,6 +736,22 @@ > > > }; > > > }; > > > > > > + aips4: bus@32c00000 { > > > + compatible = "simple-bus"; > > > + reg = <0x32c00000 0x400000>; > > > + #address-cells = <1>; > > > + #size-cells = <1>; > > > + ranges; > > > + > > > + media_blk_ctrl: clock-controller@32ec0000 { > > > > For this combo device, maybe we can directly name it as blk-ctrl@32ec0000. > > Rob, do you think if we can do that? > > > > I think it was Stephen who suggested we change it to clock-controller in the > last's version thread. > > TBH, I agree with you here, since it makes more sense to be called blk-ctrl > provided that this is not really just a clock controller. > How do you think? Regards Aisheng > > > + compatible = "fsl,imx8mp-media-blk-ctrl", "syscon"; > > > + reg = <0x32ec0000 0x10000>; > > > + > > > > Remove unnecessary blank line > > > > Will do. > > > Otherwise: > > Reviewed-by: Dong Aisheng <aisheng.dong@nxp.com> > > > > Regards > > Aisheng > > > > > + #clock-cells = <1>; > > > + #reset-cells = <1>; > > > + }; > > > + }; > > > + > > > aips5: bus@30c00000 { > > > compatible = "fsl,aips-bus", "simple-bus"; > > > reg = <0x30c00000 0x400000>; > > > -- > > > 2.7.4 > > >
On 20-08-20 09:31:27, Dong Aisheng wrote: > Hi Rob, Stephen, > > On Thu, Aug 20, 2020 at 4:37 AM Abel Vesa <abel.vesa@nxp.com> wrote: > > > > On 20-08-18 19:34:14, Dong Aisheng wrote: > > > On Fri, Aug 14, 2020 at 8:12 PM Abel Vesa <abel.vesa@nxp.com> wrote: > > > > > > > > Some of the features of the media_ctrl will be used by some > > > > different drivers in a way those drivers will know best, so adding the > > > > syscon compatible we allow those to do just that. Only the resets > > > > and the clocks are registered bit the clk-blk-ctrl driver. > > > > > > > > Signed-off-by: Abel Vesa <abel.vesa@nxp.com> > > > > --- > > > > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++++++++++ > > > > 1 file changed, 16 insertions(+) > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > > > index dede0ae..2d6d213 100644 > > > > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > > > @@ -736,6 +736,22 @@ > > > > }; > > > > }; > > > > > > > > + aips4: bus@32c00000 { > > > > + compatible = "simple-bus"; > > > > + reg = <0x32c00000 0x400000>; > > > > + #address-cells = <1>; > > > > + #size-cells = <1>; > > > > + ranges; > > > > + > > > > + media_blk_ctrl: clock-controller@32ec0000 { > > > > > > For this combo device, maybe we can directly name it as blk-ctrl@32ec0000. > > > Rob, do you think if we can do that? > > > > > > > I think it was Stephen who suggested we change it to clock-controller in the > > last's version thread. > > > > TBH, I agree with you here, since it makes more sense to be called blk-ctrl > > provided that this is not really just a clock controller. > > > > How do you think? > Stephen, can you give us an argument for leaving it as clock-controller ? > Regards > Aisheng > > > > > + compatible = "fsl,imx8mp-media-blk-ctrl", "syscon"; > > > > + reg = <0x32ec0000 0x10000>; > > > > + > > > > > > Remove unnecessary blank line > > > > > > > Will do. > > > > > Otherwise: > > > Reviewed-by: Dong Aisheng <aisheng.dong@nxp.com> > > > > > > Regards > > > Aisheng > > > > > > > + #clock-cells = <1>; > > > > + #reset-cells = <1>; > > > > + }; > > > > + }; > > > > + > > > > aips5: bus@30c00000 { > > > > compatible = "fsl,aips-bus", "simple-bus"; > > > > reg = <0x30c00000 0x400000>; > > > > -- > > > > 2.7.4 > > > >
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi index dede0ae..2d6d213 100644 --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi @@ -736,6 +736,22 @@ }; }; + aips4: bus@32c00000 { + compatible = "simple-bus"; + reg = <0x32c00000 0x400000>; + #address-cells = <1>; + #size-cells = <1>; + ranges; + + media_blk_ctrl: clock-controller@32ec0000 { + compatible = "fsl,imx8mp-media-blk-ctrl", "syscon"; + reg = <0x32ec0000 0x10000>; + + #clock-cells = <1>; + #reset-cells = <1>; + }; + }; + aips5: bus@30c00000 { compatible = "fsl,aips-bus", "simple-bus"; reg = <0x30c00000 0x400000>;
Some of the features of the media_ctrl will be used by some different drivers in a way those drivers will know best, so adding the syscon compatible we allow those to do just that. Only the resets and the clocks are registered bit the clk-blk-ctrl driver. Signed-off-by: Abel Vesa <abel.vesa@nxp.com> --- arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)