diff --git a/drivers/media/video/saa6588.c b/drivers/media/video/saa6588.c
index 4d31c320ac174bb48def96ad7176b3561cc37716..5f4e9858cb9dc1664b89f0b718ffb75649e7629b 100644
--- a/drivers/media/video/saa6588.c
+++ b/drivers/media/video/saa6588.c
@@ -33,6 +33,7 @@
 
 #include <media/rds.h>
 #include <media/v4l2-device.h>
+#include <media/v4l2-chip-ident.h>
 #include <media/v4l2-i2c-drv-legacy.h>
 
 /* Addresses to scan */
@@ -431,6 +432,13 @@ static long saa6588_ioctl(struct v4l2_subdev *sd, unsigned int cmd, void *arg)
 	return 0;
 }
 
+static int saa6588_g_chip_ident(struct v4l2_subdev *sd, struct v4l2_dbg_chip_ident *chip)
+{
+	struct i2c_client *client = v4l2_get_subdevdata(sd);
+
+	return v4l2_chip_ident_i2c_client(client, chip, V4L2_IDENT_SAA6588, 0);
+}
+
 static int saa6588_command(struct i2c_client *client, unsigned cmd, void *arg)
 {
 	return v4l2_subdev_command(i2c_get_clientdata(client), cmd, arg);
@@ -439,6 +447,7 @@ static int saa6588_command(struct i2c_client *client, unsigned cmd, void *arg)
 /* ----------------------------------------------------------------------- */
 
 static const struct v4l2_subdev_core_ops saa6588_core_ops = {
+	.g_chip_ident = saa6588_g_chip_ident,
 	.ioctl = saa6588_ioctl,
 };
 
diff --git a/include/media/v4l2-chip-ident.h b/include/media/v4l2-chip-ident.h
index 4e2182e52a8ec08c1536ccc95d8eea5e93aa8e6b..bbe2bb6a596a9c4733fe261453a0f733a885b788 100644
--- a/include/media/v4l2-chip-ident.h
+++ b/include/media/v4l2-chip-ident.h
@@ -92,6 +92,9 @@ enum {
 	/* module tea6420: just ident 6420 */
 	V4L2_IDENT_TEA6420 = 6420,
 
+	/* module saa6588: just ident 6588 */
+	V4L2_IDENT_SAA6588 = 6588,
+
 	/* module saa6752hs: reserved range 6750-6759 */
 	V4L2_IDENT_SAA6752HS = 6752,
 	V4L2_IDENT_SAA6752HS_AC3 = 6753,