| Message ID | 1371838198-7327-13-git-send-email-gsi@denx.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Dear Gerhard Sittig, > optionally dump relevant configuration of pins, matrix lines, > and delay/interval times at the very end of the probe routine > (development feature, silent by default) > > Signed-off-by: Gerhard Sittig <gsi@denx.de> > --- > drivers/input/keyboard/matrix_keypad.c | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/drivers/input/keyboard/matrix_keypad.c > b/drivers/input/keyboard/matrix_keypad.c index 275b157..69b15720 100644 > --- a/drivers/input/keyboard/matrix_keypad.c > +++ b/drivers/input/keyboard/matrix_keypad.c > @@ -800,6 +800,24 @@ static int matrix_keypad_probe(struct platform_device > *pdev) device_init_wakeup(&pdev->dev, pdata->wakeup); > platform_set_drvdata(pdev, keypad); > > + dev_dbg(&pdev->dev, "gpios col num[%u] lvl[%s] drv[%s] enc[%s]\n", > + pdata->num_col_gpios, > + pdata->col_gpios_active_low ? "low" : "high", > + pdata->col_gpios_push_pull ? "pp" : "od", > + pdata->col_gpios_binary_encoded ? "bin" : "1-1"); > + dev_dbg(&pdev->dev, "gpios row num[%u] lvl[%s]\n", > + pdata->num_row_gpios, > + pdata->row_gpios_active_low ? "low" : "high"); > + dev_dbg(&pdev->dev, "matrix cols[%u] rows[%u] code shift[%u]\n", > + pdata->num_matrix_cols, > + pdata->num_matrix_rows, > + keypad->row_shift); > + dev_dbg(&pdev->dev, "times scan[%u-%u] bounce[%u] switch[%u]\n", > + pdata->col_scan_delay_us, > + pdata->col_scan_delay_us_max, > + pdata->debounce_ms, > + pdata->col_switch_delay_ms); > + It's a bit hard to understand this kind of debug output. Besides, we already have all the DT props in /proc/device-tree where you can check them , do we really need this patch then ? > return 0; > > err_free_gpio: Best regards, Marek Vasut
On Sat, Jun 22, 2013 at 04:28 +0200, Marek Vasut wrote: > > Dear Gerhard Sittig, > > > optionally dump relevant configuration of pins, matrix lines, > > and delay/interval times at the very end of the probe routine > > (development feature, silent by default) > > > > Signed-off-by: Gerhard Sittig <gsi@denx.de> > > --- > > drivers/input/keyboard/matrix_keypad.c | 18 ++++++++++++++++++ > > 1 file changed, 18 insertions(+) > > > > diff --git a/drivers/input/keyboard/matrix_keypad.c > > b/drivers/input/keyboard/matrix_keypad.c index 275b157..69b15720 100644 > > --- a/drivers/input/keyboard/matrix_keypad.c > > +++ b/drivers/input/keyboard/matrix_keypad.c > > @@ -800,6 +800,24 @@ static int matrix_keypad_probe(struct platform_device > > *pdev) device_init_wakeup(&pdev->dev, pdata->wakeup); > > platform_set_drvdata(pdev, keypad); > > > > + dev_dbg(&pdev->dev, "gpios col num[%u] lvl[%s] drv[%s] enc[%s]\n", > > + pdata->num_col_gpios, > > + pdata->col_gpios_active_low ? "low" : "high", > > + pdata->col_gpios_push_pull ? "pp" : "od", > > + pdata->col_gpios_binary_encoded ? "bin" : "1-1"); > > + dev_dbg(&pdev->dev, "gpios row num[%u] lvl[%s]\n", > > + pdata->num_row_gpios, > > + pdata->row_gpios_active_low ? "low" : "high"); > > + dev_dbg(&pdev->dev, "matrix cols[%u] rows[%u] code shift[%u]\n", > > + pdata->num_matrix_cols, > > + pdata->num_matrix_rows, > > + keypad->row_shift); > > + dev_dbg(&pdev->dev, "times scan[%u-%u] bounce[%u] switch[%u]\n", > > + pdata->col_scan_delay_us, > > + pdata->col_scan_delay_us_max, > > + pdata->debounce_ms, > > + pdata->col_switch_delay_ms); > > + > > It's a bit hard to understand this kind of debug output. Besides, we already > have all the DT props in /proc/device-tree where you can check them , do we > really need this patch then ? This output shall reflect not only what was provided to the driver as input, but as well what the driver made of it (that is, some of the values get derived from others, some underwent normalization). The terse format mostly is a consequence of the 80 columns line length limit, and the output being for debugging purposes exclusively. But I understand that one needs to know the driver's internal working before one can read this output. And I felt that this is the only situation where one would enable debugging for the driver. That patch might as well get dropped, I have no strong feelings about it. Actually I put it at the end of the series such that nothing else builds on top of it and that it doesn't change the context of anything else, so that dropping it doesn't matter anyway. virtually yours Gerhard Sittig
diff --git a/drivers/input/keyboard/matrix_keypad.c b/drivers/input/keyboard/matrix_keypad.c index 275b157..69b15720 100644 --- a/drivers/input/keyboard/matrix_keypad.c +++ b/drivers/input/keyboard/matrix_keypad.c @@ -800,6 +800,24 @@ static int matrix_keypad_probe(struct platform_device *pdev) device_init_wakeup(&pdev->dev, pdata->wakeup); platform_set_drvdata(pdev, keypad); + dev_dbg(&pdev->dev, "gpios col num[%u] lvl[%s] drv[%s] enc[%s]\n", + pdata->num_col_gpios, + pdata->col_gpios_active_low ? "low" : "high", + pdata->col_gpios_push_pull ? "pp" : "od", + pdata->col_gpios_binary_encoded ? "bin" : "1-1"); + dev_dbg(&pdev->dev, "gpios row num[%u] lvl[%s]\n", + pdata->num_row_gpios, + pdata->row_gpios_active_low ? "low" : "high"); + dev_dbg(&pdev->dev, "matrix cols[%u] rows[%u] code shift[%u]\n", + pdata->num_matrix_cols, + pdata->num_matrix_rows, + keypad->row_shift); + dev_dbg(&pdev->dev, "times scan[%u-%u] bounce[%u] switch[%u]\n", + pdata->col_scan_delay_us, + pdata->col_scan_delay_us_max, + pdata->debounce_ms, + pdata->col_switch_delay_ms); + return 0; err_free_gpio:
optionally dump relevant configuration of pins, matrix lines, and delay/interval times at the very end of the probe routine (development feature, silent by default) Signed-off-by: Gerhard Sittig <gsi@denx.de> --- drivers/input/keyboard/matrix_keypad.c | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+)