From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751358AbbHRH1K (ORCPT ); Tue, 18 Aug 2015 03:27:10 -0400 Received: from mailout2.w1.samsung.com ([210.118.77.12]:23882 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750855AbbHRH1I (ORCPT ); Tue, 18 Aug 2015 03:27:08 -0400 X-AuditID: cbfec7f4-f79c56d0000012ee-31-55d2de49a16a Message-id: <55D2DE48.1090508@samsung.com> Date: Tue, 18 Aug 2015 09:27:04 +0200 From: Jacek Anaszewski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130804 Thunderbird/17.0.8 MIME-version: 1.0 To: Vaishali Thakkar Cc: linux-leds@vger.kernel.org, Bryan Wu , Linux Kernel Mailing List Subject: Re: [PATCH] leds: leds-fsg: Use devm_led_classdev_register References: <20150815102101.GA8323@localhost> <55D19AD6.8030506@samsung.com> In-reply-to: Content-type: text/plain; charset=UTF-8; format=flowed Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrPLMWRmVeSWpSXmKPExsVy+t/xy7qe9y6FGqxvN7Q4unMik8XlXXPY LLa+Wcdo8XzlNVYHFo+ds+6ye3zeJBfAFMVlk5Kak1mWWqRvl8CV8WXyIuaC79wVd5dfYmlg XM/ZxcjJISFgIjFx7k4WCFtM4sK99WxdjFwcQgJLGSWe921mgXCeMUp8b3nCDlLFK6AlceH/ BTYQm0VAVWLvh34wm03AUOLni9dMILaoQITEn9P7WCHqBSV+TL4HtkFEQFfi68SfYPXMApUS C3avBqsRFnCWaDvbArX5FqPE+YcHwJZxCgRL3D55jAWiwUziUcs6ZghbXmLzmrfMExgFZiHZ MQtJ2SwkZQsYmVcxiqaWJhcUJ6XnGuoVJ+YWl+al6yXn525ihATrlx2Mi49ZHWIU4GBU4uGt yLsUKsSaWFZcmXuIUYKDWUmEVz8VKMSbklhZlVqUH19UmpNafIhRmoNFSZx37q73IUIC6Ykl qdmpqQWpRTBZJg5OqQbGaC7Jykv1+sy7Gt1VbrdKnemN5Mo8Mv1u5m5N2R+tCct+Pgz43BsS srrNZ3P4/F0hGutX3lD8n3ai2lErW9XqgvbEB/uDXby2sGRfNP5tGrx3m8rk+e+Yb8Uqn9r5 fafps8Jptxgv8Jms/eg0Z3O2q8WMyOb84/cnHwg+0fpc5cz7U/KRt1cIKbEUZyQaajEXFScC AMr1OltSAgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Vaishali, On 08/18/2015 07:46 AM, Vaishali Thakkar wrote: > On Mon, Aug 17, 2015 at 2:02 PM, Vaishali Thakkar > wrote: >> >> On 17 Aug 2015 13:57, "Jacek Anaszewski" wrote: >>> >>> Hi Vaishali, >>> >>> Thanks for the patch. Unfortunately it causes build break. >>> Please fix the issues and resubmit. >> >> Oops! I am really very sorry. By mistake I sent a basic version of patch >> instead of final one. Thanks for the review. >> >> I'll send v2 with the fix of that build error. > > Hi Jacek, > > Can you please tell me what are those build errors? > After looking at my patch, I can expect that as I forgot to > remove '.remove = fsg_led_remove' from structure, > it can cause an error. Also, there is one typing mistake in > function name. So, both of these can be a reason for > breaking a build. Those was the errors I experienced too. > Also to be on safe side, I tried to cross compile it for arm > architecture. But to my surprise, it is not compiling properly. > So, it would be good if you can point me to the script you > used for compilation so that I can check it before sending a new > version. I am using arm-linux-gnueabi cross toolchain. config: arch/arm/configs/ixp4xx_defconfig Build command: make ARCH=arm CROSS_COMPILE=arm-linux-gnueabi- all After fixing the errors compilation succeeded. Do you have a hardware to test the modifications against? -- Best Regards, Jacek Anaszewski