Re: [PATCH] dsa: return error code upstream
From: Florian Fainelli Date: Thu, 27 Dec 2018 17:06:58 -0800 > Le 12/27/18 à 4:22 PM, David Miller a écrit : >> From: Kangjie Lu >> Date: Tue, 25 Dec 2018 22:08:18 -0600 >> >>> Both bcm_sf2_sw_indir_rw and mdiobus_write_nested could fail, so let's >>> return their error codes upstream. >>> >>> Signed-off-by: Kangjie Lu >> >> Applied with Subject line adjusted as per Florian's feedback. >> > > Thanks, technically bcm_sf2_sw_indir_rw() cannot fail and checking its > return value in the write (op == 0) case will always lead to returning 0 > anyway. In case this ever changes in the future, we should be more > future proof with that change anyway. Ok, thanks for the insight.
Re: [PATCH] dsa: return error code upstream
Le 12/27/18 à 4:22 PM, David Miller a écrit : > From: Kangjie Lu > Date: Tue, 25 Dec 2018 22:08:18 -0600 > >> Both bcm_sf2_sw_indir_rw and mdiobus_write_nested could fail, so let's >> return their error codes upstream. >> >> Signed-off-by: Kangjie Lu > > Applied with Subject line adjusted as per Florian's feedback. > Thanks, technically bcm_sf2_sw_indir_rw() cannot fail and checking its return value in the write (op == 0) case will always lead to returning 0 anyway. In case this ever changes in the future, we should be more future proof with that change anyway. -- Florian
Re: [PATCH] dsa: return error code upstream
From: Kangjie Lu Date: Tue, 25 Dec 2018 22:08:18 -0600 > Both bcm_sf2_sw_indir_rw and mdiobus_write_nested could fail, so let's > return their error codes upstream. > > Signed-off-by: Kangjie Lu Applied with Subject line adjusted as per Florian's feedback.
Re: [PATCH] dsa: return error code upstream
Le 12/25/18 à 8:08 PM, Kangjie Lu a écrit : > Both bcm_sf2_sw_indir_rw and mdiobus_write_nested could fail, so let's > return their error codes upstream. Thanks for your patch, the subject should be: net: dsa: bcm_sf2: Propagate error value from mdio_write to match existing commits done to that file. Thanks! -- Florian