From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751865AbdFZTNh (ORCPT ); Mon, 26 Jun 2017 15:13:37 -0400 Received: from mail-he1eur01on0097.outbound.protection.outlook.com ([104.47.0.97]:32896 "EHLO EUR01-HE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751425AbdFZTN2 (ORCPT ); Mon, 26 Jun 2017 15:13:28 -0400 Authentication-Results: redhat.com; dkim=none (message not signed) header.d=none;redhat.com; dmarc=none action=none header.from=axentia.se; Message-ID: <59515CCF.8060406@axentia.se> Date: Mon, 26 Jun 2017 21:13:19 +0200 From: Peter Rosin User-Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: linux-kernel@vger.kernel.org, nouveau@lists.freedesktop.org, dri-devel@lists.freedesktop.org, virtualization@lists.linux-foundation.org, amd-gfx@lists.freedesktop.org, Daniel Vetter , Boris Brezillon , Russell King , Ben Skeggs , Dave Airlie , intel-gfx@lists.freedesktop.org, Vincent Abriou , Philippe Cornu , Yannick Fertre , Alex Deucher , =?windows-1252?Q?Christian_K=F6nig?= , Gerd Hoffmann Subject: Re: [PATCH v2 00/14] improve the fb_setcmap helper References: <1498111597-10714-1-git-send-email-peda@axentia.se> <20170626093526.7dezn6d74m5ktsok@phenom.ffwll.local> In-Reply-To: <20170626093526.7dezn6d74m5ktsok@phenom.ffwll.local> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [90.229.234.78] X-ClientProxiedBy: HE1PR07CA0039.eurprd07.prod.outlook.com (2603:10a6:7:66::25) To AM5PR0202MB2548.eurprd02.prod.outlook.com (2603:10a6:203:6d::9) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 72944677-3397-41d7-1ef3-08d4bcc76abe X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(300000503095)(300135400095)(201703131423075)(201703031133081)(300000504095)(300135200095)(300000505095)(300135600095)(300000506067)(300135500095);SRVR:AM5PR0202MB2548; X-Microsoft-Exchange-Diagnostics: 1;AM5PR0202MB2548;3:8tnQWtPOsovEYnwSKDOJTbWGCi8FhkzT8djjeZLijAaalUBQMiThaA03zJk815POGlwXfy3FaJaVox5UTb0QctAdxTWMREmpaUJXjNRvHnP8Q1LjpHXIICmitWhynepCRD3iomX/jfyNvtgc4/VTSjoWc1HZFXkjXdCbvRFXGFp9fTEx5VqSqKSeKa5HGAOxjPzUU1+oUQGD3IFqrUjkpjDmcbvwDaTPosE0dtUC5olWVHL4gOAm7+5g9hylZYYBk43KhkJ4UAOPuTM1wLmxX3l6T1j85yXyLQSOjTwB9iAE/5yLMMGLjyVmieN+T1r06Bo/IVBmc8WOlH7HljRxgFwGcj/6kfUqCN3tMUuY+BHTgLkaLUks/djyyTkvy+i8JHWKK8Tju1xzYQYYExV720LPAqd4TqXud9UaTSI9IG3Fmrp/tPAlbypM2lrf/z/v0ljGI88OKl7QwwjT4CbIyMk3/Z5K4ae9xImPDeAZ+w2eLZxaw4t2/bqhf930DLaIQM1B+7yPejtCKNm3UU6rCj1665/Rq2GQjePgcsvV/BFTdpMvNXbjUoZA/qFHavTikDnkIf8E8sG34WKZlWOawGFWIuSLRoIHdXFitlk4OGXtBLjwCdfIHNah95QjWO7v9puX+T6OuXawlZTRI3+JQEoHwwi/XSoPktC1kp96gLGnHPra6yV4UXp8sftHm5rG6TOnSmhxwBlcsRmGFYxI9y+RzbDUP7S+uNwRS9h6nAs= X-MS-TrafficTypeDiagnostic: AM5PR0202MB2548: X-Microsoft-Exchange-Diagnostics: 1;AM5PR0202MB2548;25:GpDpqBCCvkEABspumd2MUkyE+tmzqC9RnFGtyP3JZssl4iDkd8vttCbDqki0YsxllZFFQD++/GeM0tpq/BMQ3fFPZDqniWsnfw0uCJpH86BvlYY5thtJvpybh2Tx3dKE8z20xpoEmQGS/+bKfR+HTYt5c/ZlXjZ0zQoAOgM/o1A7K4LIBjVfcbgKzyc9li61DdrqO/xrXPrn7JTtqZA5OYpoyBPu72kf1kTbydj4XP2DR1Bnc9zGcU0lZQJDe1xwVAx1ZWYNE8XparsMDoemGgitj1UYKbPOlp87ztu4xYNWwAV1ZYsW7Vz1UA8vuKmIDHpHVv2W6SfTU/Rc13m0SVp7n5aLwvnz0q2rlZMNvt5NlqZuTzEbTegHu8ANc0K0Arl1ASZe5gtOslZis5i5J3B3e06+NTMnksxsN4H3G3aLDNJtRdoFlkd5GwO+CbsPVEwpJ5HzAUD5jyyQqMyyPVhGNzDgmVE8Cz8ZtHnVrW0id231guxtMWmbSTImer7HhRLMV1cmkAuXfutZXoxQmOzJG+gTcS1/Qdxa1YRsULHGZ0Rgdq1xUqW25mM1KC7hcaYinGeh8zvSNjLs4etKFt8d6XYNqjanI/+ISuZj3tL9Uq4UjE4yRPpRpYfrpiGNCFWpPt1iilkimtV6fOr1odePI7JqK2/YrcrTMEcjS+H4S1mKBG4JaR733mH4ytAvj0VqS+ykx8B+7A2QzDR6vIGzcKWZBGGnsiQ8mXiXy0kYi+C0cQiyZMqdrOrTR8c5k4aAH+G+oHWmsS/EGXAsWhs+qa3BT+hW+iFipo5AQNs7NR/xIXwIq29Q7fBuYMeQsUqVdH72dcxhLQbp0SRlDvVMc7RbG0CmA5st6mS6xdPB0d8iBVwHfOlFV3ri7Ygch4WSuLNj87tOiSpBYZJFNrZmWRiEg5TEQZF/NvKe51k= X-Microsoft-Exchange-Diagnostics: 1;AM5PR0202MB2548;31:iBYIvxy9R0yXTqM7XKyo9vKj/71+8RCs44HyY/uvYouHV6iskRNljLWyjrkhaxzRxama27GDn4U/1/Mg+yyI3YWSyWdt+MxGClXkdovIM8K3CVXuFE8m/ywgD46NPq4EfnfrthpycOrXGI4pm9gMQUCHI2LNnxMm3DHuGaUTUHQyFeIYkaDi395bRjiGNuH8kZXnebQ+76c8Vz+4Nb/gGFSLvyNNTDVVRcSs0RBQ8zhpJa7IvLtJJ7OBVc7H056XlhDC1DwGU6sPkBRm8IEtqZTwxE/EGnuwZezZeN3DnBxI1s14R1NXQd2eneGQ7HJth8wd82lD7F65TwyPiTNglUkM9Fu1ArEbYDcL+4fCqmpS4gih2PbYiZW5JklPhEBmJXs83coXmW/GYbhsqtJjJOLvgsqgtz8jIl2bKwUjoO2ZKiw1YoLNl6jQ2x+TTZ8pSme3JCksfjTvZp5nmxhQbw05gm8wslteD6NKVcBXyfn8ivzPwfnzLYOr4Z+Q3toiceILgotdpgp8ceHuL17x8MCilbYjj+PFNAMXNrla2EZHd8m27QP+Fpu8gYxq0Fnmwsmh3qObtRPz13pIXGjPUNv6y612Uv6FGdeGB/hkaBsqrgdBnKMVZ0x+al2GoHsXGEChz9g7CC1aib0vQYcb3epbp/Z9dHSc4lXF0o9pfSEUOpJLyDkqP2XKHd2MoRD+NLDy9wjrhV+sH+QP9mXi9w== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(133145235818549)(236129657087228)(167848164394848); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(5005006)(8121501046)(100000703101)(100105400095)(93006095)(93001095)(10201501046)(3002001)(6041248)(201703131423075)(201703061421075)(20161123562025)(20161123564025)(20161123560025)(20161123555025)(20161123558100)(6072148)(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: =?Windows-1252?Q?1;AM5PR0202MB2548;4:FCcw11NziDJ/H/0vc7OjCrijfC3Uf7osFWbS?= =?Windows-1252?Q?i/Qgla6hej0rGOkuTDlvwDspJJ/RzhB4Wha+zrKzMILOEbZTnTJcGahR?= =?Windows-1252?Q?B7qMwfLpZP2BN+zZ49JpOxdyqTeGjCmC3VcOTSzErwJtbv8t6B1dPuXZ?= =?Windows-1252?Q?IKp4TiY4QHtQQo57jBSKASv7n4MaevxnHQwGGCxdR9BwiXlP+dFvTnME?= =?Windows-1252?Q?YrGb145IzaSgZx7TC8RJq2/dLsHQxJdsKJFkVBGKGdcRmYIeJvAiHjeh?= =?Windows-1252?Q?LI5PEKnrzgizzxKPBGGs4r9+heVEb55CI3nx9428Z8sFFIyAzCEotLIe?= =?Windows-1252?Q?T/M6dZo1WZv9piHmCVE9EPDxQ9Wc2ZOC46plT9QPbJ2owEVKaLGVaWuG?= =?Windows-1252?Q?fySrxxDWni78pYa0xlm6lByVJ50Wr6OqFyqOUoKznEkVSBz4Pnhucf+E?= =?Windows-1252?Q?L0dnF+qdd1BJ3WLJBqoxFIrC43uA0XMaueTrfG+BTuonFREvA2jPT4nS?= =?Windows-1252?Q?mzMgMQE495zKvU2L6dd75VZwI8g+zBkuHb2ulx9GMjU/9X0Ago1EcA2W?= =?Windows-1252?Q?5EyArvtl/ESwwIqAZcZm/43xVu5WxVORmA1X0d/d/hcp4iS5Q6bkDdgS?= =?Windows-1252?Q?LPidRq3l1dbuT4Fxc9ksQe2QkQPHx8UwcNOTcMb42vsB3YPWxlglzVxj?= =?Windows-1252?Q?wzopbxWsS3hRIu/Ng/ESnqOPnDL2CFdFMy0O0/pXpj+4LNQYB/JeZjt2?= =?Windows-1252?Q?p7U1W+v4NawDDxepLFOId1/TW4ePCYLVOLSv7E2JAY+yVIuuaQvhkprn?= =?Windows-1252?Q?rx1AgtI/MK2t8+DfHuOXXlRp5vjmrl6jiP7pnsRGHK/le7FZ/qMPac7T?= =?Windows-1252?Q?yg7Df4gjOWgmRJPsNxvvmxerTI+Z1Pew54D5fBDw4uCZ1pGilrvLKDGG?= =?Windows-1252?Q?9A/pjua2YqGk3nm4c4Yz50ocJvkgYUoxvpbX83LppSaumW4c1+jxDLWN?= =?Windows-1252?Q?0h6PphppoxEIyv1nYqECLOGui6X7IM0P8XP/LTEq9rCaFsge1mrvxWF0?= =?Windows-1252?Q?U3yFwr6C2ewLi3ouegzKQJ4GKfi3lIuChrhMtOx4l8/b2wcYGyjoP3je?= =?Windows-1252?Q?rV5vKM3NM864CeIawhkGBLci7lVQ3tHmD3rSpEzEeUzh91I+imKTUOo6?= =?Windows-1252?Q?VNCI8vkV3DWgsvRkPjOyFHElclVOa8bmQEUM4R9tioxr8EqyF4jX2kJc?= =?Windows-1252?Q?e/wYZ891tIzrGVd7eejlGDEy8jDef5oaU0DwvTo=3D?= X-Forefront-PRVS: 0350D7A55D X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(6009001)(6049001)(39400400002)(39450400003)(24454002)(377424004)(4001350100001)(53546010)(76176999)(65816999)(74482002)(50466002)(77096006)(6486002)(508600001)(59896002)(33656002)(229853002)(2906002)(99136001)(50986999)(25786009)(6116002)(23746002)(5660300001)(7416002)(86362001)(305945005)(3846002)(87266999)(54356999)(6666003)(36756003)(81166006)(2950100002)(42186005)(117156002)(64126003)(8666007)(53936002)(83506001)(80316001)(7736002)(65806001)(230700001)(47776003)(189998001)(38730400002)(65956001)(66066001)(6246003)(8676002)(921003)(42262002)(1121003);DIR:OUT;SFP:1102;SCL:1;SRVR:AM5PR0202MB2548;H:[192.168.1.169];FPR:;SPF:None;MLV:nov;PTR:InfoNoRecords;LANG:en; X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;AM5PR0202MB2548;23:Z70vNW8VwFprghpOGz9Pc82g85bhYNE1K1W?= =?Windows-1252?Q?LuxTD2eJv9gq4vmtG7NG/2OBCprPo6uPU8b6dl2x6LnSwOOIZx1dd2gy?= =?Windows-1252?Q?cSZftwzkZGUYVPr6SZDnk+1PmCUK3Bfa0n0/S5vLb7NsArzd7vp/3Tx/?= =?Windows-1252?Q?Nzhs3ssHsN1U7NjzrNjdapX1HvjZd9hNVBSK/sHCloW9AVqlYnXV03j8?= =?Windows-1252?Q?XIQwyVnAFlrDBHNb4j8FJ/l51UA6tFGI1KadQAboiC0LnFS8rZUcNGA3?= =?Windows-1252?Q?tKp+50WtfH8V2qm6lYH6pmKXKHWA6wtl3i9ZSvxwh4IZaA2xsAX0Nylq?= =?Windows-1252?Q?qPDhpz3sqZXQH8EkHogCQonlwbfNkleXAnx/8ToSNvO04W12HQjeUBt9?= =?Windows-1252?Q?K0IPs4F7IkipaCuFNc8aKCicG9PzghDTxT7zsoE9v+q3cGOXXY3pp8Rd?= =?Windows-1252?Q?6SAkjbL8MZqbkjVhGPMXGc9NYh/LKAuvMup2m3u+tvf4MjRlmkg2c8kG?= =?Windows-1252?Q?EXu15CMvtWQ3WlXgPVwnZZAXS+w03LG4JCMv8Ivkm2T3l9RfeoAkVtzJ?= =?Windows-1252?Q?yABhjnBmumfRZVIkS+tUqAD/kQ7RUV1jyHHZdyJHZPwgSUaT6hohD+O8?= =?Windows-1252?Q?BZ2AggCY209VPWIJuEIKGFmSkZZLHuvMOSEXFkEZNswOsLGWdqkHs0sD?= =?Windows-1252?Q?xeWRD18D2WP+z4mBehKpAEF/eS2nJYYP+EiI0FNgUlmfWTt/Gd9QMmiG?= =?Windows-1252?Q?5Mmho/v1bORNr8TyA4x0WmIJi/72+N03zN+0qXA9rzTKXaFyxVJ2Ffzc?= =?Windows-1252?Q?ogVW7nHSAqfwggBZuL1I5TxwgFEX23DriP5yH/ZjLLU7PeEThkFv/QBU?= =?Windows-1252?Q?YIr6bDtoqHpUBoUBq8O/TFl2O5HomBo78snMxPX8PzkSqbJy8K1QAAOj?= =?Windows-1252?Q?XKoQL8x0eSBJDJnodkcTW9qFmYeIwsdi2ViA/99TzzmJvWwwXU53Z4Q0?= =?Windows-1252?Q?uwGjKboLgcSFrtz1jOhuqI+m64Ose8teCOXplfn3OH6h/h/la3HVUt5v?= =?Windows-1252?Q?3PCjIgGrNXUF+uxVQYOKdkZrkjRwjapX5TACdek8/K19qyEhqfl8ipNt?= =?Windows-1252?Q?iZw3NvshOesgVZUg2OBTc9zWg0aS0scYyfVyYnMSXUnq07UpiA6T0OP9?= =?Windows-1252?Q?3C7GbWPm/6QXBNwVTIjAKB7IVDhisCgt0QUomXps8VLvWCu3gcqlzAEk?= =?Windows-1252?Q?S289IDdHbvsA/6i1fIf8Hgy2VAWKw8Q1L2V5CTHP5j2cD7xmngpp6DR7?= =?Windows-1252?Q?4CnofkJyhhjLCu4cquJ5+v7fqLxHsW66ds0lgKKBS2S+9pODKxg+aInj?= =?Windows-1252?Q?VaHto9eTxwMLpyIUG87RSWgAxGqWF8zwyYWelmyJFj3YXUlwA9njr3Ii?= =?Windows-1252?Q?j6oXScmxrMWylpCOUKpbgHTvhsl0ygsdbIL3mRS9HFQ=3D=3D?= X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;AM5PR0202MB2548;6:3H8evjqFsKk9MNcBAifSenLxIZH1GB0vSMey?= =?Windows-1252?Q?0RXEsmPwGRJcix44fIIYyrqp/+5jgoyOW94OptsFX6DMo07hC84eN0Tv?= =?Windows-1252?Q?9SLoZAqPPUgL9R9Ntkjjo0FWmZyvSR/oQhhpDcOBgq+ZpHKiJ3MDTx5j?= =?Windows-1252?Q?BtH5uCoqc1kOFFXWxybW5NGtrkvvdt4nysgF083+0Bfay6ZqY7q55/OC?= =?Windows-1252?Q?CqWr9UHbA5ASkOjqY39cFOdCmJj3H/vypAWXKDpWFpu+ygVlGFr/bpFA?= =?Windows-1252?Q?FTwIIe+3R90H1PDvqcVrldsxOxl66YbtNv21zjcljgasS1B6maRQsk7b?= =?Windows-1252?Q?qtJcEyzrtCAD8F0QTeGQHOtXxYoi24Bo2zd/gkuwZn8y8fZwIhJYwUcv?= =?Windows-1252?Q?PbX8DKALX6eAWi/fzLH8dGHJr/UuMPrIw5hZ8plbELf84eQj5XI2DWAi?= =?Windows-1252?Q?RLNKBwWPpaL+4Rd6raFb+E5TLskVRCvz2iKUUqR0p9ze4W3oT3QE8SNU?= =?Windows-1252?Q?3hfoga4o0Q05/ip65Hr/I3XqOwk6CRDVjdpVXigTzW9VmbixEl87SBSK?= =?Windows-1252?Q?b/Sb0/cJ9vP9KdU1l39IPK+YDbl84dIAq5DP2gFfMy3k86YuW2JhaRAJ?= =?Windows-1252?Q?OAx+PAaV+WHIFKJ+sfYdMg2bp98B+AVWtxK+RnNGcadiEmhXYRXKZC5i?= =?Windows-1252?Q?ELnnqOMt9GXPVCiJqZ/1Fc4wossx8YxNoBTzhE0r5p0S8ILbvQ7+t319?= =?Windows-1252?Q?woDfLc3PtHtpq01Gd09SSaHHhXRPeKHDbAXbZYyhrREj/KFk7BftwMlW?= =?Windows-1252?Q?fJ+JHgCsZgTxxJKzVa/11kutkwe9h9/LhvoMaByN/ANw5OE1rEsoQ4GK?= =?Windows-1252?Q?ox+1eVfexTcCVkVMzIANad1FCWTeMUuL1rCQXkK/e25Yxx8IPWTS6Kxt?= =?Windows-1252?Q?iQvTejfxm00Q8l7mTmsUg3B1Xcd86h7KsB4TTLYyF9UXWT+4hyoVyY9V?= =?Windows-1252?Q?Wbc4i0cURt0PBQoRhN6yGi5jMWngogmUqD3/AkTgqLBg1/6CrIkk9fhx?= =?Windows-1252?Q?LwVGtStx/WhxZMA=3D?= X-Microsoft-Exchange-Diagnostics: 1;AM5PR0202MB2548;5:DWP4ScVMZGAnKhEvDXyzIVY+ID+wspR1tJcBdFYsq3a2uSA051s14KY5rr3EML7mnLtEgIXX0uapfkmqgyXYNeXjPChDWPnBOQnc923JqdhfBEFEtMn1YNT5gnDn7UmixYlkfuL98MqP3V5Ej9VXDzj+dxkqMZfS19XVg/A54d6C+pfXS9ugJNKe98o8itTtS/CQOkYOlg7LAaN8lMioDcX9xEDd4AVyYolfZa0eUU6bg6pE/7KktzUHhVvfzpavZm9mb/B7F7fT+T6lL/AF1o/OZScb0ROKl6kfLWuKJ1D9FM1SqqCfiT2UKLHyfHsdzktkRrADR2BEnFlWCD43LOwrUuaHWESqrMTwrYo8Dj8TLDAUHGmzuAd3KiFZ/FS4MSEz6QOVmER+fgbkNvWQwgm9cIgB7Jvwi1dgiDeSWmvLgwCbBABh7j6XG1fU2IQDYq4Vu4W0dVTBHqw2/fIM2Vz2Xh3Crx7fcQuCAYWk/WOxDe3TN6uyFX/2qLmBrCJ9;24:qeY9oxzNMYcNX6YviS5GksoZ9z0TN4jlDwWueDazi5BsMRBxWtppvkx65upH33fCuh37nve90ywkGqPYP7qt5h6nImxzLtdySOJl9UYEfJo= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;AM5PR0202MB2548;7:R74eRUduihET+qhNOCjB5Q7vgPNjkx9vsD8Bexq5mDhg7/OaW/SChVJ1gGhLEHvFZR9UpcvxeQ34k37mwft1XJCWnfzi72D87NC/KSNxrf6Fp7pZ8sWN7/Ywrrw2S+98lJDPNJrPg5sGUs3rDuHRqE3NkAT2z/xwYQsvbqiO7z85Mez6jPprlNHTAPRNWYFpHSSmPE9xSR3G8+wfhNNc12Zk1qWhbyNYivlnm0yCjgnM2/Up8ETfsOD6kD3ZClbdU8eg7AryfBBTZ8zj2S3g5k2sggb3S2h56teGEhPO8ViNJQA4DdaaJi3Odk+F5ml/6RTHqPg2z0fH0Ds1JB/ktYh+oHYDkeSJfFRJsUs7SW2HtjeNbT/YCDU6nl7nZDaZOvrSmwg0Au9EsZrY1XyKwg9sVCaYdDl4CzXMlAXzdw9X9+HsBPsMCR4f/9UyPGYwloIpfKV4FyPgqbinOmM2yjQR4kAq3sSEeOm4NA5djFiwDYUnG+2S5NRvDu5urIyAYmCx1E+1bGAObI/U0nJ5a14ndNncW8pSu4CHbWgfcIZ9GjASLapABvlk8d3x9FSTO6i1XI/1GW9rPOJjFgxfXUXQFbpDumgKyrpW0waTJsEWtumeMTMpVVjR8FIds02BxXgmmie7XTel3PuTDh2CItg+KLlcUaoKhgtUYBXqxG7axpH/C/M7ONqvoWhxuoUVXLyElmpOsDRe+OxWpgjRRq29HpnuRm8WasEO/XjuzE+JrtBS7eGdYPsprzETvKR5jOpVkT+7CP0UcfhROmj9yfIlRTOnu/P26h1DVF8r6NI= X-OriginatorOrg: axentia.se X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jun 2017 19:13:22.7959 (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-06-26 11:35, Daniel Vetter wrote: > On Thu, Jun 22, 2017 at 08:06:23AM +0200, Peter Rosin wrote: >> Hi! >> >> While trying to get CLUT support for the atmel_hlcdc driver, and >> specifically for the emulated fbdev interface, I received some >> push-back that my feeble in-driver attempts should be solved >> by the core. This is my attempt to do it right. >> >> I have obviously not tested all of this with more than a compile, >> but patches 1 and 3 are enough to make the atmel-hlcdc driver >> do what I need (when patched to support CLUT modes). The rest is >> just lots of removals and cleanup made possible by the improved >> core. >> >> Please test, I would not be surprised if I have fouled up some >> bit-manipulation somewhere in this mostly mechanical change... >> >> Changes since v1: >> >> - Rebased to next-20170621 >> - Split 1/11 into a preparatory patch, a cleanup patch and then >> the meat in 3/14. >> - Handle pseudo-palette for FB_VISUAL_TRUECOLOR. >> - Removed the empty .gamma_get/.gamma_set fb helpers from the >> armada driver that I had somehow managed to ignore but which >> 0day found real quick. >> - Be less judgemental on drivers only providing .gamma_get and >> .gamma_set, but no .load_lut. That's actually a valid thing >> to do if you only need pseudo-palette for FB_VISUAL_TRUECOLOR. >> - Add a comment about colliding bitfields in the nouveau driver. >> - Remove gamma_set/gamma_get declarations from the radeon driver >> (the definitions were removed in v1). > > Ok some nits/questions on the first three, but in principle looks all ok I > think. The driver patches also look good (but I didn't yet carefully > review all the conversion). What we might want to do is entirely remove > driver's reliance on ->gamma_store (mostly amounts to in-lining the > load_lut functions) and only update ->gamma_store after gamma_set returned > successfully. But that's a bit more work. > > Save/restoring it instead might be simpler to fix that bug, but since it's > pre-existing also ok as follow-up. I'm traveling and cannot make progress this week. The merge window is also real close so this series will therefore probably miss it unless something unexpected happens... I'll get back to this for the next cycle, just a heads up. Cheers, peda