From: Hirokazu Takata Here is a patch to update AR camera device driver. drivers/media/video/arv.c: - Remove warnings; use module_param() instead of MODULE_PARM(), because MODULE_PARM() is deprecated. - Fix white-space damages. Signed-off-by: Hirokazu Takata Signed-off-by: Andrew Morton --- 25-akpm/drivers/media/video/arv.c | 294 +++++++++++++++++++++----------------- 1 files changed, 166 insertions(+), 128 deletions(-) diff -puN drivers/media/video/arv.c~media-update-drivers-media-video-arvc drivers/media/video/arv.c --- 25/drivers/media/video/arv.c~media-update-drivers-media-video-arvc Thu Nov 18 15:25:41 2004 +++ 25-akpm/drivers/media/video/arv.c Thu Nov 18 15:25:41 2004 @@ -54,7 +54,7 @@ */ #define USE_INT 0 /* Don't modify */ -#define VERSION "0.02" +#define VERSION "0.03" #define ar_inl(addr) inl((unsigned long)(addr)) #define ar_outl(val, addr) outl((unsigned long)(val),(unsigned long)(addr)) @@ -89,12 +89,12 @@ extern struct cpuinfo_m32r boot_cpu_data #define MAX_AR_LINE_BYTES AR_LINE_BYTES_VGA /* frame size & type */ -#define AR_FRAME_BYTES_VGA (AR_WIDTH_VGA * AR_HEIGHT_VGA * \ - AR_BYTES_PER_PIXEL) -#define AR_FRAME_BYTES_QVGA (AR_WIDTH_QVGA * AR_HEIGHT_QVGA * \ - AR_BYTES_PER_PIXEL) -#define MAX_AR_FRAME_BYTES (MAX_AR_WIDTH * MAX_AR_HEIGHT * \ - AR_BYTES_PER_PIXEL) +#define AR_FRAME_BYTES_VGA \ + (AR_WIDTH_VGA * AR_HEIGHT_VGA * AR_BYTES_PER_PIXEL) +#define AR_FRAME_BYTES_QVGA \ + (AR_WIDTH_QVGA * AR_HEIGHT_QVGA * AR_BYTES_PER_PIXEL) +#define MAX_AR_FRAME_BYTES \ + (MAX_AR_WIDTH * MAX_AR_HEIGHT * AR_BYTES_PER_PIXEL) #define AR_MAX_FRAME 15 @@ -108,13 +108,13 @@ extern struct cpuinfo_m32r boot_cpu_data struct ar_device { struct video_device *vdev; - unsigned int start_capture; // duaring capture in INT. mode. + unsigned int start_capture; /* duaring capture in INT. mode. */ #if USE_INT - unsigned char *line_buff; // DMA line buffer + unsigned char *line_buff; /* DMA line buffer */ #endif - unsigned char *frame[MAX_AR_HEIGHT]; // frame data - short size; // capture size - short mode; // capture mode + unsigned char *frame[MAX_AR_HEIGHT]; /* frame data */ + short size; /* capture size */ + short mode; /* capture mode */ int width, height; int frame_bytes, line_bytes; wait_queue_head_t wait; @@ -126,7 +126,7 @@ static unsigned char yuv[MAX_AR_FRAME_BY /* module parameters */ /* default frequency */ -#define DEFAULT_FREQ 50 // 50 or 75 (MHz) is available as BCLK +#define DEFAULT_FREQ 50 /* 50 or 75 (MHz) is available as BCLK */ static int freq = DEFAULT_FREQ; /* BCLK: available 50 or 70 (MHz) */ static int vga = 0; /* default mode(0:QVGA mode, other:VGA mode) */ static int vga_interlace = 0; /* 0 is normal mode for, else interlace mode */ @@ -136,64 +136,68 @@ MODULE_PARM(vga_interlace, "i"); static int ar_initialize(struct video_device *dev); +static inline void wait_for_vsync(void) +{ + while (ar_inl(ARVCR0) & ARVCR0_VDS) /* wait for VSYNC */ + cpu_relax(); + while (!(ar_inl(ARVCR0) & ARVCR0_VDS)) /* wait for VSYNC */ + cpu_relax(); +} + +static inline void wait_acknowledge(void) +{ + int i; + + for (i = 0; i < 1000; i++) + cpu_relax(); + while (ar_inl(PLDI2CSTS) & PLDI2CSTS_NOACK) + cpu_relax(); +} + /******************************************************************* * I2C functions *******************************************************************/ -void iic(int n, unsigned long addr, unsigned long data1, unsigned long data2, unsigned long data3) +void iic(int n, unsigned long addr, unsigned long data1, unsigned long data2, + unsigned long data3) { int i; /* Slave Address */ ar_outl(addr, PLDI2CDATA); + wait_for_vsync(); - while ( ar_inl(ARVCR0) & ARVCR0_VDS ); // wait for VSYNC - while ( !(ar_inl(ARVCR0) & ARVCR0_VDS) ); // wait for VSYNC /* Start */ ar_outl(1, PLDI2CCND); + wait_acknowledge(); - for(i=0; i<1000; i++); - while( ar_inl(PLDI2CSTS) & PLDI2CSTS_NOACK ); - - /* Trasfer data 1 */ + /* Transfer data 1 */ ar_outl(data1, PLDI2CDATA); - while ( ar_inl(ARVCR0) & ARVCR0_VDS ); // wait for VSYNC - while ( !(ar_inl(ARVCR0) & ARVCR0_VDS) ); // wait for VSYNC + wait_for_vsync(); ar_outl(PLDI2CSTEN_STEN, PLDI2CSTEN); + wait_acknowledge(); - /* Ack wait */ - for(i=0; i<1000; i++); - while( ar_inl(PLDI2CSTS) & PLDI2CSTS_NOACK ); - - /* Trasfer data 2 */ + /* Transfer data 2 */ ar_outl(data2, PLDI2CDATA); - while ( ar_inl(ARVCR0) & ARVCR0_VDS ); // wait for VSYNC - while ( !(ar_inl(ARVCR0) & ARVCR0_VDS) ); // wait for VSYNC + wait_for_vsync(); ar_outl(PLDI2CSTEN_STEN, PLDI2CSTEN); + wait_acknowledge(); - /* Ack wait */ - for(i=0; i<1000; i++); - - while( ar_inl(PLDI2CSTS) & PLDI2CSTS_NOACK ); - - if(n==3){ - /* Trasfer data 3 */ + if (n == 3) { + /* Transfer data 3 */ ar_outl(data3, PLDI2CDATA); - while ( ar_inl(ARVCR0) & ARVCR0_VDS ); // wait for VSYNC - while ( !(ar_inl(ARVCR0) & ARVCR0_VDS) ); // wait for VSYNC + wait_for_vsync(); ar_outl(PLDI2CSTEN_STEN, PLDI2CSTEN); - - /* Ack wait */ - for(i=0; i<10000; i++); - - while( ar_inl(PLDI2CSTS) & PLDI2CSTS_NOACK ); + wait_acknowledge(); } /* Stop */ - for(i=0; i<100; i++); + for (i = 0; i < 100; i++) + cpu_relax(); ar_outl(2, PLDI2CCND); ar_outl(2, PLDI2CCND); - while( ar_inl(PLDI2CSTS) & PLDI2CSTS_BB ); + while (ar_inl(PLDI2CSTS) & PLDI2CSTS_BB) + cpu_relax(); } @@ -216,7 +220,7 @@ void init_iic(void) } else if (freq == 50) { ar_outl(244, PLDI2CFREQ); /* BCLK = 50MHz */ } else { - ar_outl(244, PLDI2CFREQ); /* default:BCLK = 50MHz */ + ar_outl(244, PLDI2CFREQ); /* default: BCLK = 50MHz */ } ar_outl(0x1, PLDI2CCR); /* I2CCR Enable */ } @@ -226,6 +230,22 @@ void init_iic(void) * Video4Linux Interface functions * **************************************************************************/ + +static inline void disable_dma(void) +{ + ar_outl(0x8000, M32R_DMAEN_PORTL); /* disable DMA0 */ +} + +static inline void enable_dma(void) +{ + ar_outl(0x8080, M32R_DMAEN_PORTL); /* enable DMA0 */ +} + +static inline void clear_dma_status(void) +{ + ar_outl(0x8000, M32R_DMAEDET_PORTL); /* clear status */ +} + static inline void wait_for_vertical_sync(int exp_line) { #if CHECK_LOST @@ -233,16 +253,18 @@ static inline void wait_for_vertical_syn int l; /* - * check HCOUNT because we can not check vertual sync. + * check HCOUNT because we cannot check vertical sync. */ for (; tmout >= 0; tmout--) { l = ar_inl(ARVHCOUNT); - if (l == exp_line) break; + if (l == exp_line) + break; } if (tmout < 0) printk("arv: lost %d -> %d\n", exp_line, l); #else - while (ar_inl(ARVHCOUNT) != exp_line) ; + while (ar_inl(ARVHCOUNT) != exp_line) + cpu_relax(); #endif } @@ -262,24 +284,26 @@ static ssize_t ar_read(struct file *file DEBUG(1, "ar_read()\n"); - if (ar->size == AR_SIZE_QVGA) arvcr1 |= ARVCR1_QVGA; - if (ar->mode == AR_MODE_NORMAL) arvcr1 |= ARVCR1_NORMAL; + if (ar->size == AR_SIZE_QVGA) + arvcr1 |= ARVCR1_QVGA; + if (ar->mode == AR_MODE_NORMAL) + arvcr1 |= ARVCR1_NORMAL; down(&ar->lock); #if USE_INT local_irq_save(flags); - ar_outl(0x80000, M32R_DMAEN_PORTL); // disable DMA0 + disable_dma(); ar_outl(0xa1871300, M32R_DMA0CR0_PORTL); ar_outl(0x01000000, M32R_DMA0CR1_PORTL); - // set AR FIFO address as source(BSEL5) - ar_outl(ARDATA32, M32R_DMA0CSA_PORTL); // - ar_outl(ARDATA32, M32R_DMA0RSA_PORTL); // - ar_outl(ar->line_buff, M32R_DMA0CDA_PORTL); // destination address - ar_outl(ar->line_buff, M32R_DMA0RDA_PORTL); // reload address - ar_outl(ar->line_bytes, M32R_DMA0CBCUT_PORTL); // byte count(bytes) - ar_outl(ar->line_bytes, M32R_DMA0RBCUT_PORTL); // reload count (bytes) + /* set AR FIFO address as source(BSEL5) */ + ar_outl(ARDATA32, M32R_DMA0CSA_PORTL); + ar_outl(ARDATA32, M32R_DMA0RSA_PORTL); + ar_outl(ar->line_buff, M32R_DMA0CDA_PORTL); /* destination addr. */ + ar_outl(ar->line_buff, M32R_DMA0RDA_PORTL); /* reload address */ + ar_outl(ar->line_bytes, M32R_DMA0CBCUT_PORTL); /* byte count (bytes) */ + ar_outl(ar->line_bytes, M32R_DMA0RBCUT_PORTL); /* reload count (bytes) */ /* * Okey , kicks AR LSI to invoke an interrupt @@ -297,7 +321,7 @@ static ssize_t ar_read(struct file *file #else /* ! USE_INT */ /* polling */ ar_outl(arvcr1, ARVCR1); - ar_outl(0x80000, M32R_DMAEN_PORTL); // disable DMA0 + disable_dma(); ar_outl(0x8000, M32R_DMAEDET_PORTL); ar_outl(0xa0861300, M32R_DMA0CR0_PORTL); ar_outl(0x01000000, M32R_DMA0CR1_PORTL); @@ -307,7 +331,8 @@ static ssize_t ar_read(struct file *file ar_outl(ar->line_bytes, M32R_DMA0RBCUT_PORTL); local_irq_save(flags); - while (ar_inl(ARVHCOUNT) != 0) ; // wait for 0 + while (ar_inl(ARVHCOUNT) != 0) /* wait for 0 */ + cpu_relax(); if (ar->mode == AR_MODE_INTERLACE && ar->size == AR_SIZE_VGA) { for (h = 0; h < ar->height; h++) { wait_for_vertical_sync(h); @@ -316,20 +341,22 @@ static ssize_t ar_read(struct file *file else l = (((h - (AR_HEIGHT_VGA/2)) << 1) + 1); ar_outl(virt_to_phys(ar->frame[l]), M32R_DMA0CDA_PORTL); - ar_outl(0x8080, M32R_DMAEN_PORTL); // enable DMA0 - while (!(ar_inl(M32R_DMAEDET_PORTL) & 0x8000)) ; - ar_outl(0x80000, M32R_DMAEN_PORTL); // disable DMA0 - ar_outl(0x8000, M32R_DMAEDET_PORTL);// clear status + enable_dma(); + while (!(ar_inl(M32R_DMAEDET_PORTL) & 0x8000)) + cpu_relax(); + disable_dma(); + clear_dma_status(); ar_outl(0xa0861300, M32R_DMA0CR0_PORTL); } } else { for (h = 0; h < ar->height; h++) { wait_for_vertical_sync(h); ar_outl(virt_to_phys(ar->frame[h]), M32R_DMA0CDA_PORTL); - ar_outl(0x8080, M32R_DMAEN_PORTL); // enable DMA0 - while (!(ar_inl(M32R_DMAEDET_PORTL) & 0x8000)) ; - ar_outl(0x80000, M32R_DMAEN_PORTL); // disable DMA0 - ar_outl(0x8000, M32R_DMAEDET_PORTL);// clear status + enable_dma(); + while (!(ar_inl(M32R_DMAEDET_PORTL) & 0x8000)) + cpu_relax(); + disable_dma(); + clear_dma_status(); ar_outl(0xa0861300, M32R_DMA0CR0_PORTL); } } @@ -426,8 +453,8 @@ static int ar_do_ioctl(struct inode *ino { struct video_window *w = arg; DEBUG(1, "VIDIOCSWIN:\n"); - if(w->width != AR_WIDTH_QVGA && w->height != AR_HEIGHT_QVGA) - if(w->width != AR_WIDTH_VGA && w->height != AR_HEIGHT_VGA) + if ((w->width != AR_WIDTH_VGA || w->height != AR_HEIGHT_VGA) && + (w->width != AR_WIDTH_QVGA || w->height != AR_HEIGHT_QVGA)) return -EINVAL; down(&ar->lock); @@ -514,7 +541,8 @@ static int ar_do_ioctl(struct inode *ino return 0; } -static int ar_ioctl(struct inode *inode, struct file *file, unsigned int cmd, unsigned long arg) +static int ar_ioctl(struct inode *inode, struct file *file, unsigned int cmd, + unsigned long arg) { return video_usercopy(inode, file, cmd, arg, ar_do_ioctl); } @@ -530,7 +558,7 @@ static void ar_interrupt(int irq, void * unsigned int line_number; unsigned int arvcr1; - line_count = ar_inl(ARVHCOUNT); // line number + line_count = ar_inl(ARVHCOUNT); /* line number */ if (ar->mode == AR_MODE_INTERLACE && ar->size == AR_SIZE_VGA) { /* operations for interlace mode */ if ( line_count < (AR_HEIGHT_VGA/2) ) /* even line */ @@ -547,17 +575,21 @@ static void ar_interrupt(int irq, void * * It is an interrupt for line 0. * we have to start capture. */ - ar_outl(0x8000, M32R_DMAEN_PORTL); // disable DMA0 - //ar_outl(ar->line_buff, M32R_DMA0CDA_PORTL); // needless? + disable_dma(); +#if 0 + ar_outl(ar->line_buff, M32R_DMA0CDA_PORTL); /* needless? */ +#endif memcpy(ar->frame[0], ar->line_buff, ar->line_bytes); - //ar_outl(0xa1861300, M32R_DMA0CR0_PORTL); - ar_outl(0x8080, M32R_DMAEN_PORTL); // enable DMA0 - ar->start_capture = 1; // during capture +#if 0 + ar_outl(0xa1861300, M32R_DMA0CR0_PORTL); +#endif + enable_dma(); + ar->start_capture = 1; /* during capture */ return; } if (ar->start_capture == 1 && line_number <= (ar->height - 1)) { - ar_outl(0x8000, M32R_DMAEN_PORTL); // disable DMA0 + disable_dma(); memcpy(ar->frame[line_number], ar->line_buff, ar->line_bytes); /* @@ -570,13 +602,15 @@ static void ar_interrupt(int irq, void * /* disable AR interrupt request */ arvcr1 = ar_inl(ARVCR1); - arvcr1 &= ~ARVCR1_HIEN; // clear int. flag - ar_outl(arvcr1, ARVCR1); // disable + arvcr1 &= ~ARVCR1_HIEN; /* clear int. flag */ + ar_outl(arvcr1, ARVCR1); /* disable */ wake_up_interruptible(&ar->wait); } else { - //ar_outl(ar->line_buff, M32R_DMA0CDA_PORTL); - //ar_outl(0xa1861300, M32R_DMA0CR0_PORTL); - ar_outl(0x8080, M32R_DMAEN_PORTL); // enable DMA +#if 0 + ar_outl(ar->line_buff, M32R_DMA0CDA_PORTL); + ar_outl(0xa1861300, M32R_DMA0CR0_PORTL); +#endif + enable_dma(); } } } @@ -602,16 +636,20 @@ static int ar_initialize(struct video_de /* * initialize AR LSI */ - ar_outl(0, ARVCR0); // assert reset of AR LSI - for( i=0; i<0x18; i++); // wait for over 10 cycles @ 27MHz - ar_outl(ARVCR0_RST, ARVCR0); // negate reset of AR LSI (enable) - for( i=0; i<0x40d; i++); // wait for over 420 cycles @ 27MHz + ar_outl(0, ARVCR0); /* assert reset of AR LSI */ + for (i = 0; i < 0x18; i++) /* wait for over 10 cycles @ 27MHz */ + cpu_relax(); + ar_outl(ARVCR0_RST, ARVCR0); /* negate reset of AR LSI (enable) */ + for (i = 0; i < 0x40d; i++) /* wait for over 420 cycles @ 27MHz */ + cpu_relax(); /* AR uses INT3 of CPU as interrupt pin. */ ar_outl(ARINTSEL_INT3, ARINTSEL); - if (ar->size == AR_SIZE_QVGA) cr |= ARVCR1_QVGA; - if (ar->mode == AR_MODE_NORMAL) cr |= ARVCR1_NORMAL; + if (ar->size == AR_SIZE_QVGA) + cr |= ARVCR1_QVGA; + if (ar->mode == AR_MODE_NORMAL) + cr |= ARVCR1_NORMAL; ar_outl(cr, ARVCR1); /* @@ -620,18 +658,19 @@ static int ar_initialize(struct video_de */ init_iic(); - for( i=0; i<0x100000; i++) // > 0xa1d10, 56ms - if((ar_inl(ARVCR0) & ARVCR0_VDS)){ // VSYNC + for (i = 0; i < 0x100000; i++) { /* > 0xa1d10, 56ms */ + if ((ar_inl(ARVCR0) & ARVCR0_VDS)) { /* VSYNC */ found = 1; break; } + } - if(found == 0) + if (found == 0) return -ENODEV; printk("arv: Initializing "); - iic(2,0x78,0x11,0x01,0x00); // start + iic(2,0x78,0x11,0x01,0x00); /* start */ iic(3,0x78,0x12,0x00,0x06); iic(3,0x78,0x12,0x12,0x30); iic(3,0x78,0x12,0x15,0x58); @@ -651,7 +690,9 @@ static int ar_initialize(struct video_de printk("."); iic(2,0x78,0x8e,0x0c,0x00); iic(2,0x78,0x8f,0x00,0x00); -// iic(2,0x78,0x90,0x00,0x00); // AWB on=1 off=0 +#if 0 + iic(2,0x78,0x90,0x00,0x00); /* AWB on=1 off=0 */ +#endif iic(2,0x78,0x93,0x01,0x00); iic(2,0x78,0x94,0xcd,0x00); iic(2,0x78,0x95,0x00,0x00); @@ -668,34 +709,24 @@ static int ar_initialize(struct video_de iic(2,0x78,0x9e,0x2e,0x00); iic(2,0x78,0xb8,0x78,0x00); iic(2,0x78,0xba,0x05,0x00); -// iic(2,0x78,0x83,0x8c,0x00); // brightness - printk("."); - - // color correction #if 0 - iic(3,0x78,0x49,0x00,0x89); // a - iic(3,0x78,0x49,0x01,0x96); // b - iic(3,0x78,0x49,0x03,0x85); // c - iic(3,0x78,0x49,0x04,0x87); // d - iic(3,0x78,0x49,0x02,0x66); // e(Lo) - iic(3,0x78,0x49,0x05,0x84); // f(Lo) - iic(3,0x78,0x49,0x06,0x04); // e(Hi) - iic(3,0x78,0x49,0x07,0x04); // e(Hi) - iic(2,0x78,0x48,0x01,0x00); // on=1 off=0 -#else - iic(3,0x78,0x49,0x00,0x95); // a - iic(3,0x78,0x49,0x01,0x96); // b - iic(3,0x78,0x49,0x03,0x85); // c - iic(3,0x78,0x49,0x04,0x97); // d - iic(3,0x78,0x49,0x02,0x7e); // e(Lo) - iic(3,0x78,0x49,0x05,0xa4); // f(Lo) - iic(3,0x78,0x49,0x06,0x04); // e(Hi) - iic(3,0x78,0x49,0x07,0x04); // e(Hi) - iic(2,0x78,0x48,0x01,0x00); // on=1 off=0 + iic(2,0x78,0x83,0x8c,0x00); /* brightness */ #endif + printk("."); + + /* color correction */ + iic(3,0x78,0x49,0x00,0x95); /* a */ + iic(3,0x78,0x49,0x01,0x96); /* b */ + iic(3,0x78,0x49,0x03,0x85); /* c */ + iic(3,0x78,0x49,0x04,0x97); /* d */ + iic(3,0x78,0x49,0x02,0x7e); /* e(Lo) */ + iic(3,0x78,0x49,0x05,0xa4); /* f(Lo) */ + iic(3,0x78,0x49,0x06,0x04); /* e(Hi) */ + iic(3,0x78,0x49,0x07,0x04); /* e(Hi) */ + iic(2,0x78,0x48,0x01,0x00); /* on=1 off=0 */ printk("."); - iic(2,0x78,0x11,0x00,0x00); // end + iic(2,0x78,0x11,0x00,0x00); /* end */ printk(" done\n"); return 0; } @@ -768,12 +799,12 @@ static int __init ar_init(void) } ar->vdev = video_device_alloc(); - if(!ar->vdev){ + if (!ar->vdev) { printk(KERN_ERR "arv: video_device_alloc() failed\n"); return -ENOMEM; } memcpy(ar->vdev, &ar_template, sizeof(ar_template)); - ar->vdev->priv = ar; + ar->vdev->priv = ar; if (vga) { ar->width = AR_WIDTH_VGA; @@ -797,14 +828,14 @@ static int __init ar_init(void) init_waitqueue_head(&ar->wait); #if USE_INT - if (request_irq(M32R_IRQ_INT3, ar_interrupt, 0, "arv", ar)){ + if (request_irq(M32R_IRQ_INT3, ar_interrupt, 0, "arv", ar)) { printk("arv: request_irq(%d) failed.\n", M32R_IRQ_INT3); ret = -EIO; goto out_irq; } #endif - if (ar_initialize(ar->vdev) != 0 ){ + if (ar_initialize(ar->vdev) != 0) { printk("arv: M64278 not found.\n"); ret = -ENODEV; goto out_dev; @@ -816,7 +847,7 @@ static int __init ar_init(void) * device is named "video[0-64]". * video_register_device() initializes h/w using ar_initialize(). */ - if (video_register_device(ar->vdev, VFL_TYPE_GRABBER, video_nr)!=0) { + if (video_register_device(ar->vdev, VFL_TYPE_GRABBER, video_nr) != 0) { /* return -1, -ENFILE(full) or others */ printk("arv: register video (Colour AR) failed.\n"); ret = -ENODEV; @@ -831,13 +862,18 @@ static int __init ar_init(void) out_dev: #if USE_INT free_irq(M32R_IRQ_INT3, ar); + out_irq: #endif - for (i = 0; i < MAX_AR_HEIGHT; i++) - if (ar->frame[i]) kfree(ar->frame[i]); + for (i = 0; i < MAX_AR_HEIGHT; i++) { + if (ar->frame[i]) + kfree(ar->frame[i]); + } + out_line_buff: #if USE_INT kfree(ar->line_buff); + out_end: #endif return ret; @@ -863,8 +899,10 @@ static void __exit ar_cleanup_module(voi #if USE_INT free_irq(M32R_IRQ_INT3, ar); #endif - for (i = 0; i < MAX_AR_HEIGHT; i++) - if (ar->frame[i]) kfree(ar->frame[i]); + for (i = 0; i < MAX_AR_HEIGHT; i++) { + if (ar->frame[i]) + kfree(ar->frame[i]); + } #if USE_INT kfree(ar->line_buff); #endif _