Date:   Mon, 13 Mar 2023 20:18:20 +0500
From:   Khadija Kamran <kamrankhadijadj@gmail.com>
To:     Dan Carpenter <error27@gmail.com>
Cc:     "Fabio M. De Francesco" <fmdefrancesco@gmail.com>,
        outreachy@lists.linux.dev,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4] staging: axis-fifo: initialize read_timeout and
 write_timeout once in probe function
Message-ID: <ZA8+vC0e6EUdm9q/@khadija-virtual-machine>
References: <ZA4M3+ZeB1Rl2fbs@khadija-virtual-machine>
 <9938109.ZD9C4EEAuf@suse>
 <ZA83yzPTq+nQF9Dd@khadija-virtual-machine>
 <f545dc88-717b-4fa0-851a-733dd5fc2eff@kili.mountain>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <f545dc88-717b-4fa0-851a-733dd5fc2eff@kili.mountain>
Precedence: bulk

On Mon, Mar 13, 2023 at 06:04:45PM +0300, Dan Carpenter wrote:
> On Mon, Mar 13, 2023 at 07:48:43PM +0500, Khadija Kamran wrote:
> > Also, I had one question. Is it okay to write a long subject as I have
> > used in this patch?
> 
> People say subject should be 50 chars (I forget the exact limit) but
> there is a little bit of flexibility.  This subject is too long though.
> Perhaps I would use something like:
> 
> [PATCH v5] staging: axis-fifo: initialize timeouts in probe only
>

Thanks Dan,
I am going to use this for the next patch  O:)

> regards,
> dan carpenter
>