Re: [PATCH] mmc: allow skipping SD card initialization

2017-09-25 Thread Sascha Hauer
On Wed, Sep 20, 2017 at 05:50:44PM +0200, Philipp Zabel wrote: > This patch allows to skip SD card initialization for eMMCs by setting > the "no-sd" property in the host controller device tree node. Avoiding > two SD command timeouts speeds up detection time by well over 200 ms. > > Signed-off-by:

Re: [PATCH] remove checks for xzalloc() returning NULL

2017-09-25 Thread Sascha Hauer
On Mon, Sep 25, 2017 at 12:02:35PM +0200, Uwe Kleine-König wrote: > xzalloc() either returns memory or panics, so checking for NULL is useless. > > Signed-off-by: Uwe Kleine-König > --- > common/state/backend_bucket_circular.c | 2 -- > common/state/backend_format_dtb.c | 2 -- > common/sta

Re: [PATCH] pinctrl: mxs: fix error message to mention right property name

2017-09-25 Thread Sascha Hauer
On Sun, Sep 24, 2017 at 11:28:53PM +0200, Uwe Kleine-König wrote: > Signed-off-by: Uwe Kleine-König > --- > drivers/pinctrl/pinctrl-mxs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Applied, thanks Sascha > diff --git a/drivers/pinctrl/pinctrl-mxs.c b/drivers/pinctrl/pinctrl-mxs

Re: [PATCH] imx-bbu-nand-fcb: read_firmware_all(): Spell "Secondary firmware" with a capital "S"

2017-09-25 Thread Sascha Hauer
On Fri, Sep 22, 2017 at 03:50:23PM +0200, Marc Kleine-Budde wrote: > "Primary firmware" is writte with a capital "P" so adjust "Secondary firmware" > to use a capital "S". > > Signed-off-by: Marc Kleine-Budde > --- > common/imx-bbu-nand-fcb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(

Re: [PATCH] console_countdown: ignore errors in getchar()

2017-09-25 Thread Sascha Hauer
On Wed, Sep 20, 2017 at 12:07:16PM -0700, Aleksander Morgado wrote: > The getchar() call may return an error reported as a -1, e.g. when the > console detects a RATP message and switches to RATP mode. > > In general it probably is a good idea to ignore these errors in the > console countdown opera

Re: [PATCH] ARM: i.MX53 QSB: drop phy reset handling in board file

2017-09-25 Thread Sascha Hauer
On Fri, Sep 22, 2017 at 11:54:59AM +0200, Uwe Kleine-König wrote: > Since commit > > 5c1846b62524 ("ARM: i.mx53: Parse Reset GPIO pin in FEC driver from > Devicetree") > > the fec driver ensures a phy reset. As there is little use in resetting > twice, drop the reset from the board file. >

Re: [PATCH v2] gpiolib: check validity for gpio_info in *_active()

2017-09-25 Thread Sascha Hauer
On Thu, Sep 21, 2017 at 10:44:55PM +0200, Uwe Kleine-König wrote: > gpio_set_active, gpio_is_active and gpio_direction_active are public > functions, accordingly there should be error checking. > > If an invalid gpio number is given to these functions without checking > gpio_adjust_value is called

Re: [PATCH] gpiolib: check validity for gpio_info in *_active()

2017-09-25 Thread Sascha Hauer
On Thu, Sep 21, 2017 at 10:41:39PM +0200, Uwe Kleine-König wrote: > gpio_set_active, gpio_is_active and gpio_direction_active are public > functions, accordingly there should be error checking. > > If an invalid gpio number is given to these functions without checking > gpio_adjust_value is called

[PATCH] remove checks for xzalloc() returning NULL

2017-09-25 Thread Uwe Kleine-König
xzalloc() either returns memory or panics, so checking for NULL is useless. Signed-off-by: Uwe Kleine-König --- common/state/backend_bucket_circular.c | 2 -- common/state/backend_format_dtb.c | 2 -- common/state/backend_format_raw.c | 4 drivers/clk/mxs/clk-div.c |