Skip to content

Commit

Permalink
phy: ralink: phy-mt7621-pci: properly print pointer address
Browse files Browse the repository at this point in the history
[ Upstream commit 652a6a2 ]

The way of printing the pointer address for the 'port_base'
address got into compile warnings on some architectures
[-Wpointer-to-int-cast]. Instead of use '%08x' and cast
to an 'unsigned int' just make use of '%px' and avoid the
cast. To avoid not really needed driver verbosity on normal
behaviour change also from 'dev_info' to 'dev_dbg'.

Fixes: d87da32 ("phy: ralink: Add PHY driver for MT7621 PCIe PHY")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Link: https://lore.kernel.org/r/20210508070930.5290-7-sergio.paracuellos@gmail.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
  • Loading branch information
paraka authored and gregkh committed Jul 14, 2021
1 parent c99410c commit b25bc3e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/phy/ralink/phy-mt7621-pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -272,8 +272,8 @@ static struct phy *mt7621_pcie_phy_of_xlate(struct device *dev,

mt7621_phy->has_dual_port = args->args[0];

dev_info(dev, "PHY for 0x%08x (dual port = %d)\n",
(unsigned int)mt7621_phy->port_base, mt7621_phy->has_dual_port);
dev_dbg(dev, "PHY for 0x%px (dual port = %d)\n",
mt7621_phy->port_base, mt7621_phy->has_dual_port);

return mt7621_phy->phy;
}
Expand Down

0 comments on commit b25bc3e

Please sign in to comment.