Re: dib3000mc.c and dib7000p.c compiler warnings

2009-06-15 Thread Patrick Boettcher

Hi Hans,
On Sun, 14 Jun 2009, Hans Verkuil wrote:


Hi Patrick,

The daily build reports these warnings for dib3000mc.c and dib7000p.c:

/marune/build/v4l-dvb-master/v4l/dib3000mc.c: In
function 'dib3000mc_i2c_enumeration':
/marune/build/v4l-dvb-master/v4l/dib3000mc.c:863: warning: the frame size of
1508 bytes is larger than 1024 bytes

/marune/build/v4l-dvb-master/v4l/dib7000p.c: In
function 'dib7000p_i2c_enumeration':
/marune/build/v4l-dvb-master/v4l/dib7000p.c:1341: warning: the frame size of
1568 bytes is larger than 1024 bytes

In both cases a big state struct is allocated on the stack. Would it be
possible to optimize that?


I agree that this is ugly and in fact I already have a solution for that. 
But it's not ready as a patch nor can it published :(.


I won't be able to work on it before mid-July. But then I will provide a 
solution.


thanks for raising this important problem,
Patrick.
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


dib3000mc.c and dib7000p.c compiler warnings

2009-06-14 Thread Hans Verkuil
Hi Patrick,

The daily build reports these warnings for dib3000mc.c and dib7000p.c:

/marune/build/v4l-dvb-master/v4l/dib3000mc.c: In 
function 'dib3000mc_i2c_enumeration':
/marune/build/v4l-dvb-master/v4l/dib3000mc.c:863: warning: the frame size of 
1508 bytes is larger than 1024 bytes

/marune/build/v4l-dvb-master/v4l/dib7000p.c: In 
function 'dib7000p_i2c_enumeration':
/marune/build/v4l-dvb-master/v4l/dib7000p.c:1341: warning: the frame size of 
1568 bytes is larger than 1024 bytes

In both cases a big state struct is allocated on the stack. Would it be 
possible to optimize that?

If you are not the right person to deal with this, who might it be?

Regards,

Hans

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html