lkml.org 
[lkml]   [2016]   [Mar]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] net: phy: at803x: don't depend on GPIOLIB
    Hello Sebastian,

    On Fri, Mar 18, 2016 at 04:56:21PM +0100, Sebastian Frias wrote:
    > On 03/18/2016 01:54 PM, Uwe Kleine-König wrote:
    > > From a driver perspecitive, it would be nice if devm_gpiod_get_optional
    > > returned NULL iff the respective gpio isn't specified even with
    > > GPIOLIB=n, but this isn't sensible either because it would result in
    > > quite some gpiolib code to not being conditionally compiled on
    > > CONFIG_GPIOLIB any more.
    >
    > Let's say that was the case, what would the PHY code do?

    With reset gpios it might not be that critical, but consider an optional
    enable gpio. (Optional in the sense, that some device have it and others
    don't, e.g. because the pin is pulled into active level by hardware.)

    Now you do:

    gpiod = gpiod_get_optional("enable");

    and if gpiod now is an error pointer, you must assume that you cannot
    operate the device. And even with GPIOLIB=n (and gpiod = ERR_PTR(-ENOSYS))
    you cannot ignore the error. For consistency I'd recommend to do the
    same for reset even though there is a chance to get a working device.

    > What would you think of making at803x_link_change_notify() print a
    > message every time it should do a reset but does not has a way to do it?

    Then this question is obsolete because the device doesn't probe.

    Best regards
    Uwe

    --
    Pengutronix e.K. | Uwe Kleine-König |
    Industrial Linux Solutions | http://www.pengutronix.de/ |

    \
     
     \ /
      Last update: 2016-03-18 21:01    [W:2.253 / U:23.908 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site