samsung-jflte: Use armv7

I have tested this on xfce4 and weston. I would like to test is on plasma-mobile, but #167 (closed) is getting in the way.

I ran into the following error when compiling the kernel for armv7:

arch/arm/crypto/aesbs-core.S:449: Error: thumb conditional instruction should be in IT block -- `addeq r6,r6,#0x10'
arch/arm/crypto/aesbs-core.S:904: Error: thumb conditional instruction should be in IT block -- `addeq r6,r6,#0x10'
arch/arm/crypto/aesbs-core.S:2100: Error: thumb conditional instruction should be in IT block -- `subne r9,#0x10'

I tried to find a patch, but in the end it was easier to just disable CONFIG_CRYPTO_AES_ARM_BS. Does anyone know if this will lead to any compatibility or performance problems?

This is a WIP until #160 (closed) is complete.

Had same problems. Fixed it by disabling NEON instructions in 'Cryptographic API' section of menuconfig (just uncheck anything that says neon in its name), I'm not able to find the exact CONFIG_ options because of my slap-everything-into-one-commit bad habit :/ yet I recall there were two or three like this, under AES and SHA (maybe you need to deal with more)

edit: just read your MR text once again and you actually found it, sorry

By Konrad Dybcio on 2019-01-12T17:14:58

added 5 commits

• f658d0d5 - 1 commit from branch postmarketOS:master
• 319278ab - device-samsung-jflte: Use armv7
• 6fbb7c9c - linux-samsung-jflte: Use armv7
• dc21af62 - linux-samsung-jflte: Disable CONFIG_CRYPTO_AES_ARM_BS
• d4684f2c - firmware-samsung-jflte: Use armv7

Compare with previous version

By Grant Miller on 2019-01-14T15:07:16

Plasma-mobile "works"! The performance isn't any better than on armhf, it's still less than one frame per second and llvmpipe still doesn't work.

By Grant Miller on 2019-01-14T17:38:55

added 27 commits

• d4684f2c...879213f0 - 22 commits from branch postmarketOS:master
• 78b4299f - device-samsung-jflte: Use armv7
• 47b6fc16 - linux-samsung-jflte: Use armv7
• 3f8b0047 - linux-samsung-jflte: Disable CONFIG_CRYPTO_AES_ARM_BS
• e9cb2d13 - firmware-samsung-jflte: Use armv7
• ee4d9ceb - linux-samsung-jflte: Use gnu89 patch

Compare with previous version

By Grant Miller on 2019-01-23T04:14:37

added 3 commits

• 04e694c2 - linux-samsung-jflte: Disable CONFIG_CRYPTO_AES_ARM_BS
• fad69008 - firmware-samsung-jflte: Use armv7
• f927fa53 - linux-samsung-jflte: Use gnu89 patch

Compare with previous version

By Grant Miller on 2019-01-23T04:25:34

unmarked as a Work In Progress

By Grant Miller on 2019-01-25T05:43:45

added 8 commits

• f927fa53...9c51b261 - 6 commits from branch postmarketOS:master
• b226f616 - samsung-jflte: Use armv7 (!144 (merged))
• 6e0088fb - linux-samsung-jflte: Use gnu89 patch (!144 (merged))

Compare with previous version

By Oliver Smith on 2019-01-25T07:33:12

approved this merge request

By Oliver Smith on 2019-01-25T07:33:42

changed the description

By Oliver Smith on 2019-01-25T07:33:53

added 1 commit

• 093b6869 - linux-samsung-jflte: Use gnu89 patch (!144 (merged))

Compare with previous version

By Oliver Smith on 2019-01-25T07:34:59

approved this merge request

By Oliver Smith on 2019-01-25T07:35:26

Changes look good, thanks a lot @GrantM11235!

(also the gnu98 patch really pays off, nice to see how we can throw out a lot of patches here!)

By Oliver Smith on 2019-01-25T07:36:16

merged

By Oliver Smith on 2019-01-25T07:36:22

mentioned in commit b226f616

By Grant Miller on 2019-02-11T21:36:40

mentioned in commit 093b6869

By Grant Miller on 2019-02-11T21:36:41

mentioned in commit fc436e7e

By Grant Miller on 2019-02-14T19:42:53

mentioned in commit 7caca260

By Grant Miller on 2019-02-14T19:42:54

@GrantM11235 try -Wa,-implicit-it=thumb to CFLAGS in Makefile

By JuniorJPDJ on 2019-03-01T15:05:48

https://gitlab.com/postmarketOS/pmaports/blob/master/.shared-patches/linux/kernel-use-the-gnu89-standard-explicitly.patch This patch shows how to do this

By JuniorJPDJ on 2019-03-02T05:25:25