From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932657AbcFJNE6 (ORCPT ); Fri, 10 Jun 2016 09:04:58 -0400 Received: from mailout3.w1.samsung.com ([210.118.77.13]:25672 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932106AbcFJNE5 (ORCPT ); Fri, 10 Jun 2016 09:04:57 -0400 X-AuditID: cbfec7f4-f796c6d000001486-cc-575abaf56c78 Subject: Re: [RFC PATCH 07/15] ARM: exynos: fixup debug macros for big-endian To: Ben Dooks , Matthew Leach References: <20160608183110.13851-1-matthew@mattleach.net> <20160608183110.13851-8-matthew@mattleach.net> <575AA193.7040907@samsung.com> <17ac39d6-d451-ed43-1aa6-f2af462a61e0@codethink.co.uk> Cc: linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org From: Krzysztof Kozlowski Message-id: <575ABAF4.3030809@samsung.com> Date: Fri, 10 Jun 2016 15:04:52 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-version: 1.0 In-reply-to: <17ac39d6-d451-ed43-1aa6-f2af462a61e0@codethink.co.uk> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrELMWRmVeSWpSXmKPExsVy+t/xy7pfd0WFGzT2iFg8uNXKZPH6haHF psfXWC0u75rDZjHj/D4mi8aly1gc2DyevZzM5LF5Sb3H1A/rWDz6tqxi9Pi8SS6ANYrLJiU1 J7MstUjfLoEro//mc8aC3dwVGz4uZW9gnMHZxcjJISFgIvHqehMbhC0mceHeeiCbi0NIYCmj xIvpr1lAEkICzxglJsyyB7GFBXwlPs5ZwQRiiwgESGxsP8cM0fCPUWLbw5uMIAlmgWyJtR9P MIPYbALGEpuXLwHbwCugJXHj7CuwOIuAqsScztlgcVGBCIlZ238wQdQISvyYfA9oMQcHp4Cz xPFrdiAms4CexP2LWhDT5SU2r3nLPIFRYBaShlkIVbOQVC1gZF7FKJpamlxQnJSea6hXnJhb XJqXrpecn7uJERLSX3YwLj5mdYhRgINRiYc3YldkuBBrYllxZe4hRgkOZiUR3o4dUeFCvCmJ lVWpRfnxRaU5qcWHGKU5WJTEeefueh8iJJCeWJKanZpakFoEk2Xi4JRqYIzadk52i3dwbNir 346+gS9Oqm2tqzaR89K892J5QoXvUeNLD3IDrL2eb/m//e/8/b3v9zx27Fte7Lys8hDTp65z GZvzAqec/7c/IsW3d2fr5I/5m44K6Ns7z3utcsk47r4d039978ZDFbGVlS+sv087MImFla/k dIRdlM/PjncKJVPv1bReO6fEUpyRaKjFXFScCAC/fY/GZQIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/10/2016 03:02 PM, Ben Dooks wrote: > On 10/06/16 12:16, Krzysztof Kozlowski wrote: >> On 06/10/2016 01:12 PM, Ben Dooks wrote: >>> On 08/06/16 19:31, Matthew Leach wrote: >>>> From: Ben Dooks >>>> >>>> The exynos low-level debug macros need to be fixed if the system is being >>>> built big endian. Add the necessary endian swaps for accessing the registers >>>> to get output working again >>>> >>>> Signed-off-by: Ben Dooks >>>> --- >>>> arch/arm/include/debug/samsung.S | 8 ++++++++ >>>> 1 file changed, 8 insertions(+) >>> >>> Should this go via the maintainer or to RMK? >>> >>> If to RMK, then shall I get it submitted (and does anyone else >>> want to ack?) >> >> If there are no objections, I can take it through samsung-soc with other >> arch/arm/mach-exynos* patches. >> >> What I am missing here, is Matthew's Signed-off-by. >> >> Matthew, you touched and sent the patch so could you add the Sob? > > I've asked Matthew to re-send the core patches with the signed-off > sorted out. Thanks. > He can also put them on a public facing git server if > that would be helpful. No need. I think it will go through separate trees anyway. Except the change adding ARCH_SUPPORTS_BIG_ENDIAN, I don't see any dependencies. As for the ARCH_SUPPORTS_BIG_ENDIAN, I think it should be applied when all necessary drivers are fixed. Best regards, Krzysztof