Re: [PATCH v3 0/5] RISC-V: Add kexec/kdump support

2021-04-07 Thread Yixun Lan
Hi Nick On 4/5/21 8:57 AM, Nick Kossifidis wrote: > This patch series adds kexec/kdump and crash kernel > support on RISC-V. For testing the patches a patched > version of kexec-tools is needed (still a work in > progress) which can be found at: > >

Re: macb broken on HiFive Unleashed

2021-03-19 Thread Yixun Lan
HI Andreas, Zong: On Fri, Mar 19, 2021 at 4:51 PM Andreas Schwab wrote: > > On Mär 19 2021, Yixun Lan wrote: > > > what's the exact root cause? and any solution? > > Try reverting the five commits starting with > 732374a0b440d9a79c8412f318a25cd37ba6f4e2. > I confirm r

Re: [PATCH v7 0/5] clk: add driver for the SiFive FU740

2021-03-19 Thread Yixun Lan
; > That breaks ethernet on the fu540. > > > > > > > I would check that, thanks for the report. > > > > Hi Andreas, > > Could you please point me out how to test the ethernet from your side? > I had tried to quick test by using iperf and wget, the ethernet seems > to work fine to me. > I will give it a shot during this weekend, since I'm facing the same issue.. Yixun Lan

Re: macb broken on HiFive Unleashed

2021-03-19 Thread Yixun Lan
HI Andreas: On Fri, Mar 19, 2021 at 8:28 AM Yixun Lan wrote: > > HI Andreas: > > On Wed, Mar 17, 2021 at 4:27 PM Andreas Schwab wrote: > > > > It turned out to be a broken clock driver. > > > > what's the exact root cause? and any solution? > seems I face

Re: macb broken on HiFive Unleashed

2021-03-19 Thread Yixun Lan
HI Andreas: On Wed, Mar 17, 2021 at 4:27 PM Andreas Schwab wrote: > > It turned out to be a broken clock driver. > what's the exact root cause? and any solution? seems I face the same issue, upgrade kernel to 5.11, then eth0 fail to bring up Yixun Lan

Re: [PATCH 2/2] Bug Fix for last patch

2021-03-15 Thread Yixun Lan
Hi Jiuyang On Tue, Mar 16, 2021 at 1:56 AM Jiuyang Liu wrote: > > Sorry for the noise, Andrew gave me feedbacks, and pointed two bugs in > last patch. > 1. asid should be thread safe, which is not the intent. > 2. asid extracting logic was wrong. > > This patch fixes these bugs. > >

Re: [PATCH v5 2/3] clk: meson: add DT documentation for emmc clock controller

2018-10-25 Thread Yixun Lan
Hi Jerome, Jianxin: see my comments On 10:58 Wed 24 Oct , Jerome Brunet wrote: > On Thu, 2018-10-18 at 13:07 +0800, Jianxin Pan wrote: > > From: Yixun Lan > > > > Document the MMC sub clock controller driver, the potential consumer > > of this driver is MMC o

Re: [PATCH v5 2/3] clk: meson: add DT documentation for emmc clock controller

2018-10-25 Thread Yixun Lan
Hi Jerome, Jianxin: see my comments On 10:58 Wed 24 Oct , Jerome Brunet wrote: > On Thu, 2018-10-18 at 13:07 +0800, Jianxin Pan wrote: > > From: Yixun Lan > > > > Document the MMC sub clock controller driver, the potential consumer > > of this driver is MMC o

[PATCH v4 3/3] clk: meson: add sub MMC clock controller driver

2018-08-09 Thread Yixun Lan
G platform. To specify which clock the MMC or NAND driver may consume, the preprocessor macros in the dt-bindings/clock/amlogic,mmc-clkc.h header can be used in the device tree sources. Signed-off-by: Yixun Lan --- drivers/clk/meson/Kconfig| 10 ++ drivers/clk/meson/Makefile | 1 + drivers/clk

[PATCH v4 2/3] clk: meson: add DT documentation for emmc clock controller

2018-08-09 Thread Yixun Lan
Document the MMC sub clock controller driver, the potential consumer of this driver is MMC or NAND. Also add four clock bindings IDs which provided by this driver. Reviewed-by: Rob Herring Signed-off-by: Yixun Lan --- .../bindings/clock/amlogic,mmc-clkc.txt | 31

[PATCH v4 1/3] clk: meson: add emmc sub clock phase delay driver

2018-08-09 Thread Yixun Lan
Export the emmc sub clock phase delay ops which will be used by the emmc sub clock driver itself. Signed-off-by: Yixun Lan --- drivers/clk/meson/Makefile | 2 +- drivers/clk/meson/clk-phase-delay.c | 96 + drivers/clk/meson/clkc.h| 13 3

[PATCH v4 3/3] clk: meson: add sub MMC clock controller driver

2018-08-09 Thread Yixun Lan
G platform. To specify which clock the MMC or NAND driver may consume, the preprocessor macros in the dt-bindings/clock/amlogic,mmc-clkc.h header can be used in the device tree sources. Signed-off-by: Yixun Lan --- drivers/clk/meson/Kconfig| 10 ++ drivers/clk/meson/Makefile | 1 + drivers/clk

[PATCH v4 2/3] clk: meson: add DT documentation for emmc clock controller

2018-08-09 Thread Yixun Lan
Document the MMC sub clock controller driver, the potential consumer of this driver is MMC or NAND. Also add four clock bindings IDs which provided by this driver. Reviewed-by: Rob Herring Signed-off-by: Yixun Lan --- .../bindings/clock/amlogic,mmc-clkc.txt | 31

[PATCH v4 1/3] clk: meson: add emmc sub clock phase delay driver

2018-08-09 Thread Yixun Lan
Export the emmc sub clock phase delay ops which will be used by the emmc sub clock driver itself. Signed-off-by: Yixun Lan --- drivers/clk/meson/Makefile | 2 +- drivers/clk/meson/clk-phase-delay.c | 96 + drivers/clk/meson/clkc.h| 13 3

[PATCH v4 0/3] clk: meson: add a sub EMMC clock controller support

2018-08-09 Thread Yixun Lan
/20180710163658.6175-1-yixun@amlogic.com [4] https://lkml.kernel.org/r/20180712211244.11428-1-yixun@amlogic.com Yixun Lan (3): clk: meson: add emmc sub clock phase delay driver clk: meson: add DT documentation for emmc clock controller clk: meson: add sub MMC clock controller driver .../bindings/clock

[PATCH v4 0/3] clk: meson: add a sub EMMC clock controller support

2018-08-09 Thread Yixun Lan
/20180710163658.6175-1-yixun@amlogic.com [4] https://lkml.kernel.org/r/20180712211244.11428-1-yixun@amlogic.com Yixun Lan (3): clk: meson: add emmc sub clock phase delay driver clk: meson: add DT documentation for emmc clock controller clk: meson: add sub MMC clock controller driver .../bindings/clock

Re: [PATCH v3 2/2] clk: meson: add sub MMC clock controller driver

2018-08-08 Thread Yixun Lan
Hi Jerome On 07/30/18 16:57, Jerome Brunet wrote: > On Fri, 2018-07-27 at 09:45 -0700, Stephen Boyd wrote: >> Quoting Stephen Boyd (2018-07-27 09:41:40) >>> Quoting Yixun Lan (2018-07-27 07:52:23) >>>> HI Stephen: >>>> >>>> On 07/26/2018 11:20

Re: [PATCH v3 2/2] clk: meson: add sub MMC clock controller driver

2018-08-08 Thread Yixun Lan
Hi Jerome On 07/30/18 16:57, Jerome Brunet wrote: > On Fri, 2018-07-27 at 09:45 -0700, Stephen Boyd wrote: >> Quoting Stephen Boyd (2018-07-27 09:41:40) >>> Quoting Yixun Lan (2018-07-27 07:52:23) >>>> HI Stephen: >>>> >>>> On 07/26/2018 11:20

[PATCH v3 0/2] pinctrl: meson-g12a: add pinctrl driver support

2018-08-06 Thread Yixun Lan
patch 1,2 (documentation & header file) - explain pinctrl IP - notice GPIOE located in AO bank [1] https://lkml.kernel.org/r/20180704224511.29350-1-yixun@amlogic.com [2] https://lkml.kernel.org/r/20180714232754.5402-1-yixun@amlogic.com Yixun Lan (2): documentation: pinctrl: Add compati

[PATCH v3 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-08-06 Thread Yixun Lan
from 0 - 7, each can describe one pin, the value 0 is always devoted to GPIO function, while 1 - 7 devoted to the mux pin function. Please note, the GPIOE is actually located at AO (always on) bank. Acked-by: Martin Blumenstingl Signed-off-by: Xingyu Chen Signed-off-by: Yixun Lan --- drivers

[PATCH v3 0/2] pinctrl: meson-g12a: add pinctrl driver support

2018-08-06 Thread Yixun Lan
patch 1,2 (documentation & header file) - explain pinctrl IP - notice GPIOE located in AO bank [1] https://lkml.kernel.org/r/20180704224511.29350-1-yixun@amlogic.com [2] https://lkml.kernel.org/r/20180714232754.5402-1-yixun@amlogic.com Yixun Lan (2): documentation: pinctrl: Add compati

[PATCH v3 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-08-06 Thread Yixun Lan
from 0 - 7, each can describe one pin, the value 0 is always devoted to GPIO function, while 1 - 7 devoted to the mux pin function. Please note, the GPIOE is actually located at AO (always on) bank. Acked-by: Martin Blumenstingl Signed-off-by: Xingyu Chen Signed-off-by: Yixun Lan --- drivers

[PATCH v3 1/2] documentation: pinctrl: Add compatibles for Amlogic Meson G12A pin controllers

2018-08-06 Thread Yixun Lan
Add new compatible name for Amlogic's Meson-G12A pin controllers, add a dt-binding header file which document the detail pin names. Acked-by: Martin Blumenstingl Reviewed-by: Rob Herring Signed-off-by: Xingyu Chen Signed-off-by: Yixun Lan --- .../bindings/pinctrl/meson,pinctrl.txt

[PATCH v3 1/2] documentation: pinctrl: Add compatibles for Amlogic Meson G12A pin controllers

2018-08-06 Thread Yixun Lan
Add new compatible name for Amlogic's Meson-G12A pin controllers, add a dt-binding header file which document the detail pin names. Acked-by: Martin Blumenstingl Reviewed-by: Rob Herring Signed-off-by: Xingyu Chen Signed-off-by: Yixun Lan --- .../bindings/pinctrl/meson,pinctrl.txt

Re: [PATCH 3/4] ARM64: dts: meson-gx: add dmcbus and canvas nodes.

2018-08-06 Thread Yixun Lan
hi Maxime: On Sun, Aug 5, 2018 at 4:02 AM, Maxime Jourdan wrote: >>> + sysctrl_DMC: system-controller@0 { >>> + compatible = >>> "amlogic,meson-gx-dmc-sysctrl", "syscon", "simple-mfd"; >> >> we'd like to drop 'meson-' prefix, so better using

Re: [PATCH 3/4] ARM64: dts: meson-gx: add dmcbus and canvas nodes.

2018-08-06 Thread Yixun Lan
hi Maxime: On Sun, Aug 5, 2018 at 4:02 AM, Maxime Jourdan wrote: >>> + sysctrl_DMC: system-controller@0 { >>> + compatible = >>> "amlogic,meson-gx-dmc-sysctrl", "syscon", "simple-mfd"; >> >> we'd like to drop 'meson-' prefix, so better using

Re: [PATCH 1/4] soc: amlogic: add meson-canvas driver

2018-08-03 Thread Yixun Lan
HI Maxime thanks for contributing the patches ;-) On Thu, Aug 2, 2018 at 2:51 AM, Maxime Jourdan wrote: > Amlogic SoCs have a repository of 256 canvas which they use to > describe pixel buffers. > > They contain metadata like width, height, block mode, endianness [..] > > Many IPs within those

Re: [PATCH 1/4] soc: amlogic: add meson-canvas driver

2018-08-03 Thread Yixun Lan
HI Maxime thanks for contributing the patches ;-) On Thu, Aug 2, 2018 at 2:51 AM, Maxime Jourdan wrote: > Amlogic SoCs have a repository of 256 canvas which they use to > describe pixel buffers. > > They contain metadata like width, height, block mode, endianness [..] > > Many IPs within those

Re: [PATCH 3/4] ARM64: dts: meson-gx: add dmcbus and canvas nodes.

2018-08-03 Thread Yixun Lan
Hi Maxime great job! thanks for contributing the patches.. On Thu, Aug 2, 2018 at 2:51 AM, Maxime Jourdan wrote: > Wrap the canvas node in a syscon node. > > Signed-off-by: Maxime Jourdan > --- > arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 17 + > 1 file changed, 17

Re: [PATCH 3/4] ARM64: dts: meson-gx: add dmcbus and canvas nodes.

2018-08-03 Thread Yixun Lan
Hi Maxime great job! thanks for contributing the patches.. On Thu, Aug 2, 2018 at 2:51 AM, Maxime Jourdan wrote: > Wrap the canvas node in a syscon node. > > Signed-off-by: Maxime Jourdan > --- > arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 17 + > 1 file changed, 17

Re: [RFC PATCH v2 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-08-02 Thread Yixun Lan
Hi Boris On 08/02/2018 05:50 AM, Boris Brezillon wrote: > Hi Yixun, > > On Thu, 19 Jul 2018 17:46:12 +0800 > Yixun Lan wrote: > > I haven't finished reviewing the driver yet (I'll try to do that later > this week), but I already pointed a few things to fix/improve. >

Re: [RFC PATCH v2 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-08-02 Thread Yixun Lan
Hi Boris On 08/02/2018 05:50 AM, Boris Brezillon wrote: > Hi Yixun, > > On Thu, 19 Jul 2018 17:46:12 +0800 > Yixun Lan wrote: > > I haven't finished reviewing the driver yet (I'll try to do that later > this week), but I already pointed a few things to fix/improve. >

[PATCH] clk: meson-axg: pcie: drop the mpll3 clock parent

2018-07-31 Thread Yixun Lan
We found the PCIe driver doesn't really work with the mpll3 clock which is actually reserved for debug, So drop it from the mux list. Fixes: 33b89db68236 ("clk: meson-axg: add clocks required by pcie driver") Tested-by: Jianxin Qin Signed-off-by: Yixun Lan --- hi Jerome: I'm sorr

[PATCH] clk: meson-axg: pcie: drop the mpll3 clock parent

2018-07-31 Thread Yixun Lan
We found the PCIe driver doesn't really work with the mpll3 clock which is actually reserved for debug, So drop it from the mux list. Fixes: 33b89db68236 ("clk: meson-axg: add clocks required by pcie driver") Tested-by: Jianxin Qin Signed-off-by: Yixun Lan --- hi Jerome: I'm sorr

Re: [PATCH v3 2/2] clk: meson: add sub MMC clock controller driver

2018-07-27 Thread Yixun Lan
HI Stephen: On 07/26/2018 11:20 PM, Stephen Boyd wrote: > Quoting Yixun Lan (2018-07-12 14:12:44) >> diff --git a/drivers/clk/meson/mmc-clkc.c b/drivers/clk/meson/mmc-clkc.c >> new file mode 100644 >> index ..36c4c7cd69a6 >> --- /dev/null >> +

Re: [PATCH v3 2/2] clk: meson: add sub MMC clock controller driver

2018-07-27 Thread Yixun Lan
HI Stephen: On 07/26/2018 11:20 PM, Stephen Boyd wrote: > Quoting Yixun Lan (2018-07-12 14:12:44) >> diff --git a/drivers/clk/meson/mmc-clkc.c b/drivers/clk/meson/mmc-clkc.c >> new file mode 100644 >> index ..36c4c7cd69a6 >> --- /dev/null >> +

Re: [PATCH v3 1/2] clk: meson: add DT documentation for emmc clock controller

2018-07-27 Thread Yixun Lan
HI Rob On 07/25/2018 07:29 AM, Rob Herring wrote: > On Thu, Jul 12, 2018 at 09:12:43PM +0000, Yixun Lan wrote: >> Document the MMC sub clock controller driver, the potential consumer >> of this driver is MMC or NAND. Also add three clock bindings IDs which >> pr

Re: [PATCH v3 1/2] clk: meson: add DT documentation for emmc clock controller

2018-07-27 Thread Yixun Lan
HI Rob On 07/25/2018 07:29 AM, Rob Herring wrote: > On Thu, Jul 12, 2018 at 09:12:43PM +0000, Yixun Lan wrote: >> Document the MMC sub clock controller driver, the potential consumer >> of this driver is MMC or NAND. Also add three clock bindings IDs which >> pr

Re: [PATCH] arm64: dts: meson: Switch simple-mfd and syscon order

2018-07-26 Thread Yixun Lan
HI Neil On 07/26/2018 10:13 PM, Neil Armstrong wrote: > The order between "syscon" and "simple-mfd" is important because in these > particular cases, the node needs to be first a "simple-mfd" to expose > it's sub-nodes, and later on a "syscon" to permit other nodes to access > this register space

Re: [PATCH] arm64: dts: meson: Switch simple-mfd and syscon order

2018-07-26 Thread Yixun Lan
HI Neil On 07/26/2018 10:13 PM, Neil Armstrong wrote: > The order between "syscon" and "simple-mfd" is important because in these > particular cases, the node needs to be first a "simple-mfd" to expose > it's sub-nodes, and later on a "syscon" to permit other nodes to access > this register space

Re: [PATCH v2 1/3] clk: meson: add DT documentation for emmc clock controller

2018-07-23 Thread Yixun Lan
HI Kevin On 07/23/2018 10:12 PM, Kevin Hilman wrote: > Yixun Lan writes: > > [...] > >>> >>>> Second, we might like to convert eMMC driver to also use mmc-clkc model. >>> >>> IMO, this should be done as part of merging this series. Otherw

Re: [PATCH v2 1/3] clk: meson: add DT documentation for emmc clock controller

2018-07-23 Thread Yixun Lan
HI Kevin On 07/23/2018 10:12 PM, Kevin Hilman wrote: > Yixun Lan writes: > > [...] > >>> >>>> Second, we might like to convert eMMC driver to also use mmc-clkc model. >>> >>> IMO, this should be done as part of merging this series. Otherw

Re: [PATCH 3/3] clk: meson-g12a: Add EE Clock controller driver

2018-07-19 Thread Yixun Lan
On 07/19/2018 10:06 PM, Kevin Hilman wrote: > Yixun Lan writes: > > [...] > >>> >>> As Jerome said, I think consistency is an important goal, so leaving off >>> the "meson-" for drivers/clk is fine with me. >>> >> ok >>

Re: [PATCH 3/3] clk: meson-g12a: Add EE Clock controller driver

2018-07-19 Thread Yixun Lan
On 07/19/2018 10:06 PM, Kevin Hilman wrote: > Yixun Lan writes: > > [...] > >>> >>> As Jerome said, I think consistency is an important goal, so leaving off >>> the "meson-" for drivers/clk is fine with me. >>> >> ok >>

Re: [RFC PATCH v2 1/2] dt-bindings: nand: meson: add Amlogic NAND controller driver

2018-07-19 Thread Yixun Lan
Hi Boris: see my comments, thanks for the quick response On 07/19/18 17:57, Boris Brezillon wrote: > On Thu, 19 Jul 2018 17:46:11 +0800 > Yixun Lan wrote: > >> From: Liang Yang >> >> Add Amlogic NAND controller dt-bindings for Meson SoC, >> Current this dri

Re: [RFC PATCH v2 1/2] dt-bindings: nand: meson: add Amlogic NAND controller driver

2018-07-19 Thread Yixun Lan
Hi Boris: see my comments, thanks for the quick response On 07/19/18 17:57, Boris Brezillon wrote: > On Thu, 19 Jul 2018 17:46:11 +0800 > Yixun Lan wrote: > >> From: Liang Yang >> >> Add Amlogic NAND controller dt-bindings for Meson SoC, >> Current this dri

Re: [PATCH 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-07-19 Thread Yixun Lan
HI Boris On 07/19/18 16:39, Boris Brezillon wrote: > Hi Yixun, > > On Thu, 19 Jul 2018 16:13:47 +0800 > Yixun Lan wrote: > >>>>> You're doing DMA on those buffers, and devm_kzalloc() is not >>>>> DMA-friendly (returned buffers are not aligned

Re: [PATCH 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-07-19 Thread Yixun Lan
HI Boris On 07/19/18 16:39, Boris Brezillon wrote: > Hi Yixun, > > On Thu, 19 Jul 2018 16:13:47 +0800 > Yixun Lan wrote: > >>>>> You're doing DMA on those buffers, and devm_kzalloc() is not >>>>> DMA-friendly (returned buffers are not aligned

[RFC PATCH v2 0/2] mtd: rawnand: meson: add Amlogic NAND driver support

2018-07-19 Thread Yixun Lan
- how to construct mtd->name [1] https://lkml.kernel.org/r/20180613161314.14894-1-yixun@amlogic.com [2] https://lkml.kernel.org/r/20180712211244.11428-1-yixun@amlogic.com Liang Yang (1): dt-bindings: nand: meson: add Amlogic NAND controller driver Yixun Lan (1): mtd: rawnand: meson

[RFC PATCH v2 1/2] dt-bindings: nand: meson: add Amlogic NAND controller driver

2018-07-19 Thread Yixun Lan
From: Liang Yang Add Amlogic NAND controller dt-bindings for Meson SoC, Current this driver support GXBB/GXL/AXG platform. Signed-off-by: Liang Yang Signed-off-by: Yixun Lan --- .../bindings/mtd/amlogic,meson-nand.txt | 95 +++ 1 file changed, 95 insertions(+) create

[RFC PATCH v2 0/2] mtd: rawnand: meson: add Amlogic NAND driver support

2018-07-19 Thread Yixun Lan
- how to construct mtd->name [1] https://lkml.kernel.org/r/20180613161314.14894-1-yixun@amlogic.com [2] https://lkml.kernel.org/r/20180712211244.11428-1-yixun@amlogic.com Liang Yang (1): dt-bindings: nand: meson: add Amlogic NAND controller driver Yixun Lan (1): mtd: rawnand: meson

[RFC PATCH v2 1/2] dt-bindings: nand: meson: add Amlogic NAND controller driver

2018-07-19 Thread Yixun Lan
From: Liang Yang Add Amlogic NAND controller dt-bindings for Meson SoC, Current this driver support GXBB/GXL/AXG platform. Signed-off-by: Liang Yang Signed-off-by: Yixun Lan --- .../bindings/mtd/amlogic,meson-nand.txt | 95 +++ 1 file changed, 95 insertions(+) create

[RFC PATCH v2 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-07-19 Thread Yixun Lan
Add initial support for the Amlogic NAND flash controller which found in the Meson-GXBB/GXL/AXG SoCs. Signed-off-by: Liang Yang Signed-off-by: Yixun Lan --- drivers/mtd/nand/raw/Kconfig | 10 + drivers/mtd/nand/raw/Makefile |1 + drivers/mtd/nand/raw/meson_nand.c | 1333

[RFC PATCH v2 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-07-19 Thread Yixun Lan
Add initial support for the Amlogic NAND flash controller which found in the Meson-GXBB/GXL/AXG SoCs. Signed-off-by: Liang Yang Signed-off-by: Yixun Lan --- drivers/mtd/nand/raw/Kconfig | 10 + drivers/mtd/nand/raw/Makefile |1 + drivers/mtd/nand/raw/meson_nand.c | 1333

Re: [PATCH 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-07-19 Thread Yixun Lan
HI Boris: thanks for the quick response. On 07/19/18 03:08, Boris Brezillon wrote: > Hi Yixun, > > On Wed, 18 Jul 2018 17:38:56 +0800 > Yixun Lan wrote: > >>>> + >>>> +#define NFC_REG_CMD 0x00 >>>> +#define NFC_RE

Re: [PATCH 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-07-19 Thread Yixun Lan
HI Boris: thanks for the quick response. On 07/19/18 03:08, Boris Brezillon wrote: > Hi Yixun, > > On Wed, 18 Jul 2018 17:38:56 +0800 > Yixun Lan wrote: > >>>> + >>>> +#define NFC_REG_CMD 0x00 >>>> +#define NFC_RE

Re: [PATCH 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-07-18 Thread Yixun Lan
Hi Roris thanks for all your suggestions! It actually takes us some time to digest all your comments ;-) and get back to you on these questions. On 06/25/18 03:38, Boris Brezillon wrote: > > > Hi Yixun, > > On Wed, 13 Jun 2018 16:13:14 +0000 > Yixun Lan wrote: >

Re: [PATCH 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

2018-07-18 Thread Yixun Lan
Hi Roris thanks for all your suggestions! It actually takes us some time to digest all your comments ;-) and get back to you on these questions. On 06/25/18 03:38, Boris Brezillon wrote: > > > Hi Yixun, > > On Wed, 13 Jun 2018 16:13:14 +0000 > Yixun Lan wrote: >

Re: [PATCH 3/3] pinctrl: meson-g12a: add pinctrl driver support

2018-07-17 Thread Yixun Lan
HI Martin On 07/14/18 23:30, Martin Blumenstingl wrote: > Hi Yixun, > > On Wed, Jul 4, 2018 at 4:50 PM Yixun Lan wrote: >> >> Add the pinctrl driver for Meson-G12A SoC which share the similar IP as >> the previous Meson-AXG SoC. > my understanding is that:

Re: [PATCH 3/3] pinctrl: meson-g12a: add pinctrl driver support

2018-07-17 Thread Yixun Lan
HI Martin On 07/14/18 23:30, Martin Blumenstingl wrote: > Hi Yixun, > > On Wed, Jul 4, 2018 at 4:50 PM Yixun Lan wrote: >> >> Add the pinctrl driver for Meson-G12A SoC which share the similar IP as >> the previous Meson-AXG SoC. > my understanding is that:

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Yixun Lan
HI Jerome On 07/16/18 17:54, Jerome Brunet wrote: > +/* uart_ao_a_ee */ +static const unsigned int uart_ao_rx_a_c2_pins[]= { GPIOC_2 }; +static const unsigned int uart_ao_tx_a_c3_pins[]= { GPIOC_3 }; >>> >>> Same comment as Martin about naming consistency ... drop c2

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Yixun Lan
HI Jerome On 07/16/18 17:54, Jerome Brunet wrote: > +/* uart_ao_a_ee */ +static const unsigned int uart_ao_rx_a_c2_pins[]= { GPIOC_2 }; +static const unsigned int uart_ao_tx_a_c3_pins[]= { GPIOC_3 }; >>> >>> Same comment as Martin about naming consistency ... drop c2

Re: [PATCH 3/3] clk: meson-g12a: Add EE Clock controller driver

2018-07-16 Thread Yixun Lan
HI Kevin just want to clarify.. On 07/16/18 21:38, Kevin Hilman wrote: > Jerome Brunet writes: > >> On Tue, 2018-07-10 at 09:21 +0800, Yixun Lan wrote: >>> >>> On 07/10/18 05:53, Martin Blumenstingl wrote: >>>> On Mon, Jul 9, 2018 at 1:14 PM Jian Hu

Re: [PATCH 3/3] clk: meson-g12a: Add EE Clock controller driver

2018-07-16 Thread Yixun Lan
HI Kevin just want to clarify.. On 07/16/18 21:38, Kevin Hilman wrote: > Jerome Brunet writes: > >> On Tue, 2018-07-10 at 09:21 +0800, Yixun Lan wrote: >>> >>> On 07/10/18 05:53, Martin Blumenstingl wrote: >>>> On Mon, Jul 9, 2018 at 1:14 PM Jian Hu

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Yixun Lan
Hi Jerome thanks for the review, see my comments below On 07/16/18 00:16, Jerome Brunet wrote: > On Sat, 2018-07-14 at 23:27 +0000, Yixun Lan wrote: >> Add the pinctrl driver for Meson-G12A SoC which share the similar IP as >> the previous Meson-AXG SoC. >> >>

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Yixun Lan
Hi Jerome thanks for the review, see my comments below On 07/16/18 00:16, Jerome Brunet wrote: > On Sat, 2018-07-14 at 23:27 +0000, Yixun Lan wrote: >> Add the pinctrl driver for Meson-G12A SoC which share the similar IP as >> the previous Meson-AXG SoC. >> >>

[PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-14 Thread Yixun Lan
to the mux pin function. Please note, the GPIOE is actually located at AO (always on) bank. Acked-by: Martin Blumenstingl Signed-off-by: Xingyu Chen Signed-off-by: Yixun Lan --- drivers/pinctrl/meson/Kconfig |6 + drivers/pinctrl/meson/Makefile |1 + drivers

[PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-14 Thread Yixun Lan
to the mux pin function. Please note, the GPIOE is actually located at AO (always on) bank. Acked-by: Martin Blumenstingl Signed-off-by: Xingyu Chen Signed-off-by: Yixun Lan --- drivers/pinctrl/meson/Kconfig |6 + drivers/pinctrl/meson/Makefile |1 + drivers

[PATCH v2 0/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-14 Thread Yixun Lan
rg/r/20180704224511.29350-1-yixun@amlogic.com Yixun Lan (2): documentation: pinctrl: Add compatibles for Amlogic Meson G12A pin controllers pinctrl: meson-g12a: add pinctrl driver support .../bindings/pinctrl/meson,pinctrl.txt|2 + drivers/pinctrl/meson/Kconfig |6 + dri

[PATCH v2 1/2] documentation: pinctrl: Add compatibles for Amlogic Meson G12A pin controllers

2018-07-14 Thread Yixun Lan
Add new compatible name for Amlogic's Meson-G12A pin controllers, add a dt-binding header file which document the detail pin names. Acked-by: Martin Blumenstingl Signed-off-by: Xingyu Chen Signed-off-by: Yixun Lan --- .../bindings/pinctrl/meson,pinctrl.txt| 2 + include/dt-bindings

[PATCH v2 0/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-14 Thread Yixun Lan
rg/r/20180704224511.29350-1-yixun@amlogic.com Yixun Lan (2): documentation: pinctrl: Add compatibles for Amlogic Meson G12A pin controllers pinctrl: meson-g12a: add pinctrl driver support .../bindings/pinctrl/meson,pinctrl.txt|2 + drivers/pinctrl/meson/Kconfig |6 + dri

[PATCH v2 1/2] documentation: pinctrl: Add compatibles for Amlogic Meson G12A pin controllers

2018-07-14 Thread Yixun Lan
Add new compatible name for Amlogic's Meson-G12A pin controllers, add a dt-binding header file which document the detail pin names. Acked-by: Martin Blumenstingl Signed-off-by: Xingyu Chen Signed-off-by: Yixun Lan --- .../bindings/pinctrl/meson,pinctrl.txt| 2 + include/dt-bindings

Re: [PATCH 2/3] dt-bindings: pinctrl: meson-g12a: document pin name

2018-07-14 Thread Yixun Lan
HI Martin thanks for the comments On 07/14/2018 10:47 PM, Martin Blumenstingl wrote: > Hi Yixun, > > On Wed, Jul 4, 2018 at 4:49 PM Yixun Lan wrote: >> >> Document the pins for Amlogic's Meson-G12A SoC. > I suggest to combine patch 1 (adding the compatible

Re: [PATCH 2/3] dt-bindings: pinctrl: meson-g12a: document pin name

2018-07-14 Thread Yixun Lan
HI Martin thanks for the comments On 07/14/2018 10:47 PM, Martin Blumenstingl wrote: > Hi Yixun, > > On Wed, Jul 4, 2018 at 4:49 PM Yixun Lan wrote: >> >> Document the pins for Amlogic's Meson-G12A SoC. > I suggest to combine patch 1 (adding the compatible

Re: [PATCH 0/3] pinctrl: meson-g12a: add pinctrl driver support

2018-07-14 Thread Yixun Lan
Hi Martin see my comments On 07/14/2018 10:36 PM, Martin Blumenstingl wrote: > On Tue, Jul 10, 2018 at 12:07 AM Martin Blumenstingl > wrote: >> >> Hi Linus, >> >> On Mon, Jul 9, 2018 at 3:35 PM Linus Walleij >> wrote: >>> >&g

Re: [PATCH 0/3] pinctrl: meson-g12a: add pinctrl driver support

2018-07-14 Thread Yixun Lan
Hi Martin see my comments On 07/14/2018 10:36 PM, Martin Blumenstingl wrote: > On Tue, Jul 10, 2018 at 12:07 AM Martin Blumenstingl > wrote: >> >> Hi Linus, >> >> On Mon, Jul 9, 2018 at 3:35 PM Linus Walleij >> wrote: >>> >&g

Re: [PATCH v2 1/3] clk: meson: add DT documentation for emmc clock controller

2018-07-12 Thread Yixun Lan
Hi Rob, Jerome, Kevin see my comments On 07/13/18 08:15, Rob Herring wrote: > On Thu, Jul 12, 2018 at 5:29 PM Yixun Lan wrote: >> >> HI Rob >> >> see my comments >> >> On 07/12/2018 10:17 PM, Rob Herring wrote: >>> On Wed, Jul 11, 2018 at 8:47 PM

Re: [PATCH v2 1/3] clk: meson: add DT documentation for emmc clock controller

2018-07-12 Thread Yixun Lan
Hi Rob, Jerome, Kevin see my comments On 07/13/18 08:15, Rob Herring wrote: > On Thu, Jul 12, 2018 at 5:29 PM Yixun Lan wrote: >> >> HI Rob >> >> see my comments >> >> On 07/12/2018 10:17 PM, Rob Herring wrote: >>> On Wed, Jul 11, 2018 at 8:47 PM

Re: [PATCH v2 1/3] clk: meson: add DT documentation for emmc clock controller

2018-07-12 Thread Yixun Lan
HI Rob see my comments On 07/12/2018 10:17 PM, Rob Herring wrote: > On Wed, Jul 11, 2018 at 8:47 PM Yixun Lan wrote: >> >> Hi Rob >> >> see my comments >> >> On 07/12/18 03:43, Rob Herring wrote: >>> On Tue, Jul 10, 2018 at 04:36:56PM +,

Re: [PATCH v2 1/3] clk: meson: add DT documentation for emmc clock controller

2018-07-12 Thread Yixun Lan
HI Rob see my comments On 07/12/2018 10:17 PM, Rob Herring wrote: > On Wed, Jul 11, 2018 at 8:47 PM Yixun Lan wrote: >> >> Hi Rob >> >> see my comments >> >> On 07/12/18 03:43, Rob Herring wrote: >>> On Tue, Jul 10, 2018 at 04:36:56PM +,

[PATCH v3 2/2] clk: meson: add sub MMC clock controller driver

2018-07-12 Thread Yixun Lan
G platform. To specify which clock the MMC or NAND driver may consume, the preprocessor macros in the dt-bindings/clock/emmc-clkc.h header can be used in the device tree sources. Signed-off-by: Yixun Lan --- drivers/clk/meson/Kconfig| 9 + drivers/clk/meson/Makefile | 1 + drivers/clk/meson/

[PATCH v3 2/2] clk: meson: add sub MMC clock controller driver

2018-07-12 Thread Yixun Lan
G platform. To specify which clock the MMC or NAND driver may consume, the preprocessor macros in the dt-bindings/clock/emmc-clkc.h header can be used in the device tree sources. Signed-off-by: Yixun Lan --- drivers/clk/meson/Kconfig| 9 + drivers/clk/meson/Makefile | 1 + drivers/clk/meson/

[PATCH v3 1/2] clk: meson: add DT documentation for emmc clock controller

2018-07-12 Thread Yixun Lan
Document the MMC sub clock controller driver, the potential consumer of this driver is MMC or NAND. Also add three clock bindings IDs which provided by this driver. Signed-off-by: Yixun Lan --- .../bindings/clock/amlogic,mmc-clkc.txt | 31 +++ .../clock/amlogic,meson-mmc

[PATCH v3 1/2] clk: meson: add DT documentation for emmc clock controller

2018-07-12 Thread Yixun Lan
Document the MMC sub clock controller driver, the potential consumer of this driver is MMC or NAND. Also add three clock bindings IDs which provided by this driver. Signed-off-by: Yixun Lan --- .../bindings/clock/amlogic,mmc-clkc.txt | 31 +++ .../clock/amlogic,meson-mmc

[PATCH v3 0/2] clk: meson: add a sub EMMC clock controller support

2018-07-12 Thread Yixun Lan
[2] https://lkml.kernel.org/r/20180703145716.31860-1-yixun@amlogic.com [3] https://lkml.kernel.org/r/20180710163658.6175-1-yixun@amlogic.com Yixun Lan (2): clk: meson: add DT documentation for emmc clock controller clk: meson: add sub MMC clock controller driver .../bindings/clock

[PATCH v3 0/2] clk: meson: add a sub EMMC clock controller support

2018-07-12 Thread Yixun Lan
[2] https://lkml.kernel.org/r/20180703145716.31860-1-yixun@amlogic.com [3] https://lkml.kernel.org/r/20180710163658.6175-1-yixun@amlogic.com Yixun Lan (2): clk: meson: add DT documentation for emmc clock controller clk: meson: add sub MMC clock controller driver .../bindings/clock

Re: [PATCH v2 3/3] clk: meson: add sub MMC clock controller driver

2018-07-12 Thread Yixun Lan
Hi Jerome thanks for the review On 07/12/18 17:09, Jerome Brunet wrote: > On Tue, 2018-07-10 at 16:36 +0000, Yixun Lan wrote: >> The patch will add a MMC clock controller driver which used by MMC or NAND, >> It provide a mux and divider clock, and three phase clocks - core, tx

Re: [PATCH v2 3/3] clk: meson: add sub MMC clock controller driver

2018-07-12 Thread Yixun Lan
Hi Jerome thanks for the review On 07/12/18 17:09, Jerome Brunet wrote: > On Tue, 2018-07-10 at 16:36 +0000, Yixun Lan wrote: >> The patch will add a MMC clock controller driver which used by MMC or NAND, >> It provide a mux and divider clock, and three phase clocks - core, tx

Re: [PATCH v2 2/3] clk: meson: add sub MMC clock dt-bindings IDs

2018-07-11 Thread Yixun Lan
Hi Rob On 07/12/18 03:45, Rob Herring wrote: > On Tue, Jul 10, 2018 at 04:36:57PM +0000, Yixun Lan wrote: >> Add two clock bindings IDs which provided by the MMC clock controller, >> These two clocks will be used by MMC or NAND driver. > > I count 3 ids. I will update this

Re: [PATCH v2 2/3] clk: meson: add sub MMC clock dt-bindings IDs

2018-07-11 Thread Yixun Lan
Hi Rob On 07/12/18 03:45, Rob Herring wrote: > On Tue, Jul 10, 2018 at 04:36:57PM +0000, Yixun Lan wrote: >> Add two clock bindings IDs which provided by the MMC clock controller, >> These two clocks will be used by MMC or NAND driver. > > I count 3 ids. I will update this

Re: [PATCH v2 1/3] clk: meson: add DT documentation for emmc clock controller

2018-07-11 Thread Yixun Lan
Hi Rob see my comments On 07/12/18 03:43, Rob Herring wrote: > On Tue, Jul 10, 2018 at 04:36:56PM +0000, Yixun Lan wrote: >> Document the MMC sub clock controller driver, the potential consumer >> of this driver is MMC or NAND. > > So you all have decided to properly m

Re: [PATCH v2 1/3] clk: meson: add DT documentation for emmc clock controller

2018-07-11 Thread Yixun Lan
Hi Rob see my comments On 07/12/18 03:43, Rob Herring wrote: > On Tue, Jul 10, 2018 at 04:36:56PM +0000, Yixun Lan wrote: >> Document the MMC sub clock controller driver, the potential consumer >> of this driver is MMC or NAND. > > So you all have decided to properly m

Re: [PATCH 3/3] clk: meson-g12a: Add EE Clock controller driver

2018-07-10 Thread Yixun Lan
Hi Jerome: just one comment On 07/10/2018 05:54 PM, Jerome Brunet wrote: > On Tue, 2018-07-10 at 09:21 +0800, Yixun Lan wrote: >> HI Martin >> >> >> On 07/10/18 05:53, Martin Blumenstingl wrote: >>> On Mon, Jul 9, 2018 at 1:14 PM Jian Hu wrote: >>&g

Re: [PATCH 3/3] clk: meson-g12a: Add EE Clock controller driver

2018-07-10 Thread Yixun Lan
Hi Jerome: just one comment On 07/10/2018 05:54 PM, Jerome Brunet wrote: > On Tue, 2018-07-10 at 09:21 +0800, Yixun Lan wrote: >> HI Martin >> >> >> On 07/10/18 05:53, Martin Blumenstingl wrote: >>> On Mon, Jul 9, 2018 at 1:14 PM Jian Hu wrote: >>&g

Re: [PATCH 0/3] pinctrl: meson-g12a: add pinctrl driver support

2018-07-10 Thread Yixun Lan
HI Martin On 07/10/18 06:07, Martin Blumenstingl wrote: > Hi Linus, > > On Mon, Jul 9, 2018 at 3:35 PM Linus Walleij wrote: >> >> On Wed, Jul 4, 2018 at 4:48 PM Yixun Lan wrote: >>> >>> This patch series try to add pinctrl driver support for >>&

Re: [PATCH 0/3] pinctrl: meson-g12a: add pinctrl driver support

2018-07-10 Thread Yixun Lan
HI Martin On 07/10/18 06:07, Martin Blumenstingl wrote: > Hi Linus, > > On Mon, Jul 9, 2018 at 3:35 PM Linus Walleij wrote: >> >> On Wed, Jul 4, 2018 at 4:48 PM Yixun Lan wrote: >>> >>> This patch series try to add pinctrl driver support for >>&

[PATCH v2 3/3] clk: meson: add sub MMC clock controller driver

2018-07-10 Thread Yixun Lan
G platform. To specify which clock the MMC or NAND driver may consume, the preprocessor macros in the dt-bindings/clock/emmc-clkc.h header can be used in the device tree sources. Signed-off-by: Yixun Lan --- drivers/clk/meson/Kconfig| 9 + drivers/clk/meson/Makefile | 1 + drivers/clk/meson/

[PATCH v2 0/3] clk: meson: add a sub EMMC clock controller support

2018-07-10 Thread Yixun Lan
- update compatible name - adjust file name - divider probe() into small functions, and re-use them [1] https://lkml.kernel.org/r/20180628090034.0637a062@xps13 [2] https://lkml.kernel.org/r/20180703145716.31860-1-yixun@amlogic.com Yixun Lan (3): clk: meson: add DT documentation for emmc

[PATCH v2 2/3] clk: meson: add sub MMC clock dt-bindings IDs

2018-07-10 Thread Yixun Lan
Add two clock bindings IDs which provided by the MMC clock controller, These two clocks will be used by MMC or NAND driver. Signed-off-by: Yixun Lan --- .../dt-bindings/clock/amlogic,meson-mmc-clkc.h | 16 1 file changed, 16 insertions(+) create mode 100644 include/dt

[PATCH v2 3/3] clk: meson: add sub MMC clock controller driver

2018-07-10 Thread Yixun Lan
G platform. To specify which clock the MMC or NAND driver may consume, the preprocessor macros in the dt-bindings/clock/emmc-clkc.h header can be used in the device tree sources. Signed-off-by: Yixun Lan --- drivers/clk/meson/Kconfig| 9 + drivers/clk/meson/Makefile | 1 + drivers/clk/meson/

  1   2   3   4   5   6   7   8   >