linux-kernel - Re: [PATCHv3 11/14] HSI: Introduce Nokia N900 modem driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140419202556.GK5148@amd.pavel.ucw.cz>
Date:	Sat, 19 Apr 2014 22:25:56 +0200
From:	Pavel Machek <pavel@....cz>
To:	Sebastian Reichel <sre@...nel.org>
Cc:	Sebastian Reichel <sre@...g0.de>,
	Linus Walleij <linus.walleij@...aro.org>,
	Shubhrajyoti Datta <omaplinuxkernel@...il.com>,
	Carlos Chinea <cch.devel@...il.com>,
	Tony Lindgren <tony@...mide.com>,
	Rob Herring <robh+dt@...nel.org>,
	Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Kumar Gala <galak@...eaurora.org>, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-omap@...r.kernel.org,
	Pali Rohár <pali.rohar@...il.com>,
	Ивайло Димитров 
	<freemangordon@....bg>,
	Joni Lapilainen <joni.lapilainen@...il.com>,
	Aaro Koskinen <aaro.koskinen@....fi>
Subject: Re: [PATCHv3 11/14] HSI: Introduce Nokia N900 modem driver

On Sat 2014-03-29 01:31:47, Sebastian Reichel wrote:
> The Nokia N900's modem is connected via Synchronous Serial Interface (SSI),
> which is a legacy version of MIPI's High-speed Synchronous Serial Interface
> (HSI).
> 
> The handles the GPIOs for enabling and resetting the modem and instanciates
> ssi-protocol for data exchange. It does not yet support exchanging voice data
> with the modem.
> 
> Signed-off-by: Sebastian Reichel <sre@...nel.org>

Reviewed-by: Pavel Machek <pavel@....cz>

> +static void do_nokia_modem_rst_ind_tasklet(unsigned long data)
> +{
> +	struct nokia_modem_device *modem = (struct nokia_modem_device *)data;
> +
> +	if (!modem)
> +		return;
> +
> +	dev_info(modem->device, "*** CMT rst line change detected ***\n");

Maybe slightly less stars would work?


> +static int nokia_modem_gpio_probe(struct device *dev)
> +{
> +	struct device_node *np = dev->of_node;
> +	struct nokia_modem_device *modem = dev_get_drvdata(dev);
> +	int gpio_count, gpio_name_count, i, err;
> +
> +	gpio_count = of_gpio_count(np);
> +	gpio_name_count = of_property_count_strings(np, "gpio-names");
> +
> +	if(gpio_count != gpio_name_count) {

"if (".

> +	modem->ssi_protocol = hsi_new_client(port, &ssip);
> +	if (modem->ssi_protocol == NULL) {

...if (! ...)

> +		dev_err(dev, "Could not register ssi-protocol device\n");
> +		goto error2;
> +	}
> +
> +	err = device_attach(&modem->ssi_protocol->device);
> +	if (err == 0) {

if (!...)

									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/