Re: [ath9k-devel] [PATCH v4 07/11] ath9k: get EEPROM contents from platform data on AHB bus

[Gabor Juhos <juhosg@openwrt.org>]

Sujith =EDrta:
> Gabor Juhos wrote:
>> +static bool ath_ahb_eeprom_read(struct ath_hal *ah, u32 off, u16 *d=
ata)
>> +{
>> +	struct ath_softc *sc =3D ah->ah_sc;
>> +	struct platform_device *pdev =3D to_platform_device(sc->dev);
>> +	struct ath9k_platform_data *pdata;
>> +
>> +	pdata =3D (struct ath9k_platform_data *) pdev->dev.platform_data;
>> +	if (off >=3D (ARRAY_SIZE(pdata->eeprom_data))) {
>> +		DPRINTF(ah->ah_sc, ATH_DBG_FATAL,
>> +			"%s: flash read failed, offset %08x is out of range\n",
>> +				__func__, off);
>> +		return false;
>> +	}
>> +
>> +	*data =3D pdata->eeprom_data[off];
>> +	return true;
>> +}
>> +
>=20
> Shouldn't pdev->dev.platform_data be initalized somewhere ?

It must be initialized by the board specific setup code,

> @@ -48,6 +69,12 @@ static int ath_ahb_probe(struct platform_device *p=
dev)
>  	int ret =3D 0;
>  	struct ath_hal *ah;
> =20
> +	if (!pdev->dev.platform_data) {
> +		dev_err(&pdev->dev, "no platform data specified\n");
> +		ret =3D -EINVAL;
> +		goto err_out;
> +	}
> +

=2E.. and we have checked it in the probe routine anyway.

-Gabor

--
To unsubscribe from this list: send the line "unsubscribe linux-wireles=
s" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html