lkml.org 
[lkml]   [2023]   [Aug]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH v4 21/28] net: wan: Add framer framework support
    Date
    Hi Linus,

    Le 20/08/2023 à 23:06, Linus Walleij a écrit :
    > On Fri, Aug 18, 2023 at 6:41 PM Christophe Leroy
    > <christophe.leroy@csgroup.eu> wrote:
    >
    >> From: Herve Codina <herve.codina@bootlin.com>
    >>
    >> A framer is a component in charge of an E1/T1 line interface.
    >> Connected usually to a TDM bus, it converts TDM frames to/from E1/T1
    >> frames. It also provides information related to the E1/T1 line.
    >>
    >> The framer framework provides a set of APIs for the framer drivers
    >> (framer provider) to create/destroy a framer and APIs for the framer
    >> users (framer consumer) to obtain a reference to the framer, and
    >> use the framer.
    >>
    >> This basic implementation provides a framer abstraction for:
    >> - power on/off the framer
    >> - get the framer status (line state)
    >> - be notified on framer status changes
    >> - get/set the framer configuration
    >>
    >> Signed-off-by: Herve Codina <herve.codina@bootlin.com>
    >> Reviewed-by: Christophe Leroy <christophe.leroy@csgroup.eu>
    >> Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu>
    >
    > I had these review comments, you must have missed them?
    > https://lore.kernel.org/netdev/CACRpkdZQ9_f6+9CseV1L_wGphHujFPAYXMjJfjUrzSZRakOBzg@mail.gmail.com/
    >

    As I said in the cover letter, this series only fixes critical build
    failures that happened when CONFIG_MODULES is set. The purpose was to
    allow robots to perform their job up to the end. Other feedback and
    comments will be taken into account by Hervé when he is back from holidays.

    Thanks
    Christophe
    \
     
     \ /
      Last update: 2023-08-21 07:20    [W:2.890 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site