[U-Boot] [PATCH 0/4] ath79: fix some minor defects

[U-Boot] [PATCH 0/4] ath79: fix some minor defects

[Wills Wang]

These series of patch based on top of mips/next, it fix some defects on
the previous patch series "add support for atheros ath79 based SOCs".


Wills Wang (4):
  ath79: spi: Remove the explicit pinctrl setting
  ar933x: serial: Remove the explicit pinctrl setting
  ath79: ar933x: use BIT macro for bit shift operation
  ath79: add readonly attribute for ath79_soc_desc

 arch/mips/mach-ath79/ar933x/ddr.c | 14 +++++++-------
 arch/mips/mach-ath79/cpu.c        |  8 ++++----
 drivers/serial/serial_ar933x.c    | 16 ++--------------
 drivers/spi/ath79_spi.c           | 12 ------------
 4 files changed, 13 insertions(+), 37 deletions(-)

-- 
1.9.1

[U-Boot] [PATCH 0/4] ath79: fix some minor defects

[Daniel Schwierzeck]

Am 12.04.2016 um 05:09 schrieb Wills Wang:
> 
> These series of patch based on top of mips/next, it fix some defects on
> the previous patch series "add support for atheros ath79 based SOCs".
> 
> 
> Wills Wang (4):
>   ath79: spi: Remove the explicit pinctrl setting
>   ar933x: serial: Remove the explicit pinctrl setting
>   ath79: ar933x: use BIT macro for bit shift operation
>   ath79: add readonly attribute for ath79_soc_desc
> 
>  arch/mips/mach-ath79/ar933x/ddr.c | 14 +++++++-------
>  arch/mips/mach-ath79/cpu.c        |  8 ++++----
>  drivers/serial/serial_ar933x.c    | 16 ++--------------
>  drivers/spi/ath79_spi.c           | 12 ------------
>  4 files changed, 13 insertions(+), 37 deletions(-)
> 

all four patches applied to u-boot-mips/next, thanks!

-- 
- Daniel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160415/f27174e1/attachment.sig>

[U-Boot] [PATCH 0/4] ath79: fix some minor defects

[Marek Vasut]

On 04/15/2016 01:19 PM, Daniel Schwierzeck wrote:
> 
> 
> Am 12.04.2016 um 05:09 schrieb Wills Wang:
>>
>> These series of patch based on top of mips/next, it fix some defects on
>> the previous patch series "add support for atheros ath79 based SOCs".
>>
>>
>> Wills Wang (4):
>>   ath79: spi: Remove the explicit pinctrl setting
>>   ar933x: serial: Remove the explicit pinctrl setting
>>   ath79: ar933x: use BIT macro for bit shift operation
>>   ath79: add readonly attribute for ath79_soc_desc
>>
>>  arch/mips/mach-ath79/ar933x/ddr.c | 14 +++++++-------
>>  arch/mips/mach-ath79/cpu.c        |  8 ++++----
>>  drivers/serial/serial_ar933x.c    | 16 ++--------------
>>  drivers/spi/ath79_spi.c           | 12 ------------
>>  4 files changed, 13 insertions(+), 37 deletions(-)
>>
> 
> all four patches applied to u-boot-mips/next, thanks!
> 
Can you please update next on top of u-boot/master , so I can submit the
ar9344 ? Thanks!

-- 
Best regards,
Marek Vasut

[U-Boot] [PATCH 0/4] ath79: fix some minor defects

[Daniel Schwierzeck]

Am 03.05.2016 um 23:28 schrieb Marek Vasut:
> On 04/15/2016 01:19 PM, Daniel Schwierzeck wrote:
>>
>>
>> Am 12.04.2016 um 05:09 schrieb Wills Wang:
>>>
>>> These series of patch based on top of mips/next, it fix some defects on
>>> the previous patch series "add support for atheros ath79 based SOCs".
>>>
>>>
>>> Wills Wang (4):
>>>   ath79: spi: Remove the explicit pinctrl setting
>>>   ar933x: serial: Remove the explicit pinctrl setting
>>>   ath79: ar933x: use BIT macro for bit shift operation
>>>   ath79: add readonly attribute for ath79_soc_desc
>>>
>>>  arch/mips/mach-ath79/ar933x/ddr.c | 14 +++++++-------
>>>  arch/mips/mach-ath79/cpu.c        |  8 ++++----
>>>  drivers/serial/serial_ar933x.c    | 16 ++--------------
>>>  drivers/spi/ath79_spi.c           | 12 ------------
>>>  4 files changed, 13 insertions(+), 37 deletions(-)
>>>
>>
>> all four patches applied to u-boot-mips/next, thanks!
>>
> Can you please update next on top of u-boot/master , so I can submit the
> ar9344 ? Thanks!

ok, done

-- 
- Daniel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160504/c87e4f3a/attachment.sig>

[U-Boot] [PATCH 0/4] ath79: fix some minor defects

[Marek Vasut]

On 05/04/2016 12:07 PM, Daniel Schwierzeck wrote:
> 
> 
> Am 03.05.2016 um 23:28 schrieb Marek Vasut:
>> On 04/15/2016 01:19 PM, Daniel Schwierzeck wrote:
>>>
>>>
>>> Am 12.04.2016 um 05:09 schrieb Wills Wang:
>>>>
>>>> These series of patch based on top of mips/next, it fix some defects on
>>>> the previous patch series "add support for atheros ath79 based SOCs".
>>>>
>>>>
>>>> Wills Wang (4):
>>>>   ath79: spi: Remove the explicit pinctrl setting
>>>>   ar933x: serial: Remove the explicit pinctrl setting
>>>>   ath79: ar933x: use BIT macro for bit shift operation
>>>>   ath79: add readonly attribute for ath79_soc_desc
>>>>
>>>>  arch/mips/mach-ath79/ar933x/ddr.c | 14 +++++++-------
>>>>  arch/mips/mach-ath79/cpu.c        |  8 ++++----
>>>>  drivers/serial/serial_ar933x.c    | 16 ++--------------
>>>>  drivers/spi/ath79_spi.c           | 12 ------------
>>>>  4 files changed, 13 insertions(+), 37 deletions(-)
>>>>
>>>
>>> all four patches applied to u-boot-mips/next, thanks!
>>>
>> Can you please update next on top of u-boot/master , so I can submit the
>> ar9344 ? Thanks!
> 
> ok, done
> 
Thank you!

-- 
Best regards,
Marek Vasut

[U-Boot] [PATCH 0/4] ath79: fix some minor defects

[Daniel Schwierzeck]

Hi Wills,

I'm getting a warning with gcc-5.3:

  CC      arch/mips/mach-ath79/ar933x/ddr.o
arch/mips/mach-ath79/ar933x/ddr.c: In function ?ddr_tap_tuning?:
arch/mips/mach-ath79/ar933x/ddr.c:329:15: warning: ?upper? may be used
uninitialized in this function [-Wmaybe-uninitialized]
  val = (upper + lower) / 2;
               ^

Could you please look at it? I don't know which default values make
sense for 'upper' and 'lower'.


Am 15.04.2016 um 13:19 schrieb Daniel Schwierzeck:
> 
> 
> Am 12.04.2016 um 05:09 schrieb Wills Wang:
>>
>> These series of patch based on top of mips/next, it fix some defects on
>> the previous patch series "add support for atheros ath79 based SOCs".
>>
>>
>> Wills Wang (4):
>>   ath79: spi: Remove the explicit pinctrl setting
>>   ar933x: serial: Remove the explicit pinctrl setting
>>   ath79: ar933x: use BIT macro for bit shift operation
>>   ath79: add readonly attribute for ath79_soc_desc
>>
>>  arch/mips/mach-ath79/ar933x/ddr.c | 14 +++++++-------
>>  arch/mips/mach-ath79/cpu.c        |  8 ++++----
>>  drivers/serial/serial_ar933x.c    | 16 ++--------------
>>  drivers/spi/ath79_spi.c           | 12 ------------
>>  4 files changed, 13 insertions(+), 37 deletions(-)
>>
> 
> all four patches applied to u-boot-mips/next, thanks!
> 

-- 
- Daniel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160506/95f2c203/attachment.sig>

[U-Boot] [PATCH 0/4] ath79: fix some minor defects

[Wills Wang]

On Friday, May 06, 2016 06:43 PM, Daniel Schwierzeck wrote:
> Hi Wills,
>
> I'm getting a warning with gcc-5.3:
>
>    CC      arch/mips/mach-ath79/ar933x/ddr.o
> arch/mips/mach-ath79/ar933x/ddr.c: In function ?ddr_tap_tuning?:
> arch/mips/mach-ath79/ar933x/ddr.c:329:15: warning: ?upper? may be used
> uninitialized in this function [-Wmaybe-uninitialized]
>    val = (upper + lower) / 2;
>                 ^
>
> Could you please look at it? I don't know which default values make
> sense for 'upper' and 'lower'.
Ok, compiler might need explicit initializer.
I will add a patch for this later.
>
> Am 15.04.2016 um 13:19 schrieb Daniel Schwierzeck:
>>
>> Am 12.04.2016 um 05:09 schrieb Wills Wang:
>>> These series of patch based on top of mips/next, it fix some defects on
>>> the previous patch series "add support for atheros ath79 based SOCs".
>>>
>>>
>>> Wills Wang (4):
>>>    ath79: spi: Remove the explicit pinctrl setting
>>>    ar933x: serial: Remove the explicit pinctrl setting
>>>    ath79: ar933x: use BIT macro for bit shift operation
>>>    ath79: add readonly attribute for ath79_soc_desc
>>>
>>>   arch/mips/mach-ath79/ar933x/ddr.c | 14 +++++++-------
>>>   arch/mips/mach-ath79/cpu.c        |  8 ++++----
>>>   drivers/serial/serial_ar933x.c    | 16 ++--------------
>>>   drivers/spi/ath79_spi.c           | 12 ------------
>>>   4 files changed, 13 insertions(+), 37 deletions(-)
>>>
>> all four patches applied to u-boot-mips/next, thanks!
>>

-- 
Best Regards
Wills