Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp3749118pxj; Mon, 7 Jun 2021 20:04:19 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxpu48ESdtO22bm9Wq6yMBOptXyis+rprO2lMFXuhBH35YiD+Hjk35iAdN+BlwvHF6UlJ2R X-Received: by 2002:a05:6402:1111:: with SMTP id u17mr23445117edv.87.1623121458994; Mon, 07 Jun 2021 20:04:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1623121458; cv=none; d=google.com; s=arc-20160816; b=NLC5VBFufQFBi9gnPdEW8KNRVs8NixPkiv2N69SFXOpFGAT5TsghjgyCmqUF1CUqE8 lx8gjGRjrKo9KeWpf54xESeppHOw5brDc4t9an11McJ5YDO6qpSP3Kl/JFJr5EkeU1+O /PUF1z64W6LvQT6OTsEob6Hc/kaTtwQBW3JjRFEIUtToweJseMGfA3O/nEs0AI0IhHA1 TI4b4mB/RTPJ964bLSpsJW/99MPW/2bSF/XxriiAifseFhNupRwHRIqJgCFprMxIyHvS NdZIwKuUhGoAL3g9rJPG+iRIlJ+GSeZHRJohqYZaYFAXOpuy4/IOK7Z2EG/d4Meane1W /yHQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to :mime-version:user-agent:date:message-id:from:cc:references:to :subject:dkim-signature; bh=4zyzSgjehYpTMwvkMAdkEk8UOEiuBvRn+Lbe9JMcg3U=; b=0Q2zm28VpqEGi8K8B02p436IZlmee3RMez2YaEKOGUQ7t4Sp0DWmX4EwhtVN3pEDEl rNMN4Dc3Ah3kWD+Ci8f/sErKPWrl5Xhwq8D8U1oFSfN+KntFhgGCEJdx/XGYUSy95yKr T79r9AEgIcUM6x5VQaeuaGKl+DrgRITNg/t0zQqUhC/fLADWNG8GkpWHNngY0sMZJ6WI KQeB4p0Smqz+q2bsX8g+TmvtS1hsctdtO6p9hvEYdZauZC951u33LwCMNMMCbgp/11Ko ZJ51zn4Dgg/pphW52wzAkJm25Ip4Z4g5fvP30vFJ6LZXA5MG0xdMVjo5jBuPXaaH9FvG ofww== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=KmwLevrV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id t14si14428720edq.26.2021.06.07.20.03.56; Mon, 07 Jun 2021 20:04:18 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=KmwLevrV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231557AbhFHDDp (ORCPT + 99 others); Mon, 7 Jun 2021 23:03:45 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49080 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231520AbhFHDDl (ORCPT ); Mon, 7 Jun 2021 23:03:41 -0400 Received: from mail-pf1-x430.google.com (mail-pf1-x430.google.com [IPv6:2607:f8b0:4864:20::430]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9D64CC061574; Mon, 7 Jun 2021 20:01:38 -0700 (PDT) Received: by mail-pf1-x430.google.com with SMTP id c12so14641380pfl.3; Mon, 07 Jun 2021 20:01:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:references:cc:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=4zyzSgjehYpTMwvkMAdkEk8UOEiuBvRn+Lbe9JMcg3U=; b=KmwLevrVyZBLk+TmP9kkEbneRprDD+cTDc0mI24ZPzZmmSOvZrTtua9iRD/F86olxj XQxcHOeBpFNwsT9mafmBM5kWyJ5XrWUflmotaFtnE7CrU+ZNcSTaL+MwfZ/XEhPwkqoh tFMfhaAChKjg9eh1pl1MkH1RhJtk5qjpF56DA3DJsM1Ch/LcuBuocp0puIiG3kwKqHUm UWfOYdYCU/z4HKw6sRU32qcaLzeoAFtB2Dj9sJGsoJuDoRumahYiN8zrIbcjhQxmxO4R T6cDnViulSmU4/FnHqs+kV1rZL5UoNK/Q/BAVrKvzVrc0muqKf+yHnUpydwdsMJPp0CX jzAw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:references:cc:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=4zyzSgjehYpTMwvkMAdkEk8UOEiuBvRn+Lbe9JMcg3U=; b=ZSuETvkqE58LrigtXRsIeLnCu07eCHZFpdHVY7kjJDmdunZtNw8jaCyO5dBIzP4me7 aOA+/QLBnDy/JYtGSiyKRFe5EkVr13ehmfayWd4mebwkKMpFIG2S6ZcPnLibUjWJcH0w v6I8pRRP4m8V9DunVBRDHHOx0v3EEjZwsSRlohDQnoowxj+8+MvjjWOLAwFq8o0McLHi bxV/Lm4vWSlu84moW46+9nlqvWPgODe7Q/PyV6Y2IcK5N1jPLvCRjQGWNGuHj5IzgW8D ktx5XLKJc78PbhSdHK99jN3Zfa9ddYZOqmCsgc3JGx7hJ3pAkgqlf7Z10A9IU2va/BmV x2bg== X-Gm-Message-State: AOAM533IrF/NE5LQgcDMowNH2DRConNdLqEAvdBJW0FlodUkylnQcQsR cVCkjzYG/qlk18irdfXH1krLJ7xE7M4= X-Received: by 2002:a63:9515:: with SMTP id p21mr18889215pgd.333.1623121297946; Mon, 07 Jun 2021 20:01:37 -0700 (PDT) Received: from [10.1.1.25] (222-152-189-137-fibre.sparkbb.co.nz. [222.152.189.137]) by smtp.gmail.com with ESMTPSA id s22sm1671639pfg.197.2021.06.07.20.01.33 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 07 Jun 2021 20:01:37 -0700 (PDT) Subject: Re: [PATCH] m68k/q40: Replace q40ide with pata_falcon and falconide To: Finn Thain References: <352239036981177050c296ca3deb84a102d9faa2.1622970906.git.fthain@linux-m68k.org> <4a32d59-ebb-9dc-fec7-ad172547d9a3@linux-m68k.org> Cc: Geert Uytterhoeven , Richard Zidlicky , Jens Axboe , "David S. Miller" , linux-m68k@lists.linux-m68k.org, linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org From: Michael Schmitz Message-ID: <3b789204-a959-a266-9fc3-1b2be09c981a@gmail.com> Date: Tue, 8 Jun 2021 15:01:30 +1200 User-Agent: Mozilla/5.0 (X11; Linux ppc; rv:45.0) Gecko/20100101 Icedove/45.4.0 MIME-Version: 1.0 In-Reply-To: <4a32d59-ebb-9dc-fec7-ad172547d9a3@linux-m68k.org> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Finn, Am 07.06.2021 um 14:07 schrieb Finn Thain: > On Mon, 7 Jun 2021, Michael Schmitz wrote: > >> Hi Finn, >> >> are there any substantial changes from what I'd tested before? > > The request_region() stuff is new, and I reworked the host->get_lock and > host->release_lock changes. So you may want to review it again. Done that - the weirdness around request_irq() that I remembered is not actually an issue, that's taken care of by setting host->get_lock. And your patch doesn't change anything for Falcon. Tested on my Falcon - I can still use both IDE and SCSI (no stress test done but I doubt it would turn out much different). Not setting host->get_lock on Q40 will correctly request the IRQ for IDE there, the same lowlevel IO routines can be shared by both, and there wasn't ever much more that these old drivers did. Please add my Reviewed-by as well as Tested-by. Cheers, Michael >> Your patch conflicts with (I think) the Mac IDE patches now in Geert's >> tree, and I'd rather avoid hand-patching the lot if at all possible: >> >> m68k_q40_Replace_q40ide_with_pata_falcon_and_falconide >> Applying: m68k/q40: Replace q40ide with pata_falcon and falconide >> error: patch failed: arch/m68k/configs/multi_defconfig:351 >> error: arch/m68k/configs/multi_defconfig: patch does not apply >> error: patch failed: drivers/ide/Kconfig:731 >> error: drivers/ide/Kconfig: patch does not apply >> error: patch failed: drivers/ide/Makefile:29 >> error: drivers/ide/Makefile: patch does not apply >> > > I should have sent these as a two-part series. Sorry about that. > You'll need to apply this patch first: > https://lore.kernel.org/linux-ide/3de9a93d-2f7-a650-1fa-c2129a4a765b@nippy.intranet/T/ > >> What's your baseline for this patch? >> > > I've used v5.12 and v5.13-rc5, but mostly I develop on the mac68k branch > in my github repository. > >> (In the alternative: just add my Tested-by if nothing changed aside from the >> IO region reservation) >> > > I have tested these patches in Aranym (including a multi-platform > atari/q40 build which required your ISA fix) but I don't have any way to > test the q40-only code. >