From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932736AbcHWWdv (ORCPT ); Tue, 23 Aug 2016 18:33:51 -0400 Received: from mail-by2nam01on0089.outbound.protection.outlook.com ([104.47.34.89]:47968 "EHLO NAM01-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753368AbcHWWda (ORCPT ); Tue, 23 Aug 2016 18:33:30 -0400 X-Greylist: delayed 939 seconds by postgrey-1.27 at vger.kernel.org; Tue, 23 Aug 2016 18:32:59 EDT Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=nxp.com; nxp.com; dkim=none (message not signed) header.d=none;nxp.com; dmarc=fail action=none header.from=nxp.com;nxp.com; dkim=none (message not signed) header.d=none; From: Stuart Yoder To: CC: , , , , , , Stuart Yoder Subject: [PATCH 00/14] staging: fsl-mc: restructuring and cleanup Date: Tue, 23 Aug 2016 17:12:55 -0500 Message-ID: <1471990375-28329-1-git-send-email-stuart.yoder@nxp.com> X-Mailer: git-send-email 1.9.0 X-EOPAttributedMessage: 0 X-Matching-Connectors: 131164641997589821;(91ab9b29-cfa4-454e-5278-08d120cd25b8);() X-Forefront-Antispam-Report: CIP:192.88.168.50;IPV:NLI;CTRY:US;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(7916002)(2980300002)(1109001)(1110001)(339900001)(199003)(189002)(36756003)(305945005)(77096005)(50466002)(7846002)(104016004)(85426001)(68736007)(11100500001)(48376002)(106466001)(31430400001)(626004)(2351001)(92566002)(356003)(229853001)(5003940100001)(2906002)(50226002)(8676002)(8666005)(110136002)(105606002)(189998001)(50986999)(8936002)(87936001)(81166006)(86362001)(5660300001)(586003)(81156014)(97736004)(4326007)(47776003)(33646002)(7059030);DIR:OUT;SFP:1101;SCL:1;SRVR:BN3PR03MB2228;H:tx30smr01.am.freescale.net;FPR:;SPF:Fail;PTR:InfoDomainNonexistent;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11FD005;1:MhyFFwLOSw3iUSPLBeo/sxr3/WkkFzqJgWG6pD2Hz6nVI5bsBnchVlmaPGyiILmhmtNijqovopu9pCMvDEvagMXEtmJjTrnlu8glEc7P8fxUBaC2wqMEy07Kulg7xgRULY3reo4jAvUn8gT6diJEE/xofJ0Ooe6swoaE48rpbdTHiAj2/hW39aypiojx2nsBre074TMtPdWPD/1W8pi8puPd5hw6A/ta6A0gpHrmfnaNXbAb7iJAE17nX5PsvuduxrHJUMxyVzfY8sDJq/KTYFDowroQ7WRUhhUpYetSqsPEhQH2EhD6TXyoJZlIwymXk3hXrTwufjw/20c+9kw3bfkFrn8k3JXQ3xC3/qCJKCGUXQwZe46zkOA3U8vhbsowYVTbdhN1TFrouMVw9ENldIy62Ut2BRHDdYtnkoCpawPfjkC6aOKszeQsnot+BkpanWfrM8zU7iwR6lQPa+ACEsgYXOUsWpWUk0snf8pd8sEhVbJt734Lsj5jzQTrCdeEiIC4gDWK23DdjTgN0diD1z4cgfnecNKPzJQU+yhD2WaWnP+f3VvsB3ErARhP8SaqiBpW52CDPJ9iD6Aqlv54nnfVJeuVrw7lOvr2zYoClRltwn9W+/Wy8PBuMa9bv7MCyOj1QpANb2TR40LDaFZChWj3aaFHw8Rdhanxr22whDqcEkCSO/ZwreAXNyf/LshPCdryxHCTXYqC0Iqy7DPNPhY6hyIgXWpP52vJxSvLVqA= MIME-Version: 1.0 Content-Type: text/plain X-MS-Office365-Filtering-Correlation-Id: 1b2e900d-04fe-487a-20d4-08d3cba327bd X-Microsoft-Exchange-Diagnostics: 1;BN3PR03MB2228;2:3xxJF2WRy8Apyj/LvCRypHMFcNCDXvG+dmAJosp0qH27LGkYnggk/8EIA8ibeRu2T8mYzu1mpp7QjLwLGCeACHHL03+BfFdgY3/OK32bhT7u25ch9P0k4VFQeN/c5OL7C3rtAlRyoFGLbIssNJ0JsKBC57E+X6TSELdPYx/IRISbEHsDoFsPRmXjlSFHEaqY;3:Y27gRPNV47XRGtV7qteJNG22HAiE9142d/OYD8mcntnfOdUYVBKkKkgRn8ajWI8pQUp31dnvJjBCPL1O0CxjIoRNkIV53HIodIQbIlXjRy/8KKKcXLKraYlBOG235VmMEohukfnMawUveBhpS7XvDgSwojTR0SIRnGC12Cre1UmnOEzCgsflGV0aphR3aSv0qdu7QtyVE6TIO9ON77HeHWEwx2IW38RJTO5uSC9UqTk= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BN3PR03MB2228; X-Microsoft-Exchange-Diagnostics: 1;BN3PR03MB2228;25:X5SCPyY+lEaDp5vTwoKCI7HlLM+ajqzDrjtKHg/S0gJrV37lmXXJX0qlIjW7+viXvLvH5jxL+sE5C7R+cA/lAbDPAwIEiMNO/b/47bJOKAaG9bBcr1SWSXG9iAK7a2v4UuYSCQXKd01s9O4iH1pP1YC2gujIvsrLwH/qrxe3g0wr/6SKylUK4I8lkvDoVSFdr/eF7RcL5eJHrR0X1Z1ePTRkTmdSMomPCLC/OObnom1069HFjc1Y1ovhk/pkgHQ3C4x/S3Hc+8mzRzVKmawLpSUf+rhJtK2awj59M3H+d7MIJq17JHf50s1kr2z2EthLr6oUdOEpcQnoex2td1HH2CvL2o8UNMXfYhioDovrkp4yJZjoqHGrjwUGw+GkGZHGGMUd9Vn3KOEo+xsu3zUOVPO6ytrGv1lIYopUHK9qAF55Ywk6Htu3cE6toxAjYAk092yFtpc74U8H0mLu4JNEuou8CjYzFMIZhSZ9adCejr+voxWJQenNGxSFWyARoY2MjVJAzSiYGzgFquIhKmLemVIAmlA9xzbrq9ZF2gwYuwYMAc/Zjr+RSlsAxy6iUkQlB8b5pBt3VXjgIN+3cqPsnk4hQDrIFtLUFuuwx7kPf/6VS96/DSVCkblNTi0dCLC9dq74zrupPwcssnHvq47DAaMNRxtZ+07O11M++fC4TjuNvrhVDC2KICTTWD8paC2ClSmAhcB4+YJAluftNGgP8K5ysLhvBI6xhkTwmW97hf4= X-Microsoft-Exchange-Diagnostics: 1;BN3PR03MB2228;31:ufb2lqkB776OgfLMFI1nL6/NYP0V/b32t2vjYv2LFbJ84K6UCSNpYlAb5Y+QYVb/bT6TytgpWom+0F+u6SgJfnih09k6QPsDC6iIGIbD7VCUSrUzAciTqFLLzY9n99kx396wJSghLKBk7QZpKBSzvJPS4+p7gCcaXxsIzNQD/2fAaHhESXhBK6RFW2aMBQQj3UfkQyWQ+jQ9ZGCeP1k3kkdH5EgoXpRbPFFO+7w3F08=;4:r79hymSO4ZUOFiEnSxurErZD77QekVCO4J8qS2NCAenBrdXt08ELJkfo5JpLgXFPrNWAVby7Wc3aHo08ooOF4l7gJbq8ZV0j9rxF3KR9ftd3LPOjXksjocmlb1NznkwZxwSIHvSMOgUIrKBigFfmLWmT2rS0gqjigl9KTA4BVftmc+aqGmsPKdx7MKzMENuu4SuYY7R1txH8xJ47Id+ah1xhU4436Rh08VbizBRjb3HvC2aXTGDRqVp/w0M64VRcXWufp/q/O4q2ILmNggOWspm226ZI3MRdIJM86DMQlq+slbavnj8cyVOt9Kt6oZsH3RgrRm4XV5XmsTz7GajhwOTOyz9SbTjMBrrgcPVoaMBN9+ov4Ab8NlQU3/LZJINhL/FTXvHrDtgh3vUn/G5lNLykyT9BXxv+ZfNfSfP5JE9Q33YfGJZSjcX8DAJyfmjRRAeEwUDL63oHS2pQRWkWdY7Xd72dDMbpUCXKEZHnK65a1Jx9cp5URWVAN8Tdbw4D X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040176)(601004)(2401047)(13015025)(13017025)(13023025)(13024025)(13018025)(5005006)(8121501046)(3002001)(10201501046)(6055026);SRVR:BN3PR03MB2228;BCL:0;PCL:0;RULEID:(400006);SRVR:BN3PR03MB2228; X-Forefront-PRVS: 004395A01C X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BN3PR03MB2228;23:b8Yt3sUn2WIcJpdg/qGdooM0qne69TKasQ/MZW2kA?= =?us-ascii?Q?L1G0yqu8/piLFyQ8yLFj/lbIP/XuH7FoSytxXvIZA8vy7nj/N9JXcOS6zpYB?= =?us-ascii?Q?Hj0g6SPKfhSpTvtgRjIcDs6XLJoqQEotECcLXDyKN022oKkeBVcuO+3UhZE4?= =?us-ascii?Q?GLf2J70OwGMwu5Sd0//38HwmasdPjQfbWLXQL8lVMmFhRPH3R6lLqnqvf+i4?= =?us-ascii?Q?JoIzvsPSAvU7CxV302rISphu+vTOJp/v0g7GChWVWUeKdn76ETPm23QVBGpe?= =?us-ascii?Q?bSn95fusjhwD/V9pGRZpMUEIFFyc6t6uhwuZtf9/02m/Tvpv+r8ok9qQMVxy?= =?us-ascii?Q?+CbDeEbr28SjC2R0I7pMBX3G1c9hEkw8n2II7tf/UXTHzCIh1DxMq0XQmWhv?= =?us-ascii?Q?35Z1mjbQoUN5puKi+7kY/yf19YM5uWgMs1zTF9cfFfiLO7c5T7Outk6snq4I?= =?us-ascii?Q?Whhm1J6I1PStQBC7ni0Thlfshzn3uQvTWmtfezlpl21c7R8kAhKyNIiQshVA?= =?us-ascii?Q?kPOXUzjmqJurkKswGqECTh63CpHh6qO3vwxOB9DodPXcLpOrvzDmwiSxWYCh?= =?us-ascii?Q?VZy0pgmOII2x7NAehyzRJrrq7BJnuU/LlXCeXm7BBavsb+K8UI9bFaeyPy5Q?= =?us-ascii?Q?G6eSd4SpTnAscScKEvd+nwr2EV33xsjT4Btuv2oIKn+iAtdV70YLfcBTYaJ6?= =?us-ascii?Q?v4HGOLD2Cg2SVk3FCdTxdVrK6XzcWRhFXdxJ0O5xJlwWCxZCJNLNMaEcjfdq?= =?us-ascii?Q?RgLQHb6EDidFywhnC90AZ0FSBVY743MpN8Hsl8WAzVeKI8TIkoNaVyDxmYBk?= =?us-ascii?Q?Da0v3cm1WShHyVOO9dKIdhbIg26K+QQ2IJXLYGVXbDxR/XCqHd3Ij4qjuFX0?= =?us-ascii?Q?1BAdo0CXCbOrMUFt/hW/qThQbWWCSVl3yoU9jQUmcS6lwMGs1CxKhA/rR7ht?= =?us-ascii?Q?RKvFGCtayYKvd2QMpvd6r/hafemDz7MgQhLF7lDrNtt9gfsbyukJbGB3DnI1?= =?us-ascii?Q?hclEdq+Y16TAg1vjBh1DQFXTEshrSSu6qkchIltyZ+rgEH2unovreGob3j3w?= =?us-ascii?Q?AFNe+VYVmAWYe3GZNyX9mcFN8xIQw//FDIoqz+8ZcVr+i1bXA=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1;BN3PR03MB2228;6:WmJW+0PvChfAWLMX6bif+83c1oOYnt1IAdjIzhrXIj3lQps5t/I60wOISWaDEaetcmMwLNXiTVnYlRYIeATWmBMYmvCs1CKWToTUYnlzfDV2H87pH2IgRl8dROneDCFzmbPq8PjzugU5Q9w7xxH77KMxj2FBMKRCZXFeShcPXMQyJI/nuuqBYe65m4eVgSZOJnGYy1svNcPQjr+qiYb+TpSxgNo+hcb8V7eTNI+DfFDg087BCaY5Yz0Hb+vYkhDcrR/uRpzqc0ctp4AmVbJdU05u8ofDDE0uSErCCDqid04=;5:TRNqEkZAyErEV4AEqXGNy5UvYxr0nlaHmK7LSSgAACbS5BfBCFme3WvXc5HmPSbTiuhKW9com4BIYhWjnAH8eFOQ/4im++UK6Yw5DFGhsdiE5JOQGzLiPTdhW6HMYTYsRHEk0SwtTyPZYIUmIghhRiby2b/CaBZ//O7dJl/m808=;24:26wmZvvzwRZ9X/848yXPJpJ1J13CxyyjPYd1BmDpUmpqLJP5oMOYrhghlPz7B6zwHwiUE2j+xP1RUsNvzXu/h0sHgvLaqYnSECyqh34gDhs=;7:+Z6vU6zXKq6QPG2alYmTSmmJbwG3eCqbGynBDqB8jbh4XNd4rNtNPLLTjqGKP5m9lT3803TjsoRsS8lGOD2g4plUvJecU0jDGM6nabW7l62UFFVQ++bMj9hpYNP5TrEPJ+DmOMYBIaaF7OnkFxbWqpocotdUn2+LX+VmWtxaQnFUjg0KrceQZGVr5hDxzjxag71nD1YvrgYDFeYwhFemfH5ULEJKSQ6yMLW4LkKyrwfYuTWZ7JQJLQHPXokvH2y/ SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Aug 2016 22:16:39.4469 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e;Ip=[192.88.168.50];Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR03MB2228 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org In preparation for moving this driver out of staging a review of the .h files intended to be public in include/linux/fsl turned up some needed cleanup. This patch series makes no functional changes to the driver, but does some restructuring to put public/private definitions in the right places and have a clear split between driver sub-components. -update the public mc.h and mc-bus.h files to contain only public definitions -move all private definitions to their corresponding .c files or to the new fsl-mc-private.h file for definitions private to the bus driver -rename some source files to improve clarity -create a new mc-io component to contain all APIs related to the mc-io abstraction -remove unneeded includes and forward definitions Stuart Yoder (14): staging: fsl-mc: rename mc-private.h to mc-bus.h staging: fsl-mc: improve naming of source files staging: fsl-mc: move allocator private definitions out of mc-bus.h staging: fsl-mc: move resource pool init/cleanup into allocator staging: fsl-mc: create private include for fsl-mc bus components staging: fsl-mc: move dprc-driver private definitions out of mc-bus.h staging: fsl-mc: move bus private definitions into fsl-mc-bus.c staging: fsl-mc: make fsl_mc_get_root_dprc public staging: fsl-mc: remove unneeded forward declarations staging: fsl-mc: move bus definitions from mc.h to mc-bus.h staging: fsl-mc: remove unneeded includes staging: fsl-mc: split mc-io definitions into a separate file staging: fsl-mc: explicitly include files where they are needed staging: fsl-mc: make order of includes consistent drivers/staging/fsl-mc/bus/Makefile | 7 +- drivers/staging/fsl-mc/bus/dpmcp.c | 1 + drivers/staging/fsl-mc/bus/dpmng.c | 1 + drivers/staging/fsl-mc/bus/dprc-driver.c | 67 +---- drivers/staging/fsl-mc/bus/dprc.c | 1 + .../bus/{mc-allocator.c => fsl-mc-allocator.c} | 202 ++++--------- .../staging/fsl-mc/bus/{mc-bus.c => fsl-mc-bus.c} | 43 ++- .../staging/fsl-mc/bus/{mc-msi.c => fsl-mc-msi.c} | 4 +- drivers/staging/fsl-mc/bus/fsl-mc-private.h | 52 ++++ .../staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c | 4 +- drivers/staging/fsl-mc/bus/mc-io.c | 320 +++++++++++++++++++++ drivers/staging/fsl-mc/bus/mc-sys.c | 155 +--------- .../fsl-mc/include/{mc-private.h => mc-bus.h} | 91 ++---- drivers/staging/fsl-mc/include/mc-sys.h | 15 - drivers/staging/fsl-mc/include/mc.h | 20 -- 15 files changed, 520 insertions(+), 463 deletions(-) rename drivers/staging/fsl-mc/bus/{mc-allocator.c => fsl-mc-allocator.c} (80%) rename drivers/staging/fsl-mc/bus/{mc-bus.c => fsl-mc-bus.c} (94%) rename drivers/staging/fsl-mc/bus/{mc-msi.c => fsl-mc-msi.c} (98%) create mode 100644 drivers/staging/fsl-mc/bus/fsl-mc-private.h create mode 100644 drivers/staging/fsl-mc/bus/mc-io.c rename drivers/staging/fsl-mc/include/{mc-private.h => mc-bus.h} (53%) -- 1.9.0