How to post cleanup patches ?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Mark

Now codec driver and component driver has duplicated callback function,
and snd_soc_register_codec() just copied it to component side.

	int snd_soc_register_codec(xxx) {
		...
 
		if (codec_drv->controls) {
			codec->component.controls = codec_drv->controls;
			codec->component.num_controls = codec_drv->num_controls;
		}
		if (codec_drv->dapm_widgets) {
			codec->component.dapm_widgets = codec_drv->dapm_widgets;
			codec->component.num_dapm_widgets = codec_drv->num_dapm_widgets;
		}
		if (codec_drv->dapm_routes) {
			codec->component.dapm_routes = codec_drv->dapm_routes;
			codec->component.num_dapm_routes = codec_drv->num_dapm_routes;
		}
		...
	}

I think we can cleanup this duplicated function, similar things happen on
.probe, .remove, and platform side too.

I would like to post cleanup patches for these.
As 1st step, remove codec duplicate callback.

But my concern is that you hate big-patch-set, and it is almost 70 patches.
Can I post these patches to ML ? or should I use git pull request ? or I shouldn't post ?

I will post main patches as sample patch

Best regards
---
Kuninori Morimoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux