Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1359626 > unrolled thread

[PATCH v3 0/2] irqchip/mbigen: fix the IO remap problem in mbigen driver.

Back to article view | Back to linux.kernel

Contents

  [PATCH v3 0/2] irqchip/mbigen: fix the IO remap problem in mbigen driver. MaJun <majun258@huawei.com> - 2016-03-17 09:40 +0100
    Re: [PATCH v3 0/2] irqchip/mbigen: fix the IO remap problem in mbigen  driver. Thomas Gleixner <tglx@linutronix.de> - 2016-03-21 11:40 +0100
      Re: [PATCH v3 0/2] irqchip/mbigen: fix the IO remap problem in mbigen  driver. "majun (F)" <majun258@huawei.com> - 2016-03-22 04:20 +0100

#1359626 — [PATCH v3 0/2] irqchip/mbigen: fix the IO remap problem in mbigen driver.

From: Ma Jun <majun258@huawei.com>

This patch set is used to fix the problem of remap a set of registers
repeatedly in current mbigen driver.

Changes in v3:
--- Change the log to make more detail description about
	the IO remap problem.

Changes in v2:
--- Change the mbigen device node definition as Mark suggested.
--- Change the mbigen driver based on the new mbigen dts structure.

Ma Jun (2):
  irqchip/mbigen:Change the mbigen node definition in dt binding file
  irqchip/mbigen:Change the mbigen driver based on the new mbigen node definition

 .../interrupt-controller/hisilicon,mbigen-v2.txt   |   22 +++++++++++---
 drivers/irqchip/irq-mbigen.c                       |   30 ++++++++++++++------
 2 files changed, 38 insertions(+), 14 deletions(-)

[toc] | [next] | [standalone]

#1361684 — Re: [PATCH v3 0/2] irqchip/mbigen: fix the IO remap problem in mbigen driver.

On Thu, 17 Mar 2016, MaJun wrote:
> This patch set is used to fix the problem of remap a set of registers
> repeatedly in current mbigen driver.
>
>   irqchip/mbigen:Change the mbigen node definition in dt binding file
>   irqchip/mbigen:Change the mbigen driver based on the new mbigen node definition

These subject lines are still horrible as they tell nothing about the nature
of the change. I fixed them up along with the changelogs and applied them to
irq/urgent. Can you spot the difference?

While at it. The config switch for this driver is horrible. It's just in the
middle of the device driver configs. Why is this configurable by the user at
all? It simply should be selected by arm64 or some subarch configuration
there. Please clean that up.

Thanks,

	tglx

[toc] | [prev] | [next] | [standalone]

#1362310 — Re: [PATCH v3 0/2] irqchip/mbigen: fix the IO remap problem in mbigen driver.

在 2016/3/21 18:29, Thomas Gleixner 写道:
> On Thu, 17 Mar 2016, MaJun wrote:
>> This patch set is used to fix the problem of remap a set of registers
>> repeatedly in current mbigen driver.
>>
>>   irqchip/mbigen:Change the mbigen node definition in dt binding file
>>   irqchip/mbigen:Change the mbigen driver based on the new mbigen node definition
> 
> These subject lines are still horrible as they tell nothing about the nature
> of the change. I fixed them up along with the changelogs and applied them to
> irq/urgent. Can you spot the difference?

Yes, after you changing, the subject and change log clearly show why we need to
do this change.
I have learned a lot from you :)

> 
> While at it. The config switch for this driver is horrible. It's just in the
> middle of the device driver configs. Why is this configurable by the user at
> all? It simply should be selected by arm64 or some subarch configuration
> there. Please clean that up.
> 
will do.

Thanks!
MaJun

> Thanks,
> 
> 	tglx
> 
> .
> 

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web