On 21 Jun 2016, at 09:58, Aravind Sirsikar <Arvind.Sirsikar(a)radisys.com> wrote: Hi Holger, The cs_current_trans is made public to align with existing variables like block[RLC_MAX_LEN], len and others to have consistency. will it make sense to have cs and cs_current_trans as private?. We can rename cs to cs_last_tx as suggested in review of other patch. But it will have modification in other source files to address compilation errors.