From mboxrd@z Thu Jan 1 00:00:00 1970
From: Philipp Zabel
Subject: Re: [PATCH 3/5] [media] tc358743: support probe from device tree
Date: Fri, 17 Jul 2015 13:00:11 +0200
Message-ID: <1437130811.3254.1.camel@pengutronix.de>
References: <1436533897-3060-1-git-send-email-p.zabel@pengutronix.de>
<1436533897-3060-3-git-send-email-p.zabel@pengutronix.de>
<55A39982.3030006@xs4all.nl> <1436868605.3793.24.camel@pengutronix.de>
<55A4E154.8020309@xs4all.nl>
Mime-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Return-path:
In-Reply-To: <55A4E154.8020309@xs4all.nl>
Sender: linux-media-owner@vger.kernel.org
To: Hans Verkuil
Cc: Mats Randgaard , linux-media@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de
List-Id: devicetree@vger.kernel.org
Am Dienstag, den 14.07.2015, 12:15 +0200 schrieb Hans Verkuil:
[...]
> As you said, it's not public and without the formulas there is nothing you
> can do but hardcode it.
>
> If I understand this correctly these values depend on the link frequency,
> so the DT should contain the link frequency and the driver can hardcode the
> values based on that. Which means that if someone needs to support a new
> link frequency the driver needs to be extended for that frequency.
>
> As long as Toshiba keeps the formulas under NDA there isn't much else you can
> do.
Ok.
[...]
> >>> /* platform data */
> >>> - if (!pdata) {
> >>> - v4l_err(client, "No platform data!\n");
> >>> - return -ENODEV;
> >>> + if (pdata) {
> >>> + state->pdata = *pdata;
> >>> + } else {
> >>> + err = tc358743_probe_of(state);
> >>> + if (err == -ENODEV)
> >>> + v4l_err(client, "No platform data!\n");
> >>
> >> I'd replace this with "No device tree data!" or something like that.
> >
> > I'll do that, thank you.
On second thought, I'll keep it as is. The tc358743_probe_of function
prints its own error messages. In the platform data case it returns
-ENODEV, so that'd still be the correct message, then.
regards
Philipp