From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751966AbdHCWti (ORCPT ); Thu, 3 Aug 2017 18:49:38 -0400 Received: from mail-he1eur01on0134.outbound.protection.outlook.com ([104.47.0.134]:7168 "EHLO EUR01-HE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751736AbdHCWth (ORCPT ); Thu, 3 Aug 2017 18:49:37 -0400 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=peda@axentia.se; Subject: Re: [PATCH v5 03/14] drm/fb-helper: separate the fb_setcmap helper into atomic and legacy paths To: linux-kernel@vger.kernel.org, Boris Brezillon , dri-devel@lists.freedesktop.org, Daniel Vetter References: <20170713162538.22788-1-peda@axentia.se> <20170713162538.22788-4-peda@axentia.se> <20170714135423.f4uodzns24p7u675@phenom.ffwll.local> From: Peter Rosin Organization: Axentia Technologies AB Message-ID: <9fd40518-03bc-0568-b822-512c5f6a479c@axentia.se> Date: Fri, 4 Aug 2017 00:49:29 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20170714135423.f4uodzns24p7u675@phenom.ffwll.local> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [81.224.168.30] X-ClientProxiedBy: DB6PR06CA0009.eurprd06.prod.outlook.com (2603:10a6:6:1::22) To AM5PR0202MB2548.eurprd02.prod.outlook.com (2603:10a6:203:6d::9) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: d83fd439-f102-4cef-3f57-08d4dac1e8c4 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(2017082002075)(300000503095)(300135400095)(201703131423075)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);SRVR:AM5PR0202MB2548; X-Microsoft-Exchange-Diagnostics: 1;AM5PR0202MB2548;3:hKSoG/uGyEX15j1eXcn9enNWN4eyTEKUHQKPIrxTvLn9pILUl1nujFcjWY8MjNwPN3w/ZylZtbwVmiSWX5EIpHlWwSxx70qhz3DCtDh9bVnPzH0zM4hR5gRWDLR02UPk0boehO9hPjldvBf4idXaJ5ockEddXHWeFt/iPxQfFnh7f3ydV0pMBdBo2Ecmj8VZ3nIkrQVJncL1BVbLPzZ4JymPkd5vJrq1bbgysrG4iQecoJfBSzE85klyMCiUY5pC;25:lNxzmqpALuSUZXTlB4z10WMO8oQHT/on0AtU5AuEyPY6DA8D9r422IqTvH6ZjJvxk7cvf5fYUPs47o5soHUzuXX4k/jxqmyVvfDq3k0eTCIow01PwHOPjxKkqN9dRATnng+fLCtZuS4J27lyIKeh6rWKbwjUtS++4A7/je4HZ3DEtYzjBm08Tl5uBspCYQ81fHFuA+QsnpBcoF18k9QfBxABy7M4s0UO0dPFSpE00IKkc+23GYn1svoWc2HiCeH+dxicbRnVekAO/gNs+CGDY0nHyse9ioaRqoaPdujQSjNtz0GRGq3g1gEggWXAhqr/uGwbfekATJfaNX/QFgliXg==;31:QMXXqXcFidvxejyttvJjyjZ3E61XjrBtRWfesYhqUoLu1GKDr3sg1+VSPXuN1OPLe6K+Ua9lnBHlafErk1F7/vZHpVrdMYVEom32qCkGRh5lsZg68wfTgIAjDmARTi0GdvrVWZfEOul0RTP7sQ+qi2meElhnrK4Wy/o6ewJ+1mwmPfOtfbciw6sVcCPNVh20XQytlMqKN0fHuHGJh3mpyUtXpM4RCkzgltYJPao4E2s= X-MS-TrafficTypeDiagnostic: AM5PR0202MB2548: X-Exchange-Antispam-Report-Test: UriScan:; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(8121501046)(5005006)(3002001)(93006095)(93001095)(10201501046)(100000703101)(100105400095)(6041248)(2016111802025)(20161123558100)(20161123560025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123555025)(20161123562025)(20161123564025)(6072148)(6043046)(100000704101)(100105200095)(100000705101)(100105500095);SRVR:AM5PR0202MB2548;BCL:0;PCL:0;RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);SRVR:AM5PR0202MB2548; X-Microsoft-Exchange-Diagnostics: 1;AM5PR0202MB2548;4:HjwYcifQ9WASMAwIN+rIyG1l/zXRLBzOsVI/0BzXIGdmvLTiAAoaH+27cyL8ZiQxjPuzJRcZe0sglE10LWtPBkZ5OvcdCUmZALPCsm58koOaxRPSUnZVojxqAWEK4ZBXjqDTUjDbhimW4rFAUcnYvNUNrF9EwJfSeg2BbPbanvTSw9gOJQjbQpdR2jiF8FKVH5mtM4xCs99UPI5Myy2tyx+TZhDSOFLxkhHYn5sPV2Il58EmO8SBeRckf031Kfgp X-Forefront-PRVS: 03883BD916 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(7370300001)(6049001)(6009001)(39830400002)(39400400002)(39410400002)(39450400003)(189002)(24454002)(377424004)(199003)(97736004)(117156002)(53936002)(31696002)(189998001)(65956001)(66066001)(65806001)(230700001)(86362001)(68736007)(74482002)(7736002)(23676002)(81156014)(6116002)(3260700006)(81166006)(3846002)(31686004)(478600001)(53546010)(83506001)(305945005)(33646002)(101416001)(105586002)(8676002)(25786009)(42186005)(50986999)(6246003)(76176999)(54356999)(7350300001)(38730400002)(229853002)(64126003)(4001350100001)(65826007)(2950100002)(5660300001)(6666003)(6486002)(77096006)(47776003)(2906002)(36756003)(50466002)(106356001)(42262002);DIR:OUT;SFP:1102;SCL:1;SRVR:AM5PR0202MB2548;H:[192.168.13.3];FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtBTTVQUjAyMDJNQjI1NDg7MjM6VnhPODJtcXpRRSs2OFhaN2QxMURiaFdo?= =?utf-8?B?VGNjcTJaK0ZzcXdJMS8rSXhKUFVydnRuVFFzY1R0cEZlZkJIZ094bzNmV1hO?= =?utf-8?B?MzI5RTNnS2grYmlrdEQreWJoeUw1dSs1OEp4N204V2g3ZlVGbnlKS2VRZFZ5?= =?utf-8?B?LzNVWFNjWnFCNTZwUlZRQTk0bCs3RVc2aGttMGQzMUg1VzZ6VnIrL3hkSGE0?= =?utf-8?B?T2ZtdzgvNDloZmtaQ0FmSThOazMvUU1adE1KRTAzWUN5d0NzbGlkbmFjQVFi?= =?utf-8?B?WkYwYkFmMS9zVitFNFgvN3A4WGFGT20zeDlobUIvbjBJWVZUUi9Id0ltOGp4?= =?utf-8?B?ZlUwbGtUbStIdnE3K2c0Mll1MFRZU0dud1RtZEdxTTVlM2gwSXF6UTMrRmJx?= =?utf-8?B?OHVDbWVVWHBNekRXZktkV0xIU21CV2pHSG83MzJDUzZrWGQ5OTJiZFh2VDJ3?= =?utf-8?B?anp1SG9mOHFUNmw4aEU0MUFOcUFsSHZ2TWFKcXF3bzU0emRMN2c5NUJraS9i?= =?utf-8?B?Q29jcEJITXA0d2I3MmhZTTFJZ2hPbStLZVBzWEhqQ21mUm5JVkJDbjFQcXc3?= =?utf-8?B?THNUMUZQdlM1T3JwWkE0NDc4bmpmWDJMV2R6U0lGVEtjN3g3UTAxWWVzaHNF?= =?utf-8?B?eStOeFpCV0FPZWRualh4U3BPZE5tclZQVCtON3JFbmZXWDNBd3AwZStRWlBn?= =?utf-8?B?NlRQY2h0VGNaYkhjM1RlNHc4SzZnQlN0bURpNEJLdUhGVmdIa0xFRlppN1h1?= =?utf-8?B?bzl1cm9hc3lDa2N3NzNUblBiSWZSM0RqbnFxeHQyWE1yYTNMaW1DL29McXpQ?= =?utf-8?B?Zmo2cUMzZmtiSkQ0SWhKM1RzSzRQZXV6clN6djlNTEwzZ2NsQkwvNjU3NEox?= =?utf-8?B?M3gwSkgyeFhVM2Q2YWxRNllXaTl3clRlZGFoUFlJbG1zU3hLWVFWeWg5V1FP?= =?utf-8?B?Yk0ycCtXZUwyMndYRlRWMG1HVlFWZEJybUZxRzNObkZCTmlsQklkMWJZVWt1?= =?utf-8?B?WmRKZUs4d0RhN2N1d3FwS3ErMXRkZm9iQ0dUVGRMcFNxamcyWWs3bHBWMWov?= =?utf-8?B?Y01ZOW5TUW45ODVCbnA2NHpWaVFIUm51RUhNZmNhRXZWTlNPMGRTbG1HTlpk?= =?utf-8?B?dFZjeFplZENhem1wQTZocUN0MVVYdytXYS81bGNHdDV2U3Zzc0dHTW1aMWY4?= =?utf-8?B?SDNhVUpqdHliVVhDbW9panZYbU9Kb2k0ZDI2anFFRjlJeXhxTVcwZDJXRk5v?= =?utf-8?B?NTVkWU9GRmlFcGZFOGV2bm56NEJtdUMxeERhWHJ4QWJmRnhOckZYUEFpMzIw?= =?utf-8?B?c1lWUjVrK2VsQ0s2djFZSWhCYUhvVVNwMDk0ckYxS2tNd2QrMUV6a1BrN08y?= =?utf-8?B?dVBKcFVTdGNrRjh2Rkt3dm9BYUk4dUxPZTBIWUJtK1dEKzduMFBveHNvV0wr?= =?utf-8?B?R211WWVwZXB6U3hBZ2RkUXVJaHR0YTA1dTFqU1l1WDRRTjM4czduRk5KcVl6?= =?utf-8?B?N21WL1NhVE96cndlNWt6OWJvdXcrNzNPektxaEd6YklEeU9uSFFGNXNBT3ds?= =?utf-8?B?alJMTmZvb0JWMlQ2Wm90Ym5kd2ZrTlg3YWloeHVoNzloVWpRMHpNYzZvUFZX?= =?utf-8?B?aEJxeFJWc1BtUGMvQ0tvcnM1VmV3ckJGVDQrRW1oWnZLcmlhTFNjZ0N1ZUIw?= =?utf-8?B?bGVDN0lFMWdybWxlaVl6RWtHaUR1T3dJazA0MDNLMzJHVVB0eTFQSVN5MTk4?= =?utf-8?B?MWlqY2hjTVl1dEhuVHl1elNzNmN3czVkZTZvdlFMNnJXcHp3cHJJdHprR2RV?= =?utf-8?B?Umc1Q0VobzR0SHMzQlRKdHZBVmZCUS9JOEF2NnVYa2VIdTJEaXBLRnYxTWYx?= =?utf-8?B?YWI4SUNrTjg4RFZpczBNQjJENHdEOTdJMDFLQlVNdHNOVjJRMzZKTGUvbGlh?= =?utf-8?B?VDNzakRpT3ZtSVdIUEY0ZXd5QUcwWjlUVjN5bVRjRmdMQ21aZE5RcGFHWWdG?= =?utf-8?B?dElHemplbnF6N3FzQ2ZqS1pkMFdJOEJiNGdENFhrUVRhaG5YWW9rMkxEK0JB?= =?utf-8?Q?mQUdbDgOH0A1PQomP/bUAm2mqYE?= X-Microsoft-Exchange-Diagnostics: 1;AM5PR0202MB2548;6:BdLEWrwbgY72OKeQuiAjcRZkYTicYxAUsb5PsbydCjuPhDJKLquQchZ/sD1W5BEGwK9Z/tojPNeug5W1LjGt+aLjoncgFYvO94QuhlA9RiC1ynvi+gqXbjbmS4B5P51XQABRUGBzxcO4yEQElUwzDZGhLGx1191lmcq4PcTXDZITG/xMVcOY5wrGd/U0Y2o7Kmkrj0MlPCJPhj/y6rAslBKiQEelGqZy3xcJFDC+IXZkZYquQIVOhGc2RnSkgjLUpsb059MRWSZubEOrf55K6925zo6qoowX37yHqmBKgefra6IXsAMoP39u2q0SXuiU58XDwquqAnHDhMRkfyAelg==;5:JJFC4D1sNTFykT78U97LEk3a2m9LpBwjZK+4n05oMlVNYhaHSbA+ysY/Vuc+CRO+TtX4nsIg5CKUnZpgGySTZt/tWykXTkoeRLZJ1dHx0/rETHTpf+NvJ8NjI3aM/aiimgHA4Zpynx+2LQpDRLlvRA==;24:M6F0MvOFei/wjqHBZtQvutHde/I00Bqs9mfELdwzSXXEcPbfl6sCyUck2qFMSG0+WOTUg5iL8qiAaTOtQeakgd1Cy0MqGaQ7DcLgQi8tJYw=;7:RMUxktfJrcCZ6oPb7mbFwavoYD4F4fuWS9h8GUHQd3R+vy/f4bGsxmSo5lMqjMD6dIEaGIPRvyY/bywOgJfWQptNcZScieGWf5EV05X1RCorOUzBuTAyJuTrpZza3zUB2J3bqhflucp+UxYv7lVXxC/oYM3QBBtnbQvcgeKG0Uw8GQ2e9afmtH9cwJEtnTckeaocaDIn1h489GljnR/CymMqe3wYlasWdS+QwGkFy3c= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: axentia.se X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Aug 2017 22:49:32.8394 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5PR0202MB2548 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2017-07-14 15:54, Daniel Vetter wrote: > On Thu, Jul 13, 2017 at 06:25:27PM +0200, Peter Rosin wrote: >> The legacy path implements setcmap in terms of crtc .gamma_set. >> >> The atomic path implements setcmap by directly updating the crtc gamma_lut >> property. >> >> This has a couple of benefits: >> - it makes the redundant fb helpers .load_lut, .gamma_set and .gamma_get >> completely obsolete. They are now unused and subject for removal. >> - atomic drivers that support clut modes get fbdev support for those from >> the drm core. This includes atmel-hlcdc, but perhaps others as well? >> >> Signed-off-by: Peter Rosin > > Ok, I merged the core parts. I'll wait with the driver stuff for a bit > more (maybe 1-2 weeks) for more acks. Pls remind me in case I forget to > pull them in. > > Thanks a lot for doing this, great work! > -Daniel I don't see the rest it in drm-misc-next and you asked for it, so ping :-) Cheers, Peter