Re: [PATCH] viafb: I2C/DDC LCD detection for VIA framebuffer

From: Dzianis Kahanovich
Date: Mon Nov 22 2010 - 09:20:23 EST


Sorry for mistake. I sure to remove "const" but revert to older code later. I
repost fixed patch ("v2") to avoid formal problems. Sorry.

Dan Carpenter wrote:
> On Mon, Nov 22, 2010 at 02:31:55PM +0200, Dzianis Kahanovich wrote:
>> +static void via_i2c_detect(void)
>> +{
>> + const u8 *edid;
> ^^^^^
>
>> +free_edid:
>> + kfree(edid);
> ^^^^^^^^^^^
>
> That probably shouldn't be const.
>
>> + if (n > 1)
>> + break;
>> + }
>> +}
>> +
>>
>
> regards,
> dan carpenter
>
>


--
WBR, Dzianis Kahanovich AKA Denis Kaganovich, http://mahatma.bspu.unibel.by/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/