Re: [U-Boot] Pull request: u-boot-spi/next

2017-05-08 Thread Tom Rini 
On Mon, May 08, 2017 at 09:46:41PM +0530, Jagan Teki wrote:

> Hi Tom,
> 
> Please take this PR.
> 
> thanks!
> Jagan.
> 
> The following changes since commit ac6991fb5fc1a694b29f4ebd5e07baebf818d7cc:
> 
>   zynq: spi: Honour the activation / deactivation delay (2017-05-03 11:03:04 
> +0530)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-spi.git next
> 
> for you to fetch changes up to 0b2e5bbe6a4f3d25522675198b8beb0093a2e750:
> 
>   env_sf: use DIV_ROUND_UP to calculate number of sectors to erase 
> (2017-05-03 11:18:20 +0530)
> 

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-spi/next

2016-01-06 Thread Jagan Teki 
On 3 January 2016 at 21:26, Tom Rini  wrote:
> On Sun, Jan 03, 2016 at 09:10:14PM +0530, Jagan Teki wrote:
>> On 3 January 2016 at 20:14, Tom Rini  wrote:
>> > On Sun, Jan 03, 2016 at 09:26:38AM +0530, Jagan Teki wrote:
>> >> On 2 January 2016 at 22:34, Tom Rini  wrote:
>> >> > On Wed, Dec 30, 2015 at 04:04:23PM +0530, Jagan Teki wrote:
>> >> >
>> >> >> Tom,
>> >> >>
>> >> >> Please note that subject shows u-boot-spi/next but it actually
>> >> >> u-boot-spi/master and of course PR shows the repo as master (not
>> >> >> next).
>> >> >
>> >> > OK, what in here is a clear bugfix rather than new feature that we
>> >> > should have taken in sooner?  As-is this needs some rework on samsung
>> >> > stuff to make trats, trats2 and s5pc210_universal work.  Thanks!
>> >>
>> >> How come these sf and ti patches are blocking these samsung boards to
>> >> work? few of these were bug fixes and others were features (ti
>> >> patches) but those initiated in last MW.
>> >
>> > The patch to use BIT() in spi.h causes undefined references on trats2,
>> > hence the first patch I posted in reply to the PR.  This however broke
>> > trats/s5pc210_universal.
>>
>> Build looks fine for me [1]
>>
>> [1] http://paste.ubuntu.com/14387171/
>
> Yes, that and trats are fine with your series, trats2 is not.  My first
> pass at fixing trats2 broke trats/s5p..

The patch "power: pmic.h: Drop include " will fix trats2.

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

Re: [U-Boot] Pull request: u-boot-spi/next

2016-01-03 Thread Jagan Teki 
On 3 January 2016 at 20:14, Tom Rini  wrote:
> On Sun, Jan 03, 2016 at 09:26:38AM +0530, Jagan Teki wrote:
>> On 2 January 2016 at 22:34, Tom Rini  wrote:
>> > On Wed, Dec 30, 2015 at 04:04:23PM +0530, Jagan Teki wrote:
>> >
>> >> Tom,
>> >>
>> >> Please note that subject shows u-boot-spi/next but it actually
>> >> u-boot-spi/master and of course PR shows the repo as master (not
>> >> next).
>> >
>> > OK, what in here is a clear bugfix rather than new feature that we
>> > should have taken in sooner?  As-is this needs some rework on samsung
>> > stuff to make trats, trats2 and s5pc210_universal work.  Thanks!
>>
>> How come these sf and ti patches are blocking these samsung boards to
>> work? few of these were bug fixes and others were features (ti
>> patches) but those initiated in last MW.
>
> The patch to use BIT() in spi.h causes undefined references on trats2,
> hence the first patch I posted in reply to the PR.  This however broke
> trats/s5pc210_universal.

Build looks fine for me [1]

[1] http://paste.ubuntu.com/14387171/

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

Re: [U-Boot] Pull request: u-boot-spi/next

2016-01-03 Thread Tom Rini 
On Sun, Jan 03, 2016 at 09:10:14PM +0530, Jagan Teki wrote:
> On 3 January 2016 at 20:14, Tom Rini  wrote:
> > On Sun, Jan 03, 2016 at 09:26:38AM +0530, Jagan Teki wrote:
> >> On 2 January 2016 at 22:34, Tom Rini  wrote:
> >> > On Wed, Dec 30, 2015 at 04:04:23PM +0530, Jagan Teki wrote:
> >> >
> >> >> Tom,
> >> >>
> >> >> Please note that subject shows u-boot-spi/next but it actually
> >> >> u-boot-spi/master and of course PR shows the repo as master (not
> >> >> next).
> >> >
> >> > OK, what in here is a clear bugfix rather than new feature that we
> >> > should have taken in sooner?  As-is this needs some rework on samsung
> >> > stuff to make trats, trats2 and s5pc210_universal work.  Thanks!
> >>
> >> How come these sf and ti patches are blocking these samsung boards to
> >> work? few of these were bug fixes and others were features (ti
> >> patches) but those initiated in last MW.
> >
> > The patch to use BIT() in spi.h causes undefined references on trats2,
> > hence the first patch I posted in reply to the PR.  This however broke
> > trats/s5pc210_universal.
> 
> Build looks fine for me [1]
> 
> [1] http://paste.ubuntu.com/14387171/

Yes, that and trats are fine with your series, trats2 is not.  My first
pass at fixing trats2 broke trats/s5p..

-- 
Tom


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

Re: [U-Boot] Pull request: u-boot-spi/next

2016-01-03 Thread Tom Rini 
On Sun, Jan 03, 2016 at 09:26:38AM +0530, Jagan Teki wrote:
> On 2 January 2016 at 22:34, Tom Rini  wrote:
> > On Wed, Dec 30, 2015 at 04:04:23PM +0530, Jagan Teki wrote:
> >
> >> Tom,
> >>
> >> Please note that subject shows u-boot-spi/next but it actually
> >> u-boot-spi/master and of course PR shows the repo as master (not
> >> next).
> >
> > OK, what in here is a clear bugfix rather than new feature that we
> > should have taken in sooner?  As-is this needs some rework on samsung
> > stuff to make trats, trats2 and s5pc210_universal work.  Thanks!
> 
> How come these sf and ti patches are blocking these samsung boards to
> work? few of these were bug fixes and others were features (ti
> patches) but those initiated in last MW.

The patch to use BIT() in spi.h causes undefined references on trats2,
hence the first patch I posted in reply to the PR.  This however broke
trats/s5pc210_universal.

-- 
Tom


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

Re: [U-Boot] Pull request: u-boot-spi/next

2016-01-02 Thread Tom Rini 
On Wed, Dec 30, 2015 at 04:04:23PM +0530, Jagan Teki wrote:

> Tom,
> 
> Please note that subject shows u-boot-spi/next but it actually
> u-boot-spi/master and of course PR shows the repo as master (not
> next).

OK, what in here is a clear bugfix rather than new feature that we
should have taken in sooner?  As-is this needs some rework on samsung
stuff to make trats, trats2 and s5pc210_universal work.  Thanks!

-- 
Tom


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

Re: [U-Boot] Pull request: u-boot-spi/next

2016-01-02 Thread Jagan Teki 
On 2 January 2016 at 22:34, Tom Rini  wrote:
> On Wed, Dec 30, 2015 at 04:04:23PM +0530, Jagan Teki wrote:
>
>> Tom,
>>
>> Please note that subject shows u-boot-spi/next but it actually
>> u-boot-spi/master and of course PR shows the repo as master (not
>> next).
>
> OK, what in here is a clear bugfix rather than new feature that we
> should have taken in sooner?  As-is this needs some rework on samsung
> stuff to make trats, trats2 and s5pc210_universal work.  Thanks!

How come these sf and ti patches are blocking these samsung boards to
work? few of these were bug fixes and others were features (ti
patches) but those initiated in last MW.

thanks!
-- 
Jagan.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Pull request: u-boot-spi/next

2015-12-31 Thread Jagan Teki 
On 30 December 2015 at 16:04, Jagan Teki  wrote:
> Tom,

Please pull this asap, in case if you miss.

>
> Please note that subject shows u-boot-spi/next but it actually
> u-boot-spi/master and of course PR shows the repo as master (not
> next).
>
> On 30 December 2015 at 16:01, Jagan Teki  wrote:
>> Hi Tom,
>>
>> Please pull this PR.
>>
>> thanks!
>> Jagan.
>>
>> The following changes since commit 312db9f75f50b339310474d72c8524cef2bd7931:
>>
>>   armv7: omap-common: Rework SPL board_mmc_init() (2015-12-14 13:42:54 -0500)
>>
>> are available in the git repository at:
>>
>>   git://git.denx.de/u-boot-spi.git master
>>
>> for you to fetch changes up to 25aade812e40f2e869575199c64e35ebcdf5554b:
>>
>>   defconfig: am437x_sk_evm: enable spi driver model (2015-12-29 13:27:46 
>> +0530)
>>
>> 
>> Jagan Teki (24):
>>   sf: Get spi locally from spi_flash
>>   spi: make mode visible to both dm and non-dm
>>   spi: Use mode instead of op_mode_tx
>>   spi: Rename SPI_TX_BP|QPP to SPI_TX_BYTE|QUAD
>>   sf: Move spi_read_cmds_array locally
>>   sf: Rename spi_flash_set_* functions
>>   sf: Rename bank_end to bar_end in read_bar
>>   sf: Remove spi_flash_remove
>>   sf: Minor cleanup
>>   sf: Use BIT macro
>>   sf: Fix quad bit set for micron devices
>>   sf: Read back and check once spansion quad bit set
>>   sf: Read back and check once macronix quad bit set
>>   sf: Write quad bit along with read status
>>   spi: Remove SPI_OPM_RX_EXTN
>>   spi: Remove SPI_OPM_RX_DIO|QIOF
>>   spi: Rename SPI_OPM_RX_* to SPI_RX_*
>>   spi: Rename op_mode_rx to mode_rx
>>   spi: Move flags macro's to spi_slave{} members
>>   spi: Fix bit assignment with flags
>>   spi: Use BIT macro
>>   spi: Minor cleanup
>>   sf: Make IO modes at last in read modes
>>   spi: Add SPI_TX_DUAL mode
>>
>> Mugunthan V N (17):
>>   drivers: spi: ti_qspi: do not hard code chip select for memory map 
>> configuration
>>   drivers: spi:ti_qspi: change ti_qspi_slave to ti_qspi_priv for driver 
>> model conversion
>>   drivers: spi: ti_qspi: prepare driver for DM conversion
>>   dm: core: Add a new api to get indexed device address
>>   spi: Add support for dual and quad mode
>>   dra7xx_evm: qspi: do not define DM_SPI and DM_SPI_FLASH for spl
>>   dts: dra7: add spi alias for qspi
>>   drivers: spi: ti_qspi: convert driver to adopt device driver model
>>   arm: dts: dra7: add qspi register maps for memory map and control 
>> module
>>   arm: dts: am437x-gp-evm: add spi-flash comaptible for flash
>>   arm: dts: dra7-evm: add spi-flash comaptible for flash
>>   arm: dts: dra72-evm: add spi-flash comaptible for flash
>>   defconfig: dra72_evm: enable spi driver model
>>   defconfig: dra74_evm: enable spi driver model
>>   am43xx_evm: qspi: do not define DM_SPI and DM_SPI_FLASH for spl
>>   arm: dts: am4372: add qspi register maps for memory map
>>   defconfig: am437x_sk_evm: enable spi driver model
>>
>>  arch/arm/dts/am4372.dtsi|   5 +-
>>  arch/arm/dts/am437x-sk-evm.dts  |   2 +-
>>  arch/arm/dts/dra7-evm.dts   |   2 +-
>>  arch/arm/dts/dra7.dtsi  |   8 +-
>>  arch/arm/dts/dra72-evm.dts  |   2 +-
>>  configs/am437x_sk_evm_defconfig |   3 +
>>  configs/dra72_evm_defconfig |   2 +
>>  configs/dra74_evm_defconfig |   2 +
>>  drivers/core/device.c   |  31 ++-
>>  drivers/mtd/spi/sf-uclass.c |   7 +-
>>  drivers/mtd/spi/sf_internal.h   |  55 +++--
>>  drivers/mtd/spi/sf_probe.c  |   2 +-
>>  drivers/mtd/spi/spi_flash.c | 185 +-
>>  drivers/spi/ich.c   |   4 +-
>>  drivers/spi/spi-uclass.c|  38 ++-
>>  drivers/spi/ti_qspi.c   | 524 
>> 
>>  include/configs/am43xx_evm.h|   2 +
>>  include/configs/dra7xx_evm.h|   5 +
>>  include/dm/device.h |  11 +
>>  include/spi.h   |  75 +++---
>>  include/spi_flash.h |   2 -
>>  21 files changed, 667 insertions(+), 300 deletions(-)
>
> thanks!
> --
> Jagan.



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

Re: [U-Boot] Pull request: u-boot-spi/next

2015-12-30 Thread Jagan Teki 
Tom,

Please note that subject shows u-boot-spi/next but it actually
u-boot-spi/master and of course PR shows the repo as master (not
next).

On 30 December 2015 at 16:01, Jagan Teki  wrote:
> Hi Tom,
>
> Please pull this PR.
>
> thanks!
> Jagan.
>
> The following changes since commit 312db9f75f50b339310474d72c8524cef2bd7931:
>
>   armv7: omap-common: Rework SPL board_mmc_init() (2015-12-14 13:42:54 -0500)
>
> are available in the git repository at:
>
>   git://git.denx.de/u-boot-spi.git master
>
> for you to fetch changes up to 25aade812e40f2e869575199c64e35ebcdf5554b:
>
>   defconfig: am437x_sk_evm: enable spi driver model (2015-12-29 13:27:46 
> +0530)
>
> 
> Jagan Teki (24):
>   sf: Get spi locally from spi_flash
>   spi: make mode visible to both dm and non-dm
>   spi: Use mode instead of op_mode_tx
>   spi: Rename SPI_TX_BP|QPP to SPI_TX_BYTE|QUAD
>   sf: Move spi_read_cmds_array locally
>   sf: Rename spi_flash_set_* functions
>   sf: Rename bank_end to bar_end in read_bar
>   sf: Remove spi_flash_remove
>   sf: Minor cleanup
>   sf: Use BIT macro
>   sf: Fix quad bit set for micron devices
>   sf: Read back and check once spansion quad bit set
>   sf: Read back and check once macronix quad bit set
>   sf: Write quad bit along with read status
>   spi: Remove SPI_OPM_RX_EXTN
>   spi: Remove SPI_OPM_RX_DIO|QIOF
>   spi: Rename SPI_OPM_RX_* to SPI_RX_*
>   spi: Rename op_mode_rx to mode_rx
>   spi: Move flags macro's to spi_slave{} members
>   spi: Fix bit assignment with flags
>   spi: Use BIT macro
>   spi: Minor cleanup
>   sf: Make IO modes at last in read modes
>   spi: Add SPI_TX_DUAL mode
>
> Mugunthan V N (17):
>   drivers: spi: ti_qspi: do not hard code chip select for memory map 
> configuration
>   drivers: spi:ti_qspi: change ti_qspi_slave to ti_qspi_priv for driver 
> model conversion
>   drivers: spi: ti_qspi: prepare driver for DM conversion
>   dm: core: Add a new api to get indexed device address
>   spi: Add support for dual and quad mode
>   dra7xx_evm: qspi: do not define DM_SPI and DM_SPI_FLASH for spl
>   dts: dra7: add spi alias for qspi
>   drivers: spi: ti_qspi: convert driver to adopt device driver model
>   arm: dts: dra7: add qspi register maps for memory map and control module
>   arm: dts: am437x-gp-evm: add spi-flash comaptible for flash
>   arm: dts: dra7-evm: add spi-flash comaptible for flash
>   arm: dts: dra72-evm: add spi-flash comaptible for flash
>   defconfig: dra72_evm: enable spi driver model
>   defconfig: dra74_evm: enable spi driver model
>   am43xx_evm: qspi: do not define DM_SPI and DM_SPI_FLASH for spl
>   arm: dts: am4372: add qspi register maps for memory map
>   defconfig: am437x_sk_evm: enable spi driver model
>
>  arch/arm/dts/am4372.dtsi|   5 +-
>  arch/arm/dts/am437x-sk-evm.dts  |   2 +-
>  arch/arm/dts/dra7-evm.dts   |   2 +-
>  arch/arm/dts/dra7.dtsi  |   8 +-
>  arch/arm/dts/dra72-evm.dts  |   2 +-
>  configs/am437x_sk_evm_defconfig |   3 +
>  configs/dra72_evm_defconfig |   2 +
>  configs/dra74_evm_defconfig |   2 +
>  drivers/core/device.c   |  31 ++-
>  drivers/mtd/spi/sf-uclass.c |   7 +-
>  drivers/mtd/spi/sf_internal.h   |  55 +++--
>  drivers/mtd/spi/sf_probe.c  |   2 +-
>  drivers/mtd/spi/spi_flash.c | 185 +-
>  drivers/spi/ich.c   |   4 +-
>  drivers/spi/spi-uclass.c|  38 ++-
>  drivers/spi/ti_qspi.c   | 524 
> 
>  include/configs/am43xx_evm.h|   2 +
>  include/configs/dra7xx_evm.h|   5 +
>  include/dm/device.h |  11 +
>  include/spi.h   |  75 +++---
>  include/spi_flash.h |   2 -
>  21 files changed, 667 insertions(+), 300 deletions(-)

thanks!
-- 
Jagan.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-04 Thread Jagan Teki 
On 4 November 2015 at 16:51, Fabio Estevam  wrote:
> Hi Jagan,
>
> On Wed, Nov 4, 2015 at 9:17 AM, Jagan Teki  wrote:
>
>> Updated u32 to size_t as prototype uses the same. Does this break? it look 
>> OK?
>
> Here is the error I see on Blackfin:
>
> Building bf518f-ezbrd board...
> /home/fabio/.buildman-toolchains/gcc-4.6.3-nolibc/bfin-uclinux/bin/bfin-uclinux-size:
> './u-boot': No such file
> drivers/mtd/spi/sf_ops.c:655:5: error: conflicting types for 'stm_lock'
> drivers/mtd/spi/sf_ops.c:706:5: error: conflicting types for 'stm_unlock'
> make[1]: *** [drivers/mtd/spi/sf_ops.o] Error 1
> make: *** [drivers/mtd/spi] Error 2
> drivers/mtd/spi/sf_ops.c:655:5: error: conflicting types for 'stm_lock'
> drivers/mtd/spi/sf_internal.h:180:5: note: previous declaration of
> 'stm_lock' was here
> drivers/mtd/spi/sf_ops.c:706:5: error: conflicting types for 'stm_unlock'
> drivers/mtd/spi/sf_internal.h:183:5: note: previous declaration of
> 'stm_unlock' was here
> make[1]: *** [drivers/mtd/spi/sf_ops.o] Error 1
>
> By using size_t we fix this conflicting type error.

Yes true, stm_lock and stm_unlock should use size_t for len - Will
update the same.

With these all built fine?

thanks!
-- 
Jagan | openedev.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-04 Thread Fabio Estevam 
On Wed, Nov 4, 2015 at 9:30 AM, Jagan Teki  wrote:

> Yes true, stm_lock and stm_unlock should use size_t for len - Will
> update the same.
>
> With these all built fine?

After applying this size_t change plus the three attached bitops
patches I sent you for mips, arc. avr32 I do not see any more build
issues here.

Also, when you apply the three patches to your tree, please make sure
they are applied prior to the "compat: Remove is_power_of_2()
definition" so that we keep bisectability.

Thanks,

Fabio Estevam
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-04 Thread Fabio Estevam 
On Wed, Nov 4, 2015 at 9:35 AM, Fabio Estevam  wrote:

> After applying this size_t change plus the three attached bitops
> patches I sent you for mips, arc. avr32 I do not see any more build
> issues here.
>
> Also, when you apply the three patches to your tree, please make sure
> they are applied prior to the "compat: Remove is_power_of_2()
> definition" so that we keep bisectability.

Also, in the v5 I sent I split microblaze and sh changes as you pointed out.

However, you seem to have picked the previous version:
http://git.denx.de/?p=u-boot/u-boot-spi.git;a=commit;h=9918a2c36b10675f9f12e3e3dcfff8f41727b586
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-04 Thread Fabio Estevam 
Hi Tom,

On Wed, Nov 4, 2015 at 1:45 AM, Fabio Estevam  wrote:
> On Tue, Nov 3, 2015 at 11:47 PM, Tom Rini  wrote:
>
>> NAK.
>>
>> So, this blows up badly for a lot of things when I do:
>> ./tools/buildman/buildman -b master --force-build --step 0 -Cdvel
>> 'blackfin|microblaze|m68k|nds32|x86|aarch64|sandbox|mips|avr32|arm|powerpc|sh4|sparc'
>
> It seems I missed to add bitops for avr32 and mips. Will work on it tomorrow.

I don't see issues after applying the three attached patches.

Thanks
From 09b289e8a372b21d1ffe0c1484d1bb3c392e01c3 Mon Sep 17 00:00:00 2001
From: Fabio Estevam 
Date: Wed, 4 Nov 2015 08:35:02 -0200
Subject: [PATCH 1/3] mips: Use the generic bitops headers

The generic bitops headers are required when calling logarithimic
functions, such as ilog2().

Signed-off-by: Fabio Estevam 
---
 arch/mips/include/asm/bitops.h | 5 +
 1 file changed, 5 insertions(+)

diff --git a/arch/mips/include/asm/bitops.h b/arch/mips/include/asm/bitops.h
index b5c2a63..c31ff6e 100644
--- a/arch/mips/include/asm/bitops.h
+++ b/arch/mips/include/asm/bitops.h
@@ -17,6 +17,11 @@
 #include 
 #include 
 
+#include 
+#include 
+#include 
+#include 
+
 /*
  * clear_bit() doesn't provide any barrier for the compiler.
  */
-- 
1.9.1

From 6e5e4e9d87073d012fd0b5eb999cf7fc04f7002f Mon Sep 17 00:00:00 2001
From: Fabio Estevam 
Date: Wed, 4 Nov 2015 08:35:36 -0200
Subject: [PATCH 2/3] arc: Use the generic bitops headers

The generic bitops headers are required when calling logarithimic
functions, such as ilog2().

Signed-off-by: Fabio Estevam 
---
 arch/arc/include/asm/bitops.h | 5 +
 1 file changed, 5 insertions(+)

diff --git a/arch/arc/include/asm/bitops.h b/arch/arc/include/asm/bitops.h
index 85721aa..370cb46 100644
--- a/arch/arc/include/asm/bitops.h
+++ b/arch/arc/include/asm/bitops.h
@@ -16,4 +16,9 @@
 #define hweight16(x) generic_hweight16(x)
 #define hweight8(x) generic_hweight8(x)
 
+#include 
+#include 
+#include 
+#include 
+
 #endif /* __ASM_ARC_BITOPS_H */
-- 
1.9.1

From ebbbfa0cd47881646b160f372a220773691853c5 Mon Sep 17 00:00:00 2001
From: Fabio Estevam 
Date: Wed, 4 Nov 2015 08:38:45 -0200
Subject: [PATCH 3/3] avr32: Use the generic bitops headers

The generic bitops headers are required when calling logarithimic
functions, such as ilog2().

Signed-off-by: Fabio Estevam 
---
 arch/avr32/include/asm/bitops.h | 5 +
 1 file changed, 5 insertions(+)

diff --git a/arch/avr32/include/asm/bitops.h b/arch/avr32/include/asm/bitops.h
index 0ec6784..0d425c2 100644
--- a/arch/avr32/include/asm/bitops.h
+++ b/arch/avr32/include/asm/bitops.h
@@ -6,4 +6,9 @@
 #ifndef __ASM_AVR32_BITOPS_H
 #define __ASM_AVR32_BITOPS_H
 
+#include 
+#include 
+#include 
+#include 
+
 #endif /* __ASM_AVR32_BITOPS_H */
-- 
1.9.1

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

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-04 Thread Fabio Estevam 
On Wed, Nov 4, 2015 at 8:46 AM, Fabio Estevam  wrote:
> Hi Tom,
>
> On Wed, Nov 4, 2015 at 1:45 AM, Fabio Estevam  wrote:
>> On Tue, Nov 3, 2015 at 11:47 PM, Tom Rini  wrote:
>>
>>> NAK.
>>>
>>> So, this blows up badly for a lot of things when I do:
>>> ./tools/buildman/buildman -b master --force-build --step 0 -Cdvel
>>> 'blackfin|microblaze|m68k|nds32|x86|aarch64|sandbox|mips|avr32|arm|powerpc|sh4|sparc'
>>
>> It seems I missed to add bitops for avr32 and mips. Will work on it tomorrow.
>
> I don't see issues after applying the three attached patches.

Ops, I do see a build issue in blackfin that didn't happen in previous
versions. Will work on fixing it.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-04 Thread Fabio Estevam 
Hi Jagan,

On Wed, Nov 4, 2015 at 8:47 AM, Fabio Estevam  wrote:
> On Wed, Nov 4, 2015 at 8:46 AM, Fabio Estevam  wrote:
>> Hi Tom,
>>
>> On Wed, Nov 4, 2015 at 1:45 AM, Fabio Estevam  wrote:
>>> On Tue, Nov 3, 2015 at 11:47 PM, Tom Rini  wrote:
>>>
 NAK.

 So, this blows up badly for a lot of things when I do:
 ./tools/buildman/buildman -b master --force-build --step 0 -Cdvel
 'blackfin|microblaze|m68k|nds32|x86|aarch64|sandbox|mips|avr32|arm|powerpc|sh4|sparc'
>>>
>>> It seems I missed to add bitops for avr32 and mips. Will work on it 
>>> tomorrow.
>>
>> I don't see issues after applying the three attached patches.
>
> Ops, I do see a build issue in blackfin that didn't happen in previous
> versions. Will work on fixing it.

When you reworked my series you missed to updated these functions:

diff --git a/drivers/mtd/spi/sf_ops.c b/drivers/mtd/spi/sf_ops.c
index 31c79c6..d832464 100644
--- a/drivers/mtd/spi/sf_ops.c
+++ b/drivers/mtd/spi/sf_ops.c
@@ -652,7 +652,7 @@ int stm_is_locked(struct spi_flash *flash, u32
ofs, size_t len)
  *
  * Returns negative on errors, 0 on success.
  */
-int stm_lock(struct spi_flash *flash, u32 ofs, u32 len)
+int stm_lock(struct spi_flash *flash, u32 ofs, size_t len)
 {
 u8 status_old, status_new;
 u8 mask = SR_BP2 | SR_BP1 | SR_BP0;
@@ -703,7 +703,7 @@ int stm_lock(struct spi_flash *flash, u32 ofs, u32 len)
  *
  * Returns negative on errors, 0 on success.
  */
-int stm_unlock(struct spi_flash *flash, u32 ofs, u32 len)
+int stm_unlock(struct spi_flash *flash, u32 ofs, size_t len)
 {
 uint8_t status_old, status_new;
 u8 mask = SR_BP2 | SR_BP1 | SR_BP0;
-- 

With this change and the other 3 bitops patches I sent I don't see any
more build issues here.

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

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-04 Thread Jagan Teki 
Fabio,

On 4 November 2015 at 16:38, Fabio Estevam  wrote:
> Hi Jagan,
>
> On Wed, Nov 4, 2015 at 8:47 AM, Fabio Estevam  wrote:
>> On Wed, Nov 4, 2015 at 8:46 AM, Fabio Estevam  wrote:
>>> Hi Tom,
>>>
>>> On Wed, Nov 4, 2015 at 1:45 AM, Fabio Estevam  wrote:
 On Tue, Nov 3, 2015 at 11:47 PM, Tom Rini  wrote:

> NAK.
>
> So, this blows up badly for a lot of things when I do:
> ./tools/buildman/buildman -b master --force-build --step 0 -Cdvel
> 'blackfin|microblaze|m68k|nds32|x86|aarch64|sandbox|mips|avr32|arm|powerpc|sh4|sparc'

 It seems I missed to add bitops for avr32 and mips. Will work on it 
 tomorrow.
>>>
>>> I don't see issues after applying the three attached patches.
>>
>> Ops, I do see a build issue in blackfin that didn't happen in previous
>> versions. Will work on fixing it.
>
> When you reworked my series you missed to updated these functions:
>
> diff --git a/drivers/mtd/spi/sf_ops.c b/drivers/mtd/spi/sf_ops.c
> index 31c79c6..d832464 100644
> --- a/drivers/mtd/spi/sf_ops.c
> +++ b/drivers/mtd/spi/sf_ops.c
> @@ -652,7 +652,7 @@ int stm_is_locked(struct spi_flash *flash, u32
> ofs, size_t len)
>   *
>   * Returns negative on errors, 0 on success.
>   */
> -int stm_lock(struct spi_flash *flash, u32 ofs, u32 len)
> +int stm_lock(struct spi_flash *flash, u32 ofs, size_t len)
>  {
>  u8 status_old, status_new;
>  u8 mask = SR_BP2 | SR_BP1 | SR_BP0;
> @@ -703,7 +703,7 @@ int stm_lock(struct spi_flash *flash, u32 ofs, u32 len)
>   *
>   * Returns negative on errors, 0 on success.
>   */
> -int stm_unlock(struct spi_flash *flash, u32 ofs, u32 len)
> +int stm_unlock(struct spi_flash *flash, u32 ofs, size_t len)

Updated u32 to size_t as prototype uses the same. Does this break? it look OK?

thanks!
-- 
Jagan | openedev.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-04 Thread Fabio Estevam 
Hi Jagan,

On Wed, Nov 4, 2015 at 9:17 AM, Jagan Teki  wrote:

> Updated u32 to size_t as prototype uses the same. Does this break? it look OK?

Here is the error I see on Blackfin:

Building bf518f-ezbrd board...
/home/fabio/.buildman-toolchains/gcc-4.6.3-nolibc/bfin-uclinux/bin/bfin-uclinux-size:
'./u-boot': No such file
drivers/mtd/spi/sf_ops.c:655:5: error: conflicting types for 'stm_lock'
drivers/mtd/spi/sf_ops.c:706:5: error: conflicting types for 'stm_unlock'
make[1]: *** [drivers/mtd/spi/sf_ops.o] Error 1
make: *** [drivers/mtd/spi] Error 2
drivers/mtd/spi/sf_ops.c:655:5: error: conflicting types for 'stm_lock'
drivers/mtd/spi/sf_internal.h:180:5: note: previous declaration of
'stm_lock' was here
drivers/mtd/spi/sf_ops.c:706:5: error: conflicting types for 'stm_unlock'
drivers/mtd/spi/sf_internal.h:183:5: note: previous declaration of
'stm_unlock' was here
make[1]: *** [drivers/mtd/spi/sf_ops.o] Error 1

By using size_t we fix this conflicting type error.

Regards,

Fabio Estevam
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-03 Thread Tom Rini 
On Tue, Nov 03, 2015 at 11:39:56PM +0530, Jagan Teki wrote:

> Hi Tom,
> 
> Please pull this series.
> 
> thanks!
> Jagan.
> 
> The following changes since commit 96d59e9d6aa74e35c63dc74da10e41f8ba0f6de4:
> 
>   mpc85xx/t2081: enable parsing DDR ratio for T2081 rev1.1 (2015-11-02 
> 08:51:50 -0800)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-spi.git next
> 
> for you to fetch changes up to e39e68f155b4fcd47f62ab2cf0e199d04f38446d:
> 
>   sf: Add SPI NOR protection mechanism (2015-11-03 21:51:37 +0530)

NAK.

So, this blows up badly for a lot of things when I do:
./tools/buildman/buildman -b master --force-build --step 0 -Cdvel
'blackfin|microblaze|m68k|nds32|x86|aarch64|sandbox|mips|avr32|arm|powerpc|sh4|sparc'

And most of these have a toolchain that buildman can fetch, if you don't
have one handy.

-- 
Tom


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

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-03 Thread Bin Meng 
Hi Jagan,

On Wed, Nov 4, 2015 at 9:47 AM, Tom Rini  wrote:
> On Tue, Nov 03, 2015 at 11:39:56PM +0530, Jagan Teki wrote:
>
>> Hi Tom,
>>
>> Please pull this series.
>>
>> thanks!
>> Jagan.
>>
>> The following changes since commit 96d59e9d6aa74e35c63dc74da10e41f8ba0f6de4:
>>
>>   mpc85xx/t2081: enable parsing DDR ratio for T2081 rev1.1 (2015-11-02 
>> 08:51:50 -0800)
>>
>> are available in the git repository at:
>>
>>   git://git.denx.de/u-boot-spi.git next
>>
>> for you to fetch changes up to e39e68f155b4fcd47f62ab2cf0e199d04f38446d:
>>
>>   sf: Add SPI NOR protection mechanism (2015-11-03 21:51:37 +0530)
>
> NAK.
>
> So, this blows up badly for a lot of things when I do:
> ./tools/buildman/buildman -b master --force-build --step 0 -Cdvel
> 'blackfin|microblaze|m68k|nds32|x86|aarch64|sandbox|mips|avr32|arm|powerpc|sh4|sparc'
>
> And most of these have a toolchain that buildman can fetch, if you don't
> have one handy.
>

I believe you need really use buildman to build the commits. One of
the previous review comments to your patch series was regard to the
'bisectability', and buildman can help you with that.

+Simon for buildman if you want some help.

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

Re: [U-Boot] Pull request: u-boot-spi/next

2015-11-03 Thread Fabio Estevam 
On Tue, Nov 3, 2015 at 11:47 PM, Tom Rini  wrote:

> NAK.
>
> So, this blows up badly for a lot of things when I do:
> ./tools/buildman/buildman -b master --force-build --step 0 -Cdvel
> 'blackfin|microblaze|m68k|nds32|x86|aarch64|sandbox|mips|avr32|arm|powerpc|sh4|sparc'

It seems I missed to add bitops for avr32 and mips. Will work on it tomorrow.

In the meantime, if you have the build warnings/errors to share it
would be helpful.

Regards,

Fabio Estevam
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot