Re: [U-Boot] [PATCH v7 4/6] drivers: Enable FPGA driver build on SPL

[Chee, Tien Fong]

On Sab, 2017-06-03 at 13:29 +0200, Marek Vasut wrote:
> On 05/29/2017 06:00 AM, tien.fong.c...@intel.com wrote:
> > 
> > From: Tien Fong Chee 
> > 
> > Enable FPGA driver build for SPL because FPGA driver is needed for
> > SPL
> > to configure and getting DDR up before loading U-boot into DDR and
> > booting from there.
> > 
> > Signed-off-by: Tien Fong Chee 
> The thing I fail to understand is why you're sending the patches in
> seemingly random order. You add/change piece of code, then enable
> something and then add the relevant code you're enabling. I really do
> not understand that.
> 
> The patches should have some logical order -- change existing code,
> add
> new code, enable stuff.
> 
> Anyway, please collect the Acks, fix the nit and resubmit.
> 
Sorry for confusing, i enabled the stuff just to detect any break and
bugs as early as possible. I will follow the logical order next time :)
.
> > 
> > ---
> >  drivers/Makefile | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/Makefile b/drivers/Makefile
> > index 64c39d3..4478212 100644
> > --- a/drivers/Makefile
> > +++ b/drivers/Makefile
> > @@ -48,6 +48,7 @@ obj-$(CONFIG_OMAP_USB_PHY) += usb/phy/
> >  obj-$(CONFIG_SPL_SATA_SUPPORT) += block/
> >  obj-$(CONFIG_SPL_USB_HOST_SUPPORT) += block/
> >  obj-$(CONFIG_SPL_MMC_SUPPORT) += block/
> > +obj-$(CONFIG_SPL_FPGA_SUPPORT) += fpga/
> >  endif
> >  
> >  ifdef CONFIG_TPL_BUILD
> > 
> 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v7 3/6] arm: socfpga: Enable FPGA driver on SPL

[Chee, Tien Fong]

On Isn, 2017-06-05 at 11:32 +0800, Chee, Tien Fong wrote:
> > > 
> > 
> On Sab, 2017-06-03 at 13:27 +0200, Marek Vasut wrote:
> > 
> > On 05/29/2017 06:00 AM, tien.fong.c...@intel.com wrote:
> > > 
> > > 
> > > From: Tien Fong Chee 
> > > 
> > > This patch is for enabling the FPGA driver support on SPL.
> > > 
> > > Signed-off-by: Tien Fong Chee 
> > > ---
> > >  include/configs/socfpga_common.h | 1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/include/configs/socfpga_common.h
> > > b/include/configs/socfpga_common.h
> > > index da7e4ad..e74e7eb 100644
> > > --- a/include/configs/socfpga_common.h
> > > +++ b/include/configs/socfpga_common.h
> > > @@ -110,6 +110,7 @@
> > >  #define CONFIG_FPGA
> > >  #define CONFIG_FPGA_ALTERA
> > >  #define CONFIG_FPGA_SOCFPGA
> > > +#define CONFIG_SPL_FPGA_SUPPORT
> > This should be Kconfig symbol selected in board config. Why is it
> > not
> > Kconfig symbol ?
> > 
Ahhaijust recalled there is a very important reason i did this
because CONFIG_CMD_FPGA which is wrapping all these FPGA related CONFIG
is not part of kconfig. During the build, kconfig and Macro #define are
in two different phase/order, so we might having build issue during
kconfig build(1st phase) when some Macro #define which is 2nd phase are
required.

> I have a patch that is specific for converting all FPGA related Macro
> config by tool created which will be submited after this patch set.
> Let me know if you prefer kconfig symbol.
> > > 
> > >  #define CONFIG_FPGA_COUNT1
> > >  #endif
> > >  #endif
> > > 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 1/2] ARM: dts: exynos5422-odroidxu3: add the LDO's nodes

[Minkyu Kang]

Hi

2017. 5. 30. 15:08에 "Jaehoon Chung" 님이 작성:

Add the LDO's nodes that taken from Linux Kernel.

Signed-off-by: Jaehoon Chung 
---
 arch/arm/dts/exynos5422-odroidxu3.dts | 209 ++

 1 file changed, 209 insertions(+)

diff --git a/arch/arm/dts/exynos5422-odroidxu3.dts
b/arch/arm/dts/exynos5422-odroidxu3.dts
index 690c747..b6f4333 100644
--- a/arch/arm/dts/exynos5422-odroidxu3.dts
+++ b/arch/arm/dts/exynos5422-odroidxu3.dts
@@ -40,6 +40,215 @@
s2mps11_pmic@66 {
compatible = "samsung,s2mps11-pmic";
reg = <0x66>;
+   voltage-regulators {
+   ldo1_reg: LDO1 {
+   regulator-name = "vdd_ldo1";
+   regulator-min-microvolt = <100>;
+   regulator-max-microvolt = <100>;
+   regulator-always-on;
+   };
+
+   ldo3_reg: LDO3 {
+   regulator-name = "vddq_mmc0";
+   regulator-min-microvolt = <180>;
+   regulator-max-microvolt = <180>;
+   };
+
+   ldo4_reg: LDO4 {
+   regulator-name = "vdd_adc";
+   regulator-min-microvolt = <180>;
+   regulator-max-microvolt = <180>;
+   };
+
+   ldo5_reg: LDO5 {
+   regulator-name = "vdd_ldo5";
+   regulator-min-microvolt = <180>;
+   regulator-max-microvolt = <180>;
+   regulator-always-on;
+   };
+
+   ldo6_reg: LDO6 {
+   regulator-name = "vdd_ldo6";
+   regulator-min-microvolt = <100>;
+   regulator-max-microvolt = <100>;
+   regulator-always-on;
+   };
+
+   ldo7_reg: LDO7 {
+   regulator-name = "vdd_ldo7";
+   regulator-min-microvolt = <180>;
+   regulator-max-microvolt = <180>;
+   regulator-always-on;
+   };
+
+   ldo8_reg: LDO8 {
+   regulator-name = "vdd_ldo8";
+   regulator-min-microvolt = <180>;
+   regulator-max-microvolt = <180>;
+   regulator-always-on;
+   };
+
+   ldo9_reg: LDO9 {
+   regulator-name = "vdd_ldo9";
+   regulator-min-microvolt = <300>;
+   regulator-max-microvolt = <300>;
+   regulator-always-on;
+   };
+
+   ldo10_reg: LDO10 {
+   regulator-name = "vdd_ldo10";
+   regulator-min-microvolt = <180>;
+   regulator-max-microvolt = <180>;
+   regulator-always-on;
+   };
+
+   ldo11_reg: LDO11 {
+   regulator-name = "vdd_ldo11";
+   regulator-min-microvolt = <100>;
+   regulator-max-microvolt = <100>;
+   regulator-always-on;
+   };
+
+   ldo12_reg: LDO12 {
+   regulator-name = "vdd_ldo12";
+   regulator-min-microvolt = <180>;
+   regulator-max-microvolt = <180>;
+   regulator-always-on;
+   };
+
+   ldo13_reg: LDO13 {
+   regulator-name = "vddq_mmc2";
+   regulator-min-microvolt = <280>;
+   regulator-max-microvolt = <280>;
+   };
+
+

Re: [U-Boot] [PATCH 1/2] ARM: dts: exynos4: change the nodes relevant to mmc/sd

[Minkyu Kang]

Hi

2017. 5. 30. 10:05에 "Jaehoon Chung" 님이 작성:

Change the nodes relevant to mmc/sd for using DM.
compatible are also changed to each SoCs.

Signed-off-by: Jaehoon Chung 
---
 arch/arm/dts/exynos4.dtsi  | 25 +--
 arch/arm/dts/exynos4210-origen.dts | 24 --
 arch/arm/dts/exynos4210-trats.dts  | 40 +---
 arch/arm/dts/exynos4210-universal_c210.dts | 36 +
 arch/arm/dts/exynos4412-odroid.dts | 50
--
 arch/arm/dts/exynos4412-trats2.dts | 49
+++--
 arch/arm/dts/exynos4x12.dtsi   | 13 
 7 files changed, 94 insertions(+), 143 deletions(-)

diff --git a/arch/arm/dts/exynos4.dtsi b/arch/arm/dts/exynos4.dtsi
index a5a00c8..53d39dc 100644
--- a/arch/arm/dts/exynos4.dtsi
+++ b/arch/arm/dts/exynos4.dtsi
@@ -122,44 +122,49 @@
interrupts = <7 63 0>;
};

-   sdhci@1251 {
+   sdhci0: sdhci@1251 {
#address-cells = <1>;
#size-cells = <0>;
-   compatible = "samsung,exynos-mmc";
+   compatible = "samsung,exynos4412-sdhci";
reg = <0x1251 0x1000>;
interrupts = <0 75 0>;
+   status = "disabled";
};

-   sdhci@1252 {
+   sdhci1: sdhci@1252 {
#address-cells = <1>;
#size-cells = <0>;
-   compatible = "samsung,exynos-mmc";
+   compatible = "samsung,exynos4412-sdhci";
reg = <0x1252 0x1000>;
interrupts = <0 76 0>;
+   status = "disabled";
};

-   sdhci@1253 {
+   sdhci2: sdhci@1253 {
#address-cells = <1>;
#size-cells = <0>;
-   compatible = "samsung,exynos-mmc";
+   compatible = "samsung,exynos4412-sdhci";
reg = <0x1253 0x1000>;
interrupts = <0 77 0>;
+   status = "disabled";
};

-   sdhci@1254 {
+   sdhci3: sdhci@1254 {
#address-cells = <1>;
#size-cells = <0>;
-   compatible = "samsung,exynos-mmc";
+   compatible = "samsung,exynos4412-sdhci";
reg = <0x1254 0x1000>;
interrupts = <0 78 0>;
+   status = "disabled";
};

-   dwmmc@1255 {
+   mshc_0: dwmmc@1255 {
#address-cells = <1>;
#size-cells = <0>;
-   compatible = "samsung,exynos-dwmmc";
+   compatible = "samsung,exynos4412-dw-mshc";
reg = <0x1255 0x1000>;
interrupts = <0 131 0>;
+   status = "disabled";
};

 };
diff --git a/arch/arm/dts/exynos4210-origen.dts b/arch/arm/dts/exynos4210-
origen.dts
index 26c4d7f..a13d033 100644
--- a/arch/arm/dts/exynos4210-origen.dts
+++ b/arch/arm/dts/exynos4210-origen.dts
@@ -22,24 +22,12 @@
aliases {
serial0 = "/serial@1380";
console = "/serial@1382";
-   mmc2 = "/sdhci@1253";
-   };
-
-   sdhci@1251 {
-   status = "disabled";
-   };
-
-   sdhci@1252 {
-   status = "disabled";
-   };
-
-   sdhci@1253 {
-   samsung,bus-width = <4>;
-   samsung,timing = <1 2 3>;
-   cd-gpios = < 2 0>;
};
+};

-   sdhci@1254 {
-   status = "disabled";
-   };
+ {
+   samsung,bus-width = <4>;
+   samsung,timing = <1 2 3>;
+   cd-gpios = < 2 0>;
+   status = "okay";
 };
diff --git a/arch/arm/dts/exynos4210-trats.dts b/arch/arm/dts/exynos4210-
trats.dts
index ac422e4..2abe3e9 100644
--- a/arch/arm/dts/exynos4210-trats.dts
+++ b/arch/arm/dts/exynos4210-trats.dts
@@ -21,8 +21,6 @@
aliases {
serial0 = "/serial@1380";
console = "/serial@1382";
-   mmc0 = "/sdhci@1251";
-   mmc2 = "/sdhci@1253";
i2c8 = _fg;
};

@@ -91,30 +89,6 @@
samsung,dsim-device-reverse-panel = <1>;
};

-   sdhci@1251 {
-   samsung,bus-width = <8>;
-   samsung,timing = <1 3 3>;
-   pwr-gpios = < 2 0>;
-   };
-
-   sdhci@1252 {
-   status = "disabled";
-   };
-
-   sdhci@1253 {
-   samsung,bus-width = <4>;
-   samsung,timing = <1 2 3>;
-   cd-gpios = < 4 0>;
-   };
-
-   sdhci@1254 {
-   status = "disabled";
-   };
-
-   dwmmc@1255 {
-   status = "disabled";
-   };
-
i2c_fg: fuel-gauge {
compatible = "i2c-gpio";
gpios = < 1 0>,/* sda */
@@ -265,3 +239,17 @@
};
};
 };
+
+ 

Re: [U-Boot] [PATCH 2/2] configs: enable the CONFIG_DM_MMC for exynos4 series

[Minkyu Kang]

Hi


2017. 5. 30. 10:04에 "Jaehoon Chung" 님이 작성:

Enable the CONFIG_DM_MMC for exynos4 series.

Signed-off-by: Jaehoon Chung 
---
 configs/odroid_defconfig| 1 +
 configs/origen_defconfig| 1 +
 configs/s5pc210_universal_defconfig | 1 +
 configs/trats2_defconfig| 1 +
 configs/trats_defconfig | 1 +
 5 files changed, 5 insertions(+)

diff --git a/configs/odroid_defconfig b/configs/odroid_defconfig
index a28058a..45c8904 100644
--- a/configs/odroid_defconfig
+++ b/configs/odroid_defconfig
@@ -39,6 +39,7 @@ CONFIG_ISO_PARTITION=y
 CONFIG_OF_CONTROL=y
 CONFIG_DFU_MMC=y
 CONFIG_DM_I2C_COMPAT=y
+CONFIG_DM_MMC=y
 CONFIG_MMC_DW=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_SDMA=y
diff --git a/configs/origen_defconfig b/configs/origen_defconfig
index dfb08fb..573c9e2 100644
--- a/configs/origen_defconfig
+++ b/configs/origen_defconfig
@@ -32,6 +32,7 @@ CONFIG_CMD_FS_GENERIC=y
 CONFIG_ISO_PARTITION=y
 CONFIG_OF_CONTROL=y
 CONFIG_DFU_MMC=y
+CONFIG_DM_MMC=y
 CONFIG_MMC_DW=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_SDMA=y
diff --git a/configs/s5pc210_universal_defconfig
b/configs/s5pc210_universal_defconfig
index c342fa9..021e6c5 100644
--- a/configs/s5pc210_universal_defconfig
+++ b/configs/s5pc210_universal_defconfig
@@ -33,6 +33,7 @@ CONFIG_ISO_PARTITION=y
 CONFIG_OF_CONTROL=y
 CONFIG_DFU_MMC=y
 CONFIG_SYS_I2C_S3C24X0=y
+CONFIG_DM_MMC=y
 CONFIG_MMC_DW=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_SDMA=y
diff --git a/configs/trats2_defconfig b/configs/trats2_defconfig
index 8c92f2f..16ac27f 100644
--- a/configs/trats2_defconfig
+++ b/configs/trats2_defconfig
@@ -37,6 +37,7 @@ CONFIG_OF_CONTROL=y
 CONFIG_DFU_MMC=y
 CONFIG_DM_I2C_GPIO=y
 CONFIG_SYS_I2C_S3C24X0=y
+CONFIG_DM_MMC=y
 CONFIG_MMC_DW=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_SDMA=y
diff --git a/configs/trats_defconfig b/configs/trats_defconfig
index 09797c3..0f2445b 100644
--- a/configs/trats_defconfig
+++ b/configs/trats_defconfig
@@ -36,6 +36,7 @@ CONFIG_OF_CONTROL=y
 CONFIG_DFU_MMC=y
 CONFIG_DM_I2C_GPIO=y
 CONFIG_SYS_I2C_S3C24X0=y
+CONFIG_DM_MMC=y
 CONFIG_MMC_DW=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_SDMA=y
--
2.10.2

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


applied to u-boot-samsung

Thanks
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v7 3/6] arm: socfpga: Enable FPGA driver on SPL

[Chee, Tien Fong]

On Sab, 2017-06-03 at 13:27 +0200, Marek Vasut wrote:
> On 05/29/2017 06:00 AM, tien.fong.c...@intel.com wrote:
> > 
> > From: Tien Fong Chee 
> > 
> > This patch is for enabling the FPGA driver support on SPL.
> > 
> > Signed-off-by: Tien Fong Chee 
> > ---
> >  include/configs/socfpga_common.h | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/include/configs/socfpga_common.h
> > b/include/configs/socfpga_common.h
> > index da7e4ad..e74e7eb 100644
> > --- a/include/configs/socfpga_common.h
> > +++ b/include/configs/socfpga_common.h
> > @@ -110,6 +110,7 @@
> >  #define CONFIG_FPGA
> >  #define CONFIG_FPGA_ALTERA
> >  #define CONFIG_FPGA_SOCFPGA
> > +#define CONFIG_SPL_FPGA_SUPPORT
> This should be Kconfig symbol selected in board config. Why is it not
> Kconfig symbol ?
> 
I have a patch that is specific for converting all FPGA related Macro
config by tool created which will be submited after this patch set.
Let me know if you prefer kconfig symbol.
> > 
> >  #define CONFIG_FPGA_COUNT  1
> >  #endif
> >  #endif
> > 
> 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Please pull u-boot-fdt (take 4)

[Tom Rini]

On Fri, Jun 02, 2017 at 05:59:04PM -0600, Simon Glass wrote:

> Hi Tom,
> 
> This includes moving the error message to stderr so that buildman picks it up.
> 
> 
> 
> The following changes since commit 46bac66b20da6d50e757bdca74703153f233090b:
> 
>   sandbox: Move to use live tree (2017-06-01 07:03:17 -0600)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to 99ed4a2e979150879fb70aea71898709536375d3:
> 
>   fdt: Drop fdt_select.py (2017-06-02 10:18:20 -0600)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 01/33] mmc: select the available type from host_caps and card_caps

[Jaehoon Chung]

Hi Ziyuan,

On 06/05/2017 09:50 AM, Ziyuan wrote:
> hi Jaehoon,
> 
> On 05/25/2017 09:08 PM, Jaehoon Chung wrote:
>> Hi Ziyuan,
>>
>> On 05/25/2017 05:12 PM, Ziyuan wrote:
>>> hi Jaehoon,
>>>
>>> On 05/16/2017 09:55 AM, Jaehoon Chung wrote:
 Hi Ziyuan,

 On 05/16/2017 10:15 AM, Ziyuan wrote:
> hi Simon & Jaehoon,
>
> On 05/16/2017 08:18 AM, Simon Glass wrote:
>> Hi Ziyuan,
>>
>> On 15 May 2017 at 00:06, Ziyuan Xu  wrote:
>>> The original implementation select HS timing by default, add available
>>> type selection for higher speed mode compatibility, such as hs200,
>>> hs400, hs400es.
>>>
>>> By the way, we assume that card run at 1.8V or 1.2V I/O when its timing
>>> is ddr52/hs200/hs400(es).
>>>
>>> Signed-off-by: Ziyuan Xu 
>>> ---
>>>
>>> drivers/mmc/mmc.c | 59 
>>> ++-
>>> include/mmc.h | 16 +++
>>> 2 files changed, 74 insertions(+), 1 deletion(-)
>>>
>> Is there a cover letter for this series, please?
> This patchset is used for hs200/hs400/ddr52 mode of eMMC device, and 
> fixes some bug for dw_mmc & sdhci controller.
> It's only valid in U-Boot stage, we still use 'High Speed' in SPL.
>
> I tested it on evb-rk3288 board(eMMC 4.5) and evb-rk3388 board(eMMC 5.0).
>> I just reviewed an MMC series at add higher speed support. I'm not
>> sure but I suspect these overlap.
> Ha, I just reviewed Vignesh's patches, it focuses on uhs mode of sd card. 
> It looks to me.
> But some details are not the same as mine. Anyway, what do you think?
 I didn't review both yet..After checking, i will share my opinion. how 
 about?
>>> *ping*...
>>> Did you test this patchset on Samsung platform? As far as I know, Samsung 
>>> SoCs also make use of dw_mmc and sdhci controllers.
>> some details are not the same as yours..but some points are duplicated.
>> DWMMC and  sdhci controller side are not duplicated..
>>
>> As i am feeling..your patches are based on Linux kernel code..right?
> 
> Yup, the kernel's mmc driver is stabilized and it was tested on so many 
> various boards.
> Simon had review Jean-jacques's patch,  but a little glitch need to be 
> debugged.  So I should wait for him, and rebase?

I will make a branch for testing about HS200 and UHS-I..then you can make the 
patches based on it.
Well, after making branch, i will share it at mailing. Thanks for your effort!

Best Regards,
Jaehoon Chung

> 
>> Regards,
>> Simon
>>
>>
>>
>
>
>


>>>
>>>
>>>
>>>
>>
>>
>>
> 
> 
> 
> 
> 

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] Please pull u-boot-x86

[Bin Meng]

Hi Tom,

The following changes since commit 5cafcbab580bac6c7f44ca4a164c422b7e1ecf20:

  Merge git://git.denx.de/u-boot-net (2017-06-03 18:05:28 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-x86.git

for you to fetch changes up to d7f7ba36b2edecb16393f8373cca833adb8ea6c3:

  x86: fsp: Remove the call to set up internal uart in fsp_init()
(2017-06-05 08:55:22 +0800)


Bin Meng (6):
  x86: baytrail: Change "fsp, mrc-init-tseg-size" default value to 1
  x86: baytrail: Remove "serial-debug-port-*" settings
  x86: baytrail: Use macros instead of magic numbers for FSP settings
  x86: baytrail: Change lpe/lpss-sio/scc FSP properties to integer
  x86: baytrail: Fix boot hang with a debug build
  x86: fsp: Remove the call to set up internal uart in fsp_init()

 arch/x86/cpu/baytrail/Kconfig|  4 
 arch/x86/cpu/baytrail/early_uart.c   |  5 +
 arch/x86/cpu/baytrail/fsp_configs.c  | 50
++
 arch/x86/dts/bayleybay.dts   | 25
-
 arch/x86/dts/baytrail_som-db5800-som-6867.dts| 21 ++---
 arch/x86/dts/conga-qeval20-qa3-e3845.dts | 35
++-
 arch/x86/dts/dfi-bt700.dtsi  | 35
++-
 arch/x86/dts/minnowmax.dts   | 37
++---
 arch/x86/include/asm/arch-baytrail/fsp/fsp_configs.h | 87
+++
 arch/x86/include/asm/arch-baytrail/fsp/fsp_vpd.h |  9 -
 arch/x86/lib/fsp/fsp_support.c   |  4 
 doc/device-tree-bindings/misc/intel,baytrail-fsp.txt | 88

 12 files changed, 230 insertions(+), 170 deletions(-)

Regards,
Bin
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 01/33] mmc: select the available type from host_caps and card_caps

[Ziyuan]

hi Jaehoon,

On 05/25/2017 09:08 PM, Jaehoon Chung wrote:

Hi Ziyuan,

On 05/25/2017 05:12 PM, Ziyuan wrote:

hi Jaehoon,

On 05/16/2017 09:55 AM, Jaehoon Chung wrote:

Hi Ziyuan,

On 05/16/2017 10:15 AM, Ziyuan wrote:

hi Simon & Jaehoon,

On 05/16/2017 08:18 AM, Simon Glass wrote:

Hi Ziyuan,

On 15 May 2017 at 00:06, Ziyuan Xu  wrote:

The original implementation select HS timing by default, add available
type selection for higher speed mode compatibility, such as hs200,
hs400, hs400es.

By the way, we assume that card run at 1.8V or 1.2V I/O when its timing
is ddr52/hs200/hs400(es).

Signed-off-by: Ziyuan Xu 
---

drivers/mmc/mmc.c | 59 
++-
include/mmc.h | 16 +++
2 files changed, 74 insertions(+), 1 deletion(-)


Is there a cover letter for this series, please?

This patchset is used for hs200/hs400/ddr52 mode of eMMC device, and fixes some bug 
for dw_mmc & sdhci controller.
It's only valid in U-Boot stage, we still use 'High Speed' in SPL.

I tested it on evb-rk3288 board(eMMC 4.5) and evb-rk3388 board(eMMC 5.0).

I just reviewed an MMC series at add higher speed support. I'm not
sure but I suspect these overlap.

Ha, I just reviewed Vignesh's patches, it focuses on uhs mode of sd card. It 
looks to me.
But some details are not the same as mine. Anyway, what do you think?

I didn't review both yet..After checking, i will share my opinion. how about?

*ping*...
Did you test this patchset on Samsung platform? As far as I know, Samsung SoCs 
also make use of dw_mmc and sdhci controllers.

some details are not the same as yours..but some points are duplicated.
DWMMC and  sdhci controller side are not duplicated..

As i am feeling..your patches are based on Linux kernel code..right?


Yup, the kernel's mmc driver is stabilized and it was tested on so many 
various boards.
Simon had review Jean-jacques's patch,  but a little glitch need to be 
debugged.  So I should wait for him, and rebase?



Regards,
Simon






















___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v4 1/5] GPT: read partition table from device into a data structure

[alison]

From: Alison Chaiken 

Make the partition table available for modification by reading it from
the user-specified device into a linked list.   Provide an accessor
function for command-line testing.

Changes since v3:
Fixed problem with NULL termination of the newly created uuid[]
member of the partitions list in allocate_disk_part.   This
version has been tested in the u-boot sandbox.

Signed-off-by: Alison Chaiken 
---
 cmd/gpt.c  | 116 +
 include/part.h |   7 
 2 files changed, 123 insertions(+)

diff --git a/cmd/gpt.c b/cmd/gpt.c
index 4d00a35..e898f35 100644
--- a/cmd/gpt.c
+++ b/cmd/gpt.c
@@ -19,6 +19,9 @@
 #include 
 #include 
 #include 
+#include 
+
+static LIST_HEAD(disk_partitions);
 
 /**
  * extract_env(): Expand env name from string format '&{env_name}'
@@ -151,6 +154,115 @@ static bool found_key(const char *str, const char *key)
return result;
 }
 
+static void del_gpt_info(void)
+{
+   struct list_head *pos = _partitions;
+   struct disk_part *curr;
+   while (!list_empty(pos)) {
+   curr = list_entry(pos->next, struct disk_part, list);
+   list_del(pos->next);
+   free(curr);
+   }
+}
+
+static struct disk_part *allocate_disk_part(disk_partition_t *info, int 
partnum)
+{
+   struct disk_part *newpart;
+   newpart = (struct disk_part *)malloc(sizeof(*newpart));
+   if (!newpart)
+   return ERR_PTR(-ENOMEM);
+   memset(newpart, '\0', sizeof(newpart));
+
+   newpart->gpt_part_info.start = info->start;
+   newpart->gpt_part_info.size = info->size;
+   newpart->gpt_part_info.blksz = info->blksz;
+   strncpy((char *)newpart->gpt_part_info.name, (const char *)info->name,
+   PART_NAME_LEN);
+   newpart->gpt_part_info.name[PART_NAME_LEN - 1] = '\0';
+   strncpy((char *)newpart->gpt_part_info.type, (const char *)info->type,
+   PART_TYPE_LEN);
+   newpart->gpt_part_info.type[PART_TYPE_LEN - 1] = '\0';
+   newpart->gpt_part_info.bootable = info->bootable;
+#ifdef CONFIG_PARTITION_UUIDS
+   strncpy(newpart->gpt_part_info.uuid, (const char *)info->uuid,
+   UUID_STR_LEN);
+   /* UUID_STR_LEN is correct, as uuid[]'s length is UUID_STR_LEN+1 chars 
*/
+   newpart->gpt_part_info.uuid[UUID_STR_LEN] = '\0';
+#endif
+   newpart->partnum = partnum;
+
+   return newpart;
+}
+
+static void print_gpt_info(void)
+{
+   struct list_head *pos;
+   struct disk_part *curr;
+
+   list_for_each(pos, _partitions) {
+   curr = list_entry(pos, struct disk_part, list);
+   printf("Partition %d:\n", curr->partnum);
+   printf("1st block %x, size %x\n", 
(unsigned)curr->gpt_part_info.start,
+  (unsigned)curr->gpt_part_info.size);
+   printf("Block size %lu, name %s\n", curr->gpt_part_info.blksz,
+  curr->gpt_part_info.name);
+   printf("Type %s, bootable %d\n", curr->gpt_part_info.type,
+  curr->gpt_part_info.bootable);
+#ifdef CONFIG_PARTITION_UUIDS
+   printf("UUID %s\n", curr->gpt_part_info.uuid);
+#endif
+   printf("\n");
+   }
+}
+
+/*
+ * read partition info into disk_partitions list where
+ * it can be printed or modified
+ */
+static int get_gpt_info(struct blk_desc *dev_desc)
+{
+   /* start partition numbering at 1, as U-Boot does */
+   int valid_parts = 1, p, ret;
+   disk_partition_t info;
+   struct disk_part *new_disk_part;
+
+   if (disk_partitions.next == NULL)
+   INIT_LIST_HEAD(_partitions);
+
+   for (p = 1; p <= MAX_SEARCH_PARTITIONS; p++) {
+   ret = part_get_info(dev_desc, p, );
+   if (ret)
+   continue;
+
+   new_disk_part = allocate_disk_part(, valid_parts);
+   if (IS_ERR(new_disk_part) && valid_parts >= 2)
+   return -ENODEV;
+
+   list_add_tail(_disk_part->list, _partitions);
+   valid_parts++;
+   }
+   if (!valid_parts) {
+   printf("** No valid partitions found **\n");
+   del_gpt_info();
+   return -ENODEV;
+   }
+   return --valid_parts;
+}
+
+/* a wrapper to test get_gpt_info */
+static int do_get_gpt_info(struct blk_desc *dev_desc)
+{
+   int ret;
+
+   ret = get_gpt_info(dev_desc);
+   if (ret > 0) {
+   print_gpt_info();
+   del_gpt_info();
+   }
+   return ret;
+}
+
+
 /**
  * set_gpt_info(): Fill partition information from string
  * function allocates memory, remember to free!
@@ -455,6 +567,8 @@ static int do_gpt(cmd_tbl_t *cmdtp, int flag, int argc, 
char * const argv[])
printf("Verify GPT: ");
} else if (strcmp(argv[1], "guid") == 0) {
return 

[U-Boot] [PATCH 4/5] sandbox: README: fix partition command invocation

[alison]

From: Alison Chaiken 

The instructions for creating a disk image that are presently in
README.sandbox fail because sfdisk doesn't know about GPT.

Signed-off-by: Alison Chaiken 
---
 board/sandbox/README.sandbox | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/board/sandbox/README.sandbox b/board/sandbox/README.sandbox
index 02d8ab3..9dc2eb0 100644
--- a/board/sandbox/README.sandbox
+++ b/board/sandbox/README.sandbox
@@ -333,7 +333,7 @@ the contents of the root directory on the second partion of 
the image
 A disk image can be created using the following commands:
 
 $> truncate -s 1200M ./disk.raw
-$> echo -e "label: gpt\n,64M,U\n,,L" | /usr/sbin/sfdisk  ./disk.raw
+$> echo -e "label: gpt\n,64M,U\n,,L" | /usr/sbin/sgdisk  ./disk.raw
 $> lodev=`sudo losetup -P -f --show ./disk.raw`
 $> sudo mkfs.vfat -n EFI -v ${lodev}p1
 $> sudo mkfs.ext4 -L ROOT -v ${lodev}p2
-- 
2.1.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v4 3/5] GPT: fix error in partitions string doc

[alison]

From: Alison Chaiken 

The existing partitions-list parsing in cmd/gpt.c passes a value
from gpt_default() to set_gpt_info() that README.gpt suggests
should begin with 'partitions='.  Partition-list strings should
in fact begin with 'uuid_disk', as otherwise the call from
set_gpt_info() to extract_val() to find 'uuid_disk' will fail.
Change README.gpt accordingly.

Changes since v3:
-- The part of this patch dealing with the partition-flipping
   behavior was squashed into the patch that introduces that
   feature.

Signed-off-by: Alison Chaiken 
---
 doc/README.gpt | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/doc/README.gpt b/doc/README.gpt
index e29b188..754e490 100644
--- a/doc/README.gpt
+++ b/doc/README.gpt
@@ -156,10 +156,10 @@ Creating GPT partitions in U-Boot:
 To restore GUID partition table one needs to:
 1. Define partition layout in the environment.
Format of partitions layout:
- "partitions=uuid_disk=...;name=u-boot,size=60MiB,uuid=...;
+ "uuid_disk=...;name=u-boot,size=60MiB,uuid=...;
name=kernel,size=60MiB,uuid=...;"
  or
- "partitions=uuid_disk=${uuid_gpt_disk};name=${uboot_name},
+ "uuid_disk=${uuid_gpt_disk};name=${uboot_name},
size=${uboot_size},uuid=${uboot_uuid};"
 
The fields 'name' and 'size' are mandatory for every partition.
@@ -233,7 +233,7 @@ PARTITION_BASIC_DATA_GUID 
(EBD0A0A2-B9E5-4433-87C0-68B6B72699C7).
 If you define 'CONFIG_PARTITION_TYPE_GUID', a optionnal parameter 'type'
 can specify a other partition type guid:
 
- "partitions=uuid_disk=...;name=u-boot,size=60MiB,uuid=...;
+ "uuid_disk=...;name=u-boot,size=60MiB,uuid=...;
name=kernel,size=60MiB,uuid=...,
type=0FC63DAF-8483-4772-8E79-3D69D8477DE4;"
 
@@ -255,7 +255,7 @@ Some strings can be also used at the place of known GUID :
"lvm"= PARTITION_LINUX_LVM_GUID
   (E6D6D379-F507-44C2-A23C-238F2A3DF928)
 
-"partitions=uuid_disk=...;name=u-boot,size=60MiB,uuid=...;
+"uuid_disk=...;name=u-boot,size=60MiB,uuid=...;
name=kernel,size=60MiB,uuid=...,type=linux;"
 
 They are also used to display the type of partition in "part list" command.
-- 
2.1.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 5/5] cmd gpt: test in sandbox

[alison]

From: Alison Chaiken 

Make minor changes to README.gpt and sandbox_defconfig to support
testing of the gpt command's functionality in the sandbox.

Signed-off-by: Alison Chaiken 
---
 doc/README.gpt | 10 ++
 1 file changed, 10 insertions(+)

diff --git a/doc/README.gpt b/doc/README.gpt
index 754e490..7b3936b 100644
--- a/doc/README.gpt
+++ b/doc/README.gpt
@@ -224,6 +224,16 @@ U-BOOT> gpt flip mmc 0
 
 Choose different partition names by modifying these strings in gpt.c.
 
+The GPT functionality may be tested with the 'sandbox' board by
+creating a disk image as described under 'Block Device Emulation' in
+board/sandbox/README.sandbox:
+
+=>host bind 0 ./disk.raw
+=> gpt read host 0
+[ . . . ]
+=> gpt flip host 0
+[ . . . ]
+
 Partition type GUID:
 
 
-- 
2.1.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Pull request: u-boot-net.git master

[Tom Rini]

On Fri, Jun 02, 2017 at 02:45:15PM -0500, Joe Hershberger wrote:

> Hi Tom,
> 
> The following changes since commit 46bac66b20da6d50e757bdca74703153f233090b:
> 
>   sandbox: Move to use live tree (2017-06-01 07:03:17 -0600)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-net.git master
> 
> for you to fetch changes up to 2303bff7d55df47105740e5d635d50ef9f6856b6:
> 
>   net: pch_gbe: Add cache maintenance (2017-06-02 14:44:20 -0500)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Pull Request: u-boot-sunxi/master

[Tom Rini]

On Thu, Jun 01, 2017 at 01:43:17PM +, Jagan Teki wrote:

> Hi Tom,
> 
> Please take this PR.
> 
> thanks,
> Jagan.
> 
> The following changes since commit 4c78028737c3185f49f5691183aeac3478b5f699:
> 
>   mksunxi_fit_atf.sh: Allow for this to complete when bl31.bin is missing 
> (2017-05-22 20:18:01 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-sunxi.git master
> 
> for you to fetch changes up to 2dbe9c1362c7481fac93524f65340a48c79eed9a:
> 
>   sun50i: a64: Add initial Banana Pi M64 support (2017-06-01 09:26:05 +)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 3/7] configs: k2g_evm: make sure config fallbacks take effect

[Tom Rini]

On Fri, Jun 02, 2017 at 05:54:01PM +0530, Sekhar Nori wrote:

> Since config fallbacks contained in include/config_fallbacks.h
> come into k2g_evm.h file through ti_armv7_keystone2.h, it should
> be the last file included.
> 
> Without this, #define of FAT_WRITE when environment is in FAT
> does not happen as the environment location is decided later
> in the file.
> 
> Similar issues can come with other config fallbacks implemented.
> 
> Signed-off-by: Sekhar Nori 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] Kconfig: Migrate FS_FAT / FAT_WRITE

[Tom Rini]

On Sat, Jun 03, 2017 at 06:00:04PM -0400, Tom Rini wrote:

> Now that these symbols are in Kconfig, migrate all users.  Use imply on
> a number of platforms that default to having this enabled.  As part of
> this we must migrate some straglers for CMD_FAT and DOS_PARTITION.
> 
> Signed-off-by: Tom Rini 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 4/7] config_fallbacks: add additional fallbacks for fat filesystem

[Tom Rini]

On Fri, Jun 02, 2017 at 05:54:02PM +0530, Sekhar Nori wrote:

> Add fallbacks needed to keep all boards building
> while they are migrated to use Kconfig symbols
> instead of defines in _config.h files for
> FAT filesystem.
> 
> These should eventually go away once Kconfig select
> or imply statements are put in place and duplicated
> defines in _config.h removed.
> 
> Signed-off-by: Sekhar Nori 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 2/7] configs: k2*_evm: let each board decide env location

[Tom Rini]

On Fri, Jun 02, 2017 at 05:54:00PM +0530, Sekhar Nori wrote:

> Not all TI Keystone2 EVMs want environment in NAND flash.
> K2G EVM which has an MMC/SD slot, keep environment in a
> FAT partition on SD card.
> 
> Since ti_armv7_keystone2.h defines environment is in NAND,
> boards which do not follow that have to #undef'ine that
> configuration. This leads to ugly ordering issues around
> where exactly the include of ti_armv7_keystone2.h can come
> in within the k2*_evm.h files.
> 
> Move environment location to config file of each board.
> This should make it easy to change it for any one board
> without affecting all other boards.
> 
> Signed-off-by: Sekhar Nori 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH RESEND 1/7] fs: fat: add kbuild configuration support

[Tom Rini]

On Fri, Jun 02, 2017 at 05:53:59PM +0530, Sekhar Nori wrote:

> Add Kconfig symbols for various configurations
> supported by FAT filesystem support code.
> 
> CONFIG_SUPPORT_VFAT has been left out since its
> force enabled in include/fat.h and probably
> should get removed at some point.
> 
> Signed-off-by: Sekhar Nori 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 6/7] board: ti: enable support for writing to fat partition

[Tom Rini]

On Fri, Jun 02, 2017 at 05:54:04PM +0530, Sekhar Nori wrote:

> Enable support for writing to FAT partitions on
> TI's boards.
> 
> Signed-off-by: Sekhar Nori 
> ---
>  board/ti/common/Kconfig   | 1 +
>  include/configs/am43xx_evm.h  | 1 -
>  include/configs/ti_armv7_common.h | 8 
>  3 files changed, 1 insertion(+), 9 deletions(-)

I included the Kconfig logic here in the first patch in the series, and
then migrated it with everything else.

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] armv7m: Disable D-cache when booting nommu(ARMv7M) Linux kernel

[Toshifumi NISHINAGA]

Disable D-Cache is required when booting nommu Linux kernel.
(please see Linux kernel source "arch/arm/kernel/head-nommu.S")

U-Boot is enabled D-cache and I-Cache at startup.
However, it does not disable D-Cache before
booting nommu Linux kernel.
Therefore, I call dcache_disable()
when the CPU is ARMv7M to fix this problem.

Signed-off-by: Toshifumi NISHINAGA 
---
 arch/arm/lib/bootm.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c
index 4dbe6a5303..d14a8b5737 100644
--- a/arch/arm/lib/bootm.c
+++ b/arch/arm/lib/bootm.c
@@ -359,6 +359,7 @@ static void boot_jump_linux(bootm_headers_t *images, int 
flag)
 #ifdef CONFIG_CPU_V7M
ulong addr = (ulong)kernel_entry | 1;
kernel_entry = (void *)addr;
+   dcache_disable();
 #endif
s = getenv("machid");
if (s) {
-- 
2.13.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 10/10] mips: bmips: enable the SPI flash on the Netgear CG3100D

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> It's a Spansion (s25fl064a) 8 MB SPI flash.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: no changes
>  v2: remove spi alias
> 
>  arch/mips/dts/netgear,cg3100d.dts | 12 
>  configs/netgear_cg3100d_ram_defconfig |  8 
>  2 files changed, 20 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 09/10] mips: bmips: enable the SPI flash on the Sagem F@ST1704

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> It's a Winbond (w25x32) 4 MB SPI flash.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: rename BCM6338 SPI driver to BCM6348
>  v2: remove spi alias
> 
>  arch/mips/dts/sagem,f...@st1704.dts | 12 
>  configs/sagem_f@st1704_ram_defconfig |  8 
>  2 files changed, 20 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 08/10] mips: bmips: add bcm63xx-spi driver support for BCM63268

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> This driver manages the low speed SPI controller present on this SoC.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: no changes
>  v2: add spi alias
> 
>  arch/mips/dts/brcm,bcm63268.dtsi | 17 +
>  1 file changed, 17 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 07/10] mips: bmips: add bcm63xx-spi driver support for BCM3380

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> This driver manages the SPI controller present on this SoC.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: no changes
>  v2: add spi alias
> 
>  arch/mips/dts/brcm,bcm3380.dtsi | 17 +
>  1 file changed, 17 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 06/10] mips: bmips: add bcm63xx-spi driver support for BCM6358

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> This driver manages the SPI controller present on this SoC.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: no changes
>  v2: add spi alias
> 
>  arch/mips/dts/brcm,bcm6358.dtsi | 17 +
>  1 file changed, 17 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 05/10] mips: bmips: add bcm63xx-spi driver support for BCM6348

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> This driver manages the SPI controller present on this SoC.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: rename BCM6338 SPI driver to BCM6348
>  v2: add spi alias
> 
>  arch/mips/dts/brcm,bcm6348.dtsi | 17 +
>  1 file changed, 17 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 04/10] mips: bmips: add bcm63xx-spi driver support for BCM6338

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> This driver manages the SPI controller present on this SoC.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: rename BCM6338 SPI driver to BCM6348
>  v2: add spi alias
> 
>  arch/mips/dts/brcm,bcm6338.dtsi | 17 +
>  1 file changed, 17 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 03/10] dm: spi: add BCM63xx SPI driver

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> This driver is a simplified version of linux/drivers/spi/spi-bcm63xx.c
> Instead of supporting both HW revisions of the controller in a single build,
> support has been split by the selected config to save space.
> 
> Signed-off-by: Álvaro Fernández Rojas 
> Reviewed-by: Simon Glass 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: rename BCM6338 SPI driver to BCM6348
>   switch to devfdt_get_addr_size_index()
>  v2: no changes
> 
>  drivers/spi/Kconfig   |  21 +++
>  drivers/spi/Makefile  |   1 +
>  drivers/spi/bcm63xx_spi.c | 404 
> ++
>  3 files changed, 426 insertions(+)
>  create mode 100644 drivers/spi/bcm63xx_spi.c
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 02/10] drivers: spi: consider command bytes when sending transfers

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> Command bytes are part of the written bytes and they should be taken into
> account when sending a spi transfer.
> 
> Signed-off-by: Álvaro Fernández Rojas 
> Reviewed-by: Simon Glass 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: Fix bug introduced in v2: sizeof(cmd) vs len
>  v2: Introduce changes requested by Simon Glass:
>   - Always include command bytes when determining max write size.
> 
>  drivers/mtd/spi/spi_flash.c | 2 +-
>  include/spi.h   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 01/10] drivers: spi: allow limiting reads

[Daniel Schwierzeck]

Am 03.06.2017 um 11:57 schrieb Álvaro Fernández Rojas:
> For some SPI controllers it's not possible to keep the CS active between
> transfers and they are limited to a known number of bytes.
> This splits spi_flash reads into different iterations in order to respect
> the SPI controller limits.
> 
> Signed-off-by: Álvaro Fernández Rojas 
> Reviewed-by: Simon Glass 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v3: no changes
>  v2: no changes
> 
>  drivers/mtd/spi/spi_flash.c | 3 +++
>  include/spi.h   | 3 +++
>  2 files changed, 6 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2 4/4] mips: bmips: enable the SPI flash on the Comtrend AR-5387un

[Daniel Schwierzeck]

Am 03.06.2017 um 12:16 schrieb Álvaro Fernández Rojas:
> It's a Macronix (mx25l12805d) 16 MB SPI flash.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v2: no changes
> 
>  arch/mips/dts/comtrend,ar-5387un.dts| 12 
>  configs/comtrend_ar5387un_ram_defconfig |  8 
>  2 files changed, 20 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2 3/4] mips: bmips: add bcm63xx-hsspi driver support for BCM63268

[Daniel Schwierzeck]

Am 03.06.2017 um 12:16 schrieb Álvaro Fernández Rojas:
> This driver manages the high speed SPI controller present on this SoC.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v2: no changes
> 
>  arch/mips/dts/brcm,bcm63268.dtsi | 21 +
>  1 file changed, 21 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2 2/4] mips: bmips: add bcm63xx-hsspi driver support for BCM6328

[Daniel Schwierzeck]

Am 03.06.2017 um 12:16 schrieb Álvaro Fernández Rojas:
> This driver manages the SPI controller present on this SoC.
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
>  v2: no changes
> 
>  arch/mips/dts/brcm,bcm6328.dtsi | 24 
>  1 file changed, 24 insertions(+)
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2 1/4] dm: spi: add BCM63xx HSSPI driver

[Daniel Schwierzeck]

Am 03.06.2017 um 12:16 schrieb Álvaro Fernández Rojas:
> This driver is a simplified version of linux/drivers/spi/spi-bcm63xx-hsspi.c
> 
> Signed-off-by: Álvaro Fernández Rojas 

Reviewed-by: Daniel Schwierzeck 

> ---
> v2: Introduce changes suggested by Simon Glass:
>  - Split bcm63xx_hsspi_xfer() into smaller functions.
>  - Check possible clock errors.
>  - Check possible reset errors.
>  - Switch to devfdt_get_addr_size_index().
>  - Use setbits32_be() for clock gate.
> 
>  drivers/spi/Kconfig |   8 +
>  drivers/spi/Makefile|   1 +
>  drivers/spi/bcm63xx_hsspi.c | 413 
> 
>  3 files changed, 422 insertions(+)
>  create mode 100644 drivers/spi/bcm63xx_hsspi.c
> 

-- 
- Daniel



signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot