[0/5] riscv: minor fixes and cleanups

Message

Johan Hovold Jan. 18, 2019, 2:03 p.m. UTC

This series fix up some minor issues found through inspection.

Note that the fourth patch changes which cpu (hart) devicetree nodes
are enabled by following the Linux convention of considering nodes
without a status property as enabled.

These patches are against the riscv-next (and fixes) branch with Andreas
node-reference fix applied:

	https://lore.kernel.org/lkml/[e-mail]/

and have been tested using QEMU.

Johan


Johan Hovold (5):
  riscv: add missing newlines to printk messages
  riscv: use pr_info and friends
  riscv: fix riscv_of_processor_hartid() comment
  riscv: treat cpu devicetree nodes without status as enabled
  riscv: use for_each_of_cpu_node iterator

 arch/riscv/kernel/cpu.c        | 28 ++++++++++++----------------
 arch/riscv/kernel/cpufeature.c | 13 +++++++------
 arch/riscv/kernel/ftrace.c     |  2 +-
 arch/riscv/kernel/setup.c      |  6 +++---
 arch/riscv/kernel/smpboot.c    |  4 ++--
 5 files changed, 25 insertions(+), 28 deletions(-)

Comments

Johan Hovold Feb. 11, 2019, 9:34 a.m. UTC | #1

On Fri, Jan 18, 2019 at 03:03:03PM +0100, Johan Hovold wrote:
> This series fix up some minor issues found through inspection.
> 
> Note that the fourth patch changes which cpu (hart) devicetree nodes
> are enabled by following the Linux convention of considering nodes
> without a status property as enabled.
> 
> These patches are against the riscv-next (and fixes) branch with Andreas
> node-reference fix applied:
> 
> 	https://lore.kernel.org/lkml/[e-mail]/
> 
> and have been tested using QEMU.

> Johan Hovold (5):
>   riscv: add missing newlines to printk messages
>   riscv: use pr_info and friends
>   riscv: fix riscv_of_processor_hartid() comment
>   riscv: treat cpu devicetree nodes without status as enabled
>   riscv: use for_each_of_cpu_node iterator

Are these still in your queue, Palmer?

Note that the node-reference dependency mentioned above is now in
Linus's tree.

Johan

Palmer Dabbelt Feb. 11, 2019, 7:59 p.m. UTC | #2

On Mon, 11 Feb 2019 01:34:06 PST (-0800), [e-mail] wrote:
> On Fri, Jan 18, 2019 at 03:03:03PM +0100, Johan Hovold wrote:
>> This series fix up some minor issues found through inspection.
>>
>> Note that the fourth patch changes which cpu (hart) devicetree nodes
>> are enabled by following the Linux convention of considering nodes
>> without a status property as enabled.
>>
>> These patches are against the riscv-next (and fixes) branch with Andreas
>> node-reference fix applied:
>>
>> 	https://lore.kernel.org/lkml/[e-mail]/
>>
>> and have been tested using QEMU.
>
>> Johan Hovold (5):
>>   riscv: add missing newlines to printk messages
>>   riscv: use pr_info and friends
>>   riscv: fix riscv_of_processor_hartid() comment
>>   riscv: treat cpu devicetree nodes without status as enabled
>>   riscv: use for_each_of_cpu_node iterator
>
> Are these still in your queue, Palmer?
>
> Note that the node-reference dependency mentioned above is now in
> Linus's tree.

They're still in my queue.  With any luck I'll have time tonight to go through 
by Linux patch review backlog.