* [PATCH] dt-bindings: versal: Add new PM_DEV_I2C_PMC macro @ 2021-11-30 12:56 Michal Simek 2021-11-30 16:56 ` Tom Rini 2022-01-05 9:28 ` Michal Simek 0 siblings, 2 replies; 6+ messages in thread From: Michal Simek @ 2021-11-30 12:56 UTC (permalink / raw) To: u-boot, git; +Cc: Sandeep Gundlupet Raju From: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> Add new macro for PMC I2C power domain. Signed-off-by: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> Signed-off-by: Michal Simek <michal.simek@xilinx.com> --- include/dt-bindings/power/xlnx-versal-power.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/include/dt-bindings/power/xlnx-versal-power.h b/include/dt-bindings/power/xlnx-versal-power.h index 1b75175edce5..4a727754ad02 100644 --- a/include/dt-bindings/power/xlnx-versal-power.h +++ b/include/dt-bindings/power/xlnx-versal-power.h @@ -1,6 +1,6 @@ /* SPDX-License-Identifier: GPL-2.0 */ /* - * Copyright (C) 2019 - 2020 Xilinx, Inc. + * Copyright (C) 2019 - 2021 Xilinx, Inc. */ #ifndef _DT_BINDINGS_VERSAL_POWER_H @@ -26,6 +26,7 @@ #define PM_DEV_OSPI (0x1822402aU) #define PM_DEV_QSPI (0x1822402bU) #define PM_DEV_GPIO_PMC (0x1822402cU) +#define PM_DEV_I2C_PMC (0x1822402dU) #define PM_DEV_SDIO_0 (0x1822402eU) #define PM_DEV_SDIO_1 (0x1822402fU) #define PM_DEV_RTC (0x18224034U) -- 2.33.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: versal: Add new PM_DEV_I2C_PMC macro 2021-11-30 12:56 [PATCH] dt-bindings: versal: Add new PM_DEV_I2C_PMC macro Michal Simek @ 2021-11-30 16:56 ` Tom Rini 2021-11-30 17:48 ` Michal Simek 2022-01-05 9:28 ` Michal Simek 1 sibling, 1 reply; 6+ messages in thread From: Tom Rini @ 2021-11-30 16:56 UTC (permalink / raw) To: Michal Simek; +Cc: u-boot, git, Sandeep Gundlupet Raju [-- Attachment #1: Type: text/plain, Size: 1315 bytes --] On Tue, Nov 30, 2021 at 01:56:58PM +0100, Michal Simek wrote: > From: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> > > Add new macro for PMC I2C power domain. > > Signed-off-by: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> > Signed-off-by: Michal Simek <michal.simek@xilinx.com> > --- > > include/dt-bindings/power/xlnx-versal-power.h | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/include/dt-bindings/power/xlnx-versal-power.h b/include/dt-bindings/power/xlnx-versal-power.h > index 1b75175edce5..4a727754ad02 100644 > --- a/include/dt-bindings/power/xlnx-versal-power.h > +++ b/include/dt-bindings/power/xlnx-versal-power.h > @@ -1,6 +1,6 @@ > /* SPDX-License-Identifier: GPL-2.0 */ > /* > - * Copyright (C) 2019 - 2020 Xilinx, Inc. > + * Copyright (C) 2019 - 2021 Xilinx, Inc. > */ > > #ifndef _DT_BINDINGS_VERSAL_POWER_H > @@ -26,6 +26,7 @@ > #define PM_DEV_OSPI (0x1822402aU) > #define PM_DEV_QSPI (0x1822402bU) > #define PM_DEV_GPIO_PMC (0x1822402cU) > +#define PM_DEV_I2C_PMC (0x1822402dU) > #define PM_DEV_SDIO_0 (0x1822402eU) > #define PM_DEV_SDIO_1 (0x1822402fU) > #define PM_DEV_RTC (0x18224034U) So, where is this binding upstreamed, is it U-Boot specific? -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: versal: Add new PM_DEV_I2C_PMC macro 2021-11-30 16:56 ` Tom Rini @ 2021-11-30 17:48 ` Michal Simek 2021-11-30 17:52 ` Tom Rini 0 siblings, 1 reply; 6+ messages in thread From: Michal Simek @ 2021-11-30 17:48 UTC (permalink / raw) To: Tom Rini, Michal Simek; +Cc: u-boot, git, Sandeep Gundlupet Raju On 11/30/21 17:56, Tom Rini wrote: > On Tue, Nov 30, 2021 at 01:56:58PM +0100, Michal Simek wrote: > >> From: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> >> >> Add new macro for PMC I2C power domain. >> >> Signed-off-by: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> >> Signed-off-by: Michal Simek <michal.simek@xilinx.com> >> --- >> >> include/dt-bindings/power/xlnx-versal-power.h | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/include/dt-bindings/power/xlnx-versal-power.h b/include/dt-bindings/power/xlnx-versal-power.h >> index 1b75175edce5..4a727754ad02 100644 >> --- a/include/dt-bindings/power/xlnx-versal-power.h >> +++ b/include/dt-bindings/power/xlnx-versal-power.h >> @@ -1,6 +1,6 @@ >> /* SPDX-License-Identifier: GPL-2.0 */ >> /* >> - * Copyright (C) 2019 - 2020 Xilinx, Inc. >> + * Copyright (C) 2019 - 2021 Xilinx, Inc. >> */ >> >> #ifndef _DT_BINDINGS_VERSAL_POWER_H >> @@ -26,6 +26,7 @@ >> #define PM_DEV_OSPI (0x1822402aU) >> #define PM_DEV_QSPI (0x1822402bU) >> #define PM_DEV_GPIO_PMC (0x1822402cU) >> +#define PM_DEV_I2C_PMC (0x1822402dU) >> #define PM_DEV_SDIO_0 (0x1822402eU) >> #define PM_DEV_SDIO_1 (0x1822402fU) >> #define PM_DEV_RTC (0x18224034U) > > So, where is this binding upstreamed, is it U-Boot specific? I didn't push this file to upstream Linux because didn't push any versal specific dtsi file there yet. But this file is just IDs which are used by power domain driver which is upstream already. Here you can see support for xlnx,versal-firmware driver. https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/firmware/xilinx/zynqmp.c?h=v5.16-rc3#n1493 Here you can see full structure used on versal. https://github.com/Xilinx/u-boot-xlnx/blob/master/arch/arm/dts/versal-clk.dtsi#L153 Thanks, Michal ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: versal: Add new PM_DEV_I2C_PMC macro 2021-11-30 17:48 ` Michal Simek @ 2021-11-30 17:52 ` Tom Rini 2021-11-30 17:55 ` Michal Simek 0 siblings, 1 reply; 6+ messages in thread From: Tom Rini @ 2021-11-30 17:52 UTC (permalink / raw) To: Michal Simek; +Cc: u-boot, git, Sandeep Gundlupet Raju [-- Attachment #1: Type: text/plain, Size: 2258 bytes --] On Tue, Nov 30, 2021 at 06:48:02PM +0100, Michal Simek wrote: > > > On 11/30/21 17:56, Tom Rini wrote: > > On Tue, Nov 30, 2021 at 01:56:58PM +0100, Michal Simek wrote: > > > > > From: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> > > > > > > Add new macro for PMC I2C power domain. > > > > > > Signed-off-by: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> > > > Signed-off-by: Michal Simek <michal.simek@xilinx.com> > > > --- > > > > > > include/dt-bindings/power/xlnx-versal-power.h | 3 ++- > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > diff --git a/include/dt-bindings/power/xlnx-versal-power.h b/include/dt-bindings/power/xlnx-versal-power.h > > > index 1b75175edce5..4a727754ad02 100644 > > > --- a/include/dt-bindings/power/xlnx-versal-power.h > > > +++ b/include/dt-bindings/power/xlnx-versal-power.h > > > @@ -1,6 +1,6 @@ > > > /* SPDX-License-Identifier: GPL-2.0 */ > > > /* > > > - * Copyright (C) 2019 - 2020 Xilinx, Inc. > > > + * Copyright (C) 2019 - 2021 Xilinx, Inc. > > > */ > > > #ifndef _DT_BINDINGS_VERSAL_POWER_H > > > @@ -26,6 +26,7 @@ > > > #define PM_DEV_OSPI (0x1822402aU) > > > #define PM_DEV_QSPI (0x1822402bU) > > > #define PM_DEV_GPIO_PMC (0x1822402cU) > > > +#define PM_DEV_I2C_PMC (0x1822402dU) > > > #define PM_DEV_SDIO_0 (0x1822402eU) > > > #define PM_DEV_SDIO_1 (0x1822402fU) > > > #define PM_DEV_RTC (0x18224034U) > > > > So, where is this binding upstreamed, is it U-Boot specific? > > I didn't push this file to upstream Linux because didn't push any versal > specific dtsi file there yet. But this file is just IDs which are used by > power domain driver which is upstream already. > > Here you can see support for xlnx,versal-firmware driver. > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/firmware/xilinx/zynqmp.c?h=v5.16-rc3#n1493 > > Here you can see full structure used on versal. > > https://github.com/Xilinx/u-boot-xlnx/blob/master/arch/arm/dts/versal-clk.dtsi#L153 OK. Yeah, I don't have a problem with this patch as-is, but please do make sure it's on the TODO list to upstream this part as well. Thanks! -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: versal: Add new PM_DEV_I2C_PMC macro 2021-11-30 17:52 ` Tom Rini @ 2021-11-30 17:55 ` Michal Simek 0 siblings, 0 replies; 6+ messages in thread From: Michal Simek @ 2021-11-30 17:55 UTC (permalink / raw) To: Tom Rini, Michal Simek; +Cc: u-boot, git, Sandeep Gundlupet Raju On 11/30/21 18:52, Tom Rini wrote: > On Tue, Nov 30, 2021 at 06:48:02PM +0100, Michal Simek wrote: >> >> >> On 11/30/21 17:56, Tom Rini wrote: >>> On Tue, Nov 30, 2021 at 01:56:58PM +0100, Michal Simek wrote: >>> >>>> From: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> >>>> >>>> Add new macro for PMC I2C power domain. >>>> >>>> Signed-off-by: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> >>>> Signed-off-by: Michal Simek <michal.simek@xilinx.com> >>>> --- >>>> >>>> include/dt-bindings/power/xlnx-versal-power.h | 3 ++- >>>> 1 file changed, 2 insertions(+), 1 deletion(-) >>>> >>>> diff --git a/include/dt-bindings/power/xlnx-versal-power.h b/include/dt-bindings/power/xlnx-versal-power.h >>>> index 1b75175edce5..4a727754ad02 100644 >>>> --- a/include/dt-bindings/power/xlnx-versal-power.h >>>> +++ b/include/dt-bindings/power/xlnx-versal-power.h >>>> @@ -1,6 +1,6 @@ >>>> /* SPDX-License-Identifier: GPL-2.0 */ >>>> /* >>>> - * Copyright (C) 2019 - 2020 Xilinx, Inc. >>>> + * Copyright (C) 2019 - 2021 Xilinx, Inc. >>>> */ >>>> #ifndef _DT_BINDINGS_VERSAL_POWER_H >>>> @@ -26,6 +26,7 @@ >>>> #define PM_DEV_OSPI (0x1822402aU) >>>> #define PM_DEV_QSPI (0x1822402bU) >>>> #define PM_DEV_GPIO_PMC (0x1822402cU) >>>> +#define PM_DEV_I2C_PMC (0x1822402dU) >>>> #define PM_DEV_SDIO_0 (0x1822402eU) >>>> #define PM_DEV_SDIO_1 (0x1822402fU) >>>> #define PM_DEV_RTC (0x18224034U) >>> >>> So, where is this binding upstreamed, is it U-Boot specific? >> >> I didn't push this file to upstream Linux because didn't push any versal >> specific dtsi file there yet. But this file is just IDs which are used by >> power domain driver which is upstream already. >> >> Here you can see support for xlnx,versal-firmware driver. >> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/firmware/xilinx/zynqmp.c?h=v5.16-rc3#n1493 >> >> Here you can see full structure used on versal. >> >> https://github.com/Xilinx/u-boot-xlnx/blob/master/arch/arm/dts/versal-clk.dtsi#L153 > > OK. Yeah, I don't have a problem with this patch as-is, but please do > make sure it's on the TODO list to upstream this part as well. Thanks! We are trying to upstream everything based on time we have. Thanks, Michal ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: versal: Add new PM_DEV_I2C_PMC macro 2021-11-30 12:56 [PATCH] dt-bindings: versal: Add new PM_DEV_I2C_PMC macro Michal Simek 2021-11-30 16:56 ` Tom Rini @ 2022-01-05 9:28 ` Michal Simek 1 sibling, 0 replies; 6+ messages in thread From: Michal Simek @ 2022-01-05 9:28 UTC (permalink / raw) To: U-Boot, git; +Cc: Sandeep Gundlupet Raju út 30. 11. 2021 v 13:57 odesílatel Michal Simek <michal.simek@xilinx.com> napsal: > > From: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> > > Add new macro for PMC I2C power domain. > > Signed-off-by: Sandeep Gundlupet Raju <sandeep.gundlupet-raju@xilinx.com> > Signed-off-by: Michal Simek <michal.simek@xilinx.com> > --- > > include/dt-bindings/power/xlnx-versal-power.h | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/include/dt-bindings/power/xlnx-versal-power.h b/include/dt-bindings/power/xlnx-versal-power.h > index 1b75175edce5..4a727754ad02 100644 > --- a/include/dt-bindings/power/xlnx-versal-power.h > +++ b/include/dt-bindings/power/xlnx-versal-power.h > @@ -1,6 +1,6 @@ > /* SPDX-License-Identifier: GPL-2.0 */ > /* > - * Copyright (C) 2019 - 2020 Xilinx, Inc. > + * Copyright (C) 2019 - 2021 Xilinx, Inc. > */ > > #ifndef _DT_BINDINGS_VERSAL_POWER_H > @@ -26,6 +26,7 @@ > #define PM_DEV_OSPI (0x1822402aU) > #define PM_DEV_QSPI (0x1822402bU) > #define PM_DEV_GPIO_PMC (0x1822402cU) > +#define PM_DEV_I2C_PMC (0x1822402dU) > #define PM_DEV_SDIO_0 (0x1822402eU) > #define PM_DEV_SDIO_1 (0x1822402fU) > #define PM_DEV_RTC (0x18224034U) > -- > 2.33.1 > Applied. M -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Xilinx Microblaze Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-01-05 9:28 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-11-30 12:56 [PATCH] dt-bindings: versal: Add new PM_DEV_I2C_PMC macro Michal Simek 2021-11-30 16:56 ` Tom Rini 2021-11-30 17:48 ` Michal Simek 2021-11-30 17:52 ` Tom Rini 2021-11-30 17:55 ` Michal Simek 2022-01-05 9:28 ` Michal Simek
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.