lkml.org 
[lkml]   [2014]   [Oct]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] ARM: keystone: add bus notifier to set dma_pfn_offset for pci devices
Date
On Friday 10 October 2014 11:15:37 Murali Karicheri wrote:
> @@ -54,6 +55,8 @@ static void __init keystone_init(void)
> keystone_pm_runtime_init();
> if (platform_nb.notifier_call)
> bus_register_notifier(&platform_bus_type, &platform_nb);
> + if (platform_nb.notifier_call)
> + bus_register_notifier(&pci_bus_type, &platform_nb);
> of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
>

No, this looks very wrong. Santosh spent an enormous effort on obsoleting
the platform notifier block by adding the range parser to the platform
device probe path.

You should really remove platform_nb and all associated code rather than
adding more code to it.

NAK

Arnd


\
 
 \ /
  Last update: 2014-10-10 21:01    [W:0.112 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site