[PATCH 01/14] arm: Use %pS printk format for symbols from direct addresses

[PATCH 01/14] arm: Use %pS printk format for symbols from direct addresses

[Helge Deller]

Use the %pS printk format for printing symbols from direct addresses.
On ARM there is actually no difference between %pS and %pF, but for consistency
throughout the kernel fix the wrong usage here too.

Signed-off-by: Helge Deller <deller@gmx.de>
Cc: Russell King <linux@armlinux.org.uk>
Cc: linux-arm-kernel at lists.infradead.org
---
 arch/arm/mm/alignment.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mm/alignment.c b/arch/arm/mm/alignment.c
index 2c96190..20d721f 100644
--- a/arch/arm/mm/alignment.c
+++ b/arch/arm/mm/alignment.c
@@ -133,7 +133,7 @@ static const char *usermode_action[] = {
 static int alignment_proc_show(struct seq_file *m, void *v)
 {
 	seq_printf(m, "User:\t\t%lu\n", ai_user);
-	seq_printf(m, "System:\t\t%lu (%pF)\n", ai_sys, ai_sys_last_pc);
+	seq_printf(m, "System:\t\t%lu (%pS)\n", ai_sys, ai_sys_last_pc);
 	seq_printf(m, "Skipped:\t%lu\n", ai_skipped);
 	seq_printf(m, "Half:\t\t%lu\n", ai_half);
 	seq_printf(m, "Word:\t\t%lu\n", ai_word);
-- 
2.1.0

[PATCH 04/14] ti_sci: Use %pS printk format for direct addresses

[Helge Deller]

Use the %pS printk format for printing symbols from direct addresses.
This is important for the ia64, ppc64 and parisc64 architectures, while on
other architectures there is no difference between %pS and %pF.
Fix it for consistency across the kernel.

Signed-off-by: Helge Deller <deller@gmx.de>
Cc: Nishanth Menon <nm@ti.com>
Cc: Tero Kristo <t-kristo@ti.com>
Cc: Santosh Shilimkar <ssantosh@kernel.org>
Cc: linux-arm-kernel at lists.infradead.org
---
 drivers/firmware/ti_sci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c
index 00cfed3..23b12d9 100644
--- a/drivers/firmware/ti_sci.c
+++ b/drivers/firmware/ti_sci.c
@@ -439,7 +439,7 @@ static inline int ti_sci_do_xfer(struct ti_sci_info *info,
 	/* And we wait for the response. */
 	timeout = msecs_to_jiffies(info->desc->max_rx_timeout_ms);
 	if (!wait_for_completion_timeout(&xfer->done, timeout)) {
-		dev_err(dev, "Mbox timedout in resp(caller: %pF)\n",
+		dev_err(dev, "Mbox timedout in resp(caller: %pS)\n",
 			(void *)_RET_IP_);
 		ret = -ETIMEDOUT;
 	}
-- 
2.1.0

[PATCH 04/14] ti_sci: Use %pS printk format for direct addresses

[Nishanth Menon]

On 20:27-20170906, Helge Deller wrote:
> Use the %pS printk format for printing symbols from direct addresses.
> This is important for the ia64, ppc64 and parisc64 architectures, while on
> other architectures there is no difference between %pS and %pF.
> Fix it for consistency across the kernel.
> 
> Signed-off-by: Helge Deller <deller@gmx.de>
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Tero Kristo <t-kristo@ti.com>
> Cc: Santosh Shilimkar <ssantosh@kernel.org>
> Cc: linux-arm-kernel at lists.infradead.org

OK with me. Thanks for doing the patch.
Acked-by: Nishanth Menon <nm@ti.com>

Santosh, Tero: maybe queue for an rc or later cycle, I think?

[...]

-- 
Regards,
Nishanth Menon

[PATCH 04/14] ti_sci: Use %pS printk format for direct addresses

[Santosh Shilimkar]

On 9/8/2017 4:30 PM, Nishanth Menon wrote:
> On 20:27-20170906, Helge Deller wrote:
>> Use the %pS printk format for printing symbols from direct addresses.
>> This is important for the ia64, ppc64 and parisc64 architectures, while on
>> other architectures there is no difference between %pS and %pF.
>> Fix it for consistency across the kernel.
>>
>> Signed-off-by: Helge Deller <deller@gmx.de>
>> Cc: Nishanth Menon <nm@ti.com>
>> Cc: Tero Kristo <t-kristo@ti.com>
>> Cc: Santosh Shilimkar <ssantosh@kernel.org>
>> Cc: linux-arm-kernel at lists.infradead.org
> 
> OK with me. Thanks for doing the patch.
> Acked-by: Nishanth Menon <nm@ti.com>
> 
> Santosh, Tero: maybe queue for an rc or later cycle, I think?
> 
Doesn't look a rc type fix so can wait for next one.