drivers/iio/adc/rzg2l_adc.c | 67 +++++++++++++++++++++++++------------ 1 file changed, 45 insertions(+), 22 deletions(-)
From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com> Hi, Series adds some cleanups for the RZ/G2L ADC driver after the support for the RZ/G3S SoC. Thank you, Claudiu Beznea Changes in v4: - open the devres group in its own function and rename the rzg2l_adc_probe() to rzg2l_adc_probe_helper() to have simpler code - collected tags Changes in v3: - in patch 2/2 use a devres group for all the devm resources acquired in the driver's probe Changes in v2: - updated cover letter - collected tags - updated patch 1/2 to drop devres APIs from the point the runtime PM is enabled Claudiu Beznea (2): iio: adc: rzg2l_adc: Open a devres group iio: adc: rzg2l: Cleanup suspend/resume path drivers/iio/adc/rzg2l_adc.c | 67 +++++++++++++++++++++++++------------ 1 file changed, 45 insertions(+), 22 deletions(-) -- 2.43.0
On Mon, 24 Mar 2025 14:26:25 +0200 Claudiu <claudiu.beznea@tuxon.dev> wrote: > From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com> > > Hi, > > Series adds some cleanups for the RZ/G2L ADC driver after the support > for the RZ/G3S SoC. This doesn't address Dmitry's comment or highlight the outstanding question he had to Greg KH on v3. I appreciate you want to get this fixed but I'd rather we got it 'right' first time! Also, please make sure to +CC anyone who engaged with an earlier version. For reference of Greg if he sees this, Dmitry was expressing view that the fix belongs in the bus layer not the individual drivers. FWIW that feels like the right layer to me as well. https://lore.kernel.org/all/Z8k8lDxA53gUJa0n@google.com/#t Jonathan > > Thank you, > Claudiu Beznea > > Changes in v4: > - open the devres group in its own function and rename the > rzg2l_adc_probe() to rzg2l_adc_probe_helper() to have simpler code > - collected tags > > Changes in v3: > - in patch 2/2 use a devres group for all the devm resources > acquired in the driver's probe > > Changes in v2: > - updated cover letter > - collected tags > - updated patch 1/2 to drop devres APIs from the point the > runtime PM is enabled > > Claudiu Beznea (2): > iio: adc: rzg2l_adc: Open a devres group > iio: adc: rzg2l: Cleanup suspend/resume path > > drivers/iio/adc/rzg2l_adc.c | 67 +++++++++++++++++++++++++------------ > 1 file changed, 45 insertions(+), 22 deletions(-) >
Hi, Jonathan, On 27.03.2025 17:38, Jonathan Cameron wrote: > On Mon, 24 Mar 2025 14:26:25 +0200 > Claudiu <claudiu.beznea@tuxon.dev> wrote: > >> From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com> >> >> Hi, >> >> Series adds some cleanups for the RZ/G2L ADC driver after the support >> for the RZ/G3S SoC. > > This doesn't address Dmitry's comment or highlight the outstanding > question he had to Greg KH on v3. > I appreciate you want to get this fixed but I'd rather we got > it 'right' first time! My bad. As there was no input on platform bus patch I though this is not the desired way of going forward. Sorry for that. > > Also, please make sure to +CC anyone who engaged with an earlier version. Ok, will do it. > > For reference of Greg if he sees this, Dmitry was expressing view that > the fix belongs in the bus layer not the individual drivers. > FWIW that feels like the right layer to me as well. To me, too. Thank you, Claudiu > > https://lore.kernel.org/all/Z8k8lDxA53gUJa0n@google.com/#t > > Jonathan > > > >> >> Thank you, >> Claudiu Beznea >> >> Changes in v4: >> - open the devres group in its own function and rename the >> rzg2l_adc_probe() to rzg2l_adc_probe_helper() to have simpler code >> - collected tags >> >> Changes in v3: >> - in patch 2/2 use a devres group for all the devm resources >> acquired in the driver's probe >> >> Changes in v2: >> - updated cover letter >> - collected tags >> - updated patch 1/2 to drop devres APIs from the point the >> runtime PM is enabled >> >> Claudiu Beznea (2): >> iio: adc: rzg2l_adc: Open a devres group >> iio: adc: rzg2l: Cleanup suspend/resume path >> >> drivers/iio/adc/rzg2l_adc.c | 67 +++++++++++++++++++++++++------------ >> 1 file changed, 45 insertions(+), 22 deletions(-) >> >
On Thu, Mar 27, 2025 at 03:38:45PM +0000, Jonathan Cameron wrote: > On Mon, 24 Mar 2025 14:26:25 +0200 > Claudiu <claudiu.beznea@tuxon.dev> wrote: > > > From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com> > > > > Hi, > > > > Series adds some cleanups for the RZ/G2L ADC driver after the support > > for the RZ/G3S SoC. > > This doesn't address Dmitry's comment or highlight the outstanding > question he had to Greg KH on v3. > I appreciate you want to get this fixed but I'd rather we got > it 'right' first time! > > Also, please make sure to +CC anyone who engaged with an earlier version. > > For reference of Greg if he sees this, Dmitry was expressing view that > the fix belongs in the bus layer not the individual drivers. > FWIW that feels like the right layer to me as well. > > https://lore.kernel.org/all/Z8k8lDxA53gUJa0n@google.com/#t As this is a PM question, Rafael would be the best to ask. thanks, greg k-h
On Thu, 27 Mar 2025 17:22:20 +0100 Greg KH <gregkh@linuxfoundation.org> wrote: > On Thu, Mar 27, 2025 at 03:38:45PM +0000, Jonathan Cameron wrote: > > On Mon, 24 Mar 2025 14:26:25 +0200 > > Claudiu <claudiu.beznea@tuxon.dev> wrote: > > > > > From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com> > > > > > > Hi, > > > > > > Series adds some cleanups for the RZ/G2L ADC driver after the support > > > for the RZ/G3S SoC. > > > > This doesn't address Dmitry's comment or highlight the outstanding > > question he had to Greg KH on v3. > > I appreciate you want to get this fixed but I'd rather we got > > it 'right' first time! > > > > Also, please make sure to +CC anyone who engaged with an earlier version. > > > > For reference of Greg if he sees this, Dmitry was expressing view that > > the fix belongs in the bus layer not the individual drivers. > > FWIW that feels like the right layer to me as well. > > > > https://lore.kernel.org/all/Z8k8lDxA53gUJa0n@google.com/#t > > As this is a PM question, Rafael would be the best to ask. Sure. Perhaps Rafael missed previous discussion, so I've messaged him directly to draw his attention to the series. Claudiu, please include all relevant people in +CC. Don't trim it down to those effected by a particular solution as has happened here. +CC Rafael, Daniel and Ulf. Thanks, Jonathan > > thanks, > > greg k-h
Hi, Jonathan, Daniel, On 30.03.2025 18:36, Jonathan Cameron wrote: > On Thu, 27 Mar 2025 17:22:20 +0100 > Greg KH <gregkh@linuxfoundation.org> wrote: > >> On Thu, Mar 27, 2025 at 03:38:45PM +0000, Jonathan Cameron wrote: >>> On Mon, 24 Mar 2025 14:26:25 +0200 >>> Claudiu <claudiu.beznea@tuxon.dev> wrote: >>> >>>> From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com> >>>> >>>> Hi, >>>> >>>> Series adds some cleanups for the RZ/G2L ADC driver after the support >>>> for the RZ/G3S SoC. >>> >>> This doesn't address Dmitry's comment or highlight the outstanding >>> question he had to Greg KH on v3. >>> I appreciate you want to get this fixed but I'd rather we got >>> it 'right' first time! >>> >>> Also, please make sure to +CC anyone who engaged with an earlier version. >>> >>> For reference of Greg if he sees this, Dmitry was expressing view that >>> the fix belongs in the bus layer not the individual drivers. >>> FWIW that feels like the right layer to me as well. >>> >>> https://lore.kernel.org/all/Z8k8lDxA53gUJa0n@google.com/#t >> >> As this is a PM question, Rafael would be the best to ask. > > Sure. Perhaps Rafael missed previous discussion, so I've messaged > him directly to draw his attention to the series. > > Claudiu, please include all relevant people in +CC. Don't trim > it down to those effected by a particular solution as has happened > here. +CC Rafael, Daniel and Ulf. As the discussion [1] is progressing very slowly: Jonathan: do you consider having this series as a temporary solution? Daniel: do you consider having the fix in [2] as a temporary solution (of course, with the adjustments suggested by Geert)? There is also [3] for which a similar approach was proposed. From what I understood from the discussions on [3], Bjorn is OK with the current solution. I am taking the responsibility to do the necessary adjustments to all these drivers once there is a resolution for [1]. Thank you, Claudiu [1] https://lore.kernel.org/all/20250215130849.227812-1-claudiu.beznea.uj@bp.renesas.com/ [2] https://lore.kernel.org/all/20250324135701.179827-3-claudiu.beznea.uj@bp.renesas.com/ [3] https://lore.kernel.org/all/20250430103236.3511989-6-claudiu.beznea.uj@bp.renesas.com/
© 2016 - 2025 Red Hat, Inc.