Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753057AbdGCUhG (ORCPT ); Mon, 3 Jul 2017 16:37:06 -0400 Received: from fllnx209.ext.ti.com ([198.47.19.16]:18894 "EHLO fllnx209.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753009AbdGCUhF (ORCPT ); Mon, 3 Jul 2017 16:37:05 -0400 Subject: Re: [PATCH v2] drm: tilcdc: tilcdc_panel: make of_device_ids const. To: Arvind Yadav , , CC: , References: From: Jyri Sarha Message-ID: Date: Mon, 3 Jul 2017 23:36:10 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: en-GB Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1494 Lines: 43 On 06/28/17 08:38, Arvind Yadav wrote: > of_device_ids are not supposed to change at runtime. All functions > working with of_device_ids provided by work with const > of_device_ids. So mark the non-const structs as const. > > File size before: > text data bss dec hex filename > 1531 592 0 2123 84b drivers/gpu/drm/tilcdc/tilcdc_panel.o > > File size after constify: > text data bss dec hex filename > 1915 176 0 2091 82b drivers/gpu/drm/tilcdc/tilcdc_panel.o > > Signed-off-by: Arvind Yadav Thanks, I pick this up and push it in next merge window (for 4.14). However, both this and the bundled tfp410 driver is going away as soon as I have time to do it, but it is unlikely to happen before v4.14. Best regards, Jyri > --- > Changes in v1: > Subject was not correct. > > drivers/gpu/drm/tilcdc/tilcdc_panel.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/tilcdc/tilcdc_panel.c b/drivers/gpu/drm/tilcdc/tilcdc_panel.c > index 28c3e2f..2abe7c4 100644 > --- a/drivers/gpu/drm/tilcdc/tilcdc_panel.c > +++ b/drivers/gpu/drm/tilcdc/tilcdc_panel.c > @@ -419,7 +419,7 @@ static int panel_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id panel_of_match[] = { > +static const struct of_device_id panel_of_match[] = { > { .compatible = "ti,tilcdc,panel", }, > { }, > }; >