Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 28 Apr 2021 17:43:29 +0530
From:   Anupama K Patil <anupamakpatil123@gmail.com>
To:     Leon Romanovsky <leon@kernel.org>
Cc:     Greg KH <gregkh@linuxfoundation.org>,
        Jaroslav Kysela <perex@perex.cz>,
        "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
        linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
        skhan@linuxfoundation.org, B K Karthik <bkkarthik@pesu.pes.edu>,
        kernelnewbies@kernelnewbies.org
Subject: Re: [PATCH] MAINTAINERS: mark isapnp as obsolete
Message-ID: <20210428121329.o4jlx76klmzlzl5n@ubuntu>
References: <20210428110536.edy74gsobkgtd6al@ubuntu>
 <YIlCzOl/rkYeOWVW@kroah.com>
 <20210428114057.tlc53ie7crvkqqhj@ubuntu>
 <YIlLk00Wzd6Lf660@unreal>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YIlLk00Wzd6Lf660@unreal>
Precedence: bulk

On 21/04/28 02:48PM, Leon Romanovsky wrote:
> On Wed, Apr 28, 2021 at 05:10:57PM +0530, Anupama K Patil wrote:
> > On 21/04/28 01:11PM, Greg KH wrote:
> > > On Wed, Apr 28, 2021 at 04:35:36PM +0530, Anupama K Patil wrote:
> > > > isapnp code is very old and according to this link
> > > > https://en.wikipedia.org/wiki/Legacy_Plug_and_Play#Specifications
> > > > from Wikipedia, even Windows Vista disabled ISA PnP by default.
> > > > 
> > > > This change is in follow up to
> > > > https://lore.kernel.org/lkml/20210422180322.7wlyg63kv3n2k6id@ubuntu/T/#u
> > > > and https://lore.kernel.org/lkml/20210424194301.jmsqpycvsm7izbk3@ubuntu/T/#u
> > > > 
> > > > Suggested-by: B K Karthik <bkkarthik@pesu.pes.edu>
> > > > Signed-off-by: Anupama K Patil <anupamakpatil123@gmail.com>
> > > 
> > > Has the maintainer said this is unsupported?
> > > 
> > > That's who you should take suggestions from :)
> > > 
> > > thanks,
> > > 
> > > greg k-h
> > 
> > Leon said he thinks it is unsupported (I've also cc'd him). So, I've sent this patch
> > to get the feedback from the maintainer.
> 
> Leon said something different.
> https://lore.kernel.org/lkml/20210422180322.7wlyg63kv3n2k6id@ubuntu/T/#m6c837180c8d6496d5bcbb8526e274c4239844be0
> ------
> > If changes to ISA code aren't welcomed, should these be marked obsolete in the MAINTIANERS file?
> 
> I think so, but think that "Odd Fixes" better describes that Rafael wrote.
> ------
Sorry about that! but does checkpatch tell people not to send patches if it's 'Odd fixes'?
Should I wait for the maintainer's reply before doing anything else?
> 
> > 
> > Thanks,
> > Anupama