Re: [PATCH v2 0/3] Add clock driver for Actions S700 SoC

2018-07-01 Thread Parthiban Nallathambi 
Hello Andreas,

On 07/01/2018 08:05 PM, Andreas Färber wrote:
> Hi Saravannan,
> 
> Am 28.06.2018 um 21:18 schrieb Saravanan Sekar:
>> Changelog v2:
>> Fixed 0 day compilation warning
>> Modified existing dts Actions S900 clock bindings common to Actions
>> Soc family
> 
> Please post v3 without --in-reply-to. It's becoming messy to read.
> 
>> This patchset adds clock support for Actions Semi OWL series S700 SoC
>> with relevant clock bindings and device tree info. 
>>
>> Changed the UART clock using CMU instance and changes are tested in
>> cubieboard7
> 
> Did you find any better way of testing than my "reboot bootloader" from
> preinstalled Android?

Flashing bootloader.bin and u-boot-dtb.img at below offset trigger the
automatic boot from SD. But ADFU needs to long pressed before connecting
the power.

sudo dd if=bootloader.bin of=/dev/sdX bs=512 seek=4097
sudo dd if=u-boot-dtb.img of=/dev/sdX bs=1024 seek=3072

With this is place, u-boot expects GPT layout by default. I am able to boot
the kernel from FAT/ext by updating the bootcmd/u-boot env accordingly. 

I haven't tried flashing/replacing u-boot in eMMC still.

> 
> In addition to missing Actions Semi you may want to CC Cubietech.

+Added cubitech linux group.

> 
> Thanks,
> Andreas
> 

-- 
Thanks,
Parthiban N

Re: [PATCH v2 0/3] Add clock driver for Actions S700 SoC

2018-07-01 Thread Andreas Färber 
Hi Saravannan,

Am 28.06.2018 um 21:18 schrieb Saravanan Sekar:
> Changelog v2:
> Fixed 0 day compilation warning
> Modified existing dts Actions S900 clock bindings common to Actions
> Soc family

Please post v3 without --in-reply-to. It's becoming messy to read.

> This patchset adds clock support for Actions Semi OWL series S700 SoC
> with relevant clock bindings and device tree info. 
> 
> Changed the UART clock using CMU instance and changes are tested in
> cubieboard7

Did you find any better way of testing than my "reboot bootloader" from
preinstalled Android?

In addition to missing Actions Semi you may want to CC Cubietech.

Thanks,
Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)

Re: [PATCH v2 0/3] Add clock driver for Actions S700 SoC

2018-07-01 Thread Saravanan Sekar 

Hi Stephen


On 06/29/18 19:38, Stephen Boyd wrote:

Quoting Saravanan Sekar (2018-06-28 12:18:02)

Changelog v2:
Fixed 0 day compilation warning
Modified existing dts Actions S900 clock bindings common to Actions
Soc family

This patchset adds clock support for Actions Semi OWL series S700 SoC
with relevant clock bindings and device tree info.

Changed the UART clock using CMU instance and changes are tested in
cubieboard7

Thanks and regards,
Saravanan,
Parthiban

Saravanan Sekar (3):
   arm64: dts: actions: Enable clock controller for S700
   clk: actions: Add S700 SoC clock support
   dt-bindings: clock: Modify Actions Soc clock bindings

Normally this patch comes first, and then clk driver, and then dts bits.
Also, the header file for the #define constants would be part of the
bindings patch.


I will reorder the commit and split the patch

Re: [PATCH v2 0/3] Add clock driver for Actions S700 SoC

2018-06-29 Thread Stephen Boyd 
Quoting Saravanan Sekar (2018-06-28 12:18:02)
> Changelog v2:
> Fixed 0 day compilation warning
> Modified existing dts Actions S900 clock bindings common to Actions
> Soc family
> 
> This patchset adds clock support for Actions Semi OWL series S700 SoC
> with relevant clock bindings and device tree info. 
> 
> Changed the UART clock using CMU instance and changes are tested in
> cubieboard7
> 
> Thanks and regards,
> Saravanan,
> Parthiban
> 
> Saravanan Sekar (3):
>   arm64: dts: actions: Enable clock controller for S700
>   clk: actions: Add S700 SoC clock support
>   dt-bindings: clock: Modify Actions Soc clock bindings

Normally this patch comes first, and then clk driver, and then dts bits.
Also, the header file for the #define constants would be part of the
bindings patch.

[PATCH v2 0/3] Add clock driver for Actions S700 SoC

2018-06-28 Thread Saravanan Sekar 
Changelog v2:
Fixed 0 day compilation warning
Modified existing dts Actions S900 clock bindings common to Actions
Soc family

This patchset adds clock support for Actions Semi OWL series S700 SoC
with relevant clock bindings and device tree info. 

Changed the UART clock using CMU instance and changes are tested in
cubieboard7

Thanks and regards,
Saravanan,
Parthiban

Saravanan Sekar (3):
  arm64: dts: actions: Enable clock controller for S700
  clk: actions: Add S700 SoC clock support
  dt-bindings: clock: Modify Actions Soc clock bindings

 ...ions,s900-cmu.txt => actions,sx00-cmu.txt} |  18 +-
 .../boot/dts/actions/s700-cubieboard7.dts |   7 -
 arch/arm64/boot/dts/actions/s700.dtsi |   8 +
 drivers/clk/actions/Kconfig   |   7 +
 drivers/clk/actions/Makefile  |   1 +
 drivers/clk/actions/owl-s700.c| 614 ++
 include/dt-bindings/clock/actions,s700-cmu.h  | 128 
 7 files changed, 768 insertions(+), 15 deletions(-)
 rename Documentation/devicetree/bindings/clock/{actions,s900-cmu.txt => 
actions,sx00-cmu.txt} (71%)
 create mode 100644 drivers/clk/actions/owl-s700.c
 create mode 100644 include/dt-bindings/clock/actions,s700-cmu.h

-- 
2.18.0