From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758563AbbIVQHY (ORCPT ); Tue, 22 Sep 2015 12:07:24 -0400 Received: from mail-bn1bon0068.outbound.protection.outlook.com ([157.56.111.68]:52603 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1758461AbbIVQHU (ORCPT ); Tue, 22 Sep 2015 12:07:20 -0400 Authentication-Results: spf=pass (sender IP is 149.199.60.83) smtp.mailfrom=xilinx.com; lists.infradead.org; dkim=none (message not signed) header.d=none;lists.infradead.org; dmarc=bestguesspass action=none header.from=xilinx.com; Date: Tue, 22 Sep 2015 09:03:33 -0700 From: =?utf-8?B?U8O2cmVu?= Brinkmann To: Tobias Klauser CC: Anirudha Sarangi , John Linn , Michal Simek , "David S. Miller" , , Subject: Re: [PATCH] net: axinet: Use of_property_read_u32 instead of open-coding it Message-ID: <20150922160333.GD31436@xsjsorenbubuntu> References: <1442922357-18815-1-git-send-email-tklauser@distanz.ch> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1442922357-18815-1-git-send-email-tklauser@distanz.ch> User-Agent: Mutt/1.5.23 (2014-03-12) X-RCIS-Action: ALLOW X-TM-AS-Product-Ver: IMSS-7.1.0.1224-8.0.0.1202-21830.005 X-TM-AS-User-Approved-Sender: Yes;Yes X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11FD042;1:lzXt/JgGanQpQbZqjhOwyEeLNUQD5mFDoQYFjBr20S0vUySv9Jsyb9Dd6S5bDx/j4VEEU2d1iDtRC8mYwdRmmGLKo/3yWXu9RujdgXwQazWNnlO2FVjrNrfaAKYb4XAh3SOGf/yKn9rislr4iUz/fZ4DdCQaUztKk6hSwBiBh2iPvN/mmhPsvyZj6idmPuJy0LOZtCf5N6KO/uM/0x6FHGIml1wweN+1fmk5d4Bd6j+epPlIhMhhpta1K9FwMt5GXHxkxjR13nZ/NUnypu/dxjzKT+7hLEZs09tyRqDc5zOumk1b0LhXPPV9IZHK5dLQpnA/qUUkHKqhstcdhsfuT7MpSIZj2euZ6YN29ulod7Znh8D6o1sOpdLShB8Sirjy X-Forefront-Antispam-Report: CIP:149.199.60.83;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(2980300002)(438002)(189002)(377424004)(199003)(24454002)(33656002)(23676002)(76506005)(86362001)(6806004)(106466001)(189998001)(85182001)(5001960100002)(19580395003)(46102003)(50466002)(19580405001)(87936001)(83506001)(110136002)(5007970100001)(62966003)(85202003)(76176999)(2950100001)(4001540100001)(54356999)(77156002)(5001920100001)(4001350100001)(33716001)(63266004)(77096005)(64706001)(57986006)(5001830100001)(558084003)(36386004)(92566002)(81156007)(50986999)(5001860100001)(47776003)(11100500001)(107986001);DIR:OUT;SFP:1101;SCL:1;SRVR:BL2FFO11HUB045;H:xsj-pvapsmtpgw01;FPR:;SPF:Pass;PTR:unknown-60-83.xilinx.com;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11HUB045;2:KvW1I1ZlpLXpAGp/yRom2mrDqkxbenwdFCTT3gGUUfOYEikrP86tDQrShmsL8W5NZ8/U6TvKO93OMHGbNWxCybvT1bW3PILye6pMILSEIXJ7rryKi0gwlXZl1Bg657SbETDdgjqRWQ56InQozMJwqgrG1L03PppmvdzgpfhF6IQ=;3:330btOPTpTt19OukNDpzVt9MPYu3xKajCYkSHO2hd6H9Q2WYx3urevALO7V4PL2moyLpNjAi0BLsBJPFdGBtlXAIo1bwISNytJW/73c08m2/k0mVIA+XWWnOZRCPoEBYArZ07SnGhWnIEVN195YIXD32iS5Kgc2ZFolTNNs4bRzJ2KN2nz7uGxrz6qQYvfMkevRQoRfxioRH5pngxePUBTsdbIJhfxvXoS761y7m4JHkbsZZcNZBoAxMyflFHBxANJjDJVc/CVuUuqtkrNY3iA==;25:Xis11VYcaa/VQuBv+rhbcVib9IpKuElgsOswQZQh95+72xqt3F0kT+HcKngsUMPMwezDz/70dZx2CLtwX157xmfIsFXezB9cVJqFHy22eF60mROU1JKk8VrPjRzrUf1nCydrrQkZ5aVbu/2sx6knPcJbnYFmx5PcRYGyqEGzo978RJvtsiBhSC8Paru2qtsPg+MRBxAeSGWi0X5WvrpPlxCukVWyyFa0Y39dMQE+OH0rdoHudk0UjpxUILlAI1uk X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(8251501001);SRVR:BL2FFO11HUB045; X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11HUB045;20:prGAEjb3S0yC2IpG3zBRU7dKfCkM7cPk91VbhtG3n3Wa+svb7MysvRMBY9cpC6sFn/5NLzeJ8MRzxI9DYOTjTgyV1JUNZqr5KsFl2RTMC2nCTsPCWBpLOzbf9Cn8JTfz7J9Se77b86mcwKfcFM1oDeu0v3UWSM0E8dYVM4DC7ELxvYEc3H1FbqqncLck/WbkGltFlf3OmVCYWfb3tdbGyE2PkVp18xjInnanxpC3tFB0PX/qo7peOMUBCVSEpFwFnr08q07pZBJ1kT3cacC4O4IECqLv/yJ8S8xrHQuthRiJvkQ6rJ1Sq9FaoQ8kc9a8QrKAjnynAALKtz+MVSGUznnF/49QKuQjtKpIcz5x3CvU/x2FU7P77AyrZvv9QXMTFc0b7Ti6Inkf+PbHLqYO2VeFRr/siEPINvTt4JGOtG2hXzJC31KoqEEl8XZiJu7xDpfEto4jjRL1nCgq2UFdftSLHndaI5FUw+WoWvF1eVBxfOBBkYfaXgj3Nlz4AkE2;4:mLK3yqo4TvXXnKZj5xUN4UVB0gEkDfeRfqR+m7iTrvFkqw/nAzG8dtNprVQvpTaBgLTxHt68vM4y/cV6yCv5yZ7kUZ85KFRsqiMlrZ/2mwnk/lE8Fsogif+Re2xerSASkkZ4gQPu7OA0u4/CVWTkim8K2nylt30ZZw5pipJQ6C1sGkGnnQmr/eNsJsySewwccwrNnmwZXHokI9Jsi6yzQaIEbBUEvdlDU0ASpSKV4YXJ8EmHvDhR/rHLlyoJqREzQkJ7XzjtW9kAPLkiAzoE4uPu2aIyxxn/LyZjg2ttQIBYNb+SMOit2WrbPC24C07mQ2AtCViuwd1tlnJ71PecCMdk6BW0HLZxPO1Ukhz+ow4= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(520078)(8121501046)(5005006)(3002001);SRVR:BL2FFO11HUB045;BCL:0;PCL:0;RULEID:;SRVR:BL2FFO11HUB045; X-Forefront-PRVS: 0707248B64 X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCTDJGRk8xMUhVQjA0NTsyMzoxRnhidW0rckZHMW5zRFNZcmgyREJQYS85?= =?utf-8?B?QXY3REcwenBlNytSOEQ1ckRrTGJZN0FtQW15NGtBVDlBN0VoZXBZcGliS1lw?= =?utf-8?B?RFp1U3dobzg5Tm9OUHhqcDY2OWtlMlp3eFUxUWVJdWVUTWtubHhBbnZEVlU2?= =?utf-8?B?Z0xvM2o3TTF1WUJIU0hVcnI2c3ExaWh4bUR3a2V1eEpyK3E4R2V0SzNKNUJQ?= =?utf-8?B?andoeG5FcnVGNlorQVpJc0lGdHFBYVUrS0xRNmozNC85enpZTXZRRFViVnVz?= =?utf-8?B?ZDNxYko2ODcxOE8yVVRsdEh5NVFibmZ5VmhiNHBUcUVKRzhoZWRORWR2QStt?= =?utf-8?B?NHlIUHk4V0UyL05pUXRKK3dvRnkyWE5wRUtSbFZQSlBvalQxMUFVMXIrSTNL?= =?utf-8?B?dUkzbzc0NkJBRjArQnJ3cGNCVGUxTm5xekY1ck5lWllQOS9ZK3k2KzNaQWVi?= =?utf-8?B?THJsWlRDRUZhK0tqL1BCSTR1bFVGNGtucEZBQzMzTDg0UFZJTFdISldKRFVR?= =?utf-8?B?NHhDVkw2OUN3TTdlOGRybkhpMXQ4UGpuMnpBOXRVR1RsbjhOQm02YjVPRDBM?= =?utf-8?B?dmhHY3g3MjE0Y1JHKzVjZjZZdDFWSU95c3FtYlY2ZXV0d3ZScXJnQTJzUnV3?= =?utf-8?B?YnovL3phWmkwM1k1cTVPc2s4Q2YzaGVoTmR5UUY1VS83QmdsOXJSdll1c0gy?= =?utf-8?B?Z1RjOVc0dmdTU1kzWmJPbTJLQW1VaThMSERDS2hDdGRRVFhTSEt2VTEyUjNP?= =?utf-8?B?LzE1bXRkT2JLRmlXcmtuc2FzTVM4MTRtSXQ0NENqZ21qR2VrbnlCVVdJT1hG?= =?utf-8?B?ODgxL3lsNnNTL25mS053YjB0TWJHSGRTZEhBRStyc09EQTdyMEJNYXdqc1py?= =?utf-8?B?a09hbUV5TGZ3bWpqNWpRY3N5NVZWT2xoQWkzT0pzOFlqT2I2YmZoVU1hL3Vl?= =?utf-8?B?RHA2WDRTK0NIRGJSZHJCaGkvTUJhM3YxTUdRSFpPQnhNcmIwT3RFa0ZERStm?= =?utf-8?B?WU9iY0ZrRHpVZUVHZlcySWVBTVlYUDZLMzZNS3RiSzAzMzFCR3QwWllCcFNG?= =?utf-8?B?Um9NKzFVVTFWOVRTaFdNRVFhVjBReVF0WVNDbFpmeUdOeGszbEhxdk53SXg2?= =?utf-8?B?UUNjS0lZUDQ2cDFTcWVNSGQ0WHRqNTBLR01jVlh3cTRnTVVTOHE3aVBUQ2p2?= =?utf-8?B?dElrRFlPTGRkUXJBbEhlUllWd3F6WnVCRUtYTHdXclF3T0c5T3dHZmhCYmVy?= =?utf-8?B?YktCMktBUzRISm1QanB1cTZXOGFWUnFJb3ZubStCcmVkeXREMnlDcTQ4bjl5?= =?utf-8?B?elJXeHA1YXhpVERSY0ZjY0tpZUhjZzlBc2tySEJGZHBxQUwzckhvZFJqSUU1?= =?utf-8?B?allySXlMaUowRStFNjRRWkRhVnIzN1NtMTdRV3dBWTE3NWxIRlhQNWRWSS8r?= =?utf-8?B?WlBXZE01bzJmckkvSFpBSkZtTDlDdVd2Wk9wc2xLWlM5V01SMmtFUFc1aDc1?= =?utf-8?B?alp0ZDluRXMvNGYyYnd5TmlqaktXMkt6K1ZkdjRQMGRGQVU0akV6cmtheFZo?= =?utf-8?B?NkRBQkcvVjBmc0twOHZzbFFrNTMwUHZ4eldpQ0JLei90cHdOL3I2REFzU2dj?= =?utf-8?B?blNsSW1hcDhETjNlUXdSY0JxMkNteVhHK1ZpTVk5dUp0MGM4Z1hRSElGR2RP?= =?utf-8?B?NW1Rc0VyNVU3ejN0MGZTaElIOHpYMUxuVlRKWDJzZGxtaHdITkIzZ2gwajFK?= =?utf-8?B?clBtbTE0RXN4bUNreGJqNGc9PQ==?= X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11HUB045;5:StAkZ5HkQTbuyCrK8ZpK0Br+IJVZy2CPEHoqN0UVDXsHtU/S2Z50nj9tJYE6aixO7RIJkseXfgHzn4gN/HwfZYi28Eg3TDpuzvvSweg+MGc9GvScVLmGBogNVFIM2Kz3W2WpQ7vs4n677VcQH0kT4A==;24:guLH+WQljg1EtSMfJOYEgUAUIh1lE+ibGphTwbpbCjtNb0acz/mKZ57UB2j6lLPK2wPLruNiK9I/enQY2LYR++PHUG7LursS1zcwHaPJSA4= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Sep 2015 16:07:18.2733 (UTC) X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.60.83];Helo=[xsj-pvapsmtpgw01] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2FFO11HUB045 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2015-09-22 at 01:45PM +0200, Tobias Klauser wrote: > Use of_property_read_u32 instead of of_get_property with return value > checks and endianness conversion. > > Signed-off-by: Tobias Klauser Reviewed-by: Sören Brinkmann Sören