lkml.org 
[lkml]   [2018]   [Oct]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.18 008/135] net: dsa: bcm_sf2: Call setup during switch resume
    Date
    4.18-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Florian Fainelli <f.fainelli@gmail.com>

    [ Upstream commit 54baca096386d862d19c10f58f34bf787c6b3cbe ]

    There is no reason to open code what the switch setup function does, in
    fact, because we just issued a switch reset, we would make all the
    register get their default values, including for instance, having unused
    port be enabled again and wasting power and leading to an inappropriate
    switch core clock being selected.

    Fixes: 8cfa94984c9c ("net: dsa: bcm_sf2: add suspend/resume callbacks")
    Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/net/dsa/bcm_sf2.c | 10 +---------
    1 file changed, 1 insertion(+), 9 deletions(-)

    --- a/drivers/net/dsa/bcm_sf2.c
    +++ b/drivers/net/dsa/bcm_sf2.c
    @@ -698,7 +698,6 @@ static int bcm_sf2_sw_suspend(struct dsa
    static int bcm_sf2_sw_resume(struct dsa_switch *ds)
    {
    struct bcm_sf2_priv *priv = bcm_sf2_to_priv(ds);
    - unsigned int port;
    int ret;

    ret = bcm_sf2_sw_rst(priv);
    @@ -710,14 +709,7 @@ static int bcm_sf2_sw_resume(struct dsa_
    if (priv->hw_params.num_gphy == 1)
    bcm_sf2_gphy_enable_set(ds, true);

    - for (port = 0; port < DSA_MAX_PORTS; port++) {
    - if (dsa_is_user_port(ds, port))
    - bcm_sf2_port_setup(ds, port, NULL);
    - else if (dsa_is_cpu_port(ds, port))
    - bcm_sf2_imp_setup(ds, port);
    - }
    -
    - bcm_sf2_enable_acb(ds);
    + ds->ops->setup(ds);

    return 0;
    }

    \
     
     \ /
      Last update: 2018-10-16 19:45    [W:7.528 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site