Why would anyone do so for you?

The correct code can be found in this article
http://extremeelectronics.co.in/robotics/obstacle-avoiding-robot-using-avr-atmega32-%e2%80%93-part-ii/

There’s a flaw. your code will work fine UNLESS one doesnt use more than 1 channel in a row.

uint16_t ReadADC(uint8_t ch)
{
//Select ADC Channel ch must be 0-7
ch=ch&0b00000111;
ADMUX|=ch; //>>> This is just wrong!!!
.
.
}
imagine you read channel 2 after ch 1, then with your code ch 1 to ADMUX would still be written there since you havent cleared it & you would be just ‘appending’ ch 2 to ADMUX, so you end up reading channel 3! (If one uses a loop to cycle thru channels, then ultimately he would always get value from channel 7)

This is enough to give a newbie headache worth of days if he decides to upgrade to >1 channels. I wonder no one has seen this yet.

Use this instead:
ADMUX=(ADMUX & 0xF0)|ch;

Similarly, I would suggest changing
void InitADC()
{
ADMUX=(1<<REFS0);// For Aref=AVcc;
ADCSRA=(1<<ADEN)|(7<<ADPS0);
}
to
void ADC_init(void)
{
ADMUX = (ADMUX & ~(1<<REFS1)) | (1<<REFS0); // For Aref = AVcc
ADCSRA |=(1<<ADEN)|(1<<ADPS2)|(1<<ADPS1)|(1<<ADPS0); //slowest possible rate
}

which I think is the proper way.

Thanks for any comments

Regards,
Jon

Regards,
Mayank.