On Tuesday 12 December 2017 12:46 PM, Ladislav Michl wrote:
> Keerthy,
>
> On Tue, Dec 12, 2017 at 11:42:10AM +0530, Keerthy wrote:
>> Remove all the unwanted exports from the driver
>
> I'm adding event capture capability to the pwm-omap driver and so far used
> v4.15-rc3 as codebase.
>
> Intended use is an IR receiver; for that I need to measure pulses width and
> spaces between pulses. So DM timer was setup to generate interupt after
> both TCAR1 and TCAR2 are filled, values are passed to IR decoder and
> TCAR_IT_FLAG is cleared.
>
> Of course, this is just proof of concept and needs to be polished and
> generalized, but to make it at least work I need functions you just
> unexported (plus some new).
>
> Question is whenever we need this level of indirection (omap_dm_timer_ops)
> or plain exports are enough.
The general guidance is not to do plain exports and go via
omap_dm_timer_ops.
>
> Thank you,
> ladis
>
>> Signed-off-by: Keerthy <j-keerthy@xxxxxx>
>> Reviewed-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxxxx>
>> ---
>> Changes in v3:
>>
>> * Added Sebastian's Reviewed-by.
>>
>> Changes in v2:
>>
>> * No code changes in this v2 version. Only enhanced patch
>> statistics for renames.
>>
>> arch/arm/plat-omap/dmtimer.c | 27 ---------------------------
>> 1 file changed, 27 deletions(-)
>>
>> diff --git a/arch/arm/plat-omap/dmtimer.c b/arch/arm/plat-omap/dmtimer.c
>> index d443e48..72565fc 100644
>> --- a/arch/arm/plat-omap/dmtimer.c
>> +++ b/arch/arm/plat-omap/dmtimer.c
>> @@ -302,7 +302,6 @@ struct omap_dm_timer *omap_dm_timer_request(void)
>> {
>> return _omap_dm_timer_request(REQUEST_ANY, NULL);
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_request);
>>
>> struct omap_dm_timer *omap_dm_timer_request_specific(int id)
>> {
>> @@ -315,7 +314,6 @@ struct omap_dm_timer *omap_dm_timer_request_specific(int id)
>>
>> return _omap_dm_timer_request(REQUEST_BY_ID, &id);
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_request_specific);
>>
>> /**
>> * omap_dm_timer_request_by_cap - Request a timer by capability
>> @@ -330,7 +328,6 @@ struct omap_dm_timer *omap_dm_timer_request_by_cap(u32 cap)
>> {
>> return _omap_dm_timer_request(REQUEST_BY_CAP, &cap);
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_request_by_cap);
>>
>> /**
>> * omap_dm_timer_request_by_node - Request a timer by device-tree node
>> @@ -346,7 +343,6 @@ struct omap_dm_timer *omap_dm_timer_request_by_node(struct device_node *np)
>>
>> return _omap_dm_timer_request(REQUEST_BY_NODE, np);
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_request_by_node);
>>
>> int omap_dm_timer_free(struct omap_dm_timer *timer)
>> {
>> @@ -359,7 +355,6 @@ int omap_dm_timer_free(struct omap_dm_timer *timer)
>> timer->reserved = 0;
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_free);
>>
>> void omap_dm_timer_enable(struct omap_dm_timer *timer)
>> {
>> @@ -379,13 +374,11 @@ void omap_dm_timer_enable(struct omap_dm_timer *timer)
>> }
>> }
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_enable);
>>
>> void omap_dm_timer_disable(struct omap_dm_timer *timer)
>> {
>> pm_runtime_put_sync(&timer->pdev->dev);
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_disable);
>>
>> int omap_dm_timer_get_irq(struct omap_dm_timer *timer)
>> {
>> @@ -393,7 +386,6 @@ int omap_dm_timer_get_irq(struct omap_dm_timer *timer)
>> return timer->irq;
>> return -EINVAL;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_get_irq);
>>
>> #if defined(CONFIG_ARCH_OMAP1)
>> #include <mach/hardware.h>
>> @@ -429,7 +421,6 @@ __u32 omap_dm_timer_modify_idlect_mask(__u32 inputmask)
>>
>> return inputmask;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_modify_idlect_mask);
>>
>> #else
>>
>> @@ -439,7 +430,6 @@ struct clk *omap_dm_timer_get_fclk(struct omap_dm_timer *timer)
>> return timer->fclk;
>> return NULL;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_get_fclk);
>>
>> __u32 omap_dm_timer_modify_idlect_mask(__u32 inputmask)
>> {
>> @@ -447,7 +437,6 @@ __u32 omap_dm_timer_modify_idlect_mask(__u32 inputmask)
>>
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_modify_idlect_mask);
>>
>> #endif
>>
>> @@ -461,7 +450,6 @@ int omap_dm_timer_trigger(struct omap_dm_timer *timer)
>> omap_dm_timer_write_reg(timer, OMAP_TIMER_TRIGGER_REG, 0);
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_trigger);
>>
>> int omap_dm_timer_start(struct omap_dm_timer *timer)
>> {
>> @@ -482,7 +470,6 @@ int omap_dm_timer_start(struct omap_dm_timer *timer)
>> timer->context.tclr = l;
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_start);
>>
>> int omap_dm_timer_stop(struct omap_dm_timer *timer)
>> {
>> @@ -506,7 +493,6 @@ int omap_dm_timer_stop(struct omap_dm_timer *timer)
>> omap_dm_timer_disable(timer);
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_stop);
>>
>> int omap_dm_timer_set_source(struct omap_dm_timer *timer, int source)
>> {
>> @@ -569,7 +555,6 @@ int omap_dm_timer_set_source(struct omap_dm_timer *timer, int source)
>>
>> return ret;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_set_source);
>>
>> int omap_dm_timer_set_load(struct omap_dm_timer *timer, int autoreload,
>> unsigned int load)
>> @@ -595,7 +580,6 @@ int omap_dm_timer_set_load(struct omap_dm_timer *timer, int autoreload,
>> omap_dm_timer_disable(timer);
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_set_load);
>>
>> /* Optimized set_load which removes costly spin wait in timer_start */
>> int omap_dm_timer_set_load_start(struct omap_dm_timer *timer, int autoreload,
>> @@ -625,7 +609,6 @@ int omap_dm_timer_set_load_start(struct omap_dm_timer *timer, int autoreload,
>> timer->context.tcrr = load;
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_set_load_start);
>>
>> int omap_dm_timer_set_match(struct omap_dm_timer *timer, int enable,
>> unsigned int match)
>> @@ -650,7 +633,6 @@ int omap_dm_timer_set_match(struct omap_dm_timer *timer, int enable,
>> omap_dm_timer_disable(timer);
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_set_match);
>>
>> int omap_dm_timer_set_pwm(struct omap_dm_timer *timer, int def_on,
>> int toggle, int trigger)
>> @@ -676,7 +658,6 @@ int omap_dm_timer_set_pwm(struct omap_dm_timer *timer, int def_on,
>> omap_dm_timer_disable(timer);
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_set_pwm);
>>
>> int omap_dm_timer_set_prescaler(struct omap_dm_timer *timer, int prescaler)
>> {
>> @@ -699,7 +680,6 @@ int omap_dm_timer_set_prescaler(struct omap_dm_timer *timer, int prescaler)
>> omap_dm_timer_disable(timer);
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_set_prescaler);
>>
>> int omap_dm_timer_set_int_enable(struct omap_dm_timer *timer,
>> unsigned int value)
>> @@ -716,7 +696,6 @@ int omap_dm_timer_set_int_enable(struct omap_dm_timer *timer,
>> omap_dm_timer_disable(timer);
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_set_int_enable);
>>
>> /**
>> * omap_dm_timer_set_int_disable - disable timer interrupts
>> @@ -747,7 +726,6 @@ int omap_dm_timer_set_int_disable(struct omap_dm_timer *timer, u32 mask)
>> omap_dm_timer_disable(timer);
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_set_int_disable);
>>
>> unsigned int omap_dm_timer_read_status(struct omap_dm_timer *timer)
>> {
>> @@ -762,7 +740,6 @@ unsigned int omap_dm_timer_read_status(struct omap_dm_timer *timer)
>>
>> return l;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_read_status);
>>
>> int omap_dm_timer_write_status(struct omap_dm_timer *timer, unsigned int value)
>> {
>> @@ -773,7 +750,6 @@ int omap_dm_timer_write_status(struct omap_dm_timer *timer, unsigned int value)
>>
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_write_status);
>>
>> unsigned int omap_dm_timer_read_counter(struct omap_dm_timer *timer)
>> {
>> @@ -784,7 +760,6 @@ unsigned int omap_dm_timer_read_counter(struct omap_dm_timer *timer)
>>
>> return __omap_dm_timer_read_counter(timer, timer->posted);
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_read_counter);
>>
>> int omap_dm_timer_write_counter(struct omap_dm_timer *timer, unsigned int value)
>> {
>> @@ -799,7 +774,6 @@ int omap_dm_timer_write_counter(struct omap_dm_timer *timer, unsigned int value)
>> timer->context.tcrr = value;
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timer_write_counter);
>>
>> int omap_dm_timers_active(void)
>> {
>> @@ -816,7 +790,6 @@ int omap_dm_timers_active(void)
>> }
>> return 0;
>> }
>> -EXPORT_SYMBOL_GPL(omap_dm_timers_active);
>>
>> static const struct of_device_id omap_timer_match[];
>>
>> --
>> 1.9.1
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
>> the body of a message to majordomo@xxxxxxxxxxxxxxx
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
