On Tue, Jul 17, 2007 at 05:42:52PM +0900, Magnus Damm wrote:
> pata_platform: Fix NULL pointer dereference
>
> The platform-specific structures may leave pdev->dev.platform_data as NULL.
>
> Signed-off-by: Magnus Damm <damm@xxxxxxxxxx>
Looks like the change that broke this came in the libata merge, more
specifically this change:
commit 5f45bc50976ee1f408f7171af155aec646655a37
Author: Sonic Zhang <sonic.adi@xxxxxxxxx>
Date: Fri Jun 15 17:45:49 2007 +0800
Add irq_flags to struct pata_platform_info
On some embedded platforms, such as blackfin, the gpio interrupt for
IDE interface is designed to be triggered with high voltage. The gpio
port should be configured properly by set_irq_type() when register
the irq. This patch enable the generic pata platform driver to
accept platform irq flags data.
Signed-off-by: Sonic Zhang <sonic.adi@xxxxxxxxx>
Signed-off-by: Jeff Garzik <jeff@xxxxxxxxxx>
It would be great if libata people could actually be bothered to CC driver
authors on driver changes so that way these things don't get completely broken
in mainline when simple testing on the platforms that actually _rely_ on this
driver would have shown that this was broken.
I don't intend to poll the various libata branches on the off chance I'll find
something that's hosed my platforms, and I don't think it's an unreasonable
expectation to at least have to see the patch and sign off on it before it gets
merged. Every other subsystem works this way, I wonder why libata feels that
it's special.
This is also not the first time changes to pata_platform have caused fallout,
without finding out about it until after the changes were already merged.
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
