Thread (29 messages) 29 messages, 3 authors, 2021-10-16

Re: [PATCH 05/12] regulator: Introduce tps68470-regulator driver

From: Mark Brown <broonie@kernel.org>
Date: 2021-10-15 18:58:56
Also in: linux-acpi, linux-media, lkml, platform-driver-x86

On Fri, Oct 15, 2021 at 08:50:13PM +0200, Hans de Goede wrote:
Are you happy with the platform_data for this driver as defined in
patch 4/12 ? :
Some of the other review comments lead me to believe that you'd be
sending out a new version at some point?
https://lore.kernel.org/platform-driver-x86/20211008162121.6628-1-hdegoede@redhat.com/T/#m745cc1191f531a57ae7998f5c8817ba9a46f0fed (local)
I am very confused about why it's in the driver without a DMI quirk
and/or clear comments about why and saying that this is a terrible
example to copy.  I'd also expect to get compile test coverage for the
driver.

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help