From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DE9BEC10F11 for ; Wed, 10 Apr 2019 20:07:53 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id B4FC82075B for ; Wed, 10 Apr 2019 20:07:53 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726756AbfDJUHw (ORCPT ); Wed, 10 Apr 2019 16:07:52 -0400 Received: from muru.com ([72.249.23.125]:45250 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726181AbfDJUHw (ORCPT ); Wed, 10 Apr 2019 16:07:52 -0400 Received: from atomide.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id AEDE580B6; Wed, 10 Apr 2019 20:08:06 +0000 (UTC) Date: Wed, 10 Apr 2019 13:07:48 -0700 From: Tony Lindgren To: Daniel Lezcano Cc: Ladislav Michl , Nathan Chancellor , Thomas Gleixner , linux-kernel@vger.kernel.org, YueHaibing , Keerthy Subject: Re: [PATCH] clocksource/drivers/timer-ti-dm: Remove omap_dm_timer_set_load_start Message-ID: <20190410200748.GN2839@atomide.com> References: <20190327050126.12064-1-natechancellor@gmail.com> <20190327081142.GA11034@lenoch> <20190404141730.GE49658@atomide.com> <6defcea8-1e94-14ac-f9c5-e14c61ea4d99@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <6defcea8-1e94-14ac-f9c5-e14c61ea4d99@linaro.org> User-Agent: Mutt/1.11.4 (2019-03-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, * Daniel Lezcano [190410 17:02]: > can you ask for an acked-by before pulling a patch in your tree? I certainly do ask and wait for acks where possible :) Note that I have not applied this patch. I just added Keerthy to Cc on this thread so maybe you misread the message earlier. My comment "seems like no other takers" was for Ladislav regarding somebody picking up his earlier work, not for picking up this patch :) In any case, I've been waiting for you guys to pick up the patch, here's my ack if you're waiting for it: Acked-by: Tony Lindgren > On 04/04/2019 16:17, Tony Lindgren wrote: > > * Ladislav Michl [190327 08:12]: > >> Hello Nathan, > >> > >> On Tue, Mar 26, 2019 at 10:01:27PM -0700, Nathan Chancellor wrote: > >>> Commit 008258d995a6 ("clocksource/drivers/timer-ti-dm: Make > >>> omap_dm_timer_set_load_start() static") made omap_dm_time_set_load_start > >>> static because its prototype was not defined in a header. Unfortunately, > >>> this causes a build warning on multi_v7_defconfig because this function > >>> is not used anywhere in this translation unit: > >>> > >>> drivers/clocksource/timer-ti-dm.c:589:12: error: unused function > >>> 'omap_dm_timer_set_load_start' [-Werror,-Wunused-function] > >>> > >>> In fact, omap_dm_timer_set_load_start hasn't been used anywhere since > >>> commit f190be7f39a5 ("staging: tidspbridge: remove driver") and the > >>> prototype was removed in commit 592ea6bd1fad ("clocksource: timer-ti-dm: > >>> Make unexported functions static"), which is probably where this should > >>> have happened. > >> > >> Alternatively you might want to look at "clocksource: timer-ti-dm: Add event > >> capture": https://patchwork.kernel.org/patch/10237217/ (it makes use of > >> function being removed here). It is a part of an attempt to add event capture > >> for OMAP. Of course I would like this functionality to be implemented, but > >> as I do not have a time to continue, I cannot really object removing this > >> function. > >> > >> Just in case you'd be interested in finishing this task ;-) > > > > Well seems like no other takers :) We can always find the missing > > function in git history when needed, so I suggest we apply this. > > > > Adding Keerthy to Cc as he just posted a similar patch. > > > > Regards, > > > > Tony > > > > > -- > Linaro.org │ Open source software for ARM SoCs > > Follow Linaro: Facebook | > Twitter | > Blog >