diff mbox series

[4.9,095/171] mod_devicetable: fix PHY module format

Message ID	20200102220600.388131080@linuxfoundation.org
State	Superseded
Headers	show Return-Path: <SRS0=x1Uk=2X=vger.kernel.org=stable-owner@kernel.org> From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, stable@vger.kernel.org, Andrew Lunn <andrew@lunn.ch>, Russell King <rmk+kernel@armlinux.org.uk>, Florian Fainelli <f.fainelli@gmail.com>, "David S. Miller" <davem@davemloft.net> Subject: [PATCH 4.9 095/171] mod_devicetable: fix PHY module format Date: Thu, 2 Jan 2020 23:07:06 +0100 Message-Id: <20200102220600.388131080@linuxfoundation.org> In-Reply-To: <20200102220546.960200039@linuxfoundation.org> References: <20200102220546.960200039@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org Precedence: bulk
Series	None \| expand [4.9,002/171] btrfs: do not leak reloc root if we fail to read the fs root [4.9,003/171] btrfs: handle ENOENT in btrfs_uuid_tree_iterate [4.9,005/171] ALSA: hda/ca0132 - Keep power on during processing DSP response [4.9,007/171] drm: mst: Fix query_payload ack reply struct [4.9,008/171] drm/bridge: analogix-anx78xx: silence -EPROBE_DEFER warnings [4.9,010/171] spi: Add call to spi_slave_abort() function when spidev driver is released [4.9,013/171] rtlwifi: prevent memory leak in rtl_usb_probe [4.9,015/171] IB/iser: bound protection_sg size by data_sg size [4.9,018/171] media: i2c: ov2659: Fix missing 720p register config [4.9,019/171] media: ov6650: Fix stored frame format not in sync with hardware [4.9,020/171] tools/power/cpupower: Fix initializer override in hsw_ext_cstates [4.9,021/171] usb: renesas_usbhs: add suspend event support in gadget mode [4.9,023/171] regulator: max8907: Fix the usage of uninitialized variable in max8907_regulator_pr... [4.9,025/171] media: cec-funcs.h: add status_req checks [4.9,026/171] samples: pktgen: fix proc_cmd command result check logic [4.9,028/171] media: ti-vpe: vpe: fix a v4l2-compliance warning about invalid pixel format [4.9,031/171] extcon: sm5502: Reset registers during initialization [4.9,034/171] perf report: Add warning when libunwind not compiled in [4.9,035/171] usb: usbfs: Suppress problematic bind and unbind uevents. [4.9,038/171] x86/mce: Lower throttling MCE messages priority to warning [4.9,040/171] rtl8xxxu: fix RTL8723BU connection failure issue after warm reboot [4.9,041/171] x86/ioapic: Prevent inconsistent state when moving an interrupt [4.9,042/171] arm64: psci: Reduce the waiting time for cpu_psci_cpu_kill() [4.9,044/171] pinctrl: sh-pfc: sh7734: Fix duplicate TCLK1_B [4.9,047/171] spi: img-spfi: fix potential double release [4.9,048/171] ALSA: timer: Limit max amount of slave instances [4.9,050/171] perf probe: Fix to find range-only function instance [4.9,053/171] perf probe: Fix to probe an inline function which has no entry pc [4.9,055/171] perf probe: Fix to show inlined function callsite without entry_pc [4.9,057/171] perf probe: Skip overlapped location on searching variables [4.9,058/171] perf probe: Return a better scope DIE if there is no best scope [4.9,059/171] perf probe: Fix to show calling lines of inlined functions [4.9,062/171] ath10k: fix get invalid tx rate for Mesh metric [4.9,064/171] media: si470x-i2c: add missed operations in remove [4.9,066/171] spi: pxa2xx: Add missed security checks [4.9,068/171] s390/disassembler: dont hide instruction addresses [4.9,070/171] cpufreq: Register drivers only after CPU devices have been registered [4.9,072/171] iwlwifi: mvm: fix unaligned read of rx_pkt_status [4.9,073/171] spi: tegra20-slink: add missed clk_unprepare [4.9,076/171] btrfs: dont prematurely free work in run_ordered_work() [4.9,079/171] iwlwifi: check kasprintf() return value [4.9,080/171] fbtft: Make sure string is NULL terminated [4.9,083/171] libtraceevent: Fix memory leakage in copy_filter_type [4.9,084/171] net: phy: initialise phydev speed and duplex sanely [4.9,085/171] btrfs: dont prematurely free work in reada_start_machine_worker() [4.9,089/171] btrfs: do not call synchronize_srcu() in inode_tree_del [4.9,090/171] btrfs: return error pointer from alloc_test_extent_buffer [4.9,093/171] af_packet: set defaule value for tmo [4.9,095/171] mod_devicetable: fix PHY module format [4.9,098/171] net: qlogic: Fix error paths in ql_alloc_large_buffers() [4.9,099/171] net: usb: lan78xx: Fix suspend/resume PHY register access error [4.9,101/171] net: dst: Force 4-byte alignment of dst_metrics [4.9,103/171] USB: EHCI: Do not return -EPIPE when hub is disconnected [4.9,105/171] staging: comedi: gsc_hpdi: check dma_alloc_coherent() return value [4.9,106/171] ext4: fix ext4_empty_dir() for directories with holes [4.9,107/171] ext4: check for directory entries too close to block end [4.9,109/171] mmc: sdhci-of-esdhc: fix P2020 errata handling [4.9,112/171] scsi: lpfc: Fix locking on mailbox command completion [4.9,114/171] iommu/tegra-smmu: Fix page tables in > 4 GiB memory [4.9,117/171] scsi: csiostor: Dont enable IRQs too early [4.9,118/171] powerpc/pseries: Mark accumulate_stolen_time() as notrace [4.9,120/171] dma-debug: add a schedule point in debug_dma_dump_mappings() [4.9,123/171] powerpc/book3s64/hash: Add cond_resched to avoid soft lockup warning [4.9,127/171] clk: qcom: Allow constant ratio freq tables for rcg [4.9,130/171] mfd: mfd-core: Honour Device Trees request to disable a child-device [4.9,131/171] fs/quota: handle overflows of sysctl fs.quota.* and report as unsigned long [4.9,133/171] scsi: ufs: fix potential bug which ends in system hang [4.9,134/171] powerpc/pseries/cmm: Implement release() function for sysfs device [4.9,136/171] scsi: atari_scsi: sun3_scsi: Set sg_tablesize to 1 instead of SG_NONE [4.9,137/171] clk: pxa: fix one of the pxa RTC clocks [4.9,139/171] HID: Improve Windows Precision Touchpad detection. [4.9,141/171] scsi: pm80xx: Fix for SATA device discovery [4.9,144/171] gpio: mpc8xxx: Dont overwrite default irq_set_type callback [4.9,145/171] scripts/kallsyms: fix definitely-lost memory leak [4.9,146/171] cdrom: respect device capabilities during opening action [4.9,149/171] s390/cpum_sf: Check for SDBT and SDB consistency [4.9,151/171] kernel: sysctl: make drop_caches write-only [4.9,152/171] x86/mce: Fix possibly incorrect severity calculation on AMD [4.9,154/171] ALSA: hda - Downgrade error message for single-cmd fallback [4.9,157/171] filldir[64]: remove WARN_ON_ONCE() for bad directory entries [4.9,158/171] netfilter: ebtables: compat: reject all padding in matches/watchers [4.9,161/171] net: icmp: fix data-race in cmp_global_allow() [4.9,163/171] tty/serial: atmel: fix out of range clock divider handling [4.9,165/171] mmc: sdhci: Update the tuning failed messages to pr_debug level [4.9,168/171] tcp/dccp: fix possible race __inet_lookup_established() [4.9,169/171] tcp: do not send empty skb from tcp_write_xmit() [4.9,170/171] gtp: fix wrong condition in gtp_genl_dump_pdp()

Commit Message

Greg Kroah-Hartman Jan. 2, 2020, 10:07 p.m. UTC

From: Russell King <rmk+kernel@armlinux.org.uk>

[ Upstream commit d2ed49cf6c13e379c5819aa5ac20e1f9674ebc89 ]

When a PHY is probed, if the top bit is set, we end up requesting a
module with the string "mdio:-10101110000000100101000101010001" -
the top bit is printed to a signed -1 value. This leads to the module
not being loaded.

Fix the module format string and the macro generating the values for
it to ensure that we only print unsigned types and the top bit is
always 0/1. We correctly end up with
"mdio:10101110000000100101000101010001".

Fixes: 8626d3b43280 ("phylib: Support phy module autoloading")
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Reviewed-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>
---
 include/linux/mod_devicetable.h |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff mbox series

Patch

--- a/include/linux/mod_devicetable.h
+++ b/include/linux/mod_devicetable.h
@@ -502,9 +502,9 @@  struct platform_device_id {
 
 #define MDIO_MODULE_PREFIX	"mdio:"
 
-#define MDIO_ID_FMT "%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d%d"
+#define MDIO_ID_FMT "%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u%u"
 #define MDIO_ID_ARGS(_id) \
-	(_id)>>31, ((_id)>>30) & 1, ((_id)>>29) & 1, ((_id)>>28) & 1,	\
+	((_id)>>31) & 1, ((_id)>>30) & 1, ((_id)>>29) & 1, ((_id)>>28) & 1, \
 	((_id)>>27) & 1, ((_id)>>26) & 1, ((_id)>>25) & 1, ((_id)>>24) & 1, \
 	((_id)>>23) & 1, ((_id)>>22) & 1, ((_id)>>21) & 1, ((_id)>>20) & 1, \
 	((_id)>>19) & 1, ((_id)>>18) & 1, ((_id)>>17) & 1, ((_id)>>16) & 1, \