From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?utf-8?B?5pWs6ZSQ?= Subject: Re: [PATCH v4 1/6] mfd: rtsx: add func to split u32 into register Date: Mon, 8 Dec 2014 09:59:04 +0000 Message-ID: <54857668.4040902@realsil.com.cn> References: <17ba65f4bdf4707bdf038e407ecf5dc012bca281.1417758564.git.micky_ching@realsil.com.cn> <20141208084941.GC3951@x1> <54857429.5060108@realsil.com.cn> <20141208095719.GF3951@x1> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20141208095719.GF3951@x1> Content-Language: zh-CN Content-ID: <42C28E244E978A46B8E5E4D4AF9914B0@realsil.com.cn> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: Lee Jones Cc: "ulf.hansson@linaro.org" , "sameo@linux.intel.com" , "gregkh@linuxfoundation.org" , "linux-mmc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "chris@printf.net" , =?utf-8?B?546L54Kc?= , "rogerable@realtek.com" , "devel@linuxdriverproject.org" , "dan.carpenter@oracle.com" List-Id: linux-mmc@vger.kernel.org DQpPbiAxMi8wOC8yMDE0IDA1OjU3IFBNLCBMZWUgSm9uZXMgd3JvdGU6DQo+IE9uIE1vbiwgMDgg RGVjIDIwMTQsIOaVrOmUkCB3cm90ZToNCj4NCj4+IE9uIDEyLzA4LzIwMTQgMDQ6NDkgUE0sIExl ZSBKb25lcyB3cm90ZToNCj4+Pj4gZGlmZiAtLWdpdCBhL2luY2x1ZGUvbGludXgvbWZkL3J0c3hf cGNpLmggYi9pbmNsdWRlL2xpbnV4L21mZC9ydHN4X3BjaS5oDQo+Pj4+PiBpbmRleCA3NDM0NmQ1 Li45MjM0NDQ5IDEwMDY0NA0KPj4+Pj4gLS0tIGEvaW5jbHVkZS9saW51eC9tZmQvcnRzeF9wY2ku aA0KPj4+Pj4gKysrIGIvaW5jbHVkZS9saW51eC9tZmQvcnRzeF9wY2kuaA0KPj4+Pj4gQEAgLTU1 OCw2ICs1NTgsNyBAQA0KPj4+Pj4gICAjZGVmaW5lIFNEX1NBTVBMRV9QT0lOVF9DVEwJCTB4RkRB Nw0KPj4+Pj4gICAjZGVmaW5lIFNEX1BVU0hfUE9JTlRfQ1RMCQkweEZEQTgNCj4+Pj4+ICAgI2Rl ZmluZSBTRF9DTUQwCQkJCTB4RkRBOQ0KPj4+Pj4gKyNkZWZpbmUgICBTRF9DTURfU1RBUlQJCQkw eDQwDQo+Pj4gVGhpcyBpcyBhIGRpZmZlcmVudCBmb3JtYXQgYXQgdGhlIG90aGVycyBpbiB0aGUg Z3JvdXAgb24gdHdvIGNvdW50czsNCj4+PiBmaXJzdGx5IHlvdSBoYXZlIDMgd2hpdGVzcGFjZSBj aGFyYWN0ZXJzIGFmdGVyIHRoZSAjZGVmaW5lIGFuZA0KPj4+IHNlY29uZGx5IGFsbCBvZiB0aGUg b3RoZXIgaGV4IGRpZ2l0cyBpbiB0aGUgc2V0IGFyZSA0IGEgYnJlYXN0Lg0KPj4+IFBsZWFzZSBh ZGQgdGhlIGxlYWRpbmcgemVyb3MgdG8gY29uZm9ybSB0byB0aGlzIGZvcm1hdC4NCj4+Pg0KPj4g SGkgbGVlLA0KPj4NCj4+IFRoaXMgZm9ybWF0IGlzIG1vcmUgcmVhZGFibGUsIGFkZCAyIHNwYWNl IG1lYW5zIHRoaXMgdmFsdWUgaXMNCj4+IGluIGdyb3VwcyB1bmRlciByZWdpc3RlciBTRF9DTUQw LCBhbmQgZWFjaCByZWdpc3RlciB2YWx1ZSBpcyB1OCB0eXBlLA0KPj4gc28gdXNlIDIgaGV4IGRp Z2l0cyBpcyByaWdodC4NCj4+DQo+PiBUaGUgb3JpZ2luYWwgZmlsZSBtYWtlIHJlZ2lzdGVyIGFk ZHJlc3MgYW5kIGl0cyByZWxhdGVkIHZhbHVlIHNlcGFyYXRlZCwNCj4+IGFuZCBncm91cCByZWdp c3RlciB0b2dldGhlci4NCj4+DQo+PiBCdXQgSSBsaWtlIHRvIHdyaXRlIHJlZ2lzdGVyIGFkZHJl c3MgYW5kIHZhbHVlIHRvZ2V0aGVyLA0KPj4gYWRkIDIgc3BhY2UgZm9yIHZhbHVlIHRvIGluZGlj YXRlIHRoaXMgdmFsdWUgaXMgcmVsYXRlZCB0byBhYm92ZSByZWdpc3Rlci4NCj4+DQo+PiBJZiB3 ZSBhZGQgbmV3IGFkZHJlc3MvdmFsdWUsIEkgcmVjb21tZW5kIHdyaXRlIHJlZ2lzdGVyIGRlZmlu ZSBmb3JtYXQgYXMNCj4+IGJlbG93Og0KPj4NCj4+ICNkZWZpbmUgUkVHSVNURVIgICAgICAgIGFk ZHINCj4+ICNkZWZpbmUgICBSRUdfVkFMMSAgICAgIHZhbDENCj4+ICNkZWZpbmUgICBSRUdfVkFM MiAgICAgIHZhbDINCj4gT2theSwgSSBzZWUgdGhhdCB0aGVyZSBhcmUgb3RoZXIgb2NjdXJyZW5j ZXMgdXNpbmcgdGhpcyBmb3JtYXQgaW4gdGhhdA0KPiBmaWxlLiAgUGxlYXNlIGNvbnNpZGVyIGNv bnZlcnRpbmcgdGhlIGVudGlyZSBmaWxlIGludG8gdGhpcyBmb3JtYXQuICBJDQo+IGRvbid0IG1p bmQgaXQgZWl0aGVyIHdheSwgYnV0IGl0IGxvb2tzIG9kZCBpZiB0aGV5IGFyZSBtaXhlZC4NCj4N CndlIHdpbGwgcmVmb3JtYXQgaXQgaW4gdGhlIG5lYXIgZnV0dXJlLg0KDQpyZWdhcmRzLg0KX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZGV2ZWwgbWFpbGlu ZyBsaXN0CmRldmVsQGxpbnV4ZHJpdmVycHJvamVjdC5vcmcKaHR0cDovL2RyaXZlcmRldi5saW51 eGRyaXZlcnByb2plY3Qub3JnL21haWxtYW4vbGlzdGluZm8vZHJpdmVyZGV2LWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755017AbaLHJ7f (ORCPT ); Mon, 8 Dec 2014 04:59:35 -0500 Received: from rtits2.realtek.com ([60.250.210.242]:45634 "EHLO rtits2.realtek.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754997AbaLHJ7c (ORCPT ); Mon, 8 Dec 2014 04:59:32 -0500 Authenticated-By: X-SpamFilter-By: BOX Solutions SpamTrap 5.49 with qID sB89x4I6029798, This message is accepted by code: ctloc85258 From: =?utf-8?B?5pWs6ZSQ?= To: Lee Jones CC: "sameo@linux.intel.com" , "chris@printf.net" , "ulf.hansson@linaro.org" , "devel@linuxdriverproject.org" , "linux-kernel@vger.kernel.org" , "linux-mmc@vger.kernel.org" , "gregkh@linuxfoundation.org" , "dan.carpenter@oracle.com" , "rogerable@realtek.com" , =?utf-8?B?546L54Kc?= Subject: Re: [PATCH v4 1/6] mfd: rtsx: add func to split u32 into register Thread-Topic: [PATCH v4 1/6] mfd: rtsx: add func to split u32 into register Thread-Index: AQHQEE+fRoR/oTBMgUiwD+7H/IHqKZyE4RSAgAAQtYCAAAIwgIAAAH0A Date: Mon, 8 Dec 2014 09:59:04 +0000 Message-ID: <54857668.4040902@realsil.com.cn> References: <17ba65f4bdf4707bdf038e407ecf5dc012bca281.1417758564.git.micky_ching@realsil.com.cn> <20141208084941.GC3951@x1> <54857429.5060108@realsil.com.cn> <20141208095719.GF3951@x1> In-Reply-To: <20141208095719.GF3951@x1> Accept-Language: zh-CN, en-US Content-Language: zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.29.41.103] Content-Type: text/plain; charset="utf-8" Content-ID: <42C28E244E978A46B8E5E4D4AF9914B0@realsil.com.cn> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id sB89xjYw024258 On 12/08/2014 05:57 PM, Lee Jones wrote: > On Mon, 08 Dec 2014, 敬锐 wrote: > >> On 12/08/2014 04:49 PM, Lee Jones wrote: >>>> diff --git a/include/linux/mfd/rtsx_pci.h b/include/linux/mfd/rtsx_pci.h >>>>> index 74346d5..9234449 100644 >>>>> --- a/include/linux/mfd/rtsx_pci.h >>>>> +++ b/include/linux/mfd/rtsx_pci.h >>>>> @@ -558,6 +558,7 @@ >>>>> #define SD_SAMPLE_POINT_CTL 0xFDA7 >>>>> #define SD_PUSH_POINT_CTL 0xFDA8 >>>>> #define SD_CMD0 0xFDA9 >>>>> +#define SD_CMD_START 0x40 >>> This is a different format at the others in the group on two counts; >>> firstly you have 3 whitespace characters after the #define and >>> secondly all of the other hex digits in the set are 4 a breast. >>> Please add the leading zeros to conform to this format. >>> >> Hi lee, >> >> This format is more readable, add 2 space means this value is >> in groups under register SD_CMD0, and each register value is u8 type, >> so use 2 hex digits is right. >> >> The original file make register address and its related value separated, >> and group register together. >> >> But I like to write register address and value together, >> add 2 space for value to indicate this value is related to above register. >> >> If we add new address/value, I recommend write register define format as >> below: >> >> #define REGISTER addr >> #define REG_VAL1 val1 >> #define REG_VAL2 val2 > Okay, I see that there are other occurrences using this format in that > file. Please consider converting the entire file into this format. I > don't mind it either way, but it looks odd if they are mixed. > we will reformat it in the near future. regards. {.n++%ݶw{.n+{G{ayʇڙ,jfhz_(階ݢj"mG?&~iOzv^m ?I