mbox series

[0/2] thermal: renesas: Group drivers and update dependencies

Message ID 20240506154011.344324-1-niklas.soderlund+renesas@ragnatech.se (mailing list archive)
Headers show
Series thermal: renesas: Group drivers and update dependencies | expand

Message

Niklas Söderlund May 6, 2024, 3:40 p.m. UTC
Hello,

This small series moves and groups all renesas thermal drivers in a 
vendor directory as done by some other vendors. The idea is to make it 
easier to spot issues they are worked on by the same set of people.

Patch 2/2 fix such an issue where one dependency was missed for one the 
drivers.

@Biju: There is no entry for rzg2l_thermal in MAINTAINERS. Should we 
update that with an entry for you?

Niklas Söderlund (2):
  thermal: renesas: Group all renesas thermal drivers together
  thermal: renesas: rcar: Add dependency on OF

 MAINTAINERS                                   |  4 +--
 drivers/thermal/Kconfig                       | 28 ++-----------------
 drivers/thermal/Makefile                      |  4 +--
 drivers/thermal/renesas/Kconfig               | 28 +++++++++++++++++++
 drivers/thermal/renesas/Makefile              |  5 ++++
 .../thermal/{ => renesas}/rcar_gen3_thermal.c |  2 +-
 drivers/thermal/{ => renesas}/rcar_thermal.c  |  2 +-
 drivers/thermal/{ => renesas}/rzg2l_thermal.c |  2 +-
 8 files changed, 41 insertions(+), 34 deletions(-)
 create mode 100644 drivers/thermal/renesas/Kconfig
 create mode 100644 drivers/thermal/renesas/Makefile
 rename drivers/thermal/{ => renesas}/rcar_gen3_thermal.c (99%)
 rename drivers/thermal/{ => renesas}/rcar_thermal.c (99%)
 rename drivers/thermal/{ => renesas}/rzg2l_thermal.c (99%)

Comments

Biju Das May 7, 2024, 6:58 a.m. UTC | #1
Hi Niklas,

> -----Original Message-----
> From: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> Subject: [PATCH 0/2] thermal: renesas: Group drivers and update dependencies
> 
> Hello,
> 
> This small series moves and groups all renesas thermal drivers in a vendor directory as done by
> some other vendors. The idea is to make it easier to spot issues they are worked on by the same set
> of people.
> 
> Patch 2/2 fix such an issue where one dependency was missed for one the drivers.
> 
> @Biju: There is no entry for rzg2l_thermal in MAINTAINERS. Should we update that with an entry for
> you?

Yes Please.

Thanks,
Biju

> 
> Niklas Söderlund (2):
>   thermal: renesas: Group all renesas thermal drivers together
>   thermal: renesas: rcar: Add dependency on OF
> 
>  MAINTAINERS                                   |  4 +--
>  drivers/thermal/Kconfig                       | 28 ++-----------------
>  drivers/thermal/Makefile                      |  4 +--
>  drivers/thermal/renesas/Kconfig               | 28 +++++++++++++++++++
>  drivers/thermal/renesas/Makefile              |  5 ++++
>  .../thermal/{ => renesas}/rcar_gen3_thermal.c |  2 +-  drivers/thermal/{ =>
> renesas}/rcar_thermal.c  |  2 +-  drivers/thermal/{ => renesas}/rzg2l_thermal.c |  2 +-
>  8 files changed, 41 insertions(+), 34 deletions(-)  create mode 100644
> drivers/thermal/renesas/Kconfig  create mode 100644 drivers/thermal/renesas/Makefile  rename
> drivers/thermal/{ => renesas}/rcar_gen3_thermal.c (99%)  rename drivers/thermal/{ =>
> renesas}/rcar_thermal.c (99%)  rename drivers/thermal/{ => renesas}/rzg2l_thermal.c (99%)
> 
> --
> 2.45.0
Niklas Söderlund May 8, 2024, 12:26 p.m. UTC | #2
Hello Biju,

On 2024-05-07 06:58:59 +0000, Biju Das wrote:
> Hi Niklas,
> 
> > -----Original Message-----
> > From: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> > Subject: [PATCH 0/2] thermal: renesas: Group drivers and update dependencies
> > 
> > Hello,
> > 
> > This small series moves and groups all renesas thermal drivers in a vendor directory as done by
> > some other vendors. The idea is to make it easier to spot issues they are worked on by the same set
> > of people.
> > 
> > Patch 2/2 fix such an issue where one dependency was missed for one the drivers.
> > 
> > @Biju: There is no entry for rzg2l_thermal in MAINTAINERS. Should we update that with an entry for
> > you?
> 
> Yes Please.

Sounds good, once this series is merged could you send a patch adding 
yourself?

> 
> Thanks,
> Biju
> 
> > 
> > Niklas Söderlund (2):
> >   thermal: renesas: Group all renesas thermal drivers together
> >   thermal: renesas: rcar: Add dependency on OF
> > 
> >  MAINTAINERS                                   |  4 +--
> >  drivers/thermal/Kconfig                       | 28 ++-----------------
> >  drivers/thermal/Makefile                      |  4 +--
> >  drivers/thermal/renesas/Kconfig               | 28 +++++++++++++++++++
> >  drivers/thermal/renesas/Makefile              |  5 ++++
> >  .../thermal/{ => renesas}/rcar_gen3_thermal.c |  2 +-  drivers/thermal/{ =>
> > renesas}/rcar_thermal.c  |  2 +-  drivers/thermal/{ => renesas}/rzg2l_thermal.c |  2 +-
> >  8 files changed, 41 insertions(+), 34 deletions(-)  create mode 100644
> > drivers/thermal/renesas/Kconfig  create mode 100644 drivers/thermal/renesas/Makefile  rename
> > drivers/thermal/{ => renesas}/rcar_gen3_thermal.c (99%)  rename drivers/thermal/{ =>
> > renesas}/rcar_thermal.c (99%)  rename drivers/thermal/{ => renesas}/rzg2l_thermal.c (99%)
> > 
> > --
> > 2.45.0
>
Biju Das May 8, 2024, 12:31 p.m. UTC | #3
Hi Niklas,

> -----Original Message-----
> From: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> Sent: Wednesday, May 8, 2024 1:26 PM
> Subject: Re: [PATCH 0/2] thermal: renesas: Group drivers and update dependencies
> 
> Hello Biju,
> 
> On 2024-05-07 06:58:59 +0000, Biju Das wrote:
> > Hi Niklas,
> >
> > > -----Original Message-----
> > > From: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> > > Subject: [PATCH 0/2] thermal: renesas: Group drivers and update
> > > dependencies
> > >
> > > Hello,
> > >
> > > This small series moves and groups all renesas thermal drivers in a
> > > vendor directory as done by some other vendors. The idea is to make
> > > it easier to spot issues they are worked on by the same set of people.
> > >
> > > Patch 2/2 fix such an issue where one dependency was missed for one the drivers.
> > >
> > > @Biju: There is no entry for rzg2l_thermal in MAINTAINERS. Should we
> > > update that with an entry for you?
> >
> > Yes Please.
> 
> Sounds good, once this series is merged could you send a patch adding yourself?

Sure, will do.

Cheers,
Biju