Message ID | 20230911094604.14179-1-brgl@bgdev.pl (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | hte: allow building modules with COMPILE_TEST enabled | expand |
On Mon, Sep 11, 2023 at 11:46 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > Allow building all HTE modules with COMPILE_TEST Kconfig option enabled. > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > Acked-by: Linus Walleij <linus.walleij@linaro.org> > --- > This was part of a wider series but since this is independent, I'm sending > it separately. > > drivers/hte/Kconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/hte/Kconfig b/drivers/hte/Kconfig > index cf29e0218bae..083e67492bf2 100644 > --- a/drivers/hte/Kconfig > +++ b/drivers/hte/Kconfig > @@ -16,7 +16,7 @@ if HTE > > config HTE_TEGRA194 > tristate "NVIDIA Tegra194 HTE Support" > - depends on ARCH_TEGRA_194_SOC > + depends on (ARCH_TEGRA_194_SOC || COMPILE_TEST) > help > Enable this option for integrated hardware timestamping engine also > known as generic timestamping engine (GTE) support on NVIDIA Tegra194 > @@ -25,7 +25,7 @@ config HTE_TEGRA194 > > config HTE_TEGRA194_TEST > tristate "NVIDIA Tegra194 HTE Test" > - depends on HTE_TEGRA194 > + depends on (HTE_TEGRA194 || COMPILE_TEST) > help > The NVIDIA Tegra194 GTE test driver demonstrates how to use HTE > framework to timestamp GPIO and LIC IRQ lines. > -- > 2.39.2 > Hi Dipen! Any comments on this, the other patch with a comment update as well as the one addressing usage of gpiochip_find() in HTE drivers? Bart
On 9/25/23 1:37 AM, Bartosz Golaszewski wrote: > On Mon, Sep 11, 2023 at 11:46 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote: >> >> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> >> >> Allow building all HTE modules with COMPILE_TEST Kconfig option enabled. >> >> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> >> Acked-by: Linus Walleij <linus.walleij@linaro.org> >> --- >> This was part of a wider series but since this is independent, I'm sending >> it separately. >> >> drivers/hte/Kconfig | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/hte/Kconfig b/drivers/hte/Kconfig >> index cf29e0218bae..083e67492bf2 100644 >> --- a/drivers/hte/Kconfig >> +++ b/drivers/hte/Kconfig >> @@ -16,7 +16,7 @@ if HTE >> >> config HTE_TEGRA194 >> tristate "NVIDIA Tegra194 HTE Support" >> - depends on ARCH_TEGRA_194_SOC >> + depends on (ARCH_TEGRA_194_SOC || COMPILE_TEST) >> help >> Enable this option for integrated hardware timestamping engine also >> known as generic timestamping engine (GTE) support on NVIDIA Tegra194 >> @@ -25,7 +25,7 @@ config HTE_TEGRA194 >> >> config HTE_TEGRA194_TEST >> tristate "NVIDIA Tegra194 HTE Test" >> - depends on HTE_TEGRA194 >> + depends on (HTE_TEGRA194 || COMPILE_TEST) >> help >> The NVIDIA Tegra194 GTE test driver demonstrates how to use HTE >> framework to timestamp GPIO and LIC IRQ lines. >> -- >> 2.39.2 >> > > Hi Dipen! > > Any comments on this, the other patch with a comment update as well as > the one addressing usage of gpiochip_find() in HTE drivers? Sorry for the late response, This looks good to me. I will review and test rest other patches. > > Bart Acked-by: Dipen Patel <dipenp@nvidia.com>
On Thu, Sep 28, 2023 at 5:31 AM Dipen Patel <dipenp@nvidia.com> wrote: > > On 9/25/23 1:37 AM, Bartosz Golaszewski wrote: > > On Mon, Sep 11, 2023 at 11:46 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > >> > >> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > >> > >> Allow building all HTE modules with COMPILE_TEST Kconfig option enabled. > >> > >> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > >> Acked-by: Linus Walleij <linus.walleij@linaro.org> > >> --- > >> This was part of a wider series but since this is independent, I'm sending > >> it separately. > >> > >> drivers/hte/Kconfig | 4 ++-- > >> 1 file changed, 2 insertions(+), 2 deletions(-) > >> > >> diff --git a/drivers/hte/Kconfig b/drivers/hte/Kconfig > >> index cf29e0218bae..083e67492bf2 100644 > >> --- a/drivers/hte/Kconfig > >> +++ b/drivers/hte/Kconfig > >> @@ -16,7 +16,7 @@ if HTE > >> > >> config HTE_TEGRA194 > >> tristate "NVIDIA Tegra194 HTE Support" > >> - depends on ARCH_TEGRA_194_SOC > >> + depends on (ARCH_TEGRA_194_SOC || COMPILE_TEST) > >> help > >> Enable this option for integrated hardware timestamping engine also > >> known as generic timestamping engine (GTE) support on NVIDIA Tegra194 > >> @@ -25,7 +25,7 @@ config HTE_TEGRA194 > >> > >> config HTE_TEGRA194_TEST > >> tristate "NVIDIA Tegra194 HTE Test" > >> - depends on HTE_TEGRA194 > >> + depends on (HTE_TEGRA194 || COMPILE_TEST) > >> help > >> The NVIDIA Tegra194 GTE test driver demonstrates how to use HTE > >> framework to timestamp GPIO and LIC IRQ lines. > >> -- > >> 2.39.2 > >> > > > > Hi Dipen! > > > > Any comments on this, the other patch with a comment update as well as > > the one addressing usage of gpiochip_find() in HTE drivers? > Sorry for the late response, This looks good to me. I will review and test > rest other patches. > > > > > Bart > Acked-by: Dipen Patel <dipenp@nvidia.com> Do you want me to take it through the GPIO tree or your own? Either way no GPIO patches depend on it. Bart
On 9/27/23 11:53 PM, Bartosz Golaszewski wrote: > On Thu, Sep 28, 2023 at 5:31 AM Dipen Patel <dipenp@nvidia.com> wrote: >> >> On 9/25/23 1:37 AM, Bartosz Golaszewski wrote: >>> On Mon, Sep 11, 2023 at 11:46 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote: >>>> >>>> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> >>>> >>>> Allow building all HTE modules with COMPILE_TEST Kconfig option enabled. >>>> >>>> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> >>>> Acked-by: Linus Walleij <linus.walleij@linaro.org> >>>> --- >>>> This was part of a wider series but since this is independent, I'm sending >>>> it separately. >>>> >>>> drivers/hte/Kconfig | 4 ++-- >>>> 1 file changed, 2 insertions(+), 2 deletions(-) >>>> >>>> diff --git a/drivers/hte/Kconfig b/drivers/hte/Kconfig >>>> index cf29e0218bae..083e67492bf2 100644 >>>> --- a/drivers/hte/Kconfig >>>> +++ b/drivers/hte/Kconfig >>>> @@ -16,7 +16,7 @@ if HTE >>>> >>>> config HTE_TEGRA194 >>>> tristate "NVIDIA Tegra194 HTE Support" >>>> - depends on ARCH_TEGRA_194_SOC >>>> + depends on (ARCH_TEGRA_194_SOC || COMPILE_TEST) >>>> help >>>> Enable this option for integrated hardware timestamping engine also >>>> known as generic timestamping engine (GTE) support on NVIDIA Tegra194 >>>> @@ -25,7 +25,7 @@ config HTE_TEGRA194 >>>> >>>> config HTE_TEGRA194_TEST >>>> tristate "NVIDIA Tegra194 HTE Test" >>>> - depends on HTE_TEGRA194 >>>> + depends on (HTE_TEGRA194 || COMPILE_TEST) >>>> help >>>> The NVIDIA Tegra194 GTE test driver demonstrates how to use HTE >>>> framework to timestamp GPIO and LIC IRQ lines. >>>> -- >>>> 2.39.2 >>>> >>> >>> Hi Dipen! >>> >>> Any comments on this, the other patch with a comment update as well as >>> the one addressing usage of gpiochip_find() in HTE drivers? >> Sorry for the late response, This looks good to me. I will review and test >> rest other patches. >> >>> >>> Bart >> Acked-by: Dipen Patel <dipenp@nvidia.com> > > Do you want me to take it through the GPIO tree or your own? Either > way no GPIO patches depend on it. This patch I should be able to push, assuming it is for 6.7 timeframe. Do other HTE patches depend on any of your GPIO patches? > > Bart
On Thu, Sep 28, 2023 at 7:36 PM Dipen Patel <dipenp@nvidia.com> wrote: > > On 9/27/23 11:53 PM, Bartosz Golaszewski wrote: > > On Thu, Sep 28, 2023 at 5:31 AM Dipen Patel <dipenp@nvidia.com> wrote: > >> > >> On 9/25/23 1:37 AM, Bartosz Golaszewski wrote: > >>> On Mon, Sep 11, 2023 at 11:46 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > >>>> > >>>> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > >>>> > >>>> Allow building all HTE modules with COMPILE_TEST Kconfig option enabled. > >>>> > >>>> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > >>>> Acked-by: Linus Walleij <linus.walleij@linaro.org> > >>>> --- > >>>> This was part of a wider series but since this is independent, I'm sending > >>>> it separately. > >>>> > >>>> drivers/hte/Kconfig | 4 ++-- > >>>> 1 file changed, 2 insertions(+), 2 deletions(-) > >>>> > >>>> diff --git a/drivers/hte/Kconfig b/drivers/hte/Kconfig > >>>> index cf29e0218bae..083e67492bf2 100644 > >>>> --- a/drivers/hte/Kconfig > >>>> +++ b/drivers/hte/Kconfig > >>>> @@ -16,7 +16,7 @@ if HTE > >>>> > >>>> config HTE_TEGRA194 > >>>> tristate "NVIDIA Tegra194 HTE Support" > >>>> - depends on ARCH_TEGRA_194_SOC > >>>> + depends on (ARCH_TEGRA_194_SOC || COMPILE_TEST) > >>>> help > >>>> Enable this option for integrated hardware timestamping engine also > >>>> known as generic timestamping engine (GTE) support on NVIDIA Tegra194 > >>>> @@ -25,7 +25,7 @@ config HTE_TEGRA194 > >>>> > >>>> config HTE_TEGRA194_TEST > >>>> tristate "NVIDIA Tegra194 HTE Test" > >>>> - depends on HTE_TEGRA194 > >>>> + depends on (HTE_TEGRA194 || COMPILE_TEST) > >>>> help > >>>> The NVIDIA Tegra194 GTE test driver demonstrates how to use HTE > >>>> framework to timestamp GPIO and LIC IRQ lines. > >>>> -- > >>>> 2.39.2 > >>>> > >>> > >>> Hi Dipen! > >>> > >>> Any comments on this, the other patch with a comment update as well as > >>> the one addressing usage of gpiochip_find() in HTE drivers? > >> Sorry for the late response, This looks good to me. I will review and test > >> rest other patches. > >> > >>> > >>> Bart > >> Acked-by: Dipen Patel <dipenp@nvidia.com> > > > > Do you want me to take it through the GPIO tree or your own? Either > > way no GPIO patches depend on it. > This patch I should be able to push, assuming it is for 6.7 timeframe. Do other HTE patches depend on > any of your GPIO patches? > > > > > Bart > Only this one needs GPIO changes: https://lore.kernel.org/lkml/20230905185309.131295-15-brgl@bgdev.pl/ Bart
On Thu, Sep 28, 2023 at 5:31 AM Dipen Patel <dipenp@nvidia.com> wrote: > > On 9/25/23 1:37 AM, Bartosz Golaszewski wrote: > > On Mon, Sep 11, 2023 at 11:46 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > >> > >> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > >> > >> Allow building all HTE modules with COMPILE_TEST Kconfig option enabled. > >> > >> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > >> Acked-by: Linus Walleij <linus.walleij@linaro.org> > >> --- > >> This was part of a wider series but since this is independent, I'm sending > >> it separately. > >> > >> drivers/hte/Kconfig | 4 ++-- > >> 1 file changed, 2 insertions(+), 2 deletions(-) > >> > >> diff --git a/drivers/hte/Kconfig b/drivers/hte/Kconfig > >> index cf29e0218bae..083e67492bf2 100644 > >> --- a/drivers/hte/Kconfig > >> +++ b/drivers/hte/Kconfig > >> @@ -16,7 +16,7 @@ if HTE > >> > >> config HTE_TEGRA194 > >> tristate "NVIDIA Tegra194 HTE Support" > >> - depends on ARCH_TEGRA_194_SOC > >> + depends on (ARCH_TEGRA_194_SOC || COMPILE_TEST) > >> help > >> Enable this option for integrated hardware timestamping engine also > >> known as generic timestamping engine (GTE) support on NVIDIA Tegra194 > >> @@ -25,7 +25,7 @@ config HTE_TEGRA194 > >> > >> config HTE_TEGRA194_TEST > >> tristate "NVIDIA Tegra194 HTE Test" > >> - depends on HTE_TEGRA194 > >> + depends on (HTE_TEGRA194 || COMPILE_TEST) > >> help > >> The NVIDIA Tegra194 GTE test driver demonstrates how to use HTE > >> framework to timestamp GPIO and LIC IRQ lines. > >> -- > >> 2.39.2 > >> > > > > Hi Dipen! > > > > Any comments on this, the other patch with a comment update as well as > > the one addressing usage of gpiochip_find() in HTE drivers? > Sorry for the late response, This looks good to me. I will review and test > rest other patches. > > > > > Bart > Acked-by: Dipen Patel <dipenp@nvidia.com> Hey, can you pick this up into your tree as well? Thanks, Bartosz
diff --git a/drivers/hte/Kconfig b/drivers/hte/Kconfig index cf29e0218bae..083e67492bf2 100644 --- a/drivers/hte/Kconfig +++ b/drivers/hte/Kconfig @@ -16,7 +16,7 @@ if HTE config HTE_TEGRA194 tristate "NVIDIA Tegra194 HTE Support" - depends on ARCH_TEGRA_194_SOC + depends on (ARCH_TEGRA_194_SOC || COMPILE_TEST) help Enable this option for integrated hardware timestamping engine also known as generic timestamping engine (GTE) support on NVIDIA Tegra194 @@ -25,7 +25,7 @@ config HTE_TEGRA194 config HTE_TEGRA194_TEST tristate "NVIDIA Tegra194 HTE Test" - depends on HTE_TEGRA194 + depends on (HTE_TEGRA194 || COMPILE_TEST) help The NVIDIA Tegra194 GTE test driver demonstrates how to use HTE framework to timestamp GPIO and LIC IRQ lines.