[PATCH] dt-bindings: reset: Convert i.MX reset to json-schema

Convert the i.MX reset binding to DT schema format using json-schema. Signed-off-by: Anson Huang --- .../devicetree/bindings/reset/fsl,imx-src.txt | 49 -- .../devicetree/bindings/reset/fsl,imx-src.yaml | 58 ++ 2 files changed, 58 insertions(+), 49

[PATCH V2] dt-bindings: reset: Convert i.MX7 reset to json-schema

Convert the i.MX7 reset binding to DT schema format using json-schema. Signed-off-by: Anson Huang --- Changes since V1: - Improve description; - Remove i.MX8MM and i.MX8MN compatible, they use i.MX8MQ's compatible; - Keep original author as maintainer; - Remove

RE: [PATCH] dt-bindings: reset: Convert i.MX7 reset to json-schema

> Subject: RE: [PATCH] dt-bindings: reset: Convert i.MX7 reset to json-schema > > [...] > > > + > > +maintainers: > > + - Anson Huang > > + > > +description: | > > + The system reset controller can be used to reset various set of > >

RE: [PATCH] tty: serial: imx: Add return value check for platform_get_irq()

Hi, Uwe > Subject: Re: [PATCH] tty: serial: imx: Add return value check for > platform_get_irq() > > Hello Anson, > > On Mon, May 11, 2020 at 03:09:56PM +0800, Anson Huang wrote: > > RX irq is required, so add return value check for platform_get_irq(). > >

[PATCH] tty: serial: imx: Add return value check for platform_get_irq()

RX irq is required, so add return value check for platform_get_irq(). Signed-off-by: Anson Huang --- drivers/tty/serial/imx.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.c index f4d6810..f4023d9 100644 --- a/drivers/tty/serial/imx.c

[PATCH 2/2] arm64: dts: imx8mp: Add src node interrupts

Interrupts is a required property according to SRC binding, add it for SRC node. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mp.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi

[PATCH 1/2] arm64: dts: imx8mq: Add src node interrupts

Interrupts is a required property according to SRC binding, add it for SRC node. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mq.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi

[PATCH] dt-bindings: reset: Convert i.MX7 reset to json-schema

Convert the i.MX7 reset binding to DT schema format using json-schema. Signed-off-by: Anson Huang --- .../devicetree/bindings/reset/fsl,imx7-src.txt | 56 --- .../devicetree/bindings/reset/fsl,imx7-src.yaml| 64 ++ 2 files changed, 64 insertions

RE: [PATCH V4 1/5] dt-bindings: clock: Convert i.MX6Q clock to json-schema

Hi, Stephen > Subject: Re: [PATCH V4 1/5] dt-bindings: clock: Convert i.MX6Q clock to > json-schema > > Quoting Anson Huang (2020-04-29 17:51:57) > > Convert the i.MX6Q clock binding to DT schema format using json-schema. > > > > Signed-off-by: Anson Huang &g

[PATCH V4 4/5] dt-bindings: clock: Convert i.MX6SLL clock to json-schema

Convert the i.MX6SLL clock binding to DT schema format using json-schema. Signed-off-by: Anson Huang --- Changes since V3: - update license to be with (GPL-2.0-only OR BSD-2-Clause); - remove unnecessary minItem for interrupts; - remove label in example

[PATCH V4 2/5] dt-bindings: clock: Convert i.MX6SX clock to json-schema

Convert the i.MX6SX clock binding to DT schema format using json-schema. Signed-off-by: Anson Huang --- Changes since V3: - update license to be with (GPL-2.0-only OR BSD-2-Clause); - remove unnecessary minItem for interrupts; - remove label in example

[PATCH V4 1/5] dt-bindings: clock: Convert i.MX6Q clock to json-schema

Convert the i.MX6Q clock binding to DT schema format using json-schema. Signed-off-by: Anson Huang --- Changes since V3: - update license to be with (GPL-2.0-only OR BSD-2-Clause); - remove unnecessary minItem for interrupts; - remove label in example. --- .../devicetree

[PATCH V4 3/5] dt-bindings: clock: Convert i.MX6SL clock to json-schema

Convert the i.MX6SL clock binding to DT schema format using json-schema. Signed-off-by: Anson Huang --- Changes since V3: - update license to be with (GPL-2.0-only OR BSD-2-Clause); - remove unnecessary minItem for interrupts; - remove label in example

[PATCH V4 5/5] dt-bindings: clock: Convert i.MX6UL clock to json-schema

Convert the i.MX6UL clock binding to DT schema format using json-schema. Signed-off-by: Anson Huang --- Changes since V3: - update license to be with (GPL-2.0-only OR BSD-2-Clause); - remove unnecessary minItem for interrupts; - remove label in example

[PATCH V3] thermal: imx: Add missing of_node_put()

After finishing using cpu node got from of_get_cpu_node(), of_node_put() needs to be called, the cpufreq policy also needs to be put unconditionally. Signed-off-by: Anson Huang --- Changes since V2: - call cpufreq_cpu_put() unconditionally after cooling register done. --- drivers

RE: [PATCH V2] thermal: imx: Add missing of_node_put()

Hi, Amit > Subject: Re: [PATCH V2] thermal: imx: Add missing of_node_put() > > On Thu, Mar 26, 2020 at 8:06 PM Anson Huang > wrote: > > > > After finishing using cpu node got from of_get_cpu_node(), > > of_node_put() needs to be called. > > > > Signe

[PATCH 1/2] arm64: dts: imx8mm: Remove duplicated machine compatible

Machine compatible string normally is located in board DT, remove the duplicated one from SoC dtsi. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mm.dtsi | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts

[PATCH 2/2] arm64: dts: imx8mn: Remove duplicated machine compatible

Machine compatible string normally is located in board DT, remove the duplicated one from SoC dtsi. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mn.dtsi | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/boot/dts/freescale/imx8mn.dtsi b/arch/arm64/boot/dts

RE: [PATCH] cpufreq: imx-cpufreq-dt: Correct i.MX8MN's default speed grade value

Hi, Viresh > On 22-10-19, 12:39, Anson Huang wrote: > > i.MX8MN has different speed grade definition compared to > > i.MX8MQ/i.MX8MM, when fuses are NOT written, the default speed_grade > > should be set to minimum available OPP defined in DT which is 1.2GHz, > > th

[PATCH V2] cpufreq: imx-cpufreq-dt: Correct i.MX8MN's default speed grade value

: Add i.MX8MN support") Signed-off-by: Anson Huang --- Changes since V1: - Improve the coding style by removing the tab; --- drivers/cpufreq/imx-cpufreq-dt.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/cpufreq/imx-cpufreq-dt.c

[PATCH] cpufreq: imx-cpufreq-dt: Correct i.MX8MN's default speed grade value

: Add i.MX8MN support") Signed-off-by: Anson Huang --- drivers/cpufreq/imx-cpufreq-dt.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/cpufreq/imx-cpufreq-dt.c b/drivers/cpufreq/imx-cpufreq-dt.c index 35db14c..26531f0 100644 --- a/drivers/c

RE: [PATCH V6 3/5] arm64: dts: imx8qxp: Add scu key node

Hi, Shawn Dmitry has picked up the #1 and #2 patch, would you please pick up #3 ~ #5, thanks. Anson > Subject: [PATCH V6 3/5] arm64: dts: imx8qxp: Add scu key node > > Add scu key node for i.MX8QXP, disabled by default as it depends on board > design. > > Signed-o

[PATCH 3/3] arm64: dts: imx8mn: Add LPDDR4 EVK board support

. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/Makefile | 1 + arch/arm64/boot/dts/freescale/imx8mn-evk.dts | 30 2 files changed, 31 insertions(+) create mode 100644 arch/arm64/boot/dts/freescale/imx8mn-evk.dts diff --git a/arch/arm64/boot/dts

[PATCH 1/3] arm64: dts: imx8mn: Create EVK dtsi file for common use

i.MX8MN has different EVK boards to support different DDR types, the ONLY differences are DDR chips and PMIC, so most of the devices can be shared between these EVK boards, create a EVK dtsi file for common use. Signed-off-by: Anson Huang --- - This patch is based on https

[PATCH 2/3] dt-bindings: arm: imx: Add the i.MX8MN LPDDR4 EVK board

Add board binding for i.MX8MN LPDDR4 EVK board. Signed-off-by: Anson Huang --- Documentation/devicetree/bindings/arm/fsl.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml index 41db01d

[PATCH 4/5] arm64: dts: imx8mn: Move usdhc clocks assignment to board DT

usdhc's clock rate is different according to different devices connected, so clock rate assignment should be placed in board DT according to different devices connected on each usdhc port. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mn-ddr4-evk.dts | 4 arch/arm64/boot

[PATCH 3/5] arm64: dts: imx8mm: Move usdhc clocks assignment to board DT

usdhc's clock rate is different according to different devices connected, so clock rate assignment should be placed in board DT according to different devices connected on each usdhc port. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mm-evk.dts | 4 arch/arm64/boot/dts

[PATCH 5/5] ARM: dts: imx7ulp: Move usdhc clocks assignment to board DT

usdhc's clock rate is different according to different devices connected, so clock rate assignment should be placed in board DT according to different devices connected on each usdhc port. Signed-off-by: Anson Huang --- arch/arm/boot/dts/imx7ulp-evk.dts | 2 ++ arch/arm/boot/dts/imx7ulp.dtsi

[PATCH 1/5] arm64: dts: imx8qxp: Move usdhc clocks assignment to board DT

usdhc's clock rate is different according to different devices connected, so clock rate assignment should be placed in board DT according to different devices connected on each usdhc port. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8qxp-ai_ml.dts | 4 arch/arm64/boot

[PATCH 2/5] arm64: dts: imx8mq: Move usdhc clocks assignment to board DT

usdhc's clock rate is different according to different devices connected, so clock rate assignment should be placed in board DT according to different devices connected on each usdhc port. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mq-evk.dts| 4 arch

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

Hi, Fancy > Subject: [PATCH v3] clk: imx7ulp: do not export out IMX7ULP_CLK_MIPI_PLL > clock > > The mipi pll clock comes from the MIPI PHY PLL output, so it should not be a > fixed clock. > > MIPI PHY PLL is in the MIPI DSI space, and it is used as the bit clock for > transferring the pixel

[PATCH V2] clk: imx7ulp: Correct system clock source option #7

In the latest reference manual Rev.0,06/2019, the SCS's option #7 is no longer from upll, it is reserved, update clock driver accordingly. Fixes: b1260067ac3d ("clk: imx: add imx7ulp clk driver") Signed-off-by: Anson Huang Reviewed-by: Fabio Estevam --- Changes since V1:

RE: [PATCH 1/2] net: fec_main: Use platform_get_irq_byname_optional() to avoid error message

Hi, Vladimir > On Fri, 11 Oct 2019 at 04:11, Anson Huang wrote: > > > > Hi, Jakub > > > > > On Fri, 11 Oct 2019 00:38:50 +, Anson Huang wrote: > > > > > Hm. Looks like the commit you need is commit f1da567f1dc1 ("driver > core: >

RE: [PATCH] clk: imx7ulp: Correct system clock source option #7

Hi, Fabio > On Fri, Oct 11, 2019 at 5:39 AM Anson Huang > wrote: > > > > In the latest reference manual Rev.0,06/2019, the SCG1's system > > This should be SCS instead of SCG1. The reference states SCG system clock, SCS stands for system clock source, so I think it is

[PATCH] clk: imx7ulp: Correct DDR clock mux options

In the latest reference manual Rev.0,06/2019, the DDR clock mux is extended to 2 bits, and the clock options are also changed, correct them accordingly. Fixes: b1260067ac3d ("clk: imx: add imx7ulp clk driver") Signed-off-by: Anson Huang --- This patch should be base

[PATCH] clk: imx7ulp: Correct system clock source option #7

In the latest reference manual Rev.0,06/2019, the SCG1's system clock source option #7 is no longer from upll, it is reserved, update clock driver accordingly. Fixes: b1260067ac3d ("clk: imx: add imx7ulp clk driver") Signed-off-by: Anson Huang --- drivers/clk/imx/clk-imx7ulp.c | 2

RE: [PATCH 1/2] net: fec_main: Use platform_get_irq_byname_optional() to avoid error message

Hi, Jakub > On Fri, 11 Oct 2019 00:38:50 +0000, Anson Huang wrote: > > > Hm. Looks like the commit you need is commit f1da567f1dc1 ("driver core: > > > platform: Add platform_get_irq_byname_optional()") and it's > > > currently in Greg's tree. You have t

RE: [PATCH 1/2] net: fec_main: Use platform_get_irq_byname_optional() to avoid error message

Hi, Jakub > On Fri, 11 Oct 2019 00:03:22 +0000, Anson Huang wrote: > > > On Wed, 9 Oct 2019 18:15:47 +0800, Anson Huang wrote: > > > > Failed to get irq using name is NOT fatal as driver will use index > > > > to get irq instead, use platfo

RE: [PATCH 1/2] net: fec_main: Use platform_get_irq_byname_optional() to avoid error message

Hi, Jakub > On Wed, 9 Oct 2019 18:15:47 +0800, Anson Huang wrote: > > Failed to get irq using name is NOT fatal as driver will use index to > > get irq instead, use platform_get_irq_byname_optional() instead of > > platform_get_irq_byname() to avoid below error mes

[PATCH 1/2] net: fec_main: Use platform_get_irq_byname_optional() to avoid error message

.ethernet: IRQ int1 not found [0.829539] fec 30be.ethernet: IRQ int2 not found Fixes: 7723f4c5ecdb ("driver core: platform: Add an error message to platform_get_irq*()") Signed-off-by: Anson Huang --- drivers/net/ethernet/freescale/fec_main.c | 2 +- 1 file changed, 1 insertion(+),

[PATCH 2/2] net: fec_ptp: Use platform_get_irq_xxx_optional() to avoid error message

not found Fixes: 7723f4c5ecdb ("driver core: platform: Add an error message to platform_get_irq*()") Signed-off-by: Anson Huang --- drivers/net/ethernet/freescale/fec_ptp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/freescale/fe

[PATCH V2] tty: serial: imx: Use platform_get_irq_optional() for optional IRQs

not found Fixes: 7723f4c5ecdb8d83 ("driver core: platform: Add an error message to platform_get_irq*()") Signed-off-by: Anson Huang --- Changes since V1: - use platform_get_irq_optional() directly instead of getting irq count. --- drivers/tty/serial/imx.c | 4 ++-- 1 file changed, 2

RE: [PATCH] tty: serial: imx: Only get second/third IRQ when there is more than one IRQ

> On Wed, Oct 09, 2019 at 11:12:20AM +0300, Andy Shevchenko wrote: > > On Wed, Oct 9, 2019 at 9:53 AM Uwe Kleine-König > > wrote: > > > > > The patch is fine given the changed behaviour of platform_get_irq. I > > > wonder if it is sensible to introduce a variant of platform_get_irq > > > (say >

RE: [PATCH V2] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Marco > On 19-10-07 09:15, Anson Huang wrote: > > The SCU firmware does NOT always have return value stored in message > > header's function element even the API has response data, those > > special APIs are defined as void function in SCU firmware, so they > >

RE: [PATCH V2] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Marco > On 19-10-08 00:48, Anson Huang wrote: > > Hi, Marco > > > > > On 19-10-07 09:15, Anson Huang wrote: > > > > The SCU firmware does NOT always have return value stored in > > > > message header's function element even the API has respo

RE: [PATCH] tty: serial: imx: Only get second/third IRQ when there is more than one IRQ

Hi, Uwe > On Wed, Oct 09, 2019 at 07:24:57AM +0000, Anson Huang wrote: > > > On Wed, Oct 09, 2019 at 06:58:24AM +, Anson Huang wrote: > > > > > The patch is fine given the changed behaviour of > > > > > platform_get_irq. I wonder if it is

[PATCH] watchdog: imx_sc_wdt: Pretimeout should follow SCU firmware format

SCU firmware calculates pretimeout based on current time stamp instead of watchdog timeout stamp, need to convert the pretimeout to SCU firmware's timeout value. Fixes: 15f7d7fc5542 ("watchdog: imx_sc: Add pretimeout support") Signed-off-by: Anson Huang --- drivers/watchdog/imx_sc

RE: [PATCH] tty: serial: imx: Only get second/third IRQ when there is more than one IRQ

Hi, Uwe > On Wed, Oct 09, 2019 at 06:58:24AM +0000, Anson Huang wrote: > > > On Wed, Oct 09, 2019 at 02:18:31PM +0800, Anson Huang wrote: > > > > All i.MX SoCs except i.MX1 have ONLY 1 IRQ, so it is better to > > > > check the IRQ count before getting second/th

[PATCH] arm64: dts: imx8mq-evk: VDD_ARM power rail is always ON

: disabling Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mq-evk.dts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mq-evk.dts b/arch/arm64/boot/dts/freescale/imx8mq-evk.dts index 6ede46f..4e0a281 100644 --- a/arch/arm64/boot/dts

RE: [PATCH] tty: serial: imx: Only get second/third IRQ when there is more than one IRQ

Hi, Uwe > On Wed, Oct 09, 2019 at 02:18:31PM +0800, Anson Huang wrote: > > All i.MX SoCs except i.MX1 have ONLY 1 IRQ, so it is better to check > > the IRQ count before getting second/third IRQ to avoid below error > > message during probe: > > > > [0.7262

[PATCH] tty: serial: imx: Only get second/third IRQ when there is more than one IRQ

-by: Anson Huang --- drivers/tty/serial/imx.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.c index 504d81c..081fa82 100644 --- a/drivers/tty/serial/imx.c +++ b/drivers/tty/serial/imx.c @@ -2198,6 +2198,7

[PATCH 1/2] arm64: dts: imx8mq-evk: Adjust nodes following alphabetical sort

Adjust some nodes to make them follow alphabetical sort except iomuxc node which is put at the end of file because of its huge pinctrl data. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mq-evk.dts | 46 ++-- 1 file changed, 23 insertions(+), 23

[PATCH 2/2] arm64: dts: imx8mn-ddr4-evk: Move iomuxc node to end of file

All nodes are better to follow alphabetical sort except iomuxc which has huge pinctrl data, better to put it at the end of file. Signed-off-by: Anson Huang --- arch/arm64/boot/dts/freescale/imx8mn-ddr4-evk.dts | 304 +++--- 1 file changed, 152 insertions(+), 152 deletions

RE: [PATCH -next] clk: imx: clk-pll14xx: Make two variables static

Reported-by: Hulk Robot > Signed-off-by: YueHaibing Reviewed-by: Anson Huang I did NOT see this warning on my side, did you enable any special compile option? Anson > --- > drivers/clk/imx/clk-pll14xx.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git

[PATCH V2 2/3] arm64: dts: imx8mm-evk: Add i2c3 support

Enable i2c3 for i.MX8MM EVK board. Signed-off-by: Anson Huang --- No changes. --- arch/arm64/boot/dts/freescale/imx8mm-evk.dts | 14 ++ 1 file changed, 14 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mm-evk.dts b/arch/arm64/boot/dts/freescale/imx8mm-evk.dts index

[PATCH V2 3/3] arm64: dts: imx8mm-evk: Enable pca6416 on i2c3 bus

Enable pca6416 on i.MX8MM EVK board's i2c3 bus. Signed-off-by: Anson Huang --- No changes. --- arch/arm64/boot/dts/freescale/imx8mm-evk.dts | 7 +++ 1 file changed, 7 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mm-evk.dts b/arch/arm64/boot/dts/freescale/imx8mm-evk.dts

[PATCH V2 1/3] arm64: dts: imx8mm-evk: Adjust i2c nodes following alphabetical sort

The iomuxc node is being put at end of file because of its huge pinctrl data. I2C devices should be placed in alphabetical sort. Signed-off-by: Anson Huang --- new patch. --- arch/arm64/boot/dts/freescale/imx8mm-evk.dts | 124 +-- 1 file changed, 62 insertions(+), 62

RE: [PATCH 1/2] arm64: dts: imx8mm-evk: Add i2c3 support

Hi, Shawn > On Thu, Sep 19, 2019 at 05:46:47PM +0800, Anson Huang wrote: > > Enable i2c3 for i.MX8MM EVK board. > > > > Signed-off-by: Anson Huang > > --- > > arch/arm64/boot/dts/freescale/imx8mm-evk.dts | 14 ++ > > 1 file changed, 14 insert

RE: [PATCH 1/3] arm64: dts: imx8mq: Use correct clock for usdhc's ipg clk

Hi, Shawn > On Thu, Sep 19, 2019 at 01:05:57PM +0800, Anson Huang wrote: > > On i.MX8MQ, usdhc's ipg clock is from IMX8MQ_CLK_IPG_ROOT, assign it > > explicitly instead of using IMX8MQ_CLK_DUMMY. > > > > Signed-off-by: Anson Huang > > Fixes tag? OK, added them in V2. Thanks, Anson

[PATCH V2 3/3] arm64: dts: imx8mn: Use correct clock for usdhc's ipg clk

On i.MX8MN, usdhc's ipg clock is from IMX8MN_CLK_IPG_ROOT, assign it explicitly instead of using IMX8MN_CLK_DUMMY. Fixes: 6c3debcbae47 ("arm64: dts: freescale: Add i.MX8MN dtsi support") Signed-off-by: Anson Huang --- Changes since V1: - Add fixes tag. --- arch/arm64/boot/dts

[PATCH V2 2/3] arm64: dts: imx8mm: Use correct clock for usdhc's ipg clk

On i.MX8MM, usdhc's ipg clock is from IMX8MM_CLK_IPG_ROOT, assign it explicitly instead of using IMX8MM_CLK_DUMMY. Fixes: a05ea40eb384 ("arm64: dts: imx: Add i.mx8mm dtsi support") Signed-off-by: Anson Huang --- Changes since V1: - Add fixes tag. --- arch/arm64/boot/dts

[PATCH V2 1/3] arm64: dts: imx8mq: Use correct clock for usdhc's ipg clk

On i.MX8MQ, usdhc's ipg clock is from IMX8MQ_CLK_IPG_ROOT, assign it explicitly instead of using IMX8MQ_CLK_DUMMY. Fixes: 748f908cc882 ("arm64: add basic DTS for i.MX8MQ") Signed-off-by: Anson Huang --- Changes since V1: - Add fixes tag. --- arch/arm64/boot/dts/freescale/imx8m

RE: [PATCH V2] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Marco > On 19-10-07 09:15, Anson Huang wrote: > > The SCU firmware does NOT always have return value stored in message > > header's function element even the API has response data, those > > special APIs are defined as void function in SCU firmware, so they > >

[PATCH V6 4/5] arm64: dts: imx8qxp-mek: Enable scu key

Enable scu key for i.MX8QXP MEK board. Signed-off-by: Anson Huang --- No changes. --- arch/arm64/boot/dts/freescale/imx8qxp-mek.dts | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8qxp-mek.dts b/arch/arm64/boot/dts/freescale/imx8qxp-mek.dts index

[PATCH V6 5/5] arm64: defconfig: Enable CONFIG_KEYBOARD_IMX_SC_KEY as module

Select CONFIG_KEYBOARD_IMX_SC_KEY as module by default to support i.MX8QXP scu key driver. Signed-off-by: Anson Huang --- No changes. --- arch/arm64/configs/defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index c9adae4

[PATCH V6 1/5] dt-bindings: fsl: scu: add scu key binding

for i.MX system controller key driver. Signed-off-by: Anson Huang Reviewed-by: Rob Herring --- Changes since V5: - use "linux,keycodes" instead of "linux,keycode" in example. --- .../devicetree/bindings/arm/freescale/fsl,scu.txt | 14 ++ 1 file ch

[PATCH V6 2/5] input: keyboard: imx_sc: Add i.MX system controller key support

key's status. Signed-off-by: Anson Huang --- Changes since V5: - use "linux,keycodes" instead of "linux,keycode" according to DT change. --- drivers/input/keyboard/Kconfig | 7 ++ drivers/input/keyboard/Makefile | 1 + drivers/input/keyboar

[PATCH V6 3/5] arm64: dts: imx8qxp: Add scu key node

Add scu key node for i.MX8QXP, disabled by default as it depends on board design. Signed-off-by: Anson Huang --- Changes since V5: - use "linux,keycodes" instead of "linux,keycode". --- arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 7 +++ 1 file changed, 7 inser

RE: [PATCH V2 1/5] dt-bindings: fsl: scu: add scu power key binding

Hi, Shawn > On Mon, Sep 23, 2019 at 02:34:07AM +0000, Anson Huang wrote: > > Hi, Pavel > > > > > Subject: Re: [PATCH V2 1/5] dt-bindings: fsl: scu: add scu power key > > > binding > > > > > > On Tue 2019-09-03 10:03:40, Anson Huang wrote: > &g

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Marco > On 19-09-30 08:32, Anson Huang wrote: > > Hi, Marco > > > > > On 19-09-30 07:42, Anson Huang wrote: > > > > Hi, Leonard > > > > > > > > > On 2019-09-27 4:20 AM, Anson Huang wrote: > > > > > >> On

[PATCH V2] firmware: imx: Skip return value check for some special SCU firmware APIs

The SCU firmware does NOT always have return value stored in message header's function element even the API has response data, those special APIs are defined as void function in SCU firmware, so they should be treated as return success always. Signed-off-by: Anson Huang --- Changes since V1

RE: [PATCH] ARM: dts: imx7s: Correct GPT's ipg clock source

Hi, Shawn > On Mon, Sep 16, 2019 at 04:29:09PM +0800, Anson Huang wrote: > > i.MX7S/D's GPT ipg clock should be from GPT clock root and controlled > > by CCM's GPT CCGR, using correct clock source for GPT ipg clock > > instead of IMX7D_CLK_DUMMY. > > So both 'ipg' and

[PATCH V2] ARM: dts: imx7s: Correct GPT's ipg clock source

i.MX7S/D's GPT ipg clock should be from GPT clock root and controlled by CCM's GPT CCGR, using correct clock source for GPT ipg clock instead of IMX7D_CLK_DUMMY. Fixes: 3ef79ca6bd1d ("ARM: dts: imx7d: use imx7s.dtsi as base device tree") Signed-off-by: Anson Huang --- Change

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Marco > On 19-09-30 07:42, Anson Huang wrote: > > Hi, Leonard > > > > > On 2019-09-27 4:20 AM, Anson Huang wrote: > > > >> On 2019-09-26 1:06 PM, Marco Felsch wrote: > > > >>> On 19-09-26 08:03, Anson Huang wrote: > > > >>

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Leonard > > On 2019-09-27 4:20 AM, Anson Huang wrote: > > >> On 2019-09-26 1:06 PM, Marco Felsch wrote: > > >>> On 19-09-26 08:03, Anson Huang wrote: > > >>>>> On 19-09-25 18:07, Anson Huang wrote: > > >>>>>> T

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Leonard > On 2019-09-27 4:20 AM, Anson Huang wrote: > >> On 2019-09-26 1:06 PM, Marco Felsch wrote: > >>> On 19-09-26 08:03, Anson Huang wrote: > >>>>> On 19-09-25 18:07, Anson Huang wrote: > >>>>>> The SCU firmware does N

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

> On 27.09.2019 12:06, Marco Felsch wrote: > > Hi Anson, Leonard, > > > > On 19-09-27 01:20, Anson Huang wrote: > >> Hi, Leonard > >> > >>> On 2019-09-26 1:06 PM, Marco Felsch wrote: > >>>> On 19-09-26 08:03, Anson Huang wrote: > >>

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Marco > On 19-09-27 01:20, Anson Huang wrote: > > Hi, Leonard > > > > > On 2019-09-26 1:06 PM, Marco Felsch wrote: > > > > On 19-09-26 08:03, Anson Huang wrote: > > > >>> On 19-09-25 18:07, Anson Huang wrote: > > >

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Leonard > On 2019-09-26 1:06 PM, Marco Felsch wrote: > > On 19-09-26 08:03, Anson Huang wrote: > >>> On 19-09-25 18:07, Anson Huang wrote: > >>>> The SCU firmware does NOT always have return value stored in > >>>> message header's function e

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Marco > On 19-09-25 18:07, Anson Huang wrote: > > The SCU firmware does NOT always have return value stored in message > > header's function element even the API has response data, those > > special APIs are defined as void function in SCU firmware, so they > >

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

Hi, Leonard > On 25.09.2019 13:09, Anson Huang wrote: > > The SCU firmware does NOT always have return value stored in message > > header's function element even the API has response data, those > > special APIs are defined as void function in SCU firmware, so they > >

[PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

The SCU firmware does NOT always have return value stored in message header's function element even the API has response data, those special APIs are defined as void function in SCU firmware, so they should be treated as return success always. Signed-off-by: Anson Huang --- - This patch

RE: [PATCH] pwm: pwm-imx27: Use 'dev' instead of dereferencing it repeatedly

Hi, David > Subject: RE: [PATCH] pwm: pwm-imx27: Use 'dev' instead of dereferencing it > repeatedly > > From: Anson Huang > > Sent: 24 September 2019 11:03 > > Hi, David > > > > > Subject: RE: [PATCH] pwm: pwm-imx27: Use 'dev' instead of > > > der

RE: [PATCH] pwm: pwm-imx27: Use 'dev' instead of dereferencing it repeatedly

Hi, David > Subject: RE: [PATCH] pwm: pwm-imx27: Use 'dev' instead of dereferencing it > repeatedly > > From: Anson Huang > > Sent: 24 September 2019 10:00 > > Add helper variable dev = >dev to simply the code. > > > > Signed-off-by: Anson Huang &g

[PATCH 1/2] pwm: pwm-mxs: use devm_platform_ioremap_resource() to simplify code

Use the new helper devm_platform_ioremap_resource() which wraps the platform_get_resource() and devm_ioremap_resource() together, to simplify the code. Signed-off-by: Anson Huang --- - This is a resend version of patch: https://patchwork.kernel.org/patch/11048365/ --- drivers/pwm/pwm

[PATCH 2/2] pwm: pwm-mxs: Use 'dev' instead of dereferencing it repeatedly

Add helper variable dev = >dev to simply the code. Signed-off-by: Anson Huang --- drivers/pwm/pwm-mxs.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/pwm/pwm-mxs.c b/drivers/pwm/pwm-mxs.c index b14376b..6255ffc 100644 --- a/drivers/pwm/pwm-mxs.c ++

[PATCH] input: keyboard: imx_keypad: Use 'dev' instead of dereferencing it repeatedly

Add helper variable dev = >dev to simply the code. Signed-off-by: Anson Huang --- drivers/input/keyboard/imx_keypad.c | 35 ++- 1 file changed, 18 insertions(+), 17 deletions(-) diff --git a/drivers/input/keyboard/imx_keypad.c b/drivers/input/keybo

[PATCH] input: touchscreen: imx6ul_tsc: Use 'dev' instead of dereferencing it repeatedly

Add helper variable dev = >dev to simply the code. Signed-off-by: Anson Huang --- drivers/input/touchscreen/imx6ul_tsc.c | 37 +- 1 file changed, 19 insertions(+), 18 deletions(-) diff --git a/drivers/input/touchscreen/imx6ul_tsc.c b/drivers/input/touchscr

[PATCH] input: keyboard: snvs_pwrkey: Use 'dev' instead of dereferencing it repeatedly

Add helper variable dev = >dev to simply the code. Signed-off-by: Anson Huang --- drivers/input/keyboard/snvs_pwrkey.c | 29 +++-- 1 file changed, 15 insertions(+), 14 deletions(-) diff --git a/drivers/input/keyboard/snvs_pwrkey.c b/drivers/input/keyboard/snvs_pwrke

[PATCH] pwm: pwm-imx-tpm: Use 'dev' instead of dereferencing it repeatedly

Add helper variable dev = >dev to simply the code. Signed-off-by: Anson Huang --- drivers/pwm/pwm-imx-tpm.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/pwm/pwm-imx-tpm.c b/drivers/pwm/pwm-imx-tpm.c index e8385c1..4385801 100644 --- a/drivers/pwm/

[PATCH] pwm: pwm-imx27: Use 'dev' instead of dereferencing it repeatedly

Add helper variable dev = >dev to simply the code. Signed-off-by: Anson Huang --- drivers/pwm/pwm-imx27.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/pwm/pwm-imx27.c b/drivers/pwm/pwm-imx27.c index 434a351..3afee29 100644 --- a/drivers/pwm/pwm-im

[PATCH 1/3] watchdog: imx2_wdt: Remove unnecessary blank line

Remove unnecessary blank line. Signed-off-by: Anson Huang --- drivers/watchdog/imx2_wdt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/watchdog/imx2_wdt.c b/drivers/watchdog/imx2_wdt.c index 8d019a9..6711b4e 100644 --- a/drivers/watchdog/imx2_wdt.c +++ b/drivers/watchdog

[PATCH 3/3] watchdog: imx2_wdt: Use 'dev' instead of dereferencing it repeatedly

Add helper variable dev = >dev to simply the code. Signed-off-by: Anson Huang --- drivers/watchdog/imx2_wdt.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/drivers/watchdog/imx2_wdt.c b/drivers/watchdog/imx2_wdt.c index 034f32c..f8d58bf 100

[PATCH 2/3] watchdog: imx2_wdt: Use __maybe_unused instead of #if CONFIG_PM_SLEEP

Use __maybe_unused for power management related functions instead of #if CONFIG_PM_SLEEP to simply the code. Signed-off-by: Anson Huang --- drivers/watchdog/imx2_wdt.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/watchdog/imx2_wdt.c b/drivers/watchdog

RE: [PATCH v2] thermal: qoriq: add thermal monitor unit version 2 support

> -Original Message- > From: Yuantian Tang > Sent: Tuesday, September 24, 2019 11:17 AM > To: edubez...@gmail.com; rui.zh...@intel.com; Anson Huang > > Cc: daniel.lezc...@linaro.org; Leo Li ; linux- > p...@vger.kernel.org; linux-kernel@vger.kernel.org; Andy Tang &g

RE: [PATCH] thermal: qoriq: add thermal monitor unit version 2 support

Hi, Andy > Subject: RE: [PATCH] thermal: qoriq: add thermal monitor unit version 2 > support > > Hi Anson, > > Thanks for your review. Please see my reply inline. > > > -Original Message- > > From: Anson Huang > > Sent: 2019年9月24日 9:17 >

RE: [PATCH] thermal: qoriq: add thermal monitor unit version 2 support

Hi, Andy > On Mon, 2019-09-23 at 09:24 +, Andy Tang wrote: > > Hi Rui, Edubezval, > > > > Would you please review this patch? > > > CC Anson Huang. > I'd prefer all the qoriq thermal patches go through his review first. > > thanks, > rui > >

RE: [PATCH V4 2/5] input: keyboard: imx_sc: Add i.MX system controller key support

Hi, Pavel > Subject: Re: [PATCH V4 2/5] input: keyboard: imx_sc: Add i.MX system > controller key support > > Hi! > > > > > + ret = imx_scu_call_rpc(priv->key_ipc_handle, , true); > > > > + if (ret) { > > > > + dev_err(>dev, "read imx sc key failed, ret > > > > %d\n",

RE: [PATCH V2 1/5] dt-bindings: fsl: scu: add scu power key binding

Hi, Pavel > Subject: Re: [PATCH V2 1/5] dt-bindings: fsl: scu: add scu power key binding > > On Tue 2019-09-03 10:03:40, Anson Huang wrote: > > NXP i.MX8QXP is an ARMv8 SoC with a Cortex-M4 core inside as system > > controller, the system controller is in charge o

RE: [PATCH V2 1/5] dt-bindings: fsl: scu: add scu power key binding

y event etc. management, Linux kernel has to communicate > > with system controller via MU (message unit) IPC to get power key > > event, add binding doc for i.MX system controller power key driver. > > > > Signed-off-by: Anson Huang > > > +++ b/Documentation/devi

<    1   2   3   4   5   6   7   8   9   10   >