rlm_ippool and dhcp

Good day, community. I added dhcp support to rlm_ippool (i.e. now rlm_ip_pool check, if this is dhcp request and if yes -- fill dhcp attributes, instead framed-ip-address) and want to merge this changes into master branch. (also backport to 2.1.10 branch is live in small production now) [changes against master are in: https://github.com/rssh/freeradius-server/commit/546d3e69297313252c4d4adcd34... ] So, questions - is it ok to extend rlm_ippool in such way ? if yes, than I will start pull request after fixing compilation in master branch. Regards !