[gentoo-catalyst][PATCH 1/3] Add a subarch ppc64le,is the little-endian of ppc64.Also add power7,power7le,power8le.The subarch power7 and power8 are inheriting from ppc64.Power7le and Power8le are inheriting from ppc64le.As the little-endian of ppc64, the ppc64le can be a keyword,it will be used in create iso and intergrating bootloader.

[gentoo-catalyst][PATCH 1/3] Add a subarch ppc64le,is the little-endian of ppc64.Also add power7,power7le,power8le.The subarch power7 and power8 are inheriting from ppc64.Power7le and Power8le are inheriting from ppc64le.As the little-endian of ppc64, the ppc64le can be a keyword,it will be used in create iso and intergrating bootloader.

[Kevin Zhao]

Signed-off-by: Kevin Zhao <kevin.zhaoshuai@gmail.com>
---
 catalyst/arch/powerpc.py | 37 ++++++++++++++++++++++++++++++++++---
 1 file changed, 34 insertions(+), 3 deletions(-)

diff --git a/catalyst/arch/powerpc.py b/catalyst/arch/powerpc.py
index 7f4194e..bbc8941 100644
--- a/catalyst/arch/powerpc.py
+++ b/catalyst/arch/powerpc.py
@@ -28,6 +28,13 @@ class arch_ppc64(generic_ppc64):
 		self.settings["CFLAGS"]="-O2 -pipe"
 		self.settings["CHOST"]="powerpc64-unknown-linux-gnu"
 
+class arch_ppc64le(generic_ppc64):
+	"builder class for generic ppc64le"
+	def __init__(self,myspec):
+		generic_ppc64.__init__(self,myspec)
+		self.settings["CFLAGS"]="-O2 -pipe"
+		self.settings["CHOST"]="powerpc64le-unknown-linux-gnu"
+
 class arch_970(arch_ppc64):
 	"builder class for 970 aka G5 under ppc64"
 	def __init__(self,myspec):
@@ -99,12 +106,32 @@ class arch_power6(arch_ppc64):
 		self.settings["CFLAGS"]="-O2 -pipe -mcpu=power6 -mtune=power6"
 		self.settings["HOSTUSE"]=["altivec","ibm"]
 
+class arch_power7(arch_ppc64):
+	"builder class for power7 under ppc64"
+	def __init__(self,myspec):
+		arch_ppc64.__init__(self,myspec)
+		self.settings["CFLAGS"]="-O2 -pipe -mcpu=power7 -mtune=power7 -mabi=elfv2"
+		self.settings["HOSTUSE"]=["altivec","ibm"]
+
+class arch_power7le(arch_ppc64le):
+	"builder class for power7 under ppc64le"
+	def __init__(self,myspec):
+		arch_ppc64le.__init__(self,myspec)
+		self.settings["CFLAGS"]="-O2 -pipe -mcpu=power7 -mtune=power7 -mabi=elfv2"
+		self.settings["HOSTUSE"]=["altivec","ibm"]
+
 class arch_power8(arch_ppc64):
-	"builder class for power8 under ppc64le"
+	"builder class for power8 under ppc64"
 	def __init__(self,myspec):
 		arch_ppc64.__init__(self,myspec)
 		self.settings["CFLAGS"]="-O2 -pipe -mcpu=power8 -mtune=power8 -mabi=elfv2"
-		self.settings["CHOST"]="powerpc64le-unknown-linux-gnu"
+		self.settings["HOSTUSE"]=["altivec","ibm"]
+
+class arch_power8le(arch_ppc64le):
+	"builder class for power8 under ppc64le"
+	def __init__(self,myspec):
+		arch_ppc64le.__init__(self,myspec)
+		self.settings["CFLAGS"]="-O2 -pipe -mcpu=power8 -mtune=power8 -mabi=elfv2"
 		self.settings["HOSTUSE"]=["altivec","ibm"]
 
 def register():
@@ -121,7 +148,11 @@ def register():
 		"power4"	: arch_power4,
 		"power5"	: arch_power5,
 		"power6"	: arch_power6,
+		"power7"	: arch_power7,
+		"power7le"	: arch_power7le,
 		"power8"	: arch_power8,
+		"power8le"	: arch_power8le,
 		"ppc"		: arch_ppc,
-		"ppc64"		: arch_ppc64
+		"ppc64"		: arch_ppc64,
+		"ppc64le"	: arch_ppc64le
 	}, ("ppc","ppc64","ppc64le","powerpc","powerpc64","powerpc64le"))
-- 
2.1.0

[gentoo-catalyst][PATCH 2/3] Modify the parameter of ppc64 little-endian, because the ppc64 little-endian has different archtecture and new boot parameters.Don't need the hfs and mapping files as it in Big-endian.Also tweak the code of Big-endian.

[Kevin Zhao]

Signed-off-by: Kevin Zhao <kevin.zhaoshuai@gmail.com>
---
 targets/support/create-iso.sh | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/targets/support/create-iso.sh b/targets/support/create-iso.sh
index b4077c3..2e604db 100755
--- a/targets/support/create-iso.sh
+++ b/targets/support/create-iso.sh
@@ -211,8 +211,22 @@ case ${clst_hostarch} in
 			"${clst_target_path}/ppc/bootinfo.txt"
 		fi
 
+		flags=( -r -U )
 		echo ">> Running mkisofs to create iso image...."
-		run_mkisofs -r -U -chrp-boot -netatalk -hfs -probe -map "${clst_target_path}"/boot/map.hfs -part -no-desktop -hfs-volid "${clst_iso_volume_id}" -hfs-bless "${clst_target_path}"/boot -hide-hfs "zisofs" -hide-hfs "stages" -hide-hfs "distfiles" -hide-hfs "snapshots" -J ${mkisofs_zisofs_opts} -V "${clst_iso_volume_id}" -o "${1}" "${clst_target_path}"/
+		if echo ${clst_subarch} | grep -qE ".+le$"
+		then
+			flags+=(
+				-v -T -l -cache-inodes -chrp-boot
+			)
+		else
+			flags+=(
+				-chrp-boot -netatalk -hfs -probe -map "${clst_target_path}"/boot/map.hfs \ 
+				-part -no-desktop -hfs-volid "${clst_iso_volume_id}" -hfs-bless "${clst_target_path}"/boot -hide-hfs \
+				"zisofs" -hide-hfs "stages" -hide-hfs "distfiles" -hide-hfs "snapshots" -J ${mkisofs_zisofs_opts}
+			)
+		fi
+		run_mkisofs "${flags[@]}" \
+			-V "${clst_iso_volume_id}" -o "${1}" "${clst_target_path}"
 	;;
 	sparc*)
 		# Old silo (<=1.2.6) requires a specially built mkisofs
-- 
2.1.0

Re: [gentoo-catalyst][PATCH 2/3] Modify the parameter of ppc64 little-endian, because the ppc64 little-endian has different archtecture and new boot parameters.Don't need the hfs and mapping files as it in Big-endian.Also tweak the code of Big-endian.

[Mike Frysinger]

[-- Attachment #1: Type: text/plain, Size: 458 bytes --]

On 08 Jan 2016 05:04, Kevin Zhao wrote:
> +		if echo ${clst_subarch} | grep -qE ".+le$"

just use a pattern match:
	if [[ ${clst_subarch} == *le ]]

> +		then
> +			flags+=(
> +				-v -T -l -cache-inodes -chrp-boot
> +			)

-chrp-boot shows up in both, so factor that out into the common setting ?

> +			flags+=(
> +				-chrp-boot -netatalk -hfs -probe -map "${clst_target_path}"/boot/map.hfs \ 

you don't need to use \ with arrays
-mike

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [gentoo-catalyst][PATCH 2/3] Modify the parameter of ppc64 little-endian, because the ppc64 little-endian has different archtecture and new boot parameters.Don't need the hfs and mapping files as it in Big-endian.Also tweak the code of Big-endian.

[Kevin Zhao]

Hi Mike,
    Thanks for your comment.I have send a new patch cover this :-)

2016-01-09 3:50 GMT+08:00 Mike Frysinger <vapier@gentoo.org>:
> On 08 Jan 2016 05:04, Kevin Zhao wrote:
>> +             if echo ${clst_subarch} | grep -qE ".+le$"
>
> just use a pattern match:
>         if [[ ${clst_subarch} == *le ]]
>
>> +             then
>> +                     flags+=(
>> +                             -v -T -l -cache-inodes -chrp-boot
>> +                     )
>
> -chrp-boot shows up in both, so factor that out into the common setting ?
>
>> +                     flags+=(
>> +                             -chrp-boot -netatalk -hfs -probe -map "${clst_target_path}"/boot/map.hfs \
>
> you don't need to use \ with arrays
> -mike

Re: [gentoo-catalyst][PATCH 2/3] Modify the parameter of ppc64 little-endian, because the ppc64 little-endian has different archtecture and new boot parameters.Don't need the hfs and mapping files as it in Big-endian.Also tweak the code of Big-endian.

[Leno Hou]

[-- Attachment #1: Type: text/plain, Size: 1086 bytes --]

Hi Mike，

    I've tesed Kevin Zhao's patches Could you help me to review v3 patches
and merge to upstream ?
    These patches will be sent in mail list later.... Thanks

*Leno Hou*

On Tue, Jan 12, 2016 at 10:31 PM, Kevin Zhao <kevin.zhaoshuai@gmail.com>
wrote:

> Hi Mike,
>     Thanks for your comment.I have send a new patch cover this :-)
>
> 2016-01-09 3:50 GMT+08:00 Mike Frysinger <vapier@gentoo.org>:
> > On 08 Jan 2016 05:04, Kevin Zhao wrote:
> >> +             if echo ${clst_subarch} | grep -qE ".+le$"
> >
> > just use a pattern match:
> >         if [[ ${clst_subarch} == *le ]]
> >
> >> +             then
> >> +                     flags+=(
> >> +                             -v -T -l -cache-inodes -chrp-boot
> >> +                     )
> >
> > -chrp-boot shows up in both, so factor that out into the common setting ?
> >
> >> +                     flags+=(
> >> +                             -chrp-boot -netatalk -hfs -probe -map
> "${clst_target_path}"/boot/map.hfs \
> >
> > you don't need to use \ with arrays
> > -mike
>
>

[-- Attachment #2: Type: text/html, Size: 1881 bytes --]