From: John Stultz <john.stultz@linaro.org>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Arnd Bergmann <arnd@arndb.de>, David Airlie <airlied@linux.ie>,
lkml <linux-kernel@vger.kernel.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Hans Verkuil <hans.verkuil@cisco.com>,
Dan Carpenter <dan.carpenter@oracle.com>,
Bhumika Goyal <bhumirks@gmail.com>
Subject: Re: [RFC] [PATCH] drm: adv7511/33: Fix adv7511_cec_init() failure handling
Date: Mon, 20 Nov 2017 12:13:36 -0800 [thread overview]
Message-ID: <CALAqxLV5mJB3Aphdxk527c9okjdaoXAOD436OF=fji3w0wWrNQ@mail.gmail.com> (raw)
In-Reply-To: <d4e6cae8-f0f7-8ecf-b556-3cb0fa8dc0cc@xs4all.nl>
On Fri, Nov 17, 2017 at 12:43 AM, Hans Verkuil <hverkuil@xs4all.nl> wrote:
> diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> index 3a33075dbb22..56eeeea6a1fa 100644
> --- a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> +++ b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> @@ -1202,9 +1202,7 @@ static int adv7511_probe(struct i2c_client *i2c, const struct i2c_device_id *id)
> offset = adv7511->type == ADV7533 ? ADV7533_REG_CEC_OFFSET : 0;
>
> #ifdef CONFIG_DRM_I2C_ADV7511_CEC
> - ret = adv7511_cec_init(dev, adv7511, offset);
> - if (ret)
> - goto err_unregister_cec;
> + adv7511_cec_init(dev, adv7511, offset);
> #else
> regmap_write(adv7511->regmap, ADV7511_REG_CEC_CTRL + offset,
> ADV7511_CEC_CTRL_POWER_DOWN);
One tiny nit-pick I realized I should have made in my patch...
In the !CONFIG_DRM_I2C_ADV7511_CEC, can you just define adv7511_cec_init() as
{
regmap_write(adv7511->regmap, ADV7511_REG_CEC_CTRL + offset,
ADV7511_CEC_CTRL_POWER_DOWN);
}
Then we can call it either way, and don't need to have the ufly
#ifdefs in the adv7511_probe function?
thanks
-john
WARNING: multiple messages have this Message-ID (diff)
From: John Stultz <john.stultz@linaro.org>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Bhumika Goyal <bhumirks@gmail.com>, Arnd Bergmann <arnd@arndb.de>,
David Airlie <airlied@linux.ie>,
lkml <linux-kernel@vger.kernel.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
Hans Verkuil <hans.verkuil@cisco.com>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Dan Carpenter <dan.carpenter@oracle.com>
Subject: Re: [RFC] [PATCH] drm: adv7511/33: Fix adv7511_cec_init() failure handling
Date: Mon, 20 Nov 2017 12:13:36 -0800 [thread overview]
Message-ID: <CALAqxLV5mJB3Aphdxk527c9okjdaoXAOD436OF=fji3w0wWrNQ@mail.gmail.com> (raw)
In-Reply-To: <d4e6cae8-f0f7-8ecf-b556-3cb0fa8dc0cc@xs4all.nl>
On Fri, Nov 17, 2017 at 12:43 AM, Hans Verkuil <hverkuil@xs4all.nl> wrote:
> diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> index 3a33075dbb22..56eeeea6a1fa 100644
> --- a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> +++ b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> @@ -1202,9 +1202,7 @@ static int adv7511_probe(struct i2c_client *i2c, const struct i2c_device_id *id)
> offset = adv7511->type == ADV7533 ? ADV7533_REG_CEC_OFFSET : 0;
>
> #ifdef CONFIG_DRM_I2C_ADV7511_CEC
> - ret = adv7511_cec_init(dev, adv7511, offset);
> - if (ret)
> - goto err_unregister_cec;
> + adv7511_cec_init(dev, adv7511, offset);
> #else
> regmap_write(adv7511->regmap, ADV7511_REG_CEC_CTRL + offset,
> ADV7511_CEC_CTRL_POWER_DOWN);
One tiny nit-pick I realized I should have made in my patch...
In the !CONFIG_DRM_I2C_ADV7511_CEC, can you just define adv7511_cec_init() as
{
regmap_write(adv7511->regmap, ADV7511_REG_CEC_CTRL + offset,
ADV7511_CEC_CTRL_POWER_DOWN);
}
Then we can call it either way, and don't need to have the ufly
#ifdefs in the adv7511_probe function?
thanks
-john
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-11-20 20:13 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-15 12:37 [PATCH] [RFT] drm: adv7511/33: fix adv7511_cec_init() failure handling Arnd Bergmann
2017-11-15 12:37 ` Arnd Bergmann
2017-11-15 12:58 ` Hans Verkuil
2017-11-15 12:58 ` Hans Verkuil
2017-11-16 16:42 ` Naresh Kamboju
2017-11-16 21:50 ` John Stultz
2017-11-16 21:50 ` John Stultz
2017-11-16 22:20 ` John Stultz
2017-11-16 22:20 ` John Stultz
2017-11-16 22:23 ` John Stultz
2017-11-16 22:23 ` John Stultz
2017-11-16 22:59 ` [RFC][PATCH] drm: adv7511/33: Fix " John Stultz
2017-11-16 22:59 ` John Stultz
2017-11-26 12:56 ` Archit Taneja
2017-11-28 21:32 ` John Stultz
2017-11-28 21:32 ` John Stultz
2017-11-29 5:05 ` Archit Taneja
2017-11-29 5:05 ` Archit Taneja
2017-11-29 8:15 ` Arnd Bergmann
2017-11-17 8:43 ` [RFC] [PATCH] " Hans Verkuil
2017-11-17 8:43 ` Hans Verkuil
2017-11-20 15:05 ` Hans Verkuil
2017-11-20 15:05 ` Hans Verkuil
2017-11-20 15:06 ` Hans Verkuil
2017-11-20 20:13 ` John Stultz [this message]
2017-11-20 20:13 ` John Stultz
2017-11-20 20:57 ` [PATCHv2] " Hans Verkuil
2017-11-20 20:57 ` Hans Verkuil
2017-11-20 23:12 ` John Stultz
2017-11-20 23:12 ` John Stultz
2017-11-21 6:48 ` Laurent Pinchart
2017-11-21 8:18 ` Hans Verkuil
2017-11-21 8:18 ` Hans Verkuil
2017-11-21 8:17 ` [PATCHv3] " Hans Verkuil
2017-11-21 8:17 ` Hans Verkuil
2017-11-21 8:22 ` Laurent Pinchart
2017-11-23 0:22 ` John Stultz
2017-11-23 0:22 ` John Stultz
2017-11-30 7:02 ` Archit Taneja
2017-11-30 7:02 ` Archit Taneja
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CALAqxLV5mJB3Aphdxk527c9okjdaoXAOD436OF=fji3w0wWrNQ@mail.gmail.com' \
--to=john.stultz@linaro.org \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@linux.ie \
--cc=arnd@arndb.de \
--cc=bhumirks@gmail.com \
--cc=dan.carpenter@oracle.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=hans.verkuil@cisco.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.