On Sun, Feb 07, 2021 at 12:43:40PM +0900, Tetsuya Isaki wrote: > > @@ -175,9 +175,9 @@ > > .Vt audio_format_t > > structure according to given number > > .Va afp->index . > > -If there is no format with given number, return > > +If there is no format with the given number, return > > .Er EINVAL . > > -It is called at any time. > > +It can be called at any time. > > The later sounds to me "You(developer of MD driver) can call > it at any time". If so, it's incorrect.
Maybe "it can be called by the MI layer at any time" is clearer here, then? I can change it to that. > > > Similarly, if the driver supports > > .Dv SLINEAR_OE:16 > > and the upper layer chooses it, > > -the driver does not need to provide a conversion function. > > -Because the upper layer only supports conversion between > > +the driver does not need to provide a conversion function, > > +because the upper layer supports conversion between > > Is "only" a typo? or is it better to remove it in English? > > Thanks, > --- > Tetsuya Isaki <is...@pastel-flower.jp / is...@netbsd.org> I think it's clear that conversion of other formats is not supported by the rest of the paragraph, so it doesn't need to be mentioned here, where the primary purpose of the sentence is to explain why you don't need to handle conversion in that case yourself. That's just my opinion, though - I'm not an English expert, there's lots of rules I know but cannot explain. Thanks, Nia