All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Carpenter <dan.carpenter@oracle.com>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: linux-media@vger.kernel.org
Subject: re: [media] V4L: soc-camera: make (almost) all client drivers re-usable outside of the framework
Date: Mon, 7 Nov 2011 21:43:10 +0300	[thread overview]
Message-ID: <20111107184310.GY4682@mwanda> (raw)

[-- Attachment #1: Type: text/plain, Size: 978 bytes --]

Hi Guennadi,

Gcc complains about 2f0babb7e432 "[media] V4L: soc-camera: make
(almost) all client drivers re-usable outside of the framework"

include/media/soc_camera.h: In function ‘soc_camera_i2c_to_vdev’:
include/media/soc_camera.h:257:34: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]

--- a/include/media/soc_camera.h
+++ b/include/media/soc_camera.h
@@ -253,14 +253,14 @@ unsigned long soc_camera_apply_board_flags(struct soc_camera_link *icl,
 #include <linux/i2c.h>
  static inline struct video_device *soc_camera_i2c_to_vdev(const struct i2c_client *client)
   {
   -       struct soc_camera_device *icd = client->dev.platform_data;
   +       struct v4l2_subdev *sd = i2c_get_clientdata(client);
   +       struct soc_camera_device *icd = (struct soc_camera_device *)sd->grp_id;
           return icd ? icd->vdev : NULL;

sd->grp_id is a u32 so this doesn't work on 64 bit systems.

regards,
dan carpenter

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

                 reply	other threads:[~2011-11-07 18:42 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=20111107184310.GY4682@mwanda \
    --to=dan.carpenter@oracle.com \
    --cc=g.liakhovetski@gmx.de \
    --cc=linux-media@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.