On Tuesday 12 May 2009, Arnd Bergmann wrote:> +#ifdef CONFIG_HAS_DMA> ap->prd = dmam_alloc_coherent(dev, ATA_PRD_TBL_SZ, &ap->prd_dma,> GFP_KERNEL);> +#else> + ap->prd = devres_alloc(dev, ATA_PRD_TBL_SZ, GFP_KERNEL);> +#endif /* CONFIG_HAS_DMA */> if (!ap->prd)> return -ENOMEM; this should have been devm_kzalloc, not devres_alloc. Arnd <><ÿôèº{.nÇ+?·?®??+%?Ëÿ±éݶ¥?wÿº{.nÇ+?·¥?{±þ'^þ)í?æèw*jg¬±¨¶????Ý¢jÿ¾«þG«?éÿ¢¸¢·¦j:+v?¨?wèjØm¶?ÿþø¯ù®w¥þ?àþf£¢·h??â?úÿ?Ù¥