On Tue 07 Jun 09:18 PDT 2016, Peter Griffin wrote: > Some firmware and rproc drivers such as slim core don't have a > resource table. To avoid having to fudge an empty table > in the rproc driver this patch introduces a new has_rsctable > flag. rproc drivers which don't require a rsc table can set > this flag to false, and it avoids doing the initial async > firmware load in rproc_add_virtio_devices() to parse the > rsc table during rproc_add(), and also optimises rproc_fw_boot() > code path to avoid unnecessary rsc table processing of an > empty table. > > Signed-off-by: Peter Griffin <peter.griffin@xxxxxxxxxx> Hi Peter, Thanks for the patch. I do however dislike the amount of conditionals already in these code paths, so I'm not in favour of adding more. Please revert back to providing a dummy table from your driver and let's handle this topic separate of this series. Regards, Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-remoteproc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html