RE: [PATCH v3] clk: imx7ulp: do not export out IMX7ULP_CLK_MIPI_PLL clock

2019-10-15 Thread Fancy Fang
Hi Anson, Please see my below comments. Best regards, Fancy Fang -Original Message- From: Anson Huang Sent: Tuesday, October 15, 2019 11:28 AM To: Fancy Fang ; sb...@kernel.org; shawn...@kernel.org Cc: linux-...@vger.kernel.org; linux-arm-ker...@lists.infradead.org; linux-kernel

[PATCH v3] clk: imx7ulp: do not export out IMX7ULP_CLK_MIPI_PLL clock

2019-10-14 Thread Fancy Fang
MIPI DSI and not be exported out for other usages. Signed-off-by: Fancy Fang --- ChangeLog v2->v3: * Keep 'IMX7ULP_CLK_MIPI_PLL' macro definition. ChangeLog v1->v2: * Keep other clock indexes unchanged as Shawn suggested. Documentation/devicetree/bindings/clock/imx7ulp-clock.tx

[PATCH v2] clk: imx7ulp: remove IMX7ULP_CLK_MIPI_PLL clock

2019-09-18 Thread Fancy Fang
MIPI DSI and not be exported out for other usages. Signed-off-by: Fancy Fang --- ChangeLog v1->v2: * Keep other clock indexes unchanged as Shawn suggested. Documentation/devicetree/bindings/clock/imx7ulp-clock.txt | 1 - drivers/clk/imx/clk-imx7ulp.c | 3 +-- include

[PATCH v2] clk: imx7ulp: remove IMX7ULP_CLK_MIPI_PLL clock

2019-09-18 Thread Fancy Fang
MIPI DSI and not be exported out for other usages. Signed-off-by: Fancy Fang --- ChangeLog v1->v2: * Keep other clock indexes unchanged as Shawn suggested. Documentation/devicetree/bindings/clock/imx7ulp-clock.txt | 1 - drivers/clk/imx/clk-imx7ulp.c | 3 +-- include

[PATCH 2/2] clk: imx7ulp: remove IMX7ULP_CLK_MIPI_PLL clock

2019-08-22 Thread Fancy Fang
MIPI DSI and not be exported out for other usages. Signed-off-by: Fancy Fang --- .../devicetree/bindings/clock/imx7ulp-clock.txt | 1 - drivers/clk/imx/clk-imx7ulp.c | 3 +-- include/dt-bindings/clock/imx7ulp-clock.h | 15 +++ 3 files changed, 8 insertions

[PATCH 1/2] ARM: dts: imx7ulp: remove mipi pll clock node

2019-08-22 Thread Fancy Fang
According to the IMX7ULP reference manual, the mipi pll clock comes from the MIPI PHY PLL output. So it should not be defined as a fixed clock. So remove this clock node and all the references to it. Signed-off-by: Fancy Fang --- arch/arm/boot/dts/imx7ulp.dtsi | 17 - 1 file

[RESEND PATCH 2/2] clk: imx8mm: rename 'share_count_dcss' to 'share_count_disp'

2019-07-09 Thread Fancy Fang
Rename 'share_count_dcss' to 'share_count_disp', since the DCSS module does not exist on imx8mm platform. So rename it to avoid any unnecessary confusion. Signed-off-by: Fancy Fang --- drivers/clk/imx/clk-imx8mm.c | 10 +- 1 file changed, 5 insertions(+), 5 deleti

[RESEND PATCH 1/2] clk: imx8mm: rename lcdif pixel clock

2019-07-09 Thread Fancy Fang
cation which is designed after imx8mm, this same pixel root clock name has been modified from 'LCDIF_PIXEL_CLK_ROOT' to 'DISPLAY_PIXEL_CLK_ROOT'. Signed-off-by: Fancy Fang --- drivers/clk/imx/clk-imx8mm.c | 4 ++-- include/dt-bindings/clock/imx8mm-clock.h | 2

[PATCH 1/2] clk: imx8mm: rename lcdif pixel clock

2019-07-09 Thread Fancy Fang
cation which is designed after imx8mm, this same pixel root clock name has been modified from 'LCDIF_PIXEL_CLK_ROOT' to 'DISPLAY_PIXEL_CLK_ROOT'. Signed-off-by: Fancy Fang --- drivers/clk/imx/clk-imx8mm.c | 4 ++-- include/dt-bindings/clock/imx8mm-clock.h | 2

[PATCH 2/2] clk: imx8mm: rename 'share_count_dcss' to 'share_count_disp'

2019-07-09 Thread Fancy Fang
Rename 'share_count_dcss' to 'share_count_disp', since the DCSS module does not exist on imx8mm platform. So rename it to avoid any unnecessary confusion. Signed-off-by: Fancy Fang --- drivers/clk/imx/clk-imx8mm.c | 10 +- 1 file changed, 5 insertions(+), 5 deleti

RE: [PATCH] reset: Add driver for dispmix reset

2019-06-25 Thread Fancy Fang
Hi Philipp, Thanks for your comments. And please see my answers below. -Original Message- From: Philipp Zabel Sent: Tuesday, June 25, 2019 10:57 PM To: Fancy Fang ; shawn...@kernel.org; s.ha...@pengutronix.de Cc: feste...@gmail.com; ker...@pengutronix.de; linux-arm-ker

[PATCH] reset: Add driver for dispmix reset

2019-06-24 Thread Fancy Fang
them contains several reset lines to control several different modules on and off in DISPMIX which doesn't require the standard reset flow, but only line assert and deassert operations. Signed-off-by: Fancy Fang --- .../bindings/reset/nxp,dispmix-clk-en.txt | 58 +++ .../bindings/rese

[PATCH v2] [media] videobuf-dma-contig: set vm_pgoff to be zero to pass the sanity check in vm_iomap_memory().

2014-10-09 Thread Fancy Fang
hich is equal to its 'boff' value. Obviously, the 'offset' value is only used to find the matched buffer instead of to be the real offset from the buffer's physical start address as used by vm_iomap_memory(). So, in some case that if the offset is not zero, vm_iomap_m

[PATCH v2] [media] videobuf-dma-contig: set vm_pgoff to be zero to pass the sanity check in vm_iomap_memory().

2014-09-24 Thread Fancy Fang
hich is equal to its 'boff' value. Obviously, the 'offset' value is only used to find the matched buffer instead of to be the real offset from the buffer's physical start address as used by vm_iomap_memory(). So, in some case that if the offset is not zero, vm_iomap_m

[PATCH v2] [media] videobuf-dma-contig: set vm_pgoff to be zero to pass the sanity check in vm_iomap_memory().

2014-09-24 Thread Fancy Fang
hich is equal to its 'boff' value. Obviously, the 'offset' value is only used to find the matched buffer instead of to be the real offset from the buffer's physical start address as used by vm_iomap_memory(). So, in some case that if the offset is not zero, vm_iomap_m

[PATCH] [media] videobuf-dma-contig: replace vm_iomap_memory() with remap_pfn_range().

2014-09-09 Thread Fancy Fang
hich is equal to its 'boff' value. Obviously, the 'offset' value is only used to find the matched buffer instead of to be the real offset from the buffer's physical start address as used by vm_iomap_memory(). So, in some case that if the offset is not zero, vm_iomap_m

[PATCH] ARM: clk-imx6sl: correct the pxp and epdc axi clock selections

2014-09-04 Thread Fancy Fang
The parent clocks of IMX6SL_CLK_PXP_AXI_SEL and IMX6SL_CLK_EPDC_AXI_SEL clocks are not the same. So split the epdc_pxp_sels into two different clock selections 'pxp_axi_sels' and 'epdc_axi_sels'. Signed-off-by: Fancy Fang Signed-off-by: Robby Cai Acked-by: Shawn Guo --- a

[PATCH] [media] videobuf-dma-contig: replace vm_iomap_memory() with remap_pfn_range().

2014-09-04 Thread Fancy Fang
hich is equal to its 'boff' value. Obviously, the 'offset' value is only used to find the matched buffer instead of to be the real offset from the buffer's physical start address as used by vm_iomap_memory(). So, in some case that if the offset is not zero, vm_iomap_m

[PATCH] ARM: clk-imx6sl: correct the pxp and epdc axi clock selections

2014-09-04 Thread Fancy Fang
The parent clocks of IMX6SL_CLK_PXP_AXI_SEL and IMX6SL_CLK_EPDC_AXI_SEL clocks are not the same. So split the epdc_pxp_sels into two different clock selections 'pxp_axi_sels' and 'epdc_axi_sels'. Signed-off-by: Fancy Fang Signed-off-by: Robby Cai Acked-by: Shawn Guo --- a

[PATCH] [media] videobuf-dma-contig: replace vm_iomap_memory() with remap_pfn_range().

2014-09-02 Thread Fancy Fang
hich is equal to its 'boff' value. Obviously, the 'offset' value is only used to find the matched buffer instead of to be the real offset from the buffer's physical start address as used by vm_iomap_memory(). So, in some case that if the offset is not zero, vm_iomap_m