Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1712833 > unrolled thread
| Started by | Robin van der Gracht <robin@protonic.nl> |
|---|---|
| First post | 2017-08-16 12:50 +0200 |
| Last post | 2017-08-16 20:40 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] auxdisplay: ht16k33: Use generic device properties function Robin van der Gracht <robin@protonic.nl> - 2017-08-16 12:50 +0200
Re: [PATCH] auxdisplay: ht16k33: Use generic device properties function Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2017-08-16 20:40 +0200
| From | Robin van der Gracht <robin@protonic.nl> |
|---|---|
| Date | 2017-08-16 12:50 +0200 |
| Subject | [PATCH] auxdisplay: ht16k33: Use generic device properties function |
| Message-ID | <uf43E-5Wg-15@gated-at.bofh.it> |
matrix_keypad_parse_of_params() was replaced early this year. Signed-off-by: Robin van der Gracht <robin@protonic.nl> --- Early this year Dmitry Torokhov introduced a switch to using generic device properties instead of being OF-specific. Somehow this driver wasn't included in the update. For reference: https://patchwork.kernel.org/patch/9527095/ drivers/auxdisplay/ht16k33.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/auxdisplay/ht16k33.c b/drivers/auxdisplay/ht16k33.c index fbfa5b4cc567..99ce9731fc9c 100644 --- a/drivers/auxdisplay/ht16k33.c +++ b/drivers/auxdisplay/ht16k33.c @@ -354,7 +354,7 @@ static int ht16k33_keypad_probe(struct i2c_client *client, return err; } - err = matrix_keypad_parse_of_params(&client->dev, &rows, &cols); + err = matrix_keypad_parse_properties(&client->dev, &rows, &cols); if (err) return err; if (rows > HT16K33_MATRIX_KEYPAD_MAX_ROWS || -- 2.11.0
[toc] | [next] | [standalone]
| From | Dmitry Torokhov <dmitry.torokhov@gmail.com> |
|---|---|
| Date | 2017-08-16 20:40 +0200 |
| Subject | Re: [PATCH] auxdisplay: ht16k33: Use generic device properties function |
| Message-ID | <ufbot-28f-7@gated-at.bofh.it> |
| In reply to | #1712833 |
On Wed, Aug 16, 2017 at 12:22:48PM +0200, Robin van der Gracht wrote: > matrix_keypad_parse_of_params() was replaced early this year. > > Signed-off-by: Robin van der Gracht <robin@protonic.nl> > --- > > Early this year Dmitry Torokhov introduced a switch to using generic > device properties instead of being OF-specific. > > Somehow this driver wasn't included in the update. It was outside of "input" tree so I did not update directly, so I kept the old name available. I meant to send a separate patch updating ht16k33, but I guess I forgot... FWIW Reviewed-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> I think you want to send it to GregKH as he was picking up the other ht16k33 patches. > > For reference: https://patchwork.kernel.org/patch/9527095/ > > drivers/auxdisplay/ht16k33.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/auxdisplay/ht16k33.c b/drivers/auxdisplay/ht16k33.c > index fbfa5b4cc567..99ce9731fc9c 100644 > --- a/drivers/auxdisplay/ht16k33.c > +++ b/drivers/auxdisplay/ht16k33.c > @@ -354,7 +354,7 @@ static int ht16k33_keypad_probe(struct i2c_client *client, > return err; > } > > - err = matrix_keypad_parse_of_params(&client->dev, &rows, &cols); > + err = matrix_keypad_parse_properties(&client->dev, &rows, &cols); > if (err) > return err; > if (rows > HT16K33_MATRIX_KEYPAD_MAX_ROWS || > -- > 2.11.0 > Thanks. -- Dmitry
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web