Re: [linux-sunxi] [PATCH 1/3] ARM: dts: sun7i-a20: Rename bananapi as bananapi m1

2017-05-29 Thread Jagan Teki
On Tue, May 30, 2017 at 3:15 AM, Karsten Merker  wrote:
> On Mon, May 29, 2017 at 07:30:26PM +, Jagan Teki wrote:
>> From: Jagan Teki 
>>
>> from BPI(BIPAI KEJI LIMITED) products the Bananapi board
>> is named as 'Bananapi M1' and this is the starting
>> bananapi board from M1 series.
>>
>> So rename dts and suffix 'M1' on model for the same,
>> so-that next sequence on bananapi starts like M1 Plus, M2 and so..on
>>
>> Signed-off-by: Jagan Teki 
>> ---
>> Note: Bananapi BPI product site
>> http://www.banana-pi.org/product.html
>>
>>  arch/arm/boot/dts/Makefile  |   2 +-
>>  arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts | 286 
>> 
>>  arch/arm/boot/dts/sun7i-a20-bananapi.dts| 286 
>> 
>>  3 files changed, 287 insertions(+), 287 deletions(-)
>>  create mode 100644 arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
>>  delete mode 100644 arch/arm/boot/dts/sun7i-a20-bananapi.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index 45c6e65..1b086f0 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -851,7 +851,7 @@ dtb-$(CONFIG_MACH_SUN6I) += \
>>   sun6i-a31s-sinovoip-bpi-m2.dtb \
>>   sun6i-a31s-yones-toptech-bs1078-v2.dtb
>>  dtb-$(CONFIG_MACH_SUN7I) += \
>> - sun7i-a20-bananapi.dtb \
>> + sun7i-a20-bananapi-m1.dtb \
>>   sun7i-a20-bananapi-m1-plus.dtb \
>>   sun7i-a20-bananapro.dtb \
>>   sun7i-a20-cubieboard2.dtb \
>> diff --git a/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts 
>> b/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
>> new file mode 100644
>> index 000..8b97b89
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
>> @@ -0,0 +1,286 @@
>> +/*
>> + * Copyright 2014 Hans de Goede 
>> + *
>> + * Hans de Goede 
> [...]
>> +/dts-v1/;
>> +#include "sun7i-a20.dtsi"
>> +#include "sunxi-common-regulators.dtsi"
>> +
>> +#include 
>> +#include 
>> +
>> +/ {
>> + model = "LeMaker Banana Pi M1";
>> + compatible = "lemaker,bananapi", "allwinner,sun7i-a20";
> [...]
>> diff --git a/arch/arm/boot/dts/sun7i-a20-bananapi.dts 
>> b/arch/arm/boot/dts/sun7i-a20-bananapi.dts
>> deleted file mode 100644
>> index ed2f35a..000
>> --- a/arch/arm/boot/dts/sun7i-a20-bananapi.dts
>> +++ /dev/null
>
> NACK!
>
> Please neither rename the dts nor change the model string. Such a
> change would make newer kernels unusable on many existing
> installations without manual fixups by the end user.  Linux
> distributions use databases with model-specific setup information
> (such as the dtb file name, the platform-specific bootscript to
> use, usable kernel flavours (lpae or non-lpae), etc.) on kernel
> installations and kernel upgrades, and those use the model string
> as their key for finding the relevant information.  If you change
> either the dts file name or the model string inside the dts,
> you'll effectively break the proper installation of newer kernel
> versions on existing end user systems.

I understand your concerns about distribution change, but with new
change in 'bananapi' brand owned by BIPAI KEJI(BPI) the model must
need to update and this is not technically as Bananapi board it is
Bananapi  M1 [1]

These are generic changes based on the hardware vendor info.

[1] http://www.banana-pi.org/m1.html

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: [linux-sunxi] [PATCH v2 1/2] sunxi: Rename bananapi board as bananapi m1

2017-05-29 Thread Chen-Yu Tsai
Hi,

On Tue, May 30, 2017 at 5:58 AM, Karsten Merker  wrote:
> On Tue, May 30, 2017 at 01:06:54AM +0530, Jagan Teki wrote:
>> From: Jagan Teki 
>>
>> from BPI(BIPAI KEJI LIMITED) products the Bananapi board
>> is named as 'Bananapi M1' and this is the starting
>> bananapi board from M1 series.
>>
>> So rename the board defconfig, dts and suffix 'M1' on
>> model for the same  so-that next sequence on bananapi starts
>> like M1 Plus, M2 and so.on
>
> Hello,
>
> the equivalent change in the kernel has been NACKed by me as with
> the proposed change the installation of a newer kernel would
> break a large number of existing end user systems (which rely on
> the old dts file name and on the old model string).
>
> U-Boot should keep the same dts as the kernel, so this
> effectively means a NACK to this change for u-boot as well.

Do the distros also keep a list of defconfig targets to build
for U-boot?  It seems that way for Debian. I imagine just
changing the defconfig names would be troublesome for the
maintainers.

ChenYu

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: [linux-sunxi] [PATCH 1/3] ARM: dts: sun7i-a20: Rename bananapi as bananapi m1

2017-05-29 Thread Icenowy Zheng


于 2017年5月30日 GMT+08:00 上午3:30:26, Jagan Teki  写到:
>From: Jagan Teki 
>
>from BPI(BIPAI KEJI LIMITED) products the Bananapi board
>is named as 'Bananapi M1' and this is the starting
>bananapi board from M1 series.
>
>So rename dts and suffix 'M1' on model for the same,
>so-that next sequence on bananapi starts like M1 Plus, M2 and so..on
>
>Signed-off-by: Jagan Teki 
>---
>Note: Bananapi BPI product site
>http://www.banana-pi.org/product.html
>
> arch/arm/boot/dts/Makefile  |   2 +-
>arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts | 286
>
>arch/arm/boot/dts/sun7i-a20-bananapi.dts| 286
>
> 3 files changed, 287 insertions(+), 287 deletions(-)
> create mode 100644 arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
> delete mode 100644 arch/arm/boot/dts/sun7i-a20-bananapi.dts
>
>diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>index 45c6e65..1b086f0 100644
>--- a/arch/arm/boot/dts/Makefile
>+++ b/arch/arm/boot/dts/Makefile
>@@ -851,7 +851,7 @@ dtb-$(CONFIG_MACH_SUN6I) += \
>   sun6i-a31s-sinovoip-bpi-m2.dtb \
>   sun6i-a31s-yones-toptech-bs1078-v2.dtb
> dtb-$(CONFIG_MACH_SUN7I) += \
>-  sun7i-a20-bananapi.dtb \
>+  sun7i-a20-bananapi-m1.dtb \
>   sun7i-a20-bananapi-m1-plus.dtb \
>   sun7i-a20-bananapro.dtb \
>   sun7i-a20-cubieboard2.dtb \
>diff --git a/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
>b/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
>new file mode 100644
>index 000..8b97b89
>--- /dev/null
>+++ b/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
>@@ -0,0 +1,286 @@
>+/*
>+ * Copyright 2014 Hans de Goede 
>+ *
>+ * Hans de Goede 
>+ *
>+ * This file is dual-licensed: you can use it either under the terms
>+ * of the GPL or the X11 license, at your option. Note that this dual
>+ * licensing only applies to this file, and not this project as a
>+ * whole.
>+ *
>+ *  a) This file is free software; you can redistribute it and/or
>+ * modify it under the terms of the GNU General Public License as
>+ * published by the Free Software Foundation; either version 2 of
>the
>+ * License, or (at your option) any later version.
>+ *
>+ * This file is distributed in the hope that it will be useful,
>+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
>+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>+ * GNU General Public License for more details.
>+ *
>+ * Or, alternatively,
>+ *
>+ *  b) Permission is hereby granted, free of charge, to any person
>+ * obtaining a copy of this software and associated documentation
>+ * files (the "Software"), to deal in the Software without
>+ * restriction, including without limitation the rights to use,
>+ * copy, modify, merge, publish, distribute, sublicense, and/or
>+ * sell copies of the Software, and to permit persons to whom the
>+ * Software is furnished to do so, subject to the following
>+ * conditions:
>+ *
>+ * The above copyright notice and this permission notice shall be
>+ * included in all copies or substantial portions of the Software.
>+ *
>+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
>+ * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
>+ * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
>+ * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
>+ * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
>+ * OTHER DEALINGS IN THE SOFTWARE.
>+ */
>+
>+/dts-v1/;
>+#include "sun7i-a20.dtsi"
>+#include "sunxi-common-regulators.dtsi"
>+
>+#include 
>+#include 
>+
>+/ {
>+  model = "LeMaker Banana Pi M1";
>+  compatible = "lemaker,bananapi", "allwinner,sun7i-a20";

LeMaker never name this BPI M1.

The M1 tag is made by Sinovoip and Bipai.

I suggest you just set model to "Banana Pi M1" and compatible to
three:
"bananapi,bpi-m1", "lemaker,bananapi" (Sinovoip BPi
M1 is fully compatible with the original Banana Pi by LeMaker,
so here're these compatibles) and "allwinner,sun7i-a20".

>+
>+  aliases {
>+  serial0 = 
>+  serial1 = 
>+  serial2 = 
>+  };
>+
>+  chosen {
>+  stdout-path = "serial0:115200n8";
>+  };
>+
>+  leds {
>+  compatible = "gpio-leds";
>+  pinctrl-names = "default";
>+  pinctrl-0 = <_pins_bananapi>;
>+
>+  green {
>+  label = "bananapi:green:usr";
>+  gpios = < 7 24 GPIO_ACTIVE_HIGH>;
>+  };
>+  };
>+
>+  reg_gmac_3v3: gmac-3v3 {
>+  compatible = "regulator-fixed";
>+  pinctrl-names = "default";
>+ 

[linux-sunxi] [PATCH v2 2/2] sunxi: Use proper naming convention for Bananapi boards

2017-05-29 Thread Jagan Teki
From: Jagan Teki 

Since Bananapi brand is owned by BIPAI KEJI(BPI)
- Drop vendor prefix 'Sinovoip' for defconfig names
- Drop Board name 'SINOVOIP' in MAINTAINERS file
- Use small letters for defconfig files, since
  newly supported and/or longer stable defconfigs
  follow lower-case letters.

Cc: Icenowy Zheng 
Cc: Chen-Yu Tsai 
Cc: VishnuPatekar 
Signed-off-by: Jagan Teki 
---
Changes for v2:
- Drop Sinovoip and SINOVOIP vendor prefix since bananapi
  brand no longer used.

 board/sunxi/MAINTAINERS| 14 +++---
 configs/Bananapi_M1_defconfig  | 23 ---
 configs/Bananapi_M2_Ultra_defconfig| 18 --
 configs/Sinovoip_BPI_M2_Plus_defconfig | 20 
 configs/Sinovoip_BPI_M2_defconfig  | 22 --
 configs/Sinovoip_BPI_M3_defconfig  | 29 -
 configs/bananapi_m1_defconfig  | 23 +++
 configs/bananapi_m2_defconfig  | 22 ++
 configs/bananapi_m2_plus_defconfig | 20 
 configs/bananapi_m2_ultra_defconfig| 18 ++
 configs/bananapi_m3_defconfig  | 29 +
 11 files changed, 119 insertions(+), 119 deletions(-)
 delete mode 100644 configs/Bananapi_M1_defconfig
 delete mode 100644 configs/Bananapi_M2_Ultra_defconfig
 delete mode 100644 configs/Sinovoip_BPI_M2_Plus_defconfig
 delete mode 100644 configs/Sinovoip_BPI_M2_defconfig
 delete mode 100644 configs/Sinovoip_BPI_M3_defconfig
 create mode 100644 configs/bananapi_m1_defconfig
 create mode 100644 configs/bananapi_m2_defconfig
 create mode 100644 configs/bananapi_m2_plus_defconfig
 create mode 100644 configs/bananapi_m2_ultra_defconfig
 create mode 100644 configs/bananapi_m3_defconfig

diff --git a/board/sunxi/MAINTAINERS b/board/sunxi/MAINTAINERS
index c2f0227..5f60a4d 100644
--- a/board/sunxi/MAINTAINERS
+++ b/board/sunxi/MAINTAINERS
@@ -41,10 +41,10 @@ F:  configs/CSQ_CS908_defconfig
 F: configs/inet_q972_defconfig
 F: configs/Mele_A1000G_quad_defconfig
 F: configs/Mele_M9_defconfig
-F: configs/Sinovoip_BPI_M2_defconfig
+F: configs/bananapi_m2_defconfig
 F: include/configs/sun7i.h
 F: configs/A20-OLinuXino_MICRO_defconfig
-F: configs/Bananapi_M1_defconfig
+F: configs/bananapi_m1_defconfig
 F: configs/Bananapro_defconfig
 F: configs/i12-tvbox_defconfig
 F: configs/Linksprite_pcDuino3_defconfig
@@ -111,7 +111,7 @@ F:  configs/Ampe_A76_defconfig
 BANANAPI M2 ULTRA BOARD
 M: Chen-Yu Tsai 
 S: Maintained
-F: configs/Bananapi_M2_Ultra_defconfig
+F: configs/bananapi_m2_ultra_defconfig
 F: arch/arm/dts/sun8i-r40-bananapi-m2-ultra.dts
 
 COLOMBUS BOARD
@@ -284,15 +284,15 @@ S:Maintained
 F: configs/Sinlinx_SinA33_defconfig
 W: http://linux-sunxi.org/Sinlinx_SinA33
 
-SINOVOIP BPI M2 PLUS H3 BOARD
+BPI M2 PLUS H3 BOARD
 M: Icenowy Zheng 
 S: Maintained
-F: configs/Sinovoip_BPI_M2_Plus_defconfig
+F: configs/bananapi_m2_plus_defconfig
 
-SINOVOIP BPI M3 A83T BOARD
+BPI M3 A83T BOARD
 M: VishnuPatekar 
 S: Maintained
-F: configs/Sinovoip_BPI_M3_defconfig
+F: configs/bananapi_m3_defconfig
 
 SUNCHIP CX-A99 BOARD
 M: Rask Ingemann Lambertsen 
diff --git a/configs/Bananapi_M1_defconfig b/configs/Bananapi_M1_defconfig
deleted file mode 100644
index 1f8e37b..000
--- a/configs/Bananapi_M1_defconfig
+++ /dev/null
@@ -1,23 +0,0 @@
-CONFIG_ARM=y
-CONFIG_ARCH_SUNXI=y
-CONFIG_MACH_SUN7I=y
-CONFIG_DRAM_CLK=432
-CONFIG_MACPWR="PH23"
-CONFIG_VIDEO_COMPOSITE=y
-CONFIG_GMAC_TX_DELAY=3
-CONFIG_DEFAULT_DEVICE_TREE="sun7i-a20-bananapi-m1"
-CONFIG_AHCI=y
-# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
-CONFIG_SPL=y
-CONFIG_SPL_I2C_SUPPORT=y
-# CONFIG_CMD_IMLS is not set
-# CONFIG_CMD_FLASH is not set
-# CONFIG_CMD_FPGA is not set
-# CONFIG_SPL_DOS_PARTITION is not set
-# CONFIG_SPL_ISO_PARTITION is not set
-# CONFIG_SPL_EFI_PARTITION is not set
-CONFIG_NETCONSOLE=y
-CONFIG_ETH_DESIGNWARE=y
-CONFIG_RGMII=y
-CONFIG_SUN7I_GMAC=y
-CONFIG_USB_EHCI_HCD=y
diff --git a/configs/Bananapi_M2_Ultra_defconfig 
b/configs/Bananapi_M2_Ultra_defconfig
deleted file mode 100644
index 4332eca..000
--- a/configs/Bananapi_M2_Ultra_defconfig
+++ /dev/null
@@ -1,18 +0,0 @@
-CONFIG_ARM=y
-CONFIG_ARCH_SUNXI=y
-CONFIG_MACH_SUN8I_R40=y
-CONFIG_DRAM_CLK=576
-CONFIG_DRAM_ZQ=3881979
-CONFIG_DRAM_ODT_EN=y
-CONFIG_MMC0_CD_PIN="PH13"
-CONFIG_MMC_SUNXI_SLOT_EXTRA=2
-CONFIG_DEFAULT_DEVICE_TREE="sun8i-r40-bananapi-m2-ultra"
-CONFIG_AHCI=y
-# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
-CONFIG_SPL=y
-CONFIG_SPL_I2C_SUPPORT=y
-# CONFIG_CMD_IMLS is not set
-# CONFIG_CMD_FLASH is not set
-# CONFIG_CMD_FPGA is not set
-CONFIG_AXP_DLDO4_VOLT=2500

[linux-sunxi] [PATCH v2 1/2] sunxi: Rename bananapi board as bananapi m1

2017-05-29 Thread Jagan Teki
From: Jagan Teki 

from BPI(BIPAI KEJI LIMITED) products the Bananapi board
is named as 'Bananapi M1' and this is the starting
bananapi board from M1 series.

So rename the board defconfig, dts and suffix 'M1' on
model for the same  so-that next sequence on bananapi starts
like M1 Plus, M2 and so.on

Cc: Icenowy Zheng 
Signed-off-by: Jagan Teki 
---
Changes for v2:
- Newly added patch

 arch/arm/dts/sun7i-a20-bananapi-m1.dts | 293 +
 arch/arm/dts/sun7i-a20-bananapi.dts| 293 -
 board/sunxi/MAINTAINERS|   2 +-
 configs/Bananapi_M1_defconfig  |  23 +++
 configs/Bananapi_defconfig |  23 ---
 5 files changed, 317 insertions(+), 317 deletions(-)
 create mode 100644 arch/arm/dts/sun7i-a20-bananapi-m1.dts
 delete mode 100644 arch/arm/dts/sun7i-a20-bananapi.dts
 create mode 100644 configs/Bananapi_M1_defconfig
 delete mode 100644 configs/Bananapi_defconfig

diff --git a/arch/arm/dts/sun7i-a20-bananapi-m1.dts 
b/arch/arm/dts/sun7i-a20-bananapi-m1.dts
new file mode 100644
index 000..4a1d7d7
--- /dev/null
+++ b/arch/arm/dts/sun7i-a20-bananapi-m1.dts
@@ -0,0 +1,293 @@
+/*
+ * Copyright 2014 Hans de Goede 
+ *
+ * Hans de Goede 
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This file is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This file is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ * obtaining a copy of this software and associated documentation
+ * files (the "Software"), to deal in the Software without
+ * restriction, including without limitation the rights to use,
+ * copy, modify, merge, publish, distribute, sublicense, and/or
+ * sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following
+ * conditions:
+ *
+ * The above copyright notice and this permission notice shall be
+ * included in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+ * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+ * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+ * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ * OTHER DEALINGS IN THE SOFTWARE.
+ */
+
+/dts-v1/;
+#include "sun7i-a20.dtsi"
+#include "sunxi-common-regulators.dtsi"
+
+#include 
+#include 
+#include 
+
+/ {
+   model = "LeMaker Banana Pi M1";
+   compatible = "lemaker,bananapi", "allwinner,sun7i-a20";
+
+   aliases {
+   serial0 = 
+   serial1 = 
+   serial2 = 
+   };
+
+   chosen {
+   stdout-path = "serial0:115200n8";
+   };
+
+   leds {
+   compatible = "gpio-leds";
+   pinctrl-names = "default";
+   pinctrl-0 = <_pins_bananapi>;
+
+   green {
+   label = "bananapi:green:usr";
+   gpios = < 7 24 GPIO_ACTIVE_HIGH>;
+   };
+   };
+
+   reg_gmac_3v3: gmac-3v3 {
+   compatible = "regulator-fixed";
+   pinctrl-names = "default";
+   pinctrl-0 = <_power_pin_bananapi>;
+   regulator-name = "gmac-3v3";
+   regulator-min-microvolt = <330>;
+   regulator-max-microvolt = <330>;
+   startup-delay-us = <10>;
+   enable-active-high;
+   gpio = < 7 23 GPIO_ACTIVE_HIGH>;
+   };
+};
+
+ {
+   status = "okay";
+};
+
+ {
+   status = "okay";
+};
+
+ {
+   cpu-supply = <_dcdc2>;
+   operating-points = <
+   /* kHzuV */
+   96  140
+   912000  140
+   864000  135
+   72  125
+   528000  115
+   312000  110
+   144000  105
+   >;
+};
+
+ {
+   status = "okay";
+};
+
+ {
+  

[linux-sunxi] [PATCH 2/3] ARM: dts: sunxi: Update model and compatible for bananapi

2017-05-29 Thread Jagan Teki
From: Jagan Teki 

Since Bananapi brand is owned by BIPAI KEJI(BPI), the previous
Lemaker and Sinovoip no longer use the bananapi brand, hence
Updated the model and compatible for existing bananapi boards
which is denoted same like in binanapi site.

Example: for Bananapi M64 board as

model = "Banana Pi BPI-M64";
compatible = "bananapi,bpi-m64", "allwinner,sun50i-a64";

Signed-off-by: Jagan Teki 
---
Note: Bananapi BPI product site
http://www.banana-pi.org/product.html

 arch/arm/boot/dts/sun6i-a31s-sinovoip-bpi-m2.dts  | 4 ++--
 arch/arm/boot/dts/sun7i-a20-bananapi-m1-plus.dts  | 4 ++--
 arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts   | 4 ++--
 arch/arm/boot/dts/sun8i-h3-bananapi-m2-plus.dts   | 2 +-
 arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dts | 4 ++--
 5 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/arch/arm/boot/dts/sun6i-a31s-sinovoip-bpi-m2.dts 
b/arch/arm/boot/dts/sun6i-a31s-sinovoip-bpi-m2.dts
index bdfdce8..265f79d 100644
--- a/arch/arm/boot/dts/sun6i-a31s-sinovoip-bpi-m2.dts
+++ b/arch/arm/boot/dts/sun6i-a31s-sinovoip-bpi-m2.dts
@@ -46,8 +46,8 @@
 #include 
 
 / {
-   model = "Sinovoip BPI-M2";
-   compatible = "sinovoip,bpi-m2", "allwinner,sun6i-a31s";
+   model = "Banana PI BPI-M2";
+   compatible = "bananapi,bpi-m2", "allwinner,sun6i-a31s";
 
aliases {
serial0 = 
diff --git a/arch/arm/boot/dts/sun7i-a20-bananapi-m1-plus.dts 
b/arch/arm/boot/dts/sun7i-a20-bananapi-m1-plus.dts
index 08e776a..2c87236 100644
--- a/arch/arm/boot/dts/sun7i-a20-bananapi-m1-plus.dts
+++ b/arch/arm/boot/dts/sun7i-a20-bananapi-m1-plus.dts
@@ -49,8 +49,8 @@
 #include 
 
 / {
-   model = "Banana Pi BPI-M1-Plus";
-   compatible = "sinovoip,bpi-m1-plus", "allwinner,sun7i-a20";
+   model = "Banana Pi BPI-M1+";
+   compatible = "bananapi,bpi-m1-plus", "allwinner,sun7i-a20";
 
aliases {
serial0 = 
diff --git a/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts 
b/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
index 8b97b89..3d2597e 100644
--- a/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
+++ b/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
@@ -50,8 +50,8 @@
 #include 
 
 / {
-   model = "LeMaker Banana Pi M1";
-   compatible = "lemaker,bananapi", "allwinner,sun7i-a20";
+   model = "Banana Pi BPI-M1";
+   compatible = "bananapi,bpi-m1", "allwinner,sun7i-a20";
 
aliases {
serial0 = 
diff --git a/arch/arm/boot/dts/sun8i-h3-bananapi-m2-plus.dts 
b/arch/arm/boot/dts/sun8i-h3-bananapi-m2-plus.dts
index 52acbe1..643b8ec 100644
--- a/arch/arm/boot/dts/sun8i-h3-bananapi-m2-plus.dts
+++ b/arch/arm/boot/dts/sun8i-h3-bananapi-m2-plus.dts
@@ -49,7 +49,7 @@
 
 / {
model = "Banana Pi BPI-M2-Plus";
-   compatible = "sinovoip,bpi-m2-plus", "allwinner,sun8i-h3";
+   compatible = "bananapi,bpi-m2-plus", "allwinner,sun8i-h3";
 
aliases {
serial0 = 
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dts 
b/arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dts
index 6872135..949edba 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dts
@@ -47,8 +47,8 @@
 #include 
 
 / {
-   model = "BananaPi-M64";
-   compatible = "sinovoip,bananapi-m64", "allwinner,sun50i-a64";
+   model = "Banana Pi BPI-M64";
+   compatible = "bananapi,bpi-m64", "allwinner,sun50i-a64";
 
aliases {
serial0 = 
-- 
1.9.1

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[linux-sunxi] [PATCH 1/3] ARM: dts: sun7i-a20: Rename bananapi as bananapi m1

2017-05-29 Thread Jagan Teki
From: Jagan Teki 

from BPI(BIPAI KEJI LIMITED) products the Bananapi board
is named as 'Bananapi M1' and this is the starting
bananapi board from M1 series.

So rename dts and suffix 'M1' on model for the same,
so-that next sequence on bananapi starts like M1 Plus, M2 and so..on

Signed-off-by: Jagan Teki 
---
Note: Bananapi BPI product site
http://www.banana-pi.org/product.html

 arch/arm/boot/dts/Makefile  |   2 +-
 arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts | 286 
 arch/arm/boot/dts/sun7i-a20-bananapi.dts| 286 
 3 files changed, 287 insertions(+), 287 deletions(-)
 create mode 100644 arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
 delete mode 100644 arch/arm/boot/dts/sun7i-a20-bananapi.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 45c6e65..1b086f0 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -851,7 +851,7 @@ dtb-$(CONFIG_MACH_SUN6I) += \
sun6i-a31s-sinovoip-bpi-m2.dtb \
sun6i-a31s-yones-toptech-bs1078-v2.dtb
 dtb-$(CONFIG_MACH_SUN7I) += \
-   sun7i-a20-bananapi.dtb \
+   sun7i-a20-bananapi-m1.dtb \
sun7i-a20-bananapi-m1-plus.dtb \
sun7i-a20-bananapro.dtb \
sun7i-a20-cubieboard2.dtb \
diff --git a/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts 
b/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
new file mode 100644
index 000..8b97b89
--- /dev/null
+++ b/arch/arm/boot/dts/sun7i-a20-bananapi-m1.dts
@@ -0,0 +1,286 @@
+/*
+ * Copyright 2014 Hans de Goede 
+ *
+ * Hans de Goede 
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This file is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This file is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ * obtaining a copy of this software and associated documentation
+ * files (the "Software"), to deal in the Software without
+ * restriction, including without limitation the rights to use,
+ * copy, modify, merge, publish, distribute, sublicense, and/or
+ * sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following
+ * conditions:
+ *
+ * The above copyright notice and this permission notice shall be
+ * included in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+ * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+ * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+ * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ * OTHER DEALINGS IN THE SOFTWARE.
+ */
+
+/dts-v1/;
+#include "sun7i-a20.dtsi"
+#include "sunxi-common-regulators.dtsi"
+
+#include 
+#include 
+
+/ {
+   model = "LeMaker Banana Pi M1";
+   compatible = "lemaker,bananapi", "allwinner,sun7i-a20";
+
+   aliases {
+   serial0 = 
+   serial1 = 
+   serial2 = 
+   };
+
+   chosen {
+   stdout-path = "serial0:115200n8";
+   };
+
+   leds {
+   compatible = "gpio-leds";
+   pinctrl-names = "default";
+   pinctrl-0 = <_pins_bananapi>;
+
+   green {
+   label = "bananapi:green:usr";
+   gpios = < 7 24 GPIO_ACTIVE_HIGH>;
+   };
+   };
+
+   reg_gmac_3v3: gmac-3v3 {
+   compatible = "regulator-fixed";
+   pinctrl-names = "default";
+   pinctrl-0 = <_power_pin_bananapi>;
+   regulator-name = "gmac-3v3";
+   regulator-min-microvolt = <330>;
+   regulator-max-microvolt = <330>;
+   startup-delay-us = <10>;
+   enable-active-high;
+   gpio = < 7 23 GPIO_ACTIVE_HIGH>;
+   };
+};
+
+ {
+   status = "okay";
+};
+
+ {
+   status = "okay";
+};
+
+ {
+   cpu-supply = <_dcdc2>;
+   operating-points = <
+

[linux-sunxi] [PATCH 3/3] ARM: dts: sun6i-a31s: Rename sun6i-a31s-sinovoip-bpi-m2

2017-05-29 Thread Jagan Teki
From: Jagan Teki 

Since Bananapi brand is owned by BIPAI KEJI(BPI),
the previous Sinovoip no longer use the bananapi brand,
hence rename sun6i-a31s-sinovoip-bpi-m2.dts to
sun6i-a31s-bananapi-m2.dts

Signed-off-by: Jagan Teki 
---
Note: Bananapi BPI product site
http://www.banana-pi.org/product.html

 arch/arm/boot/dts/Makefile   |   2 +-
 arch/arm/boot/dts/sun6i-a31s-bananapi-m2.dts | 244 +++
 arch/arm/boot/dts/sun6i-a31s-sinovoip-bpi-m2.dts | 244 ---
 3 files changed, 245 insertions(+), 245 deletions(-)
 create mode 100644 arch/arm/boot/dts/sun6i-a31s-bananapi-m2.dts
 delete mode 100644 arch/arm/boot/dts/sun6i-a31s-sinovoip-bpi-m2.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 1b086f0..e588721 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -848,7 +848,7 @@ dtb-$(CONFIG_MACH_SUN6I) += \
sun6i-a31s-inet-q972.dtb \
sun6i-a31s-primo81.dtb \
sun6i-a31s-sina31s.dtb \
-   sun6i-a31s-sinovoip-bpi-m2.dtb \
+   sun6i-a31s-bananapi-m2.dtb \
sun6i-a31s-yones-toptech-bs1078-v2.dtb
 dtb-$(CONFIG_MACH_SUN7I) += \
sun7i-a20-bananapi-m1.dtb \
diff --git a/arch/arm/boot/dts/sun6i-a31s-bananapi-m2.dts 
b/arch/arm/boot/dts/sun6i-a31s-bananapi-m2.dts
new file mode 100644
index 000..265f79d
--- /dev/null
+++ b/arch/arm/boot/dts/sun6i-a31s-bananapi-m2.dts
@@ -0,0 +1,244 @@
+/*
+ * Copyright 2015 Hans de Goede 
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ * obtaining a copy of this software and associated documentation
+ * files (the "Software"), to deal in the Software without
+ * restriction, including without limitation the rights to use,
+ * copy, modify, merge, publish, distribute, sublicense, and/or
+ * sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following
+ * conditions:
+ *
+ * The above copyright notice and this permission notice shall be
+ * included in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+ * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+ * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+ * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ * OTHER DEALINGS IN THE SOFTWARE.
+ */
+
+/dts-v1/;
+#include "sun6i-a31s.dtsi"
+#include "sunxi-common-regulators.dtsi"
+#include 
+
+/ {
+   model = "Banana PI BPI-M2";
+   compatible = "bananapi,bpi-m2", "allwinner,sun6i-a31s";
+
+   aliases {
+   serial0 = 
+   };
+
+   chosen {
+   stdout-path = "serial0:115200n8";
+   };
+
+   leds {
+   compatible = "gpio-leds";
+   pinctrl-names = "default";
+   pinctrl-0 = <_pins_bpi_m2>;
+
+   blue {
+   label = "bpi-m2:blue:usr";
+   gpios = < 6 11 GPIO_ACTIVE_HIGH>; /* PG11 */
+   };
+
+   green {
+   label = "bpi-m2:green:usr";
+   gpios = < 6 10 GPIO_ACTIVE_HIGH>; /* PG10 */
+   };
+
+   red {
+   label = "bpi-m2:red:usr";
+   gpios = < 6 5 GPIO_ACTIVE_HIGH>; /* PG5 */
+   };
+   };
+
+   mmc2_pwrseq: mmc2_pwrseq {
+   compatible = "mmc-pwrseq-simple";
+   pinctrl-names = "default";
+   pinctrl-0 = <_pwrseq_pin_bpi_m2>;
+   reset-gpios = <_pio 0 8 GPIO_ACTIVE_LOW>; /* PL8 WIFI_EN */
+   };
+};
+
+ {
+   cpu-supply = <_dcdc3>;
+};
+
+ {
+   status = "okay";
+};
+
+ {
+   pinctrl-names = "default";
+   pinctrl-0 = <_pins_rgmii_a>, <_phy_reset_pin_bpi_m2>;
+   phy = <>;
+ 

[linux-sunxi] Re: [U-Boot] [PATCH] sunxi: sinovoip: Use proper naming convention for BPI boards

2017-05-29 Thread Chen-Yu Tsai
On Mon, May 29, 2017 at 11:37 PM, Jagan Teki  wrote:
> On Mon, May 29, 2017 at 8:27 PM, Chen-Yu Tsai  wrote:
>> On Mon, May 29, 2017 at 10:24 PM, Jagan Teki  wrote:
>>> From: Jagan Teki 
>>>
>>> - Rename Sinovoip BPI defconfigs to use small letter
>>> - To avoid naming conflict with 'LeMaker' add sinovoip
>>>   prefix to BPI M2 Ultra defconfig since bananapi is
>>>   also name used by 'LeMaker'
>>> - Add 'SINOVOIP' prefix 'BANANAPI M2 ULTRA BOARD' under
>>>   board/sunxi/MAINTAINERS file.
>>
>> No. The proper solution is to just get rid of the Sinovoip and
>> LeMaker bullshit. The brand name has been and will continue to be
>> "Banana Pi". Apart from the trademark legal drama between LeMaker,
>> Sinovoip, and Foxconn, the boards are now consistently named
>> "BPI-xyz". The original "Banana Pi" is now the "BPI-M1". The only
>> incompatible outlier is the LeMaker made BananaPro.
>
> But there is one board Bananapi_defconfig which is from LeMaker, which
> will conflict if we skip these and use binanapi for rest.

There's nothing that conflicts. Just like for the Cubieboards,
Bananapi serves as the prefix. So the defconfigs would be named:

  - Bananapi_defconfig (or Bananapi_M1_defconfig if you prefer)
  - Bananapi_M1_Plus_defconfig
  - Bananapi_M2_defconfig

and the list goes on.

>>
>> LeMaker and Sinovoip are distributors. Maybe they manufacture(d)
>> as well, but I'm not so sure.
>>
>> Also, see https://www.spinics.net/lists/devicetree/msg176001.html
>>
>> Note that "BIPAI KEJI LIMITED" is just a translation for BPI
>> Technologies limited. BIPAI is the phonetics for B-PI in English.
>> KEJI is the phonetics for "technology" in Chinese.
>
> Thanks for the info.
>
>>
>> Finally, is there a preference for all lowercase defconfig file names?
>
> Yes, for newly supported and/or longer stable defconfigs/boards follow
> lower-case defconfig file names in U-Boot.

I see. Thanks!

ChenYu

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[linux-sunxi] Re: [U-Boot] [PATCH] sunxi: sinovoip: Use proper naming convention for BPI boards

2017-05-29 Thread Jagan Teki
On Mon, May 29, 2017 at 8:27 PM, Chen-Yu Tsai  wrote:
> On Mon, May 29, 2017 at 10:24 PM, Jagan Teki  wrote:
>> From: Jagan Teki 
>>
>> - Rename Sinovoip BPI defconfigs to use small letter
>> - To avoid naming conflict with 'LeMaker' add sinovoip
>>   prefix to BPI M2 Ultra defconfig since bananapi is
>>   also name used by 'LeMaker'
>> - Add 'SINOVOIP' prefix 'BANANAPI M2 ULTRA BOARD' under
>>   board/sunxi/MAINTAINERS file.
>
> No. The proper solution is to just get rid of the Sinovoip and
> LeMaker bullshit. The brand name has been and will continue to be
> "Banana Pi". Apart from the trademark legal drama between LeMaker,
> Sinovoip, and Foxconn, the boards are now consistently named
> "BPI-xyz". The original "Banana Pi" is now the "BPI-M1". The only
> incompatible outlier is the LeMaker made BananaPro.

But there is one board Bananapi_defconfig which is from LeMaker, which
will conflict if we skip these and use binanapi for rest.

>
> LeMaker and Sinovoip are distributors. Maybe they manufacture(d)
> as well, but I'm not so sure.
>
> Also, see https://www.spinics.net/lists/devicetree/msg176001.html
>
> Note that "BIPAI KEJI LIMITED" is just a translation for BPI
> Technologies limited. BIPAI is the phonetics for B-PI in English.
> KEJI is the phonetics for "technology" in Chinese.

Thanks for the info.

>
> Finally, is there a preference for all lowercase defconfig file names?

Yes, for newly supported and/or longer stable defconfigs/boards follow
lower-case defconfig file names in U-Boot.

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[linux-sunxi] Re: [PATCH] sunxi: sinovoip: Use proper naming convention for BPI boards

2017-05-29 Thread Chen-Yu Tsai
On Mon, May 29, 2017 at 10:24 PM, Jagan Teki  wrote:
> From: Jagan Teki 
>
> - Rename Sinovoip BPI defconfigs to use small letter
> - To avoid naming conflict with 'LeMaker' add sinovoip
>   prefix to BPI M2 Ultra defconfig since bananapi is
>   also name used by 'LeMaker'
> - Add 'SINOVOIP' prefix 'BANANAPI M2 ULTRA BOARD' under
>   board/sunxi/MAINTAINERS file.

No. The proper solution is to just get rid of the Sinovoip and
LeMaker bullshit. The brand name has been and will continue to be
"Banana Pi". Apart from the trademark legal drama between LeMaker,
Sinovoip, and Foxconn, the boards are now consistently named
"BPI-xyz". The original "Banana Pi" is now the "BPI-M1". The only
incompatible outlier is the LeMaker made BananaPro.

LeMaker and Sinovoip are distributors. Maybe they manufacture(d)
as well, but I'm not so sure.

Also, see https://www.spinics.net/lists/devicetree/msg176001.html

Note that "BIPAI KEJI LIMITED" is just a translation for BPI
Technologies limited. BIPAI is the phonetics for B-PI in English.
KEJI is the phonetics for "technology" in Chinese.

Finally, is there a preference for all lowercase defconfig file names?


Regards
ChenYu

> Cc: Icenowy Zheng 
> Cc: Chen-Yu Tsai 
> Cc: VishnuPatekar 
> Signed-off-by: Jagan Teki 
> ---
>  board/sunxi/MAINTAINERS| 10 
> +-
>  .../{Sinovoip_BPI_M2_defconfig => sinovoip_bpi_m2_defconfig}   |  0
>  ...ip_BPI_M2_Plus_defconfig => sinovoip_bpi_m2_plus_defconfig} |  0
>  ...napi_M2_Ultra_defconfig => sinovoip_bpi_m2_ultra_defconfig} |  0
>  .../{Sinovoip_BPI_M3_defconfig => sinovoip_bpi_m3_defconfig}   |  0
>  5 files changed, 5 insertions(+), 5 deletions(-)
>  rename configs/{Sinovoip_BPI_M2_defconfig => sinovoip_bpi_m2_defconfig} 
> (100%)
>  rename configs/{Sinovoip_BPI_M2_Plus_defconfig => 
> sinovoip_bpi_m2_plus_defconfig} (100%)
>  rename configs/{Bananapi_M2_Ultra_defconfig => 
> sinovoip_bpi_m2_ultra_defconfig} (100%)
>  rename configs/{Sinovoip_BPI_M3_defconfig => sinovoip_bpi_m3_defconfig} 
> (100%)
>
> diff --git a/board/sunxi/MAINTAINERS b/board/sunxi/MAINTAINERS
> index a512a20..61d1479 100644
> --- a/board/sunxi/MAINTAINERS
> +++ b/board/sunxi/MAINTAINERS
> @@ -41,7 +41,7 @@ F:configs/CSQ_CS908_defconfig
>  F: configs/inet_q972_defconfig
>  F: configs/Mele_A1000G_quad_defconfig
>  F: configs/Mele_M9_defconfig
> -F: configs/Sinovoip_BPI_M2_defconfig
> +F: configs/sinovoip_bpi_m2_defconfig
>  F: include/configs/sun7i.h
>  F: configs/A20-OLinuXino_MICRO_defconfig
>  F: configs/Bananapi_defconfig
> @@ -108,10 +108,10 @@ M:Paul Kocialkowski 
>  S: Maintained
>  F: configs/Ampe_A76_defconfig
>
> -BANANAPI M2 ULTRA BOARD
> +SINOVOIP BANANAPI M2 ULTRA BOARD
>  M: Chen-Yu Tsai 
>  S: Maintained
> -F: configs/Bananapi_M2_Ultra_defconfig
> +F: configs/sinovoip_bpi_m2_ultra_defconfig
>  F: arch/arm/dts/sun8i-r40-bananapi-m2-ultra.dts
>
>  COLOMBUS BOARD
> @@ -287,12 +287,12 @@ W:http://linux-sunxi.org/Sinlinx_SinA33
>  SINOVOIP BPI M2 PLUS H3 BOARD
>  M: Icenowy Zheng 
>  S: Maintained
> -F: configs/Sinovoip_BPI_M2_Plus_defconfig
> +F: configs/sinovoip_bpi_m2_Plus_defconfig
>
>  SINOVOIP BPI M3 A83T BOARD
>  M: VishnuPatekar 
>  S: Maintained
> -F: configs/Sinovoip_BPI_M3_defconfig
> +F: configs/sinovoip_bpi_m3_defconfig
>
>  SUNCHIP CX-A99 BOARD
>  M: Rask Ingemann Lambertsen 
> diff --git a/configs/Sinovoip_BPI_M2_defconfig 
> b/configs/sinovoip_bpi_m2_defconfig
> similarity index 100%
> rename from configs/Sinovoip_BPI_M2_defconfig
> rename to configs/sinovoip_bpi_m2_defconfig
> diff --git a/configs/Sinovoip_BPI_M2_Plus_defconfig 
> b/configs/sinovoip_bpi_m2_plus_defconfig
> similarity index 100%
> rename from configs/Sinovoip_BPI_M2_Plus_defconfig
> rename to configs/sinovoip_bpi_m2_plus_defconfig
> diff --git a/configs/Bananapi_M2_Ultra_defconfig 
> b/configs/sinovoip_bpi_m2_ultra_defconfig
> similarity index 100%
> rename from configs/Bananapi_M2_Ultra_defconfig
> rename to configs/sinovoip_bpi_m2_ultra_defconfig
> diff --git a/configs/Sinovoip_BPI_M3_defconfig 
> b/configs/sinovoip_bpi_m3_defconfig
> similarity index 100%
> rename from configs/Sinovoip_BPI_M3_defconfig
> rename to configs/sinovoip_bpi_m3_defconfig
> --
> 1.9.1
>

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[linux-sunxi] [PATCH] sunxi: sinovoip: Use proper naming convention for BPI boards

2017-05-29 Thread Jagan Teki
From: Jagan Teki 

- Rename Sinovoip BPI defconfigs to use small letter
- To avoid naming conflict with 'LeMaker' add sinovoip
  prefix to BPI M2 Ultra defconfig since bananapi is
  also name used by 'LeMaker'
- Add 'SINOVOIP' prefix 'BANANAPI M2 ULTRA BOARD' under
  board/sunxi/MAINTAINERS file.

Cc: Icenowy Zheng 
Cc: Chen-Yu Tsai 
Cc: VishnuPatekar 
Signed-off-by: Jagan Teki 
---
 board/sunxi/MAINTAINERS| 10 +-
 .../{Sinovoip_BPI_M2_defconfig => sinovoip_bpi_m2_defconfig}   |  0
 ...ip_BPI_M2_Plus_defconfig => sinovoip_bpi_m2_plus_defconfig} |  0
 ...napi_M2_Ultra_defconfig => sinovoip_bpi_m2_ultra_defconfig} |  0
 .../{Sinovoip_BPI_M3_defconfig => sinovoip_bpi_m3_defconfig}   |  0
 5 files changed, 5 insertions(+), 5 deletions(-)
 rename configs/{Sinovoip_BPI_M2_defconfig => sinovoip_bpi_m2_defconfig} (100%)
 rename configs/{Sinovoip_BPI_M2_Plus_defconfig => 
sinovoip_bpi_m2_plus_defconfig} (100%)
 rename configs/{Bananapi_M2_Ultra_defconfig => 
sinovoip_bpi_m2_ultra_defconfig} (100%)
 rename configs/{Sinovoip_BPI_M3_defconfig => sinovoip_bpi_m3_defconfig} (100%)

diff --git a/board/sunxi/MAINTAINERS b/board/sunxi/MAINTAINERS
index a512a20..61d1479 100644
--- a/board/sunxi/MAINTAINERS
+++ b/board/sunxi/MAINTAINERS
@@ -41,7 +41,7 @@ F:configs/CSQ_CS908_defconfig
 F: configs/inet_q972_defconfig
 F: configs/Mele_A1000G_quad_defconfig
 F: configs/Mele_M9_defconfig
-F: configs/Sinovoip_BPI_M2_defconfig
+F: configs/sinovoip_bpi_m2_defconfig
 F: include/configs/sun7i.h
 F: configs/A20-OLinuXino_MICRO_defconfig
 F: configs/Bananapi_defconfig
@@ -108,10 +108,10 @@ M:Paul Kocialkowski 
 S: Maintained
 F: configs/Ampe_A76_defconfig
 
-BANANAPI M2 ULTRA BOARD
+SINOVOIP BANANAPI M2 ULTRA BOARD
 M: Chen-Yu Tsai 
 S: Maintained
-F: configs/Bananapi_M2_Ultra_defconfig
+F: configs/sinovoip_bpi_m2_ultra_defconfig
 F: arch/arm/dts/sun8i-r40-bananapi-m2-ultra.dts
 
 COLOMBUS BOARD
@@ -287,12 +287,12 @@ W:http://linux-sunxi.org/Sinlinx_SinA33
 SINOVOIP BPI M2 PLUS H3 BOARD
 M: Icenowy Zheng 
 S: Maintained
-F: configs/Sinovoip_BPI_M2_Plus_defconfig
+F: configs/sinovoip_bpi_m2_Plus_defconfig
 
 SINOVOIP BPI M3 A83T BOARD
 M: VishnuPatekar 
 S: Maintained
-F: configs/Sinovoip_BPI_M3_defconfig
+F: configs/sinovoip_bpi_m3_defconfig
 
 SUNCHIP CX-A99 BOARD
 M: Rask Ingemann Lambertsen 
diff --git a/configs/Sinovoip_BPI_M2_defconfig 
b/configs/sinovoip_bpi_m2_defconfig
similarity index 100%
rename from configs/Sinovoip_BPI_M2_defconfig
rename to configs/sinovoip_bpi_m2_defconfig
diff --git a/configs/Sinovoip_BPI_M2_Plus_defconfig 
b/configs/sinovoip_bpi_m2_plus_defconfig
similarity index 100%
rename from configs/Sinovoip_BPI_M2_Plus_defconfig
rename to configs/sinovoip_bpi_m2_plus_defconfig
diff --git a/configs/Bananapi_M2_Ultra_defconfig 
b/configs/sinovoip_bpi_m2_ultra_defconfig
similarity index 100%
rename from configs/Bananapi_M2_Ultra_defconfig
rename to configs/sinovoip_bpi_m2_ultra_defconfig
diff --git a/configs/Sinovoip_BPI_M3_defconfig 
b/configs/sinovoip_bpi_m3_defconfig
similarity index 100%
rename from configs/Sinovoip_BPI_M3_defconfig
rename to configs/sinovoip_bpi_m3_defconfig
-- 
1.9.1

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: [linux-sunxi] [PATCH 1/2] ARM: sun8i: v3s: add device nodes for DE2 display pipeline

2017-05-29 Thread icenowy

在 2017-05-29 16:59,Maxime Ripard 写道:

On Thu, May 25, 2017 at 10:28:24PM +0800, Icenowy Zheng wrote:

 +   compatible = "allwinner,sun8i-v3s-de2-mixer";
 +   reg = <0x0110 0x10>;
>>>
>>>The display engine also has an interrupt. Please list it
>>
>> It's a overall interrupt for DE2, not mixer interrupt.
>>
>> I will add it after we have proper interrupt chaining facility
>> for DE2.
>>
>> It cannot fit here in mixer node.
>
> Could you elaborate? If it is just shared, then listing it several
> times is OK. If the interrupt registers are not in this address
> space, then

The interrupt dealing code is still not so clear, and totally not
present in BSP DE2 code (the BSP just didn't use the interrupt).

We doesn't even know where the interrupt register is, and we cannot
promise it's not broken (as their is even no application of it in 
BSP).


Whether it's broken or not is out of the question here. Adding it
later would either break the DT compatibility, or introduce a crippled
driver for users that do not have a recent enough DT. None of the two
are really acceptable.

Especially if your plan is to add support for these.


I have no such plan -- even the BSP driver doesn't use the IRQ at all.



Maxime

--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


--
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[linux-sunxi] Re: [U-Boot] [PATCH v3] sunxi: A64/Pine64: update device tree from Linux

2017-05-29 Thread Jagan Teki
On Wed, May 24, 2017 at 3:04 PM, Andre Przywara  wrote:
> The Linux device tree for the Allwinner A64 SoC has changed a lot since
> the U-Boot version was merged.
> Let's replace the current DT with a exact copy of the Linux one as of:
> commit c6778ff813d2ca3e3c8733c87dc8b6831a64578b
> Merge: 0ff4c01 3c0e3abd
> Author: Linus Torvalds 
> Date:   Tue May 9 10:07:33 2017 -0700
>
> This is the DT used in Linux 4.12-rc1.
>
> Since U-Boot has an Ethernet driver (while Linux does not yet), we
> provide the required DT nodes for it in an ...-u-boot.dtsi file, to both
> mark them as U-Boot specific and to allow easier upgrading once Linux gets
> the driver and its own binding later.
> Compared to the existing Ethernet DT nodes we just slightly tweak the clock
> and reset nodes in there to match the new bindings used by Linux for those.
>
> Signed-off-by: Andre Przywara 
> ---
>  arch/arm/dts/sun50i-a64-pine64-plus-u-boot.dtsi|  50 ++

Yeah, may be this need to be named something like
sun50i-a64-u-boot.dtsi as of now? since it may share the similar dt on
other a64 board that we initiate the discussion about orangepi, and
BPI.

All fine expect this name.

Reviewed-by: Jagan Teki 
Tested-by: Jagan Teki 

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[linux-sunxi] Re: [PATCH 3/3] ARM: sun8i: v3s: add backlight device node for Lichee Pi Zero LCD

2017-05-29 Thread Maxime Ripard
On Mon, May 29, 2017 at 04:16:51PM +0800, Chen-Yu Tsai wrote:
> On Mon, May 29, 2017 at 3:41 PM, Maxime Ripard
>  wrote:
> > Hi,
> >
> > On Wed, May 24, 2017 at 08:13:20PM +0800, Icenowy Zheng wrote:
> >> The 40-pin LCD connector on Lichee Pi Zero has backlight pins, which is
> >> controlled by the PWM0 controller of the V3s SoC, and the controlling
> >> part is on the board.
> >>
> >> Add the PWM and backlight device nodes in the device tree file, but
> >> leave them disabled, as they can only be useful when the LCD is
> >> attached. The LCD device tree overlay files can enable these controllers
> >> and make use of them.
> >>
> >> Signed-off-by: Icenowy Zheng 
> >> ---
> >>  arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts | 14 ++
> >>  1 file changed, 14 insertions(+)
> >>
> >> diff --git a/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts 
> >> b/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts
> >> index 387fc2aa546d..e0721bf725d1 100644
> >> --- a/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts
> >> +++ b/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts
> >> @@ -52,6 +52,14 @@
> >>   serial0 = 
> >>   };
> >>
> >> + backlight: backlight {
> >> + compatible = "pwm-backlight";
> >> + pwms = < 0 5 0>;
> >> + brightness-levels = <0 10 20 30 40 50 60 70 80 90 100>;
> >
> > This is unlikely, the perceived brightness by the human eye is not
> > linear, and is actually much closer to a logarithmic function, which
> > means you should have use an exponential function here in order to
> > appear that your brightness increase is linear.
> 
> I think the backlight is a functional part of the panel. It probably
> belongs in the panel's overlay.

Yeah, that's also true.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature


[linux-sunxi] Re: [PATCH v4 00/11] drm: sun4i: Add support for the HDMI controller

2017-05-29 Thread Maxime Ripard
On Sat, May 27, 2017 at 06:09:25PM +0200, Maxime Ripard wrote:
> Hi,
> 
> Here is an attempt at getting the HDMI controller running.
> 
> This HDMI controller is found on a number of old Allwinner SoCs (A10, A10s,
> A20, A31).
> 
> This driver only supports for now the A10s because it was an easy target,
> being very close to the A13 that is already supported by our DRM driver.
> 
> There's nothing out of the extraordinary there, except maybe the clock
> setup. All the internal clocks (TMDS, DDC) have been modeled using the
> common clock framework, the TMDS clock being the parent of the DDC one.
> 
> While this might sound overkill, other SoC have a different, external
> source for the DDC clock, which will be easier to support through the clock
> framework.
> 
> The IP also supports audio (through an already supported i2s controller,
> and some missing configuration in the HDMI controller) and CEC. Both will
> come eventually.

Applied all the patches.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature


Re: [linux-sunxi] [PATCH 1/2] ARM: sun8i: v3s: add device nodes for DE2 display pipeline

2017-05-29 Thread Maxime Ripard
On Thu, May 25, 2017 at 10:28:24PM +0800, Icenowy Zheng wrote:
>  +   compatible = "allwinner,sun8i-v3s-de2-mixer";
>  +   reg = <0x0110 0x10>;
> >>>
> >>>The display engine also has an interrupt. Please list it
> >>
> >> It's a overall interrupt for DE2, not mixer interrupt.
> >>
> >> I will add it after we have proper interrupt chaining facility
> >> for DE2.
> >>
> >> It cannot fit here in mixer node.
> >
> > Could you elaborate? If it is just shared, then listing it several
> > times is OK. If the interrupt registers are not in this address
> > space, then
> 
> The interrupt dealing code is still not so clear, and totally not
> present in BSP DE2 code (the BSP just didn't use the interrupt).
> 
> We doesn't even know where the interrupt register is, and we cannot
> promise it's not broken (as their is even no application of it in BSP).

Whether it's broken or not is out of the question here. Adding it
later would either break the DT compatibility, or introduce a crippled
driver for users that do not have a recent enough DT. None of the two
are really acceptable.

Especially if your plan is to add support for these.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature


[linux-sunxi] Re: [PATCH v3] sunxi: A64/Pine64: update device tree from Linux

2017-05-29 Thread Maxime Ripard
On Wed, May 24, 2017 at 10:34:56AM +0100, Andre Przywara wrote:
> The Linux device tree for the Allwinner A64 SoC has changed a lot since
> the U-Boot version was merged.
> Let's replace the current DT with a exact copy of the Linux one as of:
> commit c6778ff813d2ca3e3c8733c87dc8b6831a64578b
> Merge: 0ff4c01 3c0e3abd
> Author: Linus Torvalds 
> Date:   Tue May 9 10:07:33 2017 -0700
> 
> This is the DT used in Linux 4.12-rc1.
> 
> Since U-Boot has an Ethernet driver (while Linux does not yet), we
> provide the required DT nodes for it in an ...-u-boot.dtsi file, to both
> mark them as U-Boot specific and to allow easier upgrading once Linux gets
> the driver and its own binding later.
> Compared to the existing Ethernet DT nodes we just slightly tweak the clock
> and reset nodes in there to match the new bindings used by Linux for those.
> 
> Signed-off-by: Andre Przywara 

Acked-by: Maxime Ripard 

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature


[linux-sunxi] Re: [PATCH 3/3] ARM: sun8i: v3s: add backlight device node for Lichee Pi Zero LCD

2017-05-29 Thread Chen-Yu Tsai
On Mon, May 29, 2017 at 3:41 PM, Maxime Ripard
 wrote:
> Hi,
>
> On Wed, May 24, 2017 at 08:13:20PM +0800, Icenowy Zheng wrote:
>> The 40-pin LCD connector on Lichee Pi Zero has backlight pins, which is
>> controlled by the PWM0 controller of the V3s SoC, and the controlling
>> part is on the board.
>>
>> Add the PWM and backlight device nodes in the device tree file, but
>> leave them disabled, as they can only be useful when the LCD is
>> attached. The LCD device tree overlay files can enable these controllers
>> and make use of them.
>>
>> Signed-off-by: Icenowy Zheng 
>> ---
>>  arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts | 14 ++
>>  1 file changed, 14 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts 
>> b/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts
>> index 387fc2aa546d..e0721bf725d1 100644
>> --- a/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts
>> +++ b/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts
>> @@ -52,6 +52,14 @@
>>   serial0 = 
>>   };
>>
>> + backlight: backlight {
>> + compatible = "pwm-backlight";
>> + pwms = < 0 5 0>;
>> + brightness-levels = <0 10 20 30 40 50 60 70 80 90 100>;
>
> This is unlikely, the perceived brightness by the human eye is not
> linear, and is actually much closer to a logarithmic function, which
> means you should have use an exponential function here in order to
> appear that your brightness increase is linear.

I think the backlight is a functional part of the panel. It probably
belongs in the panel's overlay.

ChenYu

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[linux-sunxi] Re: [PATCH 3/3] ARM: sun8i: a83t: Add device node for PRCM

2017-05-29 Thread Maxime Ripard
Hi,

On Fri, May 26, 2017 at 04:00:25PM +0800, Chen-Yu Tsai wrote:
> The A83T's PRCM has the same set of clocks and resets as the A64.
> However, a few dividers are different. And due to the lack of a low
> speed 32.768 kHz oscillator, a few of the clock parents are different.
> 
> The PRCM also has controls for various power domains. These are not
> supported yet, neither in software nor in the device tree binding.
> 
> Signed-off-by: Chen-Yu Tsai 
> ---
>  arch/arm/boot/dts/sun8i-a83t.dtsi | 9 +
>  1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi 
> b/arch/arm/boot/dts/sun8i-a83t.dtsi
> index 1dc4cfe81534..8089f36deeff 100644
> --- a/arch/arm/boot/dts/sun8i-a83t.dtsi
> +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
> @@ -270,5 +270,14 @@
>   #interrupt-cells = <3>;
>   interrupts =  IRQ_TYPE_LEVEL_HIGH)>;
>   };
> +
> + r_ccu: clock@1f01400 {
> + compatible = "allwinner,sun8i-a83t-r-ccu";
> + reg = <0x01f01400 0x400>;
> + clocks = <>, <>, <>;

I applied patches 1 and 2, but that made me realise that the
pll-periph0 is also a parent and should be listed there.

Can you send a patch adjusting the binding documentation so that we
can have it in time for 4.12, and patches for the A64 / H3 DTSs ?

Thanks,
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature


[linux-sunxi] Re: [PATCH 3/3] ARM: sun8i: v3s: add backlight device node for Lichee Pi Zero LCD

2017-05-29 Thread Maxime Ripard
Hi,

On Wed, May 24, 2017 at 08:13:20PM +0800, Icenowy Zheng wrote:
> The 40-pin LCD connector on Lichee Pi Zero has backlight pins, which is
> controlled by the PWM0 controller of the V3s SoC, and the controlling
> part is on the board.
> 
> Add the PWM and backlight device nodes in the device tree file, but
> leave them disabled, as they can only be useful when the LCD is
> attached. The LCD device tree overlay files can enable these controllers
> and make use of them.
> 
> Signed-off-by: Icenowy Zheng 
> ---
>  arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts | 14 ++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts 
> b/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts
> index 387fc2aa546d..e0721bf725d1 100644
> --- a/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts
> +++ b/arch/arm/boot/dts/sun8i-v3s-licheepi-zero.dts
> @@ -52,6 +52,14 @@
>   serial0 = 
>   };
>  
> + backlight: backlight {
> + compatible = "pwm-backlight";
> + pwms = < 0 5 0>;
> + brightness-levels = <0 10 20 30 40 50 60 70 80 90 100>;

This is unlikely, the perceived brightness by the human eye is not
linear, and is actually much closer to a logarithmic function, which
means you should have use an exponential function here in order to
appear that your brightness increase is linear.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature


[linux-sunxi] Re: [PATCH 2/3] ARM: sun8i: v3s: add PWM pinmux nodes

2017-05-29 Thread Maxime Ripard
Hi,

On Wed, May 24, 2017 at 08:13:19PM +0800, Icenowy Zheng wrote:
> Allwinner V3s have two PWM channels, the first channel can be only at
> PB4 pin, and the second channel PB5.
> 
> Add their pinmux configurations.
> 
> Signed-off-by: Icenowy Zheng 
> ---
>  arch/arm/boot/dts/sun8i-v3s.dtsi | 10 ++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun8i-v3s.dtsi 
> b/arch/arm/boot/dts/sun8i-v3s.dtsi
> index bb080c4bd22c..db3328a2c89a 100644
> --- a/arch/arm/boot/dts/sun8i-v3s.dtsi
> +++ b/arch/arm/boot/dts/sun8i-v3s.dtsi
> @@ -239,6 +239,16 @@
>   pins = "PC0", "PC1", "PC2", "PC3";
>   function = "spi0";
>   };
> +
> + pwm0_pins: pwm0 {
> + pins = "PB4";
> + function = "pwm0";
> + };
> +
> + pwm1_pins: pwm1 {
> + pins = "PB5";
> + function = "pwm1";
> + };

Please order the nodes alphabetically.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature


[linux-sunxi] Re: [PATCH 1/3] ARM: sun8i: v3s: add PWM device node

2017-05-29 Thread Maxime Ripard
Hi,

On Wed, May 24, 2017 at 08:13:18PM +0800, Icenowy Zheng wrote:
> As we have already the support for the PWM controller on V3s SoC, add
> its device node in the SoC's DTSI file.
> 
> Signed-off-by: Icenowy Zheng 
> ---
>  arch/arm/boot/dts/sun8i-v3s.dtsi | 8 
>  1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun8i-v3s.dtsi 
> b/arch/arm/boot/dts/sun8i-v3s.dtsi
> index a49ebef53c91..bb080c4bd22c 100644
> --- a/arch/arm/boot/dts/sun8i-v3s.dtsi
> +++ b/arch/arm/boot/dts/sun8i-v3s.dtsi
> @@ -255,6 +255,14 @@
>   interrupts = ;
>   };
>  
> + pwm: pwm@1c21400 {
> + compatible = "allwinner,sun7i-a20-pwm";

You need to have a v3s compatible here.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature


[linux-sunxi] Re: [PATCH 0/6] ARM: sunxi: Enable AXP PMIC power supplies

2017-05-29 Thread Maxime Ripard
On Thu, May 25, 2017 at 03:31:32PM +0800, Chen-Yu Tsai wrote:
> Hi Maxime,
> 
> This series enables the power supplies for the AXP PMICs on boards
> that I currently have and am actively testing.
> 
> The first patch re-orders the node references for the Cubieboard 2
> dts, to maintain alphabetical order.
> 
> Patches 2 through 5 enable the ACIN power supply on various boards.
> 
> Patch 6 ties the USB/VBUS power supply of the PMIC to the USB PHY.
> The PHY should have all available methods of VBUS detection configured.
> The Cubietruck already has the power supplies enabled, but the USB
> power supply was not configured to be used for VBUS detection.

Applied all 6, thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


signature.asc
Description: PGP signature