[lkml]   [2011]   [May]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 2/2] leds: Add ASIC3 LED support
    On Tue, May 3, 2011 at 10:25 PM, Paul Parsons <> wrote:
    > Hi Philipp,
    > --- On Tue, 3/5/11, Philipp Zabel <> 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.


     \ /
      Last update: 2011-05-04 09:51    [W:0.021 / U:34.868 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site