Date: Fri, 03 Jul 2015 14:35:19 +0000 (GMT)
From: Sarbojit Ganguly <ganguly.s@samsung.com>
Subject: Re: Re: [RFC] arm: Add for atomic half word exchange
To: Sarbojit Ganguly <ganguly.s@samsung.com>, Arnd Bergmann <arnd@arndb.de>
Cc: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>,
        "linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
        SUNEEL KUMAR SURIMANI <suneel@samsung.com>,
        VIKRAM MUPPARTHI <vikram.m@samsung.com>,
        "tglx@linutronix.de" <tglx@linutronix.de>,
        "mingo@redhat.com" <mingo@redhat.com>, "hpa@zytor.com" <hpa@zytor.com>,
        "peterz@infradead.org" <peterz@infradead.org>,
        "Waiman.Long@hp.com" <Waiman.Long@hp.com>,
        "oleg@redhat.com" <oleg@redhat.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        SHARAN ALLUR <sharan.allur@samsung.com>,
        "torvalds@linux-foundation.org" <torvalds@linux-foundation.org>
Reply-to: ganguly.s@samsung.com
MIME-version: 1.0
Msgkey: 20150703143444385@ganguly.s
Content-type: text/plain; charset=utf-8
MIME-version: 1.0
Message-id: <1443721553.255761435934114382.JavaMail.weblogic@ep2mlwas05b>
DLP-Filter: Pass
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 2241
Lines: 44

Sorry about that. I dug a bit deeper and found that the code was not getting executed and rectified that. The compilation indeed fails for the lack of ldrexh instruction support on ARMv6. 

Hence, my patch needs to be guarded with !CONFIG_CPU_V6 so as to allow the code run on  >=6k and above.

I will post the v2 soon.

------- Original Message -------
Sender : Arnd Bergmann<arnd@arndb.de>
Date : Jun 05, 2015 18:03 (GMT+05:30)
Title : Re: [RFC] arm: Add for atomic half word exchange

On Friday 05 June 2015 01:17:13 Sarbojit Ganguly wrote:
> Since the compilation is also a success for CONFIG_CPU_V6 with the patch,
> I think we're good to go.

I'm not following your logic. Did you create a new patch that addresses
compilation with CONFIG_CPU_V6?

The original patch should cause a compile error on v6, if it does not,
you either have a broken assembler or you forgot to enable the code
that uses these instructions. Neither of them would be a reason to
merge the patch. Please explain what is going on.

Arnd


감사합니다
사보짓   선임   삼성 전자
----------------------------------------------------------------------+
The Tao lies beyond Yin and Yang. It is silent and still as a pool of water.      |
It does not seek fame, therefore nobody knows its presence.                       |
It does not seek fortune, for it is complete within itself.                       |
It exists beyond space and time.                                                  |
----------------------------------------------------------------------+


감사합니다
사보짓   선임   삼성 전자
----------------------------------------------------------------------+
The Tao lies beyond Yin and Yang. It is silent and still as a pool of water.      |
It does not seek fame, therefore nobody knows its presence.                       |
It does not seek fortune, for it is complete within itself.                       |
It exists beyond space and time.                                                  |
----------------------------------------------------------------------+????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?