* [PATCH 06/11] s390/topology: Remove the unused parent_node() macro [not found] <1501076076-1974-1-git-send-email-douly.fnst@cn.fujitsu.com> @ 2017-07-26 13:34 ` Dou Liyang 2017-07-27 8:19 ` Heiko Carstens 0 siblings, 1 reply; 3+ messages in thread From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw) To: linux-kernel Cc: mpe, Dou Liyang, Martin Schwidefsky, Heiko Carstens, Michael Holzheu, linux-s390 Commit a7be6e5a7f8d ("mm: drop useless local parameters of __register_one_node()") removes the last user of parent_node(). The parent_node() macro in S390 platform is unnecessary. Remove it for cleanup. Reported-by: Michael Ellerman <mpe@ellerman.id.au> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com> Cc: Martin Schwidefsky <schwidefsky@de.ibm.com> Cc: Heiko Carstens <heiko.carstens@de.ibm.com> Cc: Michael Holzheu <holzheu@linux.vnet.ibm.com> Cc: linux-s390@vger.kernel.org --- arch/s390/include/asm/topology.h | 6 ------ 1 file changed, 6 deletions(-) diff --git a/arch/s390/include/asm/topology.h b/arch/s390/include/asm/topology.h index fa1bfce..5222da1 100644 --- a/arch/s390/include/asm/topology.h +++ b/arch/s390/include/asm/topology.h @@ -77,12 +77,6 @@ static inline const struct cpumask *cpumask_of_node(int node) return &node_to_cpumask_map[node]; } -/* - * Returns the number of the node containing node 'node'. This - * architecture is flat, so it is a pretty simple function! - */ -#define parent_node(node) (node) - #define pcibus_to_node(bus) __pcibus_to_node(bus) #define node_distance(a, b) __node_distance(a, b) -- 2.5.5 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 06/11] s390/topology: Remove the unused parent_node() macro 2017-07-26 13:34 ` [PATCH 06/11] s390/topology: Remove the unused parent_node() macro Dou Liyang @ 2017-07-27 8:19 ` Heiko Carstens 2017-08-16 12:56 ` Dou Liyang 0 siblings, 1 reply; 3+ messages in thread From: Heiko Carstens @ 2017-07-27 8:19 UTC (permalink / raw) To: Dou Liyang Cc: linux-kernel, mpe, Martin Schwidefsky, Michael Holzheu, linux-s390 On Wed, Jul 26, 2017 at 09:34:31PM +0800, Dou Liyang wrote: > Commit a7be6e5a7f8d ("mm: drop useless local parameters of > __register_one_node()") removes the last user of parent_node(). > > The parent_node() macro in S390 platform is unnecessary. > > Remove it for cleanup. > > Reported-by: Michael Ellerman <mpe@ellerman.id.au> > Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com> > Cc: Martin Schwidefsky <schwidefsky@de.ibm.com> > Cc: Heiko Carstens <heiko.carstens@de.ibm.com> > Cc: Michael Holzheu <holzheu@linux.vnet.ibm.com> > Cc: linux-s390@vger.kernel.org > --- Are arch maintainers supposed to pick these patches up? In any case: Acked-by: Heiko Carstens <heiko.carstens@de.ibm.com> ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 06/11] s390/topology: Remove the unused parent_node() macro 2017-07-27 8:19 ` Heiko Carstens @ 2017-08-16 12:56 ` Dou Liyang 0 siblings, 0 replies; 3+ messages in thread From: Dou Liyang @ 2017-08-16 12:56 UTC (permalink / raw) To: Heiko Carstens Cc: linux-kernel, mpe, Martin Schwidefsky, Michael Holzheu, linux-s390 Hi Heiko, At 07/27/2017 04:19 PM, Heiko Carstens wrote: > On Wed, Jul 26, 2017 at 09:34:31PM +0800, Dou Liyang wrote: >> Commit a7be6e5a7f8d ("mm: drop useless local parameters of >> __register_one_node()") removes the last user of parent_node(). >> >> The parent_node() macro in S390 platform is unnecessary. >> >> Remove it for cleanup. >> >> Reported-by: Michael Ellerman <mpe@ellerman.id.au> >> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com> >> Cc: Martin Schwidefsky <schwidefsky@de.ibm.com> >> Cc: Heiko Carstens <heiko.carstens@de.ibm.com> >> Cc: Michael Holzheu <holzheu@linux.vnet.ibm.com> >> Cc: linux-s390@vger.kernel.org >> --- > > Are arch maintainers supposed to pick these patches up? > Yes, As Michael said, It seems these patches are merging individually. can you merge this patch in s390 tree? > In any case: Acked-by: Heiko Carstens <heiko.carstens@de.ibm.com> > > Thanks, dou. ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-08-16 12:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1501076076-1974-1-git-send-email-douly.fnst@cn.fujitsu.com>
2017-07-26 13:34 ` [PATCH 06/11] s390/topology: Remove the unused parent_node() macro Dou Liyang
2017-07-27 8:19 ` Heiko Carstens
2017-08-16 12:56 ` Dou Liyang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox