Re: [PATCH v8 01/14] sizes.h: Add SZ_1T macro

2022-03-10 Thread Bjorn Helgaas
On Thu, Mar 10, 2022 at 06:09:51PM +, Christophe Leroy wrote:
> 
> 
> Le 10/03/2022 à 17:52, Bjorn Helgaas a écrit :
> > On Wed, Mar 09, 2022 at 06:44:35PM +0100, Christophe Leroy wrote:
> >> Today drivers/pci/controller/pci-xgene.c defines SZ_1T
> >>
> >> Move it into linux/sizes.h so that it can be re-used elsewhere.
> >>
> >> Link: 
> >> https://lore.kernel.org/r/575cb7164cf124c75df7cb9242ea7374733942bf.1642752946.git.christophe.le...@csgroup.eu
> >> Signed-off-by: Christophe Leroy 
> >> Signed-off-by: Lorenzo Pieralisi 
> >> Reviewed-by: Krzysztof Wilczyński 
> >> Acked-by: Bjorn Helgaas 
> >> Cc: Toan Le 
> >> Cc: linux-...@vger.kernel.org
> >> ---
> >>   This patch is already in linux-next but not in Linus' tree yet
> > 
> > What would you like me to do about this?  It's in linux-next, which
> > means it will go to Linus' tree during the next merge window.
> > 
> > But this is 01/14; are there other patches that I should be looking
> > at?  Do I need to coordinate this with other patches that depend on
> > it?
> 
> Yes sorry I should have said it. Patch 14/14 depends on it.
> 
> Don't know yet what's the merge strategy for this series, there as not 
> been any changes since v6 mid December and core parts are acked/reviewed 
> so I would be happy if at least core mm parts could go this cycle. I 
> sent a question to Michael and Andrew about it.

Since PCI is only minimally affected in this series, it would probably
make more sense for it to be merged along with the rest of the series
via a non-PCI tree.

It has my ack, so this can certainly happen.  If it does, I can easily
drop it from the PCI tree.

Bjorn


Re: [PATCH v8 01/14] sizes.h: Add SZ_1T macro

2022-03-10 Thread Christophe Leroy


Le 10/03/2022 à 17:52, Bjorn Helgaas a écrit :
> On Wed, Mar 09, 2022 at 06:44:35PM +0100, Christophe Leroy wrote:
>> Today drivers/pci/controller/pci-xgene.c defines SZ_1T
>>
>> Move it into linux/sizes.h so that it can be re-used elsewhere.
>>
>> Link: 
>> https://lore.kernel.org/r/575cb7164cf124c75df7cb9242ea7374733942bf.1642752946.git.christophe.le...@csgroup.eu
>> Signed-off-by: Christophe Leroy 
>> Signed-off-by: Lorenzo Pieralisi 
>> Reviewed-by: Krzysztof Wilczyński 
>> Acked-by: Bjorn Helgaas 
>> Cc: Toan Le 
>> Cc: linux-...@vger.kernel.org
>> ---
>>   This patch is already in linux-next but not in Linus' tree yet
> 
> What would you like me to do about this?  It's in linux-next, which
> means it will go to Linus' tree during the next merge window.
> 
> But this is 01/14; are there other patches that I should be looking
> at?  Do I need to coordinate this with other patches that depend on
> it?

Yes sorry I should have said it. Patch 14/14 depends on it.

Don't know yet what's the merge strategy for this series, there as not 
been any changes since v6 mid December and core parts are acked/reviewed 
so I would be happy if at least core mm parts could go this cycle. I 
sent a question to Michael and Andrew about it.

Christophe

Re: [PATCH v8 01/14] sizes.h: Add SZ_1T macro

2022-03-10 Thread Bjorn Helgaas
On Wed, Mar 09, 2022 at 06:44:35PM +0100, Christophe Leroy wrote:
> Today drivers/pci/controller/pci-xgene.c defines SZ_1T
> 
> Move it into linux/sizes.h so that it can be re-used elsewhere.
> 
> Link: 
> https://lore.kernel.org/r/575cb7164cf124c75df7cb9242ea7374733942bf.1642752946.git.christophe.le...@csgroup.eu
> Signed-off-by: Christophe Leroy 
> Signed-off-by: Lorenzo Pieralisi 
> Reviewed-by: Krzysztof Wilczyński 
> Acked-by: Bjorn Helgaas 
> Cc: Toan Le 
> Cc: linux-...@vger.kernel.org
> ---
>  This patch is already in linux-next but not in Linus' tree yet

What would you like me to do about this?  It's in linux-next, which
means it will go to Linus' tree during the next merge window.

But this is 01/14; are there other patches that I should be looking
at?  Do I need to coordinate this with other patches that depend on
it?

>  drivers/pci/controller/pci-xgene.c | 1 -
>  include/linux/sizes.h  | 2 ++
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/pci/controller/pci-xgene.c 
> b/drivers/pci/controller/pci-xgene.c
> index 0d5acbfc7143..77c1fe7e11f9 100644
> --- a/drivers/pci/controller/pci-xgene.c
> +++ b/drivers/pci/controller/pci-xgene.c
> @@ -49,7 +49,6 @@
>  #define EN_REG   0x0001
>  #define OB_LO_IO 0x0002
>  #define XGENE_PCIE_DEVICEID  0xE004
> -#define SZ_1T(SZ_1G*1024ULL)
>  #define PIPE_PHY_RATE_RD(src)((0xc000 & (u32)(src)) >> 0xe)
>  
>  #define XGENE_V1_PCI_EXP_CAP 0x40
> diff --git a/include/linux/sizes.h b/include/linux/sizes.h
> index 1ac79bcee2bb..84aa448d8bb3 100644
> --- a/include/linux/sizes.h
> +++ b/include/linux/sizes.h
> @@ -47,6 +47,8 @@
>  #define SZ_8G_AC(0x2, ULL)
>  #define SZ_16G   _AC(0x4, ULL)
>  #define SZ_32G   _AC(0x8, ULL)
> +
> +#define SZ_1T_AC(0x100, ULL)
>  #define SZ_64T   _AC(0x4000, ULL)
>  
>  #endif /* __LINUX_SIZES_H__ */
> -- 
> 2.34.1
> 
> 
> ___
> linux-arm-kernel mailing list
> linux-arm-ker...@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel