From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.9 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 29687C43381 for ; Sun, 24 Mar 2019 20:30:43 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id E100D2147A for ; Sun, 24 Mar 2019 20:30:42 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="HaU42Vql" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728856AbfCXUae (ORCPT ); Sun, 24 Mar 2019 16:30:34 -0400 Received: from mail-lj1-f194.google.com ([209.85.208.194]:32990 "EHLO mail-lj1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726811AbfCXUad (ORCPT ); Sun, 24 Mar 2019 16:30:33 -0400 Received: by mail-lj1-f194.google.com with SMTP id f23so6030952ljc.0; Sun, 24 Mar 2019 13:30:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=yuxIPFWyE6vKHz01aXS2PGrmp+xsZqZ4aZTXdnRi6Uk=; b=HaU42VqlUOlSwuhvlTXdTLIxnvh0CsNtUP5Nw77sA+Pz9ZDLABbFbi24wlJ1KrjFvz uexq6mmWcacHpYyMbP65IBBnc0EBgIK4z1JKVdWFor8y4/zAmF6JGxuE/Z4pZW9luffA eouVQftzT0U39iPCiBNyBlgGSIisPfxO1BFyZyRtWSX6VRDdIqs/k+7NAcqGpkbpqbKs BK6I8KiMx4/VUS8vAKr8I5q3DiX1EYcD2zNtzpJR/zKBK4XxZZersIaI5CyNRTwUELE6 u3kZy7f3X/EMYffy51uqOmfvSwCblkRqrtrc0BAK9a7H+SYanSXlL9sdHmM9qSpGUsE0 SpDw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=yuxIPFWyE6vKHz01aXS2PGrmp+xsZqZ4aZTXdnRi6Uk=; b=hhFJAmdUBDLlIDR80VptZW+HWroH18BEDhfd34PzLm8FgJVx6IiNfqGTdLrbJrKefc uIJsW+FAmAdBp8fk1Ltanh8LcEmUqICvnbGh+spOrHrSSXCz1AxpWzRPBZ1RtD34E8zT RJWq/TCgZD05E9bkUuiUTdlNALkKUwgdnmRxRmlJR0Aw88toowKDBxg2xvSMzxegH5a0 JboCqXKnr+37aD9X4pgRJha0JDlR3sZN09wmfyWtYa/wLQ3wCMbTncQuUV3WXcM9l8jS voG1U1XtraxJIn+XEj0jWd9A7nFGZMLfchLolROtDFT62MDJRAEZpdnbrV0AZqCW3eYF 27/g== X-Gm-Message-State: APjAAAXS9yOHWxKhGG+Gd8I9YxRuKQs7dSsx+0jtwhL3WOhLtBq3yvPj xFyy6Kh/tFq8tud5/hovMLs= X-Google-Smtp-Source: APXvYqyTsSGl12g7kVwQ79IOlx4bJuNI03+SseUOE8CDJ8voRbffyOzLpqQbLklLey2sfN1e3uYgDg== X-Received: by 2002:a2e:934a:: with SMTP id m10mr10985041ljh.164.1553459431459; Sun, 24 Mar 2019 13:30:31 -0700 (PDT) Received: from z50.localnet (109241207190.gdansk.vectranet.pl. [109.241.207.190]) by smtp.gmail.com with ESMTPSA id h14sm3029220ljg.10.2019.03.24.13.30.29 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 24 Mar 2019 13:30:30 -0700 (PDT) From: Janusz Krzysztofik To: Aaro Koskinen Cc: Boris Brezillon , Miquel Raynal , Richard Weinberger , David Woodhouse , Brian Norris , Marek Vasut , Tony Lindgren , linux-mtd@lists.infradead.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] mtd: rawnand: ams-delta: Drop board specific partition info Date: Sun, 24 Mar 2019 21:30:28 +0100 Message-ID: <3239616.xjhv0hSXo4@z50> In-Reply-To: <20190324185932.GA24594@darkstar.musicnaut.iki.fi> References: <20190319223718.26131-1-jmkrzyszt@gmail.com> <2631901.52Nzq63B5o@z50> <20190324185932.GA24594@darkstar.musicnaut.iki.fi> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Sunday, March 24, 2019 7:59:32 PM CET Aaro Koskinen wrote: > Hi, > > On Sun, Mar 24, 2019 at 05:48:22PM +0100, Janusz Krzysztofik wrote: > > Hi Aaro, > > > > Thanks for your review. > > > > On Wednesday, March 20, 2019 2:16:30 AM CET Aaro Koskinen wrote: > > > On Tue, Mar 19, 2019 at 11:37:18PM +0100, Janusz Krzysztofik wrote: > > > > After recent modifications, only a hardcoded partition info makes > > > > the driver device specific. Other than that, the driver uses GPIO > > > > exclusively and can be used on any hardware. > > > > > > > > Drop the partition info and use MTD partition parser with default > > > > list of partition types instead. > > > > > > > > Amstrad Delta users should append the followig partition info to their > > > ^^^^^^^^ > > > Should be "following". > > > > > > > kernel command line, possibly by embedding it in CONFIG_CMDLINE: > > > > mtdparts=ams-delta-nand:3584k(Kernel),256k(u-boot),256k(u- boot_params),\ > > > > 256k(Amstrad_LDR),27m(File_system),768k(PBL_reserved). For their > > > > convenience, select CONFIG_MTD_CMDLINE_PARTS symbol from that board > > > > Kconfig automatically if this NAND driver is also selected. > > > > > > > > Signed-off-by: Janusz Krzysztofik > > > > Cc: Tony Lindgren > > > > > > Could we move the fixed partition setup to the board file > > > instead? Otherwise this kind of change is not really nice for the users, > > > as it will likely break existing setups. The default partition layout > > > should remain the same. > > > > I'm wondering if it would be acceptable to pass partition info from a .dts > > file. I think that would be a better, more modern approach than adding a new > > header under include/linux/platform_data. > > Hmm, I thought there was some generic way to define partitions without > adding any new headers. But if that is not possible, then I guess your > CMDLINE proposal is the preferred one.. I could for example reuse (or abuse) struct gpio_nand_platdata defined in include/linux/mtd/nand-gpio.h for use with drivers/mtd/nand/raw/gpio.c, but I'm not sure if hi-jacking a header that belongs to another driver would be an elegant solution. Thanks, Janusz > > A. > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Janusz Krzysztofik Subject: Re: [PATCH] mtd: rawnand: ams-delta: Drop board specific partition info Date: Sun, 24 Mar 2019 21:30:28 +0100 Message-ID: <3239616.xjhv0hSXo4@z50> References: <20190319223718.26131-1-jmkrzyszt@gmail.com> <2631901.52Nzq63B5o@z50> <20190324185932.GA24594@darkstar.musicnaut.iki.fi> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20190324185932.GA24594@darkstar.musicnaut.iki.fi> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Aaro Koskinen Cc: linux-omap@vger.kernel.org, Boris Brezillon , Tony Lindgren , Richard Weinberger , linux-kernel@vger.kernel.org, Marek Vasut , linux-mtd@lists.infradead.org, Miquel Raynal , Brian Norris , David Woodhouse , linux-arm-kernel@lists.infradead.org List-Id: linux-omap@vger.kernel.org Hi, On Sunday, March 24, 2019 7:59:32 PM CET Aaro Koskinen wrote: > Hi, > > On Sun, Mar 24, 2019 at 05:48:22PM +0100, Janusz Krzysztofik wrote: > > Hi Aaro, > > > > Thanks for your review. > > > > On Wednesday, March 20, 2019 2:16:30 AM CET Aaro Koskinen wrote: > > > On Tue, Mar 19, 2019 at 11:37:18PM +0100, Janusz Krzysztofik wrote: > > > > After recent modifications, only a hardcoded partition info makes > > > > the driver device specific. Other than that, the driver uses GPIO > > > > exclusively and can be used on any hardware. > > > > > > > > Drop the partition info and use MTD partition parser with default > > > > list of partition types instead. > > > > > > > > Amstrad Delta users should append the followig partition info to their > > > ^^^^^^^^ > > > Should be "following". > > > > > > > kernel command line, possibly by embedding it in CONFIG_CMDLINE: > > > > mtdparts=ams-delta-nand:3584k(Kernel),256k(u-boot),256k(u- boot_params),\ > > > > 256k(Amstrad_LDR),27m(File_system),768k(PBL_reserved). For their > > > > convenience, select CONFIG_MTD_CMDLINE_PARTS symbol from that board > > > > Kconfig automatically if this NAND driver is also selected. > > > > > > > > Signed-off-by: Janusz Krzysztofik > > > > Cc: Tony Lindgren > > > > > > Could we move the fixed partition setup to the board file > > > instead? Otherwise this kind of change is not really nice for the users, > > > as it will likely break existing setups. The default partition layout > > > should remain the same. > > > > I'm wondering if it would be acceptable to pass partition info from a .dts > > file. I think that would be a better, more modern approach than adding a new > > header under include/linux/platform_data. > > Hmm, I thought there was some generic way to define partitions without > adding any new headers. But if that is not possible, then I guess your > CMDLINE proposal is the preferred one.. I could for example reuse (or abuse) struct gpio_nand_platdata defined in include/linux/mtd/nand-gpio.h for use with drivers/mtd/nand/raw/gpio.c, but I'm not sure if hi-jacking a header that belongs to another driver would be an elegant solution. Thanks, Janusz > > A. > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.7 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id BDC73C43381 for ; Sun, 24 Mar 2019 20:30:44 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 897952147A for ; Sun, 24 Mar 2019 20:30:44 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="UJF92vyQ"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="HaU42Vql" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 897952147A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=iKpt+2NYJ5OYvBLWE0QHLKX22qJ5uU63gaJzizjJGvU=; b=UJF92vyQIj/xsk pc9b3j1PmO2gCS9rXujQWU+LN4FbnHwYTGEI81qNgE+37MgQU0fM3nuApz7I2KhA+t6spk1BcqJLU FKSJV2JaXwFFvZs941h8pKaxLcFunOcOZ5Krs9UM3uurX0eRIOj9oiGgUozHOOqLAPhnwx/uYXvVI T/kNI0qMhJBXnMqHpznt2prPSMgVGRg3+s79VGsdp7QO2RbBWHQ8glh7x+DEMNaTb8CsP8yIHaMie 7yCDtLgS2ws184fi6ii/EZbCQWzU2IsXdRxYUHtKHpSnHsFfu/1R/KNWOgACi5nXwuJyBqepMwiDR N6ptKYOnHu4GwePZV8AQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1h89lN-0002Is-S1; Sun, 24 Mar 2019 20:30:37 +0000 Received: from mail-lj1-x241.google.com ([2a00:1450:4864:20::241]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1h89lK-0002Er-8q; Sun, 24 Mar 2019 20:30:35 +0000 Received: by mail-lj1-x241.google.com with SMTP id k8so5996341lja.8; Sun, 24 Mar 2019 13:30:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=yuxIPFWyE6vKHz01aXS2PGrmp+xsZqZ4aZTXdnRi6Uk=; b=HaU42VqlUOlSwuhvlTXdTLIxnvh0CsNtUP5Nw77sA+Pz9ZDLABbFbi24wlJ1KrjFvz uexq6mmWcacHpYyMbP65IBBnc0EBgIK4z1JKVdWFor8y4/zAmF6JGxuE/Z4pZW9luffA eouVQftzT0U39iPCiBNyBlgGSIisPfxO1BFyZyRtWSX6VRDdIqs/k+7NAcqGpkbpqbKs BK6I8KiMx4/VUS8vAKr8I5q3DiX1EYcD2zNtzpJR/zKBK4XxZZersIaI5CyNRTwUELE6 u3kZy7f3X/EMYffy51uqOmfvSwCblkRqrtrc0BAK9a7H+SYanSXlL9sdHmM9qSpGUsE0 SpDw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=yuxIPFWyE6vKHz01aXS2PGrmp+xsZqZ4aZTXdnRi6Uk=; b=jpBPNWiFPspy6D3R+c7siy0PfdLlH4m/6I58W72ViT9JAu6RfQR80A25cYXjvMGgyt DqxMvtkP1h9tJTP3AAmuuIjBy3QYHMhUAcP1M4L4jeZSYzZ/Peihjf7uckgjmA3OUlLb ow3ttaq2h1LvqYcO68/sK0OWW6im7xYGJ0dipHb/A6ROh0q8Ho9/uQF433aB4gcwh8sR a8tq3HkQqY/XC9vma2a/lAhPrbbTOqoRGfUvc99xktZBMnuQm8dnosAhLK2FK2GjpkW7 kslqciSDAD2/obAkgn1dRnNDdpiFi0n/OyTqouRSxlSO3qsg3ENKMycKkDzCJLg3taJj ie4A== X-Gm-Message-State: APjAAAWH5NHBenUt7x5JpK1GuMKwhYrtRVDnTIAFTgEwh1S2bK1EsrCv 4Hjm5nVFNPKKUXzI4b7iHMTE2Vi/ X-Google-Smtp-Source: APXvYqyTsSGl12g7kVwQ79IOlx4bJuNI03+SseUOE8CDJ8voRbffyOzLpqQbLklLey2sfN1e3uYgDg== X-Received: by 2002:a2e:934a:: with SMTP id m10mr10985041ljh.164.1553459431459; Sun, 24 Mar 2019 13:30:31 -0700 (PDT) Received: from z50.localnet (109241207190.gdansk.vectranet.pl. [109.241.207.190]) by smtp.gmail.com with ESMTPSA id h14sm3029220ljg.10.2019.03.24.13.30.29 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 24 Mar 2019 13:30:30 -0700 (PDT) From: Janusz Krzysztofik To: Aaro Koskinen Subject: Re: [PATCH] mtd: rawnand: ams-delta: Drop board specific partition info Date: Sun, 24 Mar 2019 21:30:28 +0100 Message-ID: <3239616.xjhv0hSXo4@z50> In-Reply-To: <20190324185932.GA24594@darkstar.musicnaut.iki.fi> References: <20190319223718.26131-1-jmkrzyszt@gmail.com> <2631901.52Nzq63B5o@z50> <20190324185932.GA24594@darkstar.musicnaut.iki.fi> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190324_133034_361321_E8FE4EA8 X-CRM114-Status: GOOD ( 25.44 ) X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-omap@vger.kernel.org, Boris Brezillon , Tony Lindgren , Richard Weinberger , linux-kernel@vger.kernel.org, Marek Vasut , linux-mtd@lists.infradead.org, Miquel Raynal , Brian Norris , David Woodhouse , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org Hi, On Sunday, March 24, 2019 7:59:32 PM CET Aaro Koskinen wrote: > Hi, > > On Sun, Mar 24, 2019 at 05:48:22PM +0100, Janusz Krzysztofik wrote: > > Hi Aaro, > > > > Thanks for your review. > > > > On Wednesday, March 20, 2019 2:16:30 AM CET Aaro Koskinen wrote: > > > On Tue, Mar 19, 2019 at 11:37:18PM +0100, Janusz Krzysztofik wrote: > > > > After recent modifications, only a hardcoded partition info makes > > > > the driver device specific. Other than that, the driver uses GPIO > > > > exclusively and can be used on any hardware. > > > > > > > > Drop the partition info and use MTD partition parser with default > > > > list of partition types instead. > > > > > > > > Amstrad Delta users should append the followig partition info to their > > > ^^^^^^^^ > > > Should be "following". > > > > > > > kernel command line, possibly by embedding it in CONFIG_CMDLINE: > > > > mtdparts=ams-delta-nand:3584k(Kernel),256k(u-boot),256k(u- boot_params),\ > > > > 256k(Amstrad_LDR),27m(File_system),768k(PBL_reserved). For their > > > > convenience, select CONFIG_MTD_CMDLINE_PARTS symbol from that board > > > > Kconfig automatically if this NAND driver is also selected. > > > > > > > > Signed-off-by: Janusz Krzysztofik > > > > Cc: Tony Lindgren > > > > > > Could we move the fixed partition setup to the board file > > > instead? Otherwise this kind of change is not really nice for the users, > > > as it will likely break existing setups. The default partition layout > > > should remain the same. > > > > I'm wondering if it would be acceptable to pass partition info from a .dts > > file. I think that would be a better, more modern approach than adding a new > > header under include/linux/platform_data. > > Hmm, I thought there was some generic way to define partitions without > adding any new headers. But if that is not possible, then I guess your > CMDLINE proposal is the preferred one.. I could for example reuse (or abuse) struct gpio_nand_platdata defined in include/linux/mtd/nand-gpio.h for use with drivers/mtd/nand/raw/gpio.c, but I'm not sure if hi-jacking a header that belongs to another driver would be an elegant solution. Thanks, Janusz > > A. > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.7 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 924E1C43381 for ; Sun, 24 Mar 2019 20:31:00 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5E71E2147A for ; Sun, 24 Mar 2019 20:31:00 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="M2N/vZ0h"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="HaU42Vql" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5E71E2147A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=s/ZMH01ofWBxOnOAThQXu1TRcnVYI9Xw0pHXO1/TgkE=; b=M2N/vZ0hqI+Oqi 4XnnuHPCdPVT/8g8jiLzlfSGBaipM/xf9rSqhrzO5xo8gTcmgRZEsoRFTl3lFcKeEISrS3HZZAzp7 ahpCSAj6kK8GE91G1Cx89Kbcd22YdTWCETGsp2gLzrslmnOI7EekGj+E6iSchjNWdhOTIV7qjV281 KnsfsEj1xNY+8ifOn9gzgcofu2Ljsttn40NaRjAemZEHix+Aw2mgvLIvr56yoklkaIafAfivCWZLO ijb2umX1V2X6Y/9CW7wLQ3H3O8XQnvVX28vYgXU0MAYQ2IzWCjRmjP3pdPyImp4Jt9XxEVWk67WFO /xW2aFL5vaVaVbjzYLiw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1h89lT-0002Nl-5y; Sun, 24 Mar 2019 20:30:43 +0000 Received: from mail-lj1-x241.google.com ([2a00:1450:4864:20::241]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1h89lK-0002Er-8q; Sun, 24 Mar 2019 20:30:35 +0000 Received: by mail-lj1-x241.google.com with SMTP id k8so5996341lja.8; Sun, 24 Mar 2019 13:30:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=yuxIPFWyE6vKHz01aXS2PGrmp+xsZqZ4aZTXdnRi6Uk=; b=HaU42VqlUOlSwuhvlTXdTLIxnvh0CsNtUP5Nw77sA+Pz9ZDLABbFbi24wlJ1KrjFvz uexq6mmWcacHpYyMbP65IBBnc0EBgIK4z1JKVdWFor8y4/zAmF6JGxuE/Z4pZW9luffA eouVQftzT0U39iPCiBNyBlgGSIisPfxO1BFyZyRtWSX6VRDdIqs/k+7NAcqGpkbpqbKs BK6I8KiMx4/VUS8vAKr8I5q3DiX1EYcD2zNtzpJR/zKBK4XxZZersIaI5CyNRTwUELE6 u3kZy7f3X/EMYffy51uqOmfvSwCblkRqrtrc0BAK9a7H+SYanSXlL9sdHmM9qSpGUsE0 SpDw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=yuxIPFWyE6vKHz01aXS2PGrmp+xsZqZ4aZTXdnRi6Uk=; b=jpBPNWiFPspy6D3R+c7siy0PfdLlH4m/6I58W72ViT9JAu6RfQR80A25cYXjvMGgyt DqxMvtkP1h9tJTP3AAmuuIjBy3QYHMhUAcP1M4L4jeZSYzZ/Peihjf7uckgjmA3OUlLb ow3ttaq2h1LvqYcO68/sK0OWW6im7xYGJ0dipHb/A6ROh0q8Ho9/uQF433aB4gcwh8sR a8tq3HkQqY/XC9vma2a/lAhPrbbTOqoRGfUvc99xktZBMnuQm8dnosAhLK2FK2GjpkW7 kslqciSDAD2/obAkgn1dRnNDdpiFi0n/OyTqouRSxlSO3qsg3ENKMycKkDzCJLg3taJj ie4A== X-Gm-Message-State: APjAAAWH5NHBenUt7x5JpK1GuMKwhYrtRVDnTIAFTgEwh1S2bK1EsrCv 4Hjm5nVFNPKKUXzI4b7iHMTE2Vi/ X-Google-Smtp-Source: APXvYqyTsSGl12g7kVwQ79IOlx4bJuNI03+SseUOE8CDJ8voRbffyOzLpqQbLklLey2sfN1e3uYgDg== X-Received: by 2002:a2e:934a:: with SMTP id m10mr10985041ljh.164.1553459431459; Sun, 24 Mar 2019 13:30:31 -0700 (PDT) Received: from z50.localnet (109241207190.gdansk.vectranet.pl. [109.241.207.190]) by smtp.gmail.com with ESMTPSA id h14sm3029220ljg.10.2019.03.24.13.30.29 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 24 Mar 2019 13:30:30 -0700 (PDT) From: Janusz Krzysztofik To: Aaro Koskinen Subject: Re: [PATCH] mtd: rawnand: ams-delta: Drop board specific partition info Date: Sun, 24 Mar 2019 21:30:28 +0100 Message-ID: <3239616.xjhv0hSXo4@z50> In-Reply-To: <20190324185932.GA24594@darkstar.musicnaut.iki.fi> References: <20190319223718.26131-1-jmkrzyszt@gmail.com> <2631901.52Nzq63B5o@z50> <20190324185932.GA24594@darkstar.musicnaut.iki.fi> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190324_133034_361321_E8FE4EA8 X-CRM114-Status: GOOD ( 25.44 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-omap@vger.kernel.org, Boris Brezillon , Tony Lindgren , Richard Weinberger , linux-kernel@vger.kernel.org, Marek Vasut , linux-mtd@lists.infradead.org, Miquel Raynal , Brian Norris , David Woodhouse , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, On Sunday, March 24, 2019 7:59:32 PM CET Aaro Koskinen wrote: > Hi, > > On Sun, Mar 24, 2019 at 05:48:22PM +0100, Janusz Krzysztofik wrote: > > Hi Aaro, > > > > Thanks for your review. > > > > On Wednesday, March 20, 2019 2:16:30 AM CET Aaro Koskinen wrote: > > > On Tue, Mar 19, 2019 at 11:37:18PM +0100, Janusz Krzysztofik wrote: > > > > After recent modifications, only a hardcoded partition info makes > > > > the driver device specific. Other than that, the driver uses GPIO > > > > exclusively and can be used on any hardware. > > > > > > > > Drop the partition info and use MTD partition parser with default > > > > list of partition types instead. > > > > > > > > Amstrad Delta users should append the followig partition info to their > > > ^^^^^^^^ > > > Should be "following". > > > > > > > kernel command line, possibly by embedding it in CONFIG_CMDLINE: > > > > mtdparts=ams-delta-nand:3584k(Kernel),256k(u-boot),256k(u- boot_params),\ > > > > 256k(Amstrad_LDR),27m(File_system),768k(PBL_reserved). For their > > > > convenience, select CONFIG_MTD_CMDLINE_PARTS symbol from that board > > > > Kconfig automatically if this NAND driver is also selected. > > > > > > > > Signed-off-by: Janusz Krzysztofik > > > > Cc: Tony Lindgren > > > > > > Could we move the fixed partition setup to the board file > > > instead? Otherwise this kind of change is not really nice for the users, > > > as it will likely break existing setups. The default partition layout > > > should remain the same. > > > > I'm wondering if it would be acceptable to pass partition info from a .dts > > file. I think that would be a better, more modern approach than adding a new > > header under include/linux/platform_data. > > Hmm, I thought there was some generic way to define partitions without > adding any new headers. But if that is not possible, then I guess your > CMDLINE proposal is the preferred one.. I could for example reuse (or abuse) struct gpio_nand_platdata defined in include/linux/mtd/nand-gpio.h for use with drivers/mtd/nand/raw/gpio.c, but I'm not sure if hi-jacking a header that belongs to another driver would be an elegant solution. Thanks, Janusz > > A. > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel