Re: [Xen-devel] [PATCH v3] ns16550-Add-command-line-parsing-adjustments

2017-04-18 Thread Jan Beulich
>>> On 17.04.17 at 17:51, wrote: > I have a question about __initconst that you mentioned. Sure, but please trim your replies. > On 4/3/2017 6:55 AM, Jan Beulich wrote: > On 31.03.17 at 17:42, wrote: >>> +/* enum struct keeping a table of all

Re: [Xen-devel] [PATCH v3] ns16550-Add-command-line-parsing-adjustments

2017-04-17 Thread Paratey, Swapnil
Hi Jan, I have a question about __initconst that you mentioned. On 4/3/2017 6:55 AM, Jan Beulich wrote: On 31.03.17 at 17:42, wrote: The title needs improvement - it doesn't really reflect what the patch does. Add name=value parsing options for com1 and com2 to

Re: [Xen-devel] [PATCH v3] ns16550-Add-command-line-parsing-adjustments

2017-04-11 Thread Jan Beulich
>>> On 10.04.17 at 20:47, wrote: > On 4/3/2017 6:55 AM, Jan Beulich wrote: > On 31.03.17 at 17:42, wrote: >> The title needs improvement - it doesn't really reflect what the >> patch does. > > I apologize for this. I kept the name same since it was

Re: [Xen-devel] [PATCH v3] ns16550-Add-command-line-parsing-adjustments

2017-04-10 Thread Paratey, Swapnil
Hi Jan, Thank you very much for your review comments. I need a few clarifications to move forward. On 4/3/2017 6:55 AM, Jan Beulich wrote: On 31.03.17 at 17:42, wrote: The title needs improvement - it doesn't really reflect what the patch does. I apologize for

Re: [Xen-devel] [PATCH v3] ns16550-Add-command-line-parsing-adjustments

2017-04-03 Thread Jan Beulich
>>> On 31.03.17 at 17:42, wrote: The title needs improvement - it doesn't really reflect what the patch does. > Add name=value parsing options for com1 and com2 to add flexibility > in setting register values for MMIO UART devices. > > Maintain backward compatibility

[Xen-devel] [PATCH v3] ns16550-Add-command-line-parsing-adjustments

2017-03-31 Thread Swapnil Paratey
Add name=value parsing options for com1 and com2 to add flexibility in setting register values for MMIO UART devices. Maintain backward compatibility with previous positional parameter specfications. eg. com1=115200,8n1,0x3f8,4 eg. com1=baud=115200,parity=n,reg_width=4,reg_shift=2,irq=4 eg.