* [PATCH v3 1/3] regulator: max14577: Add proper dt-compatible strings [not found] <CGME20200522102452eucas1p17c18de8f79e27de96474e5fcad6db5fa@eucas1p1.samsung.com> @ 2020-05-22 10:24 ` Marek Szyprowski [not found] ` <CGME20200522102455eucas1p24b695b7f6c533681f544be166ea69200@eucas1p2.samsung.com> ` (2 more replies) 0 siblings, 3 replies; 6+ messages in thread From: Marek Szyprowski @ 2020-05-22 10:24 UTC (permalink / raw) To: linux-pm, linux-kernel Cc: Marek Szyprowski, Krzysztof Kozlowski, Bartlomiej Zolnierkiewicz, Chanwoo Choi, MyungJoo Ham, Sebastian Reichel, Mark Brown Add device tree compatible strings and create proper modalias structures to let this driver load automatically if compiled as module, because max14577 MFD driver creates MFD cells with such compatible strings. Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> --- v3: - sorted of_max14577_regulator_dt_match v2: - added .of_match_table pointer --- drivers/regulator/max14577-regulator.c | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/drivers/regulator/max14577-regulator.c b/drivers/regulator/max14577-regulator.c index 07a150c9bbf2..4c9ae52b9e87 100644 --- a/drivers/regulator/max14577-regulator.c +++ b/drivers/regulator/max14577-regulator.c @@ -238,10 +238,20 @@ static const struct platform_device_id max14577_regulator_id[] = { }; MODULE_DEVICE_TABLE(platform, max14577_regulator_id); +static const struct of_device_id of_max14577_regulator_dt_match[] = { + { .compatible = "maxim,max14577-regulator", + .data = (void *)MAXIM_DEVICE_TYPE_MAX14577, }, + { .compatible = "maxim,max77836-regulator", + .data = (void *)MAXIM_DEVICE_TYPE_MAX77836, }, + { }, +}; +MODULE_DEVICE_TABLE(of, of_max14577_regulator_dt_match); + static struct platform_driver max14577_regulator_driver = { .driver = { - .name = "max14577-regulator", - }, + .name = "max14577-regulator", + .of_match_table = of_max14577_regulator_dt_match, + }, .probe = max14577_regulator_probe, .id_table = max14577_regulator_id, }; -- 2.17.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <CGME20200522102455eucas1p24b695b7f6c533681f544be166ea69200@eucas1p2.samsung.com>]
* [PATCH v3 2/3] extcon: max14577: Add proper dt-compatible strings [not found] ` <CGME20200522102455eucas1p24b695b7f6c533681f544be166ea69200@eucas1p2.samsung.com> @ 2020-05-22 10:24 ` Marek Szyprowski 2020-05-25 3:35 ` Chanwoo Choi 0 siblings, 1 reply; 6+ messages in thread From: Marek Szyprowski @ 2020-05-22 10:24 UTC (permalink / raw) To: linux-pm, linux-kernel Cc: Marek Szyprowski, Krzysztof Kozlowski, Bartlomiej Zolnierkiewicz, Chanwoo Choi, MyungJoo Ham, Sebastian Reichel, Mark Brown Add device tree compatible strings and create proper modalias structures to let this driver load automatically if compiled as module, because max14577 MFD driver creates MFD cells with such compatible strings. Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> --- v3: - sorted of_max14577_muic_dt_match v2: - added .of_match_table pointer --- drivers/extcon/extcon-max14577.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/drivers/extcon/extcon-max14577.c b/drivers/extcon/extcon-max14577.c index 32f663436e6e..03af678ddeba 100644 --- a/drivers/extcon/extcon-max14577.c +++ b/drivers/extcon/extcon-max14577.c @@ -782,9 +782,19 @@ static const struct platform_device_id max14577_muic_id[] = { }; MODULE_DEVICE_TABLE(platform, max14577_muic_id); +static const struct of_device_id of_max14577_muic_dt_match[] = { + { .compatible = "maxim,max14577-muic", + .data = (void *)MAXIM_DEVICE_TYPE_MAX14577, }, + { .compatible = "maxim,max77836-muic", + .data = (void *)MAXIM_DEVICE_TYPE_MAX77836, }, + { }, +}; +MODULE_DEVICE_TABLE(of, of_max14577_muic_dt_match); + static struct platform_driver max14577_muic_driver = { .driver = { .name = "max14577-muic", + .of_match_table = of_max14577_muic_dt_match, }, .probe = max14577_muic_probe, .remove = max14577_muic_remove, -- 2.17.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3 2/3] extcon: max14577: Add proper dt-compatible strings 2020-05-22 10:24 ` [PATCH v3 2/3] extcon: " Marek Szyprowski @ 2020-05-25 3:35 ` Chanwoo Choi 0 siblings, 0 replies; 6+ messages in thread From: Chanwoo Choi @ 2020-05-25 3:35 UTC (permalink / raw) To: Marek Szyprowski, linux-pm, linux-kernel Cc: Krzysztof Kozlowski, Bartlomiej Zolnierkiewicz, MyungJoo Ham, Sebastian Reichel, Mark Brown Hi Marek, On 5/22/20 7:24 PM, Marek Szyprowski wrote: > Add device tree compatible strings and create proper modalias structures > to let this driver load automatically if compiled as module, because > max14577 MFD driver creates MFD cells with such compatible strings. > > Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> > --- > v3: > - sorted of_max14577_muic_dt_match > v2: > - added .of_match_table pointer > --- > drivers/extcon/extcon-max14577.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/drivers/extcon/extcon-max14577.c b/drivers/extcon/extcon-max14577.c > index 32f663436e6e..03af678ddeba 100644 > --- a/drivers/extcon/extcon-max14577.c > +++ b/drivers/extcon/extcon-max14577.c > @@ -782,9 +782,19 @@ static const struct platform_device_id max14577_muic_id[] = { > }; > MODULE_DEVICE_TABLE(platform, max14577_muic_id); > > +static const struct of_device_id of_max14577_muic_dt_match[] = { > + { .compatible = "maxim,max14577-muic", > + .data = (void *)MAXIM_DEVICE_TYPE_MAX14577, }, > + { .compatible = "maxim,max77836-muic", > + .data = (void *)MAXIM_DEVICE_TYPE_MAX77836, }, > + { }, > +}; > +MODULE_DEVICE_TABLE(of, of_max14577_muic_dt_match); > + > static struct platform_driver max14577_muic_driver = { > .driver = { > .name = "max14577-muic", > + .of_match_table = of_max14577_muic_dt_match, > }, > .probe = max14577_muic_probe, > .remove = max14577_muic_remove, > Applied it. Thanks. -- Best Regards, Chanwoo Choi Samsung Electronics ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CGME20200522102455eucas1p13700db68b90ec05bd02d21014b81b656@eucas1p1.samsung.com>]
* [PATCH v3 3/3] power: charger: max14577: Add proper dt-compatible strings [not found] ` <CGME20200522102455eucas1p13700db68b90ec05bd02d21014b81b656@eucas1p1.samsung.com> @ 2020-05-22 10:24 ` Marek Szyprowski 2020-05-28 7:59 ` Sebastian Reichel 0 siblings, 1 reply; 6+ messages in thread From: Marek Szyprowski @ 2020-05-22 10:24 UTC (permalink / raw) To: linux-pm, linux-kernel Cc: Marek Szyprowski, Krzysztof Kozlowski, Bartlomiej Zolnierkiewicz, Chanwoo Choi, MyungJoo Ham, Sebastian Reichel, Mark Brown Add device tree compatible strings and create proper modalias structures to let this driver load automatically if compiled as module, because max14577 MFD driver creates MFD cells with such compatible strings. Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> --- v3: - sorted of_max14577_charger_dt_match v2: - added .of_match_table pointer --- drivers/power/supply/max14577_charger.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/drivers/power/supply/max14577_charger.c b/drivers/power/supply/max14577_charger.c index 8a59feac6468..96f4cd1941b2 100644 --- a/drivers/power/supply/max14577_charger.c +++ b/drivers/power/supply/max14577_charger.c @@ -623,9 +623,19 @@ static const struct platform_device_id max14577_charger_id[] = { }; MODULE_DEVICE_TABLE(platform, max14577_charger_id); +static const struct of_device_id of_max14577_charger_dt_match[] = { + { .compatible = "maxim,max14577-charger", + .data = (void *)MAXIM_DEVICE_TYPE_MAX14577, }, + { .compatible = "maxim,max77836-charger", + .data = (void *)MAXIM_DEVICE_TYPE_MAX77836, }, + { }, +}; +MODULE_DEVICE_TABLE(of, of_max14577_charger_dt_match); + static struct platform_driver max14577_charger_driver = { .driver = { .name = "max14577-charger", + .of_match_table = of_max14577_charger_dt_match, }, .probe = max14577_charger_probe, .remove = max14577_charger_remove, -- 2.17.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3 3/3] power: charger: max14577: Add proper dt-compatible strings 2020-05-22 10:24 ` [PATCH v3 3/3] power: charger: " Marek Szyprowski @ 2020-05-28 7:59 ` Sebastian Reichel 0 siblings, 0 replies; 6+ messages in thread From: Sebastian Reichel @ 2020-05-28 7:59 UTC (permalink / raw) To: Marek Szyprowski Cc: linux-pm, linux-kernel, Krzysztof Kozlowski, Bartlomiej Zolnierkiewicz, Chanwoo Choi, MyungJoo Ham, Mark Brown [-- Attachment #1: Type: text/plain, Size: 1832 bytes --] Hi, On Fri, May 22, 2020 at 12:24:48PM +0200, Marek Szyprowski wrote: > Add device tree compatible strings and create proper modalias structures > to let this driver load automatically if compiled as module, because > max14577 MFD driver creates MFD cells with such compatible strings. > > Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> > --- I saw some discussion for patch 1, but still queued this considering the compatible string is already documented in the DT bindings and at least one of them is already being used in DT files. TL;DR: Thanks, queued. -- Sebastian > v3: > - sorted of_max14577_charger_dt_match > v2: > - added .of_match_table pointer > --- > drivers/power/supply/max14577_charger.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/drivers/power/supply/max14577_charger.c b/drivers/power/supply/max14577_charger.c > index 8a59feac6468..96f4cd1941b2 100644 > --- a/drivers/power/supply/max14577_charger.c > +++ b/drivers/power/supply/max14577_charger.c > @@ -623,9 +623,19 @@ static const struct platform_device_id max14577_charger_id[] = { > }; > MODULE_DEVICE_TABLE(platform, max14577_charger_id); > > +static const struct of_device_id of_max14577_charger_dt_match[] = { > + { .compatible = "maxim,max14577-charger", > + .data = (void *)MAXIM_DEVICE_TYPE_MAX14577, }, > + { .compatible = "maxim,max77836-charger", > + .data = (void *)MAXIM_DEVICE_TYPE_MAX77836, }, > + { }, > +}; > +MODULE_DEVICE_TABLE(of, of_max14577_charger_dt_match); > + > static struct platform_driver max14577_charger_driver = { > .driver = { > .name = "max14577-charger", > + .of_match_table = of_max14577_charger_dt_match, > }, > .probe = max14577_charger_probe, > .remove = max14577_charger_remove, > -- > 2.17.1 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3 1/3] regulator: max14577: Add proper dt-compatible strings 2020-05-22 10:24 ` [PATCH v3 1/3] regulator: max14577: Add proper dt-compatible strings Marek Szyprowski [not found] ` <CGME20200522102455eucas1p24b695b7f6c533681f544be166ea69200@eucas1p2.samsung.com> [not found] ` <CGME20200522102455eucas1p13700db68b90ec05bd02d21014b81b656@eucas1p1.samsung.com> @ 2020-05-22 11:09 ` Mark Brown 2 siblings, 0 replies; 6+ messages in thread From: Mark Brown @ 2020-05-22 11:09 UTC (permalink / raw) To: Marek Szyprowski Cc: linux-pm, linux-kernel, Krzysztof Kozlowski, Bartlomiej Zolnierkiewicz, Chanwoo Choi, MyungJoo Ham, Sebastian Reichel [-- Attachment #1: Type: text/plain, Size: 411 bytes --] On Fri, May 22, 2020 at 12:24:46PM +0200, Marek Szyprowski wrote: > Add device tree compatible strings and create proper modalias structures > to let this driver load automatically if compiled as module, because > max14577 MFD driver creates MFD cells with such compatible strings. Same issue as before, you're putting the current Linux model into DT without adding any additional description of the hardware. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-05-28 8:00 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <CGME20200522102452eucas1p17c18de8f79e27de96474e5fcad6db5fa@eucas1p1.samsung.com> 2020-05-22 10:24 ` [PATCH v3 1/3] regulator: max14577: Add proper dt-compatible strings Marek Szyprowski [not found] ` <CGME20200522102455eucas1p24b695b7f6c533681f544be166ea69200@eucas1p2.samsung.com> 2020-05-22 10:24 ` [PATCH v3 2/3] extcon: " Marek Szyprowski 2020-05-25 3:35 ` Chanwoo Choi [not found] ` <CGME20200522102455eucas1p13700db68b90ec05bd02d21014b81b656@eucas1p1.samsung.com> 2020-05-22 10:24 ` [PATCH v3 3/3] power: charger: " Marek Szyprowski 2020-05-28 7:59 ` Sebastian Reichel 2020-05-22 11:09 ` [PATCH v3 1/3] regulator: " Mark Brown
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).