lkml.org 
[lkml]   [2011]   [May]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 2/2] leds: Add ASIC3 LED support
From
On Tue, May 3, 2011 at 10:25 PM, Paul Parsons <lost.distance@yahoo.com> wrote:
> Hi Philipp,
>
> --- On Tue, 3/5/11, Philipp Zabel <philipp.zabel@gmail.com> wrote:
>
>> About this, see the other patch I sent. This should work
>> with the GPIOs
>> configured as outputs.
> See my response on the other thread.

Will do.

>> Could be moved into struct resource definitions in
>> asic3.c.
>> Get to them with platform_get_resource() in
>> asic3_led_probe().
>> After calculating the bus shift from the resource size, you
>> have
>> everything you need to get rid of the
>> asic3_write_register() export.
> I understand, but my feeling is that implementing a more generic mechanism will add complexity for little obvious gain. If the LED driver ever needs to be more generic then this can be revisited.

Fair enough. Looks like your feeling is right - de-inlining and
exporting asic3_read/write_register costs only ~300 bytes. iomem
resources + ioremap + bus shift calculation in leds-asic3.o are
probably bigger than that.

regards
Philipp


\
 
 \ /
  Last update: 2011-05-04 09:51    [from the cache]
©2003-2011 Jasper Spaans