[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a59d4384-9cb9-baaa-e5f4-1e2111e295c9@pengutronix.de>
Date: Thu, 2 Jan 2020 15:40:36 +0100
From: Marc Kleine-Budde <mkl@...gutronix.de>
To: Dan Murphy <dmurphy@...com>
Cc: linux-can@...r.kernel.org, linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org
Subject: Re: [PATCH v2] can: tcan4x5x: Turn on the power before parsing the
config
On 12/10/19 5:32 PM, Dan Murphy wrote:
> The parse config function now performs action on the device either
> reading or writing and a reset.
BTW n+1:
Why is the function called parse_config? I don't see any parsing going on.
Please add it directly to the tcan4x5x_can_probe() function.
Marc
--
Pengutronix e.K. | Marc Kleine-Budde |
Embedded Linux | https://www.pengutronix.de |
Vertretung West/Dortmund | Phone: +49-231-2826-924 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)
Powered by blists - more mailing lists