From mboxrd@z Thu Jan  1 00:00:00 1970
From: Aneesh V <aneesh@ti.com>
Date: Tue, 17 May 2011 12:00:34 +0530
Subject: [U-Boot] [PATCH v2 01/22] mkimage: Add OMAP boot image support
In-Reply-To: <201105161442.50935.vapier@gentoo.org>
References: <1298893591-17636-1-git-send-email-aneesh@ti.com>
	<201105152255.08249.vapier@gentoo.org> <4DD0FC58.7040007@ti.com>
	<201105161442.50935.vapier@gentoo.org>
Message-ID: <4DD2160A.20903@ti.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Mike,

On Tuesday 17 May 2011 12:12 AM, Mike Frysinger wrote:
> On Monday, May 16, 2011 06:28:40 Aneesh V wrote:
>> On Monday 16 May 2011 08:25 AM, Mike Frysinger wrote:
>>> On Sunday, May 15, 2011 21:52:53 Mike Frysinger wrote:
>>>> On Sunday, May 15, 2011 11:21:19 Aneesh V wrote:
>>>>> +static void omapimage_print_header(const void *ptr)
>>>>> +{
>>>>> +	struct ch_toc *toc = (struct ch_toc *)ptr;
>>>>
>>>> you're casting away the void.  something is fundamentally broken here.
>>>
>>> err, obviously i meant "const" instead of "void" ...
>>
>> This is not my code.
>
> you're submitting the patch with only your s-o-b on it.  that means you're
> responsible for it all.

No. both From and s-o-b are John's on this patch.

>
>>    static void omapimage_print_header(const void *ptr)
>>    {
>> -	struct ch_toc *toc = (struct ch_toc *)ptr;
>> +	const struct ch_toc *toc =  (const struct ch_toc *)ptr;
>
> drop the cast entirely ... this isnt C++ after all:
> 	const struct ch_toc *toc =  ptr;

ok.

best regards,
Aneesh