Re: [PATCH v2] irqdomain: factorise irq_domain_xlate_onetwocell()

From	Sebastian Frias <sf84@laposte.net>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2] irqdomain: factorise irq_domain_xlate_onetwocell()
Date	2016-08-02 11:40 +0200
Message-ID	<s1El3-2B6-3@gated-at.bofh.it> (permalink)
References	<s1mo9-7tr-7@gated-at.bofh.it> <s1mRb-7Ds-5@gated-at.bofh.it> <s1p2F-MQ-5@gated-at.bofh.it> <s1DoZ-20O-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Thomas,

On 08/02/2016 10:31 AM, Sebastian Frias wrote:
>> So the proper way to do that is to split this into two patches:
>>
>>  #1 Change the existing code to do the masking and explain why it is correct.
>>
>>  #2 Refactor the code and get rid of the duplicated implementation.
> 
> Ok, I can do two patches.

I splitted it in two patches, one to fix it, and another one to refactor.
However, I sent the first one (the one for the fix) as a separate one, instead of as
part of a set of two patches.

I'm resending both as a set of two patches (since the second one requires the first
one), sorry for the inconvenience.

Best regards,

Sebastian

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] irqdomain: factorise irq_domain_xlate_onetwocell() Sebastian Frias <sf84@laposte.net> - 2016-08-01 16:30 +0200
  [PATCH v2] irqdomain: factorise irq_domain_xlate_onetwocell() Sebastian Frias <sf84@laposte.net> - 2016-08-01 17:00 +0200
    Re: [PATCH v2] irqdomain: factorise irq_domain_xlate_onetwocell() Thomas Gleixner <tglx@linutronix.de> - 2016-08-01 19:20 +0200
      Re: [PATCH v2] irqdomain: factorise irq_domain_xlate_onetwocell() Sebastian Frias <sf84@laposte.net> - 2016-08-02 10:40 +0200
        Re: [PATCH v2] irqdomain: factorise irq_domain_xlate_onetwocell() Sebastian Frias <sf84@laposte.net> - 2016-08-02 11:40 +0200

csiph-web