Skip to content

L
linux
Commit 029d6177 authored by Lars-Peter Clausen's avatar Lars-Peter Clausen Committed by Mauro Carvalho Chehab
Browse files
Options

[media] adv7180: Cleanup register define naming 

Consistently prefix register defines with ADV7180_REG. Also remove the "ADI"
from register names, the ADV7180 prefix should provide enough of a namespace
separation.
Signed-off-by: default avatarLars-Peter Clausen <lars@metafoo.de>
Acked-by: default avatarFederico Vaga <federico.vaga@gmail.com>
Acked-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent f3e991d4
Hide whitespace changes
Inline Side-by-side
Showing with 52 additions and 53 deletions
drivers/media/i2c/adv7180.c
View file @ 029d6177
...@@ -31,7 +31,7 @@ ...@@ -31,7 +31,7 @@
#include <media/v4l2-ctrls.h> #include <media/v4l2-ctrls.h>
#include <linux/mutex.h> #include <linux/mutex.h>
#define ADV7180_INPUT_CONTROL_REG 0x00 #define ADV7180_REG_INPUT_CONTROL 0x00
#define ADV7180_INPUT_CONTROL_AD_PAL_BG_NTSC_J_SECAM 0x00 #define ADV7180_INPUT_CONTROL_AD_PAL_BG_NTSC_J_SECAM 0x00
#define ADV7180_INPUT_CONTROL_AD_PAL_BG_NTSC_J_SECAM_PED 0x10 #define ADV7180_INPUT_CONTROL_AD_PAL_BG_NTSC_J_SECAM_PED 0x10
#define ADV7180_INPUT_CONTROL_AD_PAL_N_NTSC_J_SECAM 0x20 #define ADV7180_INPUT_CONTROL_AD_PAL_N_NTSC_J_SECAM 0x20
...@@ -50,36 +50,36 @@ ...@@ -50,36 +50,36 @@
#define ADV7180_INPUT_CONTROL_PAL_SECAM_PED 0xf0 #define ADV7180_INPUT_CONTROL_PAL_SECAM_PED 0xf0
#define ADV7180_INPUT_CONTROL_INSEL_MASK 0x0f #define ADV7180_INPUT_CONTROL_INSEL_MASK 0x0f
#define ADV7180_EXTENDED_OUTPUT_CONTROL_REG 0x04 #define ADV7180_REG_EXTENDED_OUTPUT_CONTROL 0x04
#define ADV7180_EXTENDED_OUTPUT_CONTROL_NTSCDIS 0xC5 #define ADV7180_EXTENDED_OUTPUT_CONTROL_NTSCDIS 0xC5
#define ADV7180_AUTODETECT_ENABLE_REG 0x07 #define ADV7180_REG_AUTODETECT_ENABLE 0x07
#define ADV7180_AUTODETECT_DEFAULT 0x7f #define ADV7180_AUTODETECT_DEFAULT 0x7f
/* Contrast */ /* Contrast */
#define ADV7180_CON_REG 0x08 /*Unsigned */ #define ADV7180_REG_CON 0x08 /*Unsigned */
#define ADV7180_CON_MIN 0 #define ADV7180_CON_MIN 0
#define ADV7180_CON_DEF 128 #define ADV7180_CON_DEF 128
#define ADV7180_CON_MAX 255 #define ADV7180_CON_MAX 255
/* Brightness*/ /* Brightness*/
#define ADV7180_BRI_REG 0x0a /*Signed */ #define ADV7180_REG_BRI 0x0a /*Signed */
#define ADV7180_BRI_MIN -128 #define ADV7180_BRI_MIN -128
#define ADV7180_BRI_DEF 0 #define ADV7180_BRI_DEF 0
#define ADV7180_BRI_MAX 127 #define ADV7180_BRI_MAX 127
/* Hue */ /* Hue */
#define ADV7180_HUE_REG 0x0b /*Signed, inverted */ #define ADV7180_REG_HUE 0x0b /*Signed, inverted */
#define ADV7180_HUE_MIN -127 #define ADV7180_HUE_MIN -127
#define ADV7180_HUE_DEF 0 #define ADV7180_HUE_DEF 0
#define ADV7180_HUE_MAX 128 #define ADV7180_HUE_MAX 128
#define ADV7180_ADI_CTRL_REG 0x0e #define ADV7180_REG_CTRL 0x0e
#define ADV7180_ADI_CTRL_IRQ_SPACE 0x20 #define ADV7180_CTRL_IRQ_SPACE 0x20
#define ADV7180_PWR_MAN_REG 0x0f #define ADV7180_REG_PWR_MAN 0x0f
#define ADV7180_PWR_MAN_ON 0x04 #define ADV7180_PWR_MAN_ON 0x04
#define ADV7180_PWR_MAN_OFF 0x24 #define ADV7180_PWR_MAN_OFF 0x24
#define ADV7180_PWR_MAN_RES 0x80 #define ADV7180_PWR_MAN_RES 0x80
#define ADV7180_STATUS1_REG 0x10 #define ADV7180_REG_STATUS1 0x10
#define ADV7180_STATUS1_IN_LOCK 0x01 #define ADV7180_STATUS1_IN_LOCK 0x01
#define ADV7180_STATUS1_AUTOD_MASK 0x70 #define ADV7180_STATUS1_AUTOD_MASK 0x70
#define ADV7180_STATUS1_AUTOD_NTSM_M_J 0x00 #define ADV7180_STATUS1_AUTOD_NTSM_M_J 0x00
...@@ -91,33 +91,33 @@ ...@@ -91,33 +91,33 @@
#define ADV7180_STATUS1_AUTOD_PAL_COMB 0x60 #define ADV7180_STATUS1_AUTOD_PAL_COMB 0x60
#define ADV7180_STATUS1_AUTOD_SECAM_525 0x70 #define ADV7180_STATUS1_AUTOD_SECAM_525 0x70
#define ADV7180_IDENT_REG 0x11 #define ADV7180_REG_IDENT 0x11
#define ADV7180_ID_7180 0x18 #define ADV7180_ID_7180 0x18
#define ADV7180_ICONF1_ADI 0x40 #define ADV7180_REG_ICONF1 0x40
#define ADV7180_ICONF1_ACTIVE_LOW 0x01 #define ADV7180_ICONF1_ACTIVE_LOW 0x01
#define ADV7180_ICONF1_PSYNC_ONLY 0x10 #define ADV7180_ICONF1_PSYNC_ONLY 0x10
#define ADV7180_ICONF1_ACTIVE_TO_CLR 0xC0 #define ADV7180_ICONF1_ACTIVE_TO_CLR 0xC0
/* Saturation */ /* Saturation */
#define ADV7180_SD_SAT_CB_REG 0xe3 /*Unsigned */ #define ADV7180_REG_SD_SAT_CB 0xe3 /*Unsigned */
#define ADV7180_SD_SAT_CR_REG 0xe4 /*Unsigned */ #define ADV7180_REG_SD_SAT_CR 0xe4 /*Unsigned */
#define ADV7180_SAT_MIN 0 #define ADV7180_SAT_MIN 0
#define ADV7180_SAT_DEF 128 #define ADV7180_SAT_DEF 128
#define ADV7180_SAT_MAX 255 #define ADV7180_SAT_MAX 255
#define ADV7180_IRQ1_LOCK 0x01 #define ADV7180_IRQ1_LOCK 0x01
#define ADV7180_IRQ1_UNLOCK 0x02 #define ADV7180_IRQ1_UNLOCK 0x02
#define ADV7180_ISR1_ADI 0x42 #define ADV7180_REG_ISR1 0x42
#define ADV7180_ICR1_ADI 0x43 #define ADV7180_REG_ICR1 0x43
#define ADV7180_IMR1_ADI 0x44 #define ADV7180_REG_IMR1 0x44
#define ADV7180_IMR2_ADI 0x48 #define ADV7180_REG_IMR2 0x48
#define ADV7180_IRQ3_AD_CHANGE 0x08 #define ADV7180_IRQ3_AD_CHANGE 0x08
#define ADV7180_ISR3_ADI 0x4A #define ADV7180_REG_ISR3 0x4A
#define ADV7180_ICR3_ADI 0x4B #define ADV7180_REG_ICR3 0x4B
#define ADV7180_IMR3_ADI 0x4C #define ADV7180_REG_IMR3 0x4C
#define ADV7180_IMR4_ADI 0x50 #define ADV7180_REG_IMR4 0x50
#define ADV7180_NTSC_V_BIT_END_REG 0xE6 #define ADV7180_REG_NTSC_V_BIT_END 0xE6
#define ADV7180_NTSC_V_BIT_END_MANUAL_NVEND 0x4F #define ADV7180_NTSC_V_BIT_END_MANUAL_NVEND 0x4F
struct adv7180_state { struct adv7180_state {
...@@ -196,7 +196,7 @@ static u32 adv7180_status_to_v4l2(u8 status1) ...@@ -196,7 +196,7 @@ static u32 adv7180_status_to_v4l2(u8 status1)
static int __adv7180_status(struct i2c_client *client, u32 *status, static int __adv7180_status(struct i2c_client *client, u32 *status,
v4l2_std_id *std) v4l2_std_id *std)
{ {
int status1 = i2c_smbus_read_byte_data(client, ADV7180_STATUS1_REG); int status1 = i2c_smbus_read_byte_data(client, ADV7180_REG_STATUS1);
if (status1 < 0) if (status1 < 0)
return status1; return status1;
...@@ -247,14 +247,13 @@ static int adv7180_s_routing(struct v4l2_subdev *sd, u32 input, ...@@ -247,14 +247,13 @@ static int adv7180_s_routing(struct v4l2_subdev *sd, u32 input,
if ((input & ADV7180_INPUT_CONTROL_INSEL_MASK) != input) if ((input & ADV7180_INPUT_CONTROL_INSEL_MASK) != input)
goto out; goto out;
ret = i2c_smbus_read_byte_data(client, ADV7180_INPUT_CONTROL_REG); ret = i2c_smbus_read_byte_data(client, ADV7180_REG_INPUT_CONTROL);
if (ret < 0) if (ret < 0)
goto out; goto out;
ret &= ~ADV7180_INPUT_CONTROL_INSEL_MASK; ret &= ~ADV7180_INPUT_CONTROL_INSEL_MASK;
ret = i2c_smbus_write_byte_data(client, ret = i2c_smbus_write_byte_data(client,
ADV7180_INPUT_CONTROL_REG, ret | input); ADV7180_REG_INPUT_CONTROL, ret | input);
state->input = input; state->input = input;
out: out:
mutex_unlock(&state->mutex); mutex_unlock(&state->mutex);
...@@ -284,7 +283,7 @@ static int adv7180_s_std(struct v4l2_subdev *sd, v4l2_std_id std) ...@@ -284,7 +283,7 @@ static int adv7180_s_std(struct v4l2_subdev *sd, v4l2_std_id std)
/* all standards -> autodetect */ /* all standards -> autodetect */
if (std == V4L2_STD_ALL) { if (std == V4L2_STD_ALL) {
ret = ret =
i2c_smbus_write_byte_data(client, ADV7180_INPUT_CONTROL_REG, i2c_smbus_write_byte_data(client, ADV7180_REG_INPUT_CONTROL,
ADV7180_INPUT_CONTROL_AD_PAL_BG_NTSC_J_SECAM ADV7180_INPUT_CONTROL_AD_PAL_BG_NTSC_J_SECAM
| state->input); | state->input);
if (ret < 0) if (ret < 0)
...@@ -298,7 +297,7 @@ static int adv7180_s_std(struct v4l2_subdev *sd, v4l2_std_id std) ...@@ -298,7 +297,7 @@ static int adv7180_s_std(struct v4l2_subdev *sd, v4l2_std_id std)
goto out; goto out;
ret = i2c_smbus_write_byte_data(client, ret = i2c_smbus_write_byte_data(client,
ADV7180_INPUT_CONTROL_REG, ADV7180_REG_INPUT_CONTROL,
ret | state->input); ret | state->input);
if (ret < 0) if (ret < 0)
goto out; goto out;
...@@ -322,7 +321,7 @@ static int adv7180_set_power(struct adv7180_state *state, ...@@ -322,7 +321,7 @@ static int adv7180_set_power(struct adv7180_state *state,
else else
val = ADV7180_PWR_MAN_OFF; val = ADV7180_PWR_MAN_OFF;
return i2c_smbus_write_byte_data(client, ADV7180_PWR_MAN_REG, val); return i2c_smbus_write_byte_data(client, ADV7180_REG_PWR_MAN, val);
} }
static int adv7180_s_power(struct v4l2_subdev *sd, int on) static int adv7180_s_power(struct v4l2_subdev *sd, int on)
...@@ -356,25 +355,25 @@ static int adv7180_s_ctrl(struct v4l2_ctrl *ctrl) ...@@ -356,25 +355,25 @@ static int adv7180_s_ctrl(struct v4l2_ctrl *ctrl)
val = ctrl->val; val = ctrl->val;
switch (ctrl->id) { switch (ctrl->id) {
case V4L2_CID_BRIGHTNESS: case V4L2_CID_BRIGHTNESS:
ret = i2c_smbus_write_byte_data(client, ADV7180_BRI_REG, val); ret = i2c_smbus_write_byte_data(client, ADV7180_REG_BRI, val);
break; break;
case V4L2_CID_HUE: case V4L2_CID_HUE:
/*Hue is inverted according to HSL chart */ /*Hue is inverted according to HSL chart */
ret = i2c_smbus_write_byte_data(client, ADV7180_HUE_REG, -val); ret = i2c_smbus_write_byte_data(client, ADV7180_REG_HUE, -val);
break; break;
case V4L2_CID_CONTRAST: case V4L2_CID_CONTRAST:
ret = i2c_smbus_write_byte_data(client, ADV7180_CON_REG, val); ret = i2c_smbus_write_byte_data(client, ADV7180_REG_CON, val);
break; break;
case V4L2_CID_SATURATION: case V4L2_CID_SATURATION:
/* /*
*This could be V4L2_CID_BLUE_BALANCE/V4L2_CID_RED_BALANCE *This could be V4L2_CID_BLUE_BALANCE/V4L2_CID_RED_BALANCE
*Let's not confuse the user, everybody understands saturation *Let's not confuse the user, everybody understands saturation
*/ */
ret = i2c_smbus_write_byte_data(client, ADV7180_SD_SAT_CB_REG, ret = i2c_smbus_write_byte_data(client, ADV7180_REG_SD_SAT_CB,
val); val);
if (ret < 0) if (ret < 0)
break; break;
ret = i2c_smbus_write_byte_data(client, ADV7180_SD_SAT_CR_REG, ret = i2c_smbus_write_byte_data(client, ADV7180_REG_SD_SAT_CR,
val); val);
break; break;
default: default:
...@@ -488,12 +487,12 @@ static irqreturn_t adv7180_irq(int irq, void *devid) ...@@ -488,12 +487,12 @@ static irqreturn_t adv7180_irq(int irq, void *devid)
u8 isr3; u8 isr3;
mutex_lock(&state->mutex); mutex_lock(&state->mutex);
i2c_smbus_write_byte_data(client, ADV7180_ADI_CTRL_REG, i2c_smbus_write_byte_data(client, ADV7180_REG_CTRL,
ADV7180_ADI_CTRL_IRQ_SPACE); ADV7180_CTRL_IRQ_SPACE);
isr3 = i2c_smbus_read_byte_data(client, ADV7180_ISR3_ADI); isr3 = i2c_smbus_read_byte_data(client, ADV7180_REG_ISR3);
/* clear */ /* clear */
i2c_smbus_write_byte_data(client, ADV7180_ICR3_ADI, isr3); i2c_smbus_write_byte_data(client, ADV7180_REG_ICR3, isr3);
i2c_smbus_write_byte_data(client, ADV7180_ADI_CTRL_REG, 0); i2c_smbus_write_byte_data(client, ADV7180_REG_CTRL, 0);
if (isr3 & ADV7180_IRQ3_AD_CHANGE && state->autodetect) if (isr3 & ADV7180_IRQ3_AD_CHANGE && state->autodetect)
__adv7180_status(client, NULL, &state->curr_norm); __adv7180_status(client, NULL, &state->curr_norm);
...@@ -510,7 +509,7 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state) ...@@ -510,7 +509,7 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state)
/* Enable autodetection */ /* Enable autodetection */
if (state->autodetect) { if (state->autodetect) {
ret = ret =
i2c_smbus_write_byte_data(client, ADV7180_INPUT_CONTROL_REG, i2c_smbus_write_byte_data(client, ADV7180_REG_INPUT_CONTROL,
ADV7180_INPUT_CONTROL_AD_PAL_BG_NTSC_J_SECAM ADV7180_INPUT_CONTROL_AD_PAL_BG_NTSC_J_SECAM
| state->input); | state->input);
if (ret < 0) if (ret < 0)
...@@ -518,7 +517,7 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state) ...@@ -518,7 +517,7 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state)
ret = ret =
i2c_smbus_write_byte_data(client, i2c_smbus_write_byte_data(client,
ADV7180_AUTODETECT_ENABLE_REG, ADV7180_REG_AUTODETECT_ENABLE,
ADV7180_AUTODETECT_DEFAULT); ADV7180_AUTODETECT_DEFAULT);
if (ret < 0) if (ret < 0)
return ret; return ret;
...@@ -528,7 +527,7 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state) ...@@ -528,7 +527,7 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state)
return ret; return ret;
ret = ret =
i2c_smbus_write_byte_data(client, ADV7180_INPUT_CONTROL_REG, i2c_smbus_write_byte_data(client, ADV7180_REG_INPUT_CONTROL,
ret | state->input); ret | state->input);
if (ret < 0) if (ret < 0)
return ret; return ret;
...@@ -536,14 +535,14 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state) ...@@ -536,14 +535,14 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state)
} }
/* ITU-R BT.656-4 compatible */ /* ITU-R BT.656-4 compatible */
ret = i2c_smbus_write_byte_data(client, ret = i2c_smbus_write_byte_data(client,
ADV7180_EXTENDED_OUTPUT_CONTROL_REG, ADV7180_REG_EXTENDED_OUTPUT_CONTROL,
ADV7180_EXTENDED_OUTPUT_CONTROL_NTSCDIS); ADV7180_EXTENDED_OUTPUT_CONTROL_NTSCDIS);
if (ret < 0) if (ret < 0)
return ret; return ret;
/* Manually set V bit end position in NTSC mode */ /* Manually set V bit end position in NTSC mode */
ret = i2c_smbus_write_byte_data(client, ret = i2c_smbus_write_byte_data(client,
ADV7180_NTSC_V_BIT_END_REG, ADV7180_REG_NTSC_V_BIT_END,
ADV7180_NTSC_V_BIT_END_MANUAL_NVEND); ADV7180_NTSC_V_BIT_END_MANUAL_NVEND);
if (ret < 0) if (ret < 0)
return ret; return ret;
...@@ -553,37 +552,37 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state) ...@@ -553,37 +552,37 @@ static int init_device(struct i2c_client *client, struct adv7180_state *state)
/* register for interrupts */ /* register for interrupts */
if (state->irq > 0) { if (state->irq > 0) {
ret = i2c_smbus_write_byte_data(client, ADV7180_ADI_CTRL_REG, ret = i2c_smbus_write_byte_data(client, ADV7180_REG_CTRL,
ADV7180_ADI_CTRL_IRQ_SPACE); ADV7180_CTRL_IRQ_SPACE);
if (ret < 0) if (ret < 0)
goto err; goto err;
/* config the Interrupt pin to be active low */ /* config the Interrupt pin to be active low */
ret = i2c_smbus_write_byte_data(client, ADV7180_ICONF1_ADI, ret = i2c_smbus_write_byte_data(client, ADV7180_REG_ICONF1,
ADV7180_ICONF1_ACTIVE_LOW | ADV7180_ICONF1_ACTIVE_LOW |
ADV7180_ICONF1_PSYNC_ONLY); ADV7180_ICONF1_PSYNC_ONLY);
if (ret < 0) if (ret < 0)
goto err; goto err;
ret = i2c_smbus_write_byte_data(client, ADV7180_IMR1_ADI, 0); ret = i2c_smbus_write_byte_data(client, ADV7180_REG_IMR1, 0);
if (ret < 0) if (ret < 0)
goto err; goto err;
ret = i2c_smbus_write_byte_data(client, ADV7180_IMR2_ADI, 0); ret = i2c_smbus_write_byte_data(client, ADV7180_REG_IMR2, 0);
if (ret < 0) if (ret < 0)
goto err; goto err;
/* enable AD change interrupts interrupts */ /* enable AD change interrupts interrupts */
ret = i2c_smbus_write_byte_data(client, ADV7180_IMR3_ADI, ret = i2c_smbus_write_byte_data(client, ADV7180_REG_IMR3,
ADV7180_IRQ3_AD_CHANGE); ADV7180_IRQ3_AD_CHANGE);
if (ret < 0) if (ret < 0)
goto err; goto err;
ret = i2c_smbus_write_byte_data(client, ADV7180_IMR4_ADI, 0); ret = i2c_smbus_write_byte_data(client, ADV7180_REG_IMR4, 0);
if (ret < 0) if (ret < 0)
goto err; goto err;
ret = i2c_smbus_write_byte_data(client, ADV7180_ADI_CTRL_REG, ret = i2c_smbus_write_byte_data(client, ADV7180_REG_CTRL,
0); 0);
if (ret < 0) if (ret < 0)
goto err; goto err;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7