From: Sergio Paracuellos Date: Fri, 4 May 2018 04:16:37 +0000 (+0200) Subject: staging: ks7010: change parameter types in hostif_power_mgmt_request X-Git-Tag: v5.15~8661^2~346 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0d8be3067f7f43de2eb5efb6edea467640226022;p=platform%2Fkernel%2Flinux-starfive.git staging: ks7010: change parameter types in hostif_power_mgmt_request Parameters for hostif_power_mgmt_request are declared as unsigned long and then are forced to be change to be u32. Also the caller declares explicitly unsigned long parameters just to assign them and pass into the function. Change types for those to be u32 instead so no conversion is needed at all and code gets more clear. Signed-off-by: Sergio Paracuellos Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index 08f95f7..3a6385c 100644 --- a/drivers/staging/ks7010/ks_hostif.c +++ b/drivers/staging/ks7010/ks_hostif.c @@ -1419,8 +1419,7 @@ void hostif_phy_information_request(struct ks_wlan_private *priv) static void hostif_power_mgmt_request(struct ks_wlan_private *priv, - unsigned long mode, unsigned long wake_up, - unsigned long receive_dtims) + u32 mode, u32 wake_up, u32 receive_dtims) { struct hostif_power_mgmt_request *pp; @@ -1428,9 +1427,9 @@ void hostif_power_mgmt_request(struct ks_wlan_private *priv, if (!pp) return; - pp->mode = cpu_to_le32((uint32_t)mode); - pp->wake_up = cpu_to_le32((uint32_t)wake_up); - pp->receive_dtims = cpu_to_le32((uint32_t)receive_dtims); + pp->mode = cpu_to_le32(mode); + pp->wake_up = cpu_to_le32(wake_up); + pp->receive_dtims = cpu_to_le32(receive_dtims); send_request_to_device(priv, pp, hif_align_size(sizeof(*pp))); } @@ -1884,7 +1883,7 @@ spin_unlock: static void hostif_sme_power_mgmt_set(struct ks_wlan_private *priv) { - unsigned long mode, wake_up, receive_dtims; + u32 mode, wake_up, receive_dtims; switch (priv->reg.power_mgmt) { case POWER_MGMT_SAVE1: